On Wed, Nov 16, 2016 at 11:24:14PM -0800, Denny Page wrote:
> To be clear, would I still expect to see ‘D H’ in the measurements log with 
> this change?

Yes, but the second bit in the column with four test bits should be
always zero.

> Thanks,
> Denny
> 
> 
> > On Nov 16, 2016, at 00:53, Miroslav Lichvar <mlich...@redhat.com> wrote:
> > 
> > Hm, the fix helped with the spikes I was seeking. Did we rule out the
> > possibility that in your case the spikes are due to the other issue with
> > out-of-order HW timestamps? Could you try it with this patch to make
> > sure only measurements with HW timestamps are used?
> > 
> > --- a/ntp_core.c
> > +++ b/ntp_core.c
> > @@ -1434,6 +1434,9 @@ receive_packet(NCR_Instance inst, NTP_Local_Address 
> > *local_addr,
> >        prevent a synchronisation loop */
> >     testD = message->stratum <= 1 || REF_GetMode() != REF_ModeNormal ||
> >             pkt_refid != UTI_IPToRefid(&local_addr->ip_addr);
> > +
> > +    if (inst->local_tx.source != NTP_TS_HARDWARE || rx_ts->source != 
> > NTP_TS_HARDWARE)
> > +      testB = 0;
> >   } else {
> >     offset = delay = dispersion = 0.0;
> >     sample_time = rx_ts->ts;
> > 
> 

-- 
Miroslav Lichvar

-- 
To unsubscribe email chrony-dev-requ...@chrony.tuxfamily.org with "unsubscribe" 
in the subject.
For help email chrony-dev-requ...@chrony.tuxfamily.org with "help" in the 
subject.
Trouble?  Email listmas...@chrony.tuxfamily.org.

Reply via email to