[vlc-devel] [PATCH v2 1/5] clock: add vlc_clock_main_SetInputDrift

Thomas Guillem thomas at gllm.fr
Wed Mar 10 11:28:39 UTC 2021


Hello,

I commented it but after sending the patch set to the ML.

Cf the 2 commits here:
https://code.videolan.org/tguillem/vlc/-/commit/706bb9622968c92bd4f478caa145b19353865ba1
https://code.videolan.org/tguillem/vlc/-/commit/4c76da0b76bf49e0da37512d6fd8ac66fc8b0242

On Tue, Mar 9, 2021, at 16:54, Rémi Denis-Courmont wrote:
> Le lundi 8 mars 2021, 18:13:34 EET Thomas Guillem a écrit :
> > ---
> >  src/clock/clock.c | 15 +++++++++++++--
> >  src/clock/clock.h |  1 +
> >  2 files changed, 14 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/clock/clock.c b/src/clock/clock.c
> > index cfbb5eb6f6d..0bb2a9fe375 100644
> > --- a/src/clock/clock.c
> > +++ b/src/clock/clock.c
> > @@ -45,6 +45,7 @@ struct vlc_clock_main_t
> >      average_t coeff_avg; /* Moving average to smooth out the instant coeff
> > */ double rate;
> >      double coeff;
> > +    vlc_tick_t input_drift;
> >      vlc_tick_t offset;
> >      vlc_tick_t delay;
> > 
> > @@ -88,6 +89,7 @@ static vlc_tick_t main_stream_to_system(vlc_clock_main_t
> > *main_clock, static void vlc_clock_main_reset(vlc_clock_main_t *main_clock)
> >  {
> >      AvgReset(&main_clock->coeff_avg);
> > +    main_clock->input_drift = 0;
> >      main_clock->coeff = 1.0f;
> >      main_clock->rate = 1.0f;
> >      main_clock->offset = VLC_TICK_INVALID;
> > @@ -266,7 +268,8 @@ static vlc_tick_t
> > vlc_clock_slave_to_system_locked(vlc_clock_t *clock, system =
> > vlc_clock_monotonic_to_system_locked(clock, now, ts, rate); }
> > 
> > -    return system + (clock->delay - main_clock->delay) * rate;
> > +    return system + main_clock->input_drift +
> > +        (clock->delay - main_clock->delay) * rate;
> >  }
> > 
> >  static vlc_tick_t vlc_clock_master_to_system_locked(vlc_clock_t *clock,
> > @@ -282,7 +285,7 @@ static vlc_tick_t
> > vlc_clock_master_to_system_locked(vlc_clock_t *clock, system =
> > vlc_clock_monotonic_to_system_locked(clock, now, ts, rate); }
> > 
> > -    return system + clock->delay * rate;
> > +    return system + main_clock->input_drift + clock->delay * rate;
> >  }
> > 
> >  static vlc_tick_t vlc_clock_slave_update(vlc_clock_t *clock,
> > @@ -370,6 +373,7 @@ vlc_clock_main_t *vlc_clock_main_New(void)
> >      main_clock->master = NULL;
> >      main_clock->rc = 1;
> > 
> > +    main_clock->input_drift = 0;
> >      main_clock->coeff = 1.0f;
> >      main_clock->rate = 1.0f;
> >      main_clock->offset = VLC_TICK_INVALID;
> > @@ -439,6 +443,13 @@ void vlc_clock_main_SetDejitter(vlc_clock_main_t
> > *main_clock, vlc_mutex_unlock(&main_clock->lock);
> >  }
> > 
> > +void vlc_clock_main_SetInputDrift(vlc_clock_main_t *main_clock, double
> > drift) +{
> > +    vlc_mutex_lock(&main_clock->lock);
> > +    main_clock->input_drift = drift;
> > +    vlc_mutex_unlock(&main_clock->lock);
> > +}
> > +
> >  void vlc_clock_main_ChangePause(vlc_clock_main_t *main_clock, vlc_tick_t
> > now, bool paused)
> >  {
> > diff --git a/src/clock/clock.h b/src/clock/clock.h
> > index e7373391697..bec9edd40a9 100644
> > --- a/src/clock/clock.h
> > +++ b/src/clock/clock.h
> > @@ -78,6 +78,7 @@ void vlc_clock_main_SetFirstPcr(vlc_clock_main_t
> > *main_clock, vlc_tick_t system_now, vlc_tick_t ts); void
> > vlc_clock_main_SetInputDejitter(vlc_clock_main_t *main_clock, vlc_tick_t
> > delay);
> > +void vlc_clock_main_SetInputDrift(vlc_clock_main_t *main_clock, double
> > drift);
> > 
> >  /**
> >   * This function sets the dejitter delay to absorb the clock jitter
> 
> This adds a new factor to the clock computations with zero explanation what, 
> why and how...
> 
> -- 
> Rémi Denis-Courmont
> 
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list