Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

timemaster: fix PTP poll configuration for timemaster >=4.2 #684

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

ycongal-smile
Copy link
Contributor

When using timemaster >= 4.2 (currently in Debian Sid), the chrony config looks like:

refclock SOCK /var/run/timemaster/chrony.SOCK0 poll 2 filter 4 refid PTP0 precision 1.0e-9 delay 1.0e-09 poll 0

The double poll definition triggers a weird behavior in chrony: PTP is poll'd every 4s but the Reach/LastRx computation expect a poll every second. This makes the PTP source looks unreliable (wrongly).

Fix this by using the proper config to setup the PTP poll period "ntp_poll" (sic)

New config look like:

refclock SOCK /var/run/timemaster/chrony.SOCK0 poll 0 filter 4 refid PTP0 precision 1.0e-9 delay 1.0e-09

When using timemaster >= 4.2 (currently in Debian Sid), the chrony
config looks like:
  refclock SOCK /var/run/timemaster/chrony.SOCK0 poll 2 filter 4 refid PTP0 precision 1.0e-9 delay 1.0e-09 poll 0

The double poll definition triggers a weird behavior in chrony: PTP is
poll'd every 4s but the Reach/LastRx computation expect a poll every
second. This makes the PTP source looks unreliable (wrongly).

Fix this by using the proper config to setup the PTP poll period "ntp_poll"
(sic)

New config look like:
  refclock SOCK /var/run/timemaster/chrony.SOCK0 poll 0 filter 4 refid PTP0 precision 1.0e-9 delay 1.0e-09

Signed-off-by: Yoann Congal <yoann.congal@smile.fr>
@insatomcat insatomcat force-pushed the yco/timemaster_debian_13 branch from 9ccf0a9 to 3b57e54 Compare January 10, 2025 15:04
@insatomcat insatomcat merged commit 214acf8 into seapath:main Jan 10, 2025
3 checks passed
@ycongal-smile ycongal-smile deleted the yco/timemaster_debian_13 branch January 10, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants