On Wed, Mar 25, 2020 at 01:14:29PM +0800, Dennis Law wrote: > Hello Miroslav, > > I have a follow up question regarding this. I found that the > REF_IsLeapSecondClose() function uses the system time to check if a leap > second occurrence is close. > Under the circumstances whereby system time has not yet synchronized, > wouldn't it be more correct to use timestamp from time sources for this > check?
The function checks the system time and the NTP time it is tracking. If the client was started in those 5 seconds around leap and had a significant offset, it could accept a bad sample, which could cause it to take longer to synchronize. I'll extend the check to include the sample time. Thanks, -- 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.