Author: avg
Date: Thu Nov 17 15:25:26 2016
New Revision: 308766
URL: https://svnweb.freebsd.org/changeset/base/308766

Log:
  MFC r308040,308479: nap time between pats is forced to be at most half
  of the timeout
  
  Note that in this branch the default nap period is 1 second unlike
  the head where the period is 10 seconds.

Modified:
  stable/10/usr.sbin/watchdogd/watchdogd.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.sbin/watchdogd/watchdogd.c
==============================================================================
--- stable/10/usr.sbin/watchdogd/watchdogd.c    Thu Nov 17 15:22:55 2016        
(r308765)
+++ stable/10/usr.sbin/watchdogd/watchdogd.c    Thu Nov 17 15:25:26 2016        
(r308766)
@@ -80,7 +80,8 @@ static u_int timeout = WD_TO_128SEC;
 static u_int exit_timeout = WD_TO_NEVER;
 static u_int pretimeout = 0;
 static u_int timeout_sec;
-static u_int passive = 0;
+static u_int nap = 1;
+static int passive = 0;
 static int is_daemon = 0;
 static int is_dry_run = 0;  /* do not arm the watchdog, only
                               report on timing of the watch
@@ -88,7 +89,6 @@ static int is_dry_run = 0;  /* do not ar
 static int do_timedog = 0;
 static int do_syslog = 1;
 static int fd = -1;
-static int nap = 1;
 static int carp_thresh_seconds = -1;
 static char *test_cmd = NULL;
 
@@ -685,10 +685,15 @@ seconds_to_pow2ns(int seconds)
 static void
 parseargs(int argc, char *argv[])
 {
+       struct timespec ts;
        int longindex;
        int c;
        const char *lopt;
 
+       /* Get the default value of timeout_sec from the default timeout. */
+       pow2ns_to_ts(timeout, &ts);
+       timeout_sec = ts.tv_sec;
+
        /*
         * if we end with a 'd' aka 'watchdogd' then we are the daemon program,
         * otherwise run as a command line utility.
@@ -731,9 +736,9 @@ parseargs(int argc, char *argv[])
                case 't':
                        timeout_sec = atoi(optarg);
                        timeout = parse_timeout_to_pow2ns(c, NULL, optarg);
-                       if (debugging)
-                               printf("Timeout is 2^%d nanoseconds\n",
-                                   timeout);
+                       if (debugging)
+                               printf("Timeout is 2^%d nanoseconds\n",
+                                   timeout);
                        break;
                case 'T':
                        carp_thresh_seconds =
@@ -771,6 +776,9 @@ parseargs(int argc, char *argv[])
                }
        }
 
+       if (nap > timeout_sec / 2)
+               nap = timeout_sec / 2;
+
        if (carp_thresh_seconds == -1)
                carp_thresh_seconds = nap;
 
@@ -779,10 +787,7 @@ parseargs(int argc, char *argv[])
        if (is_daemon && timeout < WD_TO_1SEC)
                errx(EX_USAGE, "-t argument is less than one second.");
        if (pretimeout_set) {
-               struct timespec ts;
-
-               pow2ns_to_ts(timeout, &ts);
-               if (pretimeout >= (uintmax_t)ts.tv_sec) {
+               if (pretimeout >= timeout_sec) {
                        errx(EX_USAGE,
                            "pretimeout (%d) >= timeout (%d -> %ld)\n"
                            "see manual section TIMEOUT RESOLUTION",
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to