Re: [chrony-dev] [GIT] chrony/chrony.git branch master updated. 3.1-65-g0773a1e

[Miroslav Lichvar]

On Sat, Jul 01, 2017 at 11:54:07AM +1200, Bryan Christianson wrote:
> Not sure if this is related to the current commits or if its an earlier 
> issue. 
> I had to exclude macOS from MOD_TAI because txc does NOT have a field called 
> tai.

It is a bug. It seems only on Linux the timex struct has the tai
field. That explains why it needs to be set via the PLL constant
field.

The TAI offset is returned in the ntptimeval struct, which can be
obtained by the ntp_gettime() call. Unfortunately, on Linux that
doesn't work. There is ntp_gettimex() for that:

ntp_gettimex() performs the same task as ntp_gettime(), but
also returns information in the tai field.

I'll try to fix it.

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.

Re: [chrony-dev] [GIT] chrony/chrony.git branch master updated. 3.1-65-g0773a1e

[Bryan Christianson]

Not sure if this is related to the current commits or if its an earlier issue. 
I had to exclude macOS from MOD_TAI because txc does NOT have a field called 
tai.


diff --git a/sys_timex.c b/sys_timex.c
index 815779e..85a2d41 100644
--- a/sys_timex.c
+++ b/sys_timex.c
@@ -115,12 +115,14 @@ set_leap(int leap, int tai_offset)
   txc.status = status;
 
 #ifdef MOD_TAI
+#if !defined(MACOSX)
   if (tai_offset && tai_offset != txc.tai) {
 txc.modes |= MOD_TAI;
 txc.constant = tai_offset;
 LOG(LOGS_INFO, "System clock TAI offset set to %d seconds", tai_offset);
   }
 #endif
+#endif

> On 1/07/2017, at 3:19 AM, g...@tuxfamily.net wrote:
> 
> This is an automated email from git. It was generated because a ref
> change was pushed to the "chrony/chrony.git" repository.
> 
> The branch, master has been updated
>   via  0773a1e630a87c4a7d311fb78f3773b50bf75c1d (commit)
>   via  4a24368763cdeacc056a29dc27e0e506bd915133 (commit)
>   via  577290c5bc6e339a883153758a78671c92fdd7de (commit)
>   via  854ff69f782875a340c15213f1bb8c4fc700f717 (commit)
>   via  29b0ad894c3ccb567d3699bcb7988416f3eaa30f (commit)
>   via  cde0a2030788d168d0dc75ac629b08f4da1088eb (commit)
>   via  a768578a26561cec24a51272f30ef9d994cfdd8f (commit)
>   via  5d838729ef9f2fe0c940ed6911c91ce06dd6c417 (commit)
>   via  d6b763dc243d8f410d7cf32b29c808ab57e234a7 (commit)
>  from  95adb52a45a7686e3ad6c847ab966dc81dd33c65 (commit)
> 
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
> 
> - Log -
> commit 0773a1e630a87c4a7d311fb78f3773b50bf75c1d
> Author: Miroslav Lichvar 
> Date:   Fri Jun 30 15:39:10 2017 +0200
> 
>ntp: fix debug message about unknown HW timestamping ifindex
> 
> commit 4a24368763cdeacc056a29dc27e0e506bd915133
> Author: Miroslav Lichvar 
> Date:   Fri Jun 30 15:18:40 2017 +0200
> 
>ntp: reverse poll tracking in interleaved symmetric mode
> 
>Unlike in the basic mode, the peer with a higher stratum needs to wait
>for a response before sending the next request in order to minimize the
>delay of the measurement and error in the measured delay.
> 
>Slightly increase the delay adjustment to make it work with older chrony
>versions.
> 
> commit 577290c5bc6e339a883153758a78671c92fdd7de
> Author: Miroslav Lichvar 
> Date:   Fri Jun 30 11:32:19 2017 +0200
> 
>ntp: fix poll interleaving with unsynchronised peers
> 
>Update the remote poll and remote stratum even for unsychronised peers,
>and handle stratum of 0 as 16, so the peers work with the opposite
>differences between their strata and can adjust their polling intervals
>in order to interleave the packets.
> 
> commit 854ff69f782875a340c15213f1bb8c4fc700f717
> Author: Miroslav Lichvar 
> Date:   Fri Jun 30 09:40:06 2017 +0200
> 
>hwclock: decrease tolerance of robust regression to 0.1 ppb
> 
> commit 29b0ad894c3ccb567d3699bcb7988416f3eaa30f
> Author: Miroslav Lichvar 
> Date:   Thu Jun 29 17:56:16 2017 +0200
> 
>reference: get TAI-UTC offset from leap second timezone
> 
>Use the timezone specified by the leapsectz directive to get the
>current TAI-UTC offset and set the offset of the system clock in order
>to provide correct TAI time to applications using ntp_adjtime(),
>ntp_gettime(), or clock_gettime(CLOCK_TAI).
> 
> commit cde0a2030788d168d0dc75ac629b08f4da1088eb
> Author: Miroslav Lichvar 
> Date:   Thu Jun 29 15:16:20 2017 +0200
> 
>sys_timex: add support for setting TAI-UTC offset
> 
> commit a768578a26561cec24a51272f30ef9d994cfdd8f
> Author: Miroslav Lichvar 
> Date:   Thu Jun 29 15:14:16 2017 +0200
> 
>local: add support for setting TAI-UTC offset
> 
> commit 5d838729ef9f2fe0c940ed6911c91ce06dd6c417
> Author: Miroslav Lichvar 
> Date:   Thu Jun 29 12:39:42 2017 +0200
> 
>reference: move static tz variables to function using them
> 
> commit d6b763dc243d8f410d7cf32b29c808ab57e234a7
> Author: Miroslav Lichvar 
> Date:   Wed Jun 28 14:05:41 2017 +0200
> 
>client: check IP address family before printing as refid
> 
> ---
> 
> Summary of changes:
> client.c |  3 ++-
> doc/chrony.conf.adoc | 26 +-
> hwclock.c|  2 +-
> local.c  |  4 ++--
> local.h  |  9 +
> localp.h |  4 ++--
> ntp_core.c   | 23 ++-
> ntp_io_linux.c   |  2 +-
> reference.c  | 52 +---
> sys_timex.c  | 29 ++---
> 10 files changed, 99 insertions(+), 55