[Git][NTPsec/ntpsec][master] Allow minpoll to be set as low as 0. Useful on fast networks.

[Eric S. Raymond]

Eric S. Raymond pushed to branch master at NTPsec / ntpsec


Commits:
a3047c7a by Eric S. Raymond at 2016-07-23T18:23:33-04:00
Allow minpoll to be set as low as 0.  Useful on fast networks.

chrony allows minpoll 0 and doesn't blow up the net, so this is
probably safe.  A higher lower limit made sense when WAN capacity was
much lower and we didn't want aggressive configurations grabbing a
lot of it. But today the worst NTP can do is spit in the ocean
comparesd to, say video streaming traffic.

Note that this does not change the default minpoll value, just the
lowest you can explicitly set.  The documentation lied about that,
claiming it was 4 when it had been 3 since an unexplained change in
2008.

- - - - -


3 changed files:

- docs/includes/assoc-options.txt
- docs/includes/clock-options.txt
- include/ntp.h


Changes:

=====================================
docs/includes/assoc-options.txt
=====================================
--- a/docs/includes/assoc-options.txt
+++ b/docs/includes/assoc-options.txt
@@ -33,7 +33,7 @@
   defaults to 10 (1,024 s), but can be increased by the _maxpoll_ option
   to an upper limit of 17 (36.4 h). The minimum poll interval defaults
   to 6 (64 s), but can be decreased by the _minpoll_ option to a lower
-  limit of 4 (16 s).
+  limit of 0 (1 s).
 
 +mode+ 'option'::
   Pass the +option+ to a reference clock driver, where +option+ is an


=====================================
docs/includes/clock-options.txt
=====================================
--- a/docs/includes/clock-options.txt
+++ b/docs/includes/clock-options.txt
@@ -30,7 +30,7 @@
     directly connected reference clocks, both _minpoll_ and _maxpoll_
     default to 6 (64 sec). For modem reference clocks, _minpoll_ defaults
     to 10 (17.1 min) and _maxpoll_ defaults to 14 (4.5 hours). The allowable
-    range is 4 (16 sec) to 17 (36.4 hours) inclusive.
+    range is 0 (1 sec) to 17 (36.4 hours) inclusive.
   +time1+ _sec_;;
     Specifies a constant to be added to the time offset produced by the
     driver, a fixed-point decimal number in seconds. This is used as a


=====================================
include/ntp.h
=====================================
--- a/include/ntp.h
+++ b/include/ntp.h
@@ -91,7 +91,7 @@ typedef unsigned int	u_int;
  * Poll interval parameters
  */
 #define NTP_UNREACH	10	/* poll unreach threshold */
-#define	NTP_MINPOLL	3	/* log2 min poll interval (8 s) */
+#define	NTP_MINPOLL	0	/* log2 min poll interval (1 s) */
 #define NTP_MINDPOLL	6	/* log2 default min poll (64 s) */
 #define NTP_MAXDPOLL	10	/* log2 default max poll (~17 m) */
 #define	NTP_MAXPOLL	17	/* log2 max poll interval (~36 h) */



View it on GitLab: https://gitlab.com/NTPsec/ntpsec/commit/a3047c7a375877436d422e04a138aace7ce1bd06
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ntpsec.org/pipermail/vc/attachments/20160723/79932d0f/attachment.html>