selftests: watchdog: avoid keepalive flood

Calling `watchdog-test [options] -p 0` results in flooding the kernel
with WDIOC_KEEPALIVE. Fix this by enforcing 1 second as minimal/default
keepalive/ping rate.

Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>

1 files changed, 6 insertions, 2 deletions

diff --git a/tools/testing/selftests/watchdog/watchdog-test.c b/tools/testing/selftests/watchdog/watchdog-test.c
index 41f40c3c4d95..9b34b319fc91 100644
--- a/tools/testing/selftests/watchdog/watchdog-test.c
+++ b/tools/testing/selftests/watchdog/watchdog-test.c
@@ -14,6 +14,8 @@
 #include <linux/types.h>
 #include <linux/watchdog.h>
 
+#define DEFAULT_PING_RATE       1
+
 int fd;
 const char v = 'V';
 static const char sopts[] = "dehp:t:";
@@ -64,7 +66,7 @@ static void usage(char *progname)
         printf(" -d, --disable       Turn off the watchdog timer\n");
         printf(" -e, --enable        Turn on the watchdog timer\n");
         printf(" -h, --help          Print the help message\n");
-        printf(" -p, --pingrate=P    Set ping rate to P seconds\n");
+        printf(" -p, --pingrate=P    Set ping rate to P seconds (default %d)\n", DEFAULT_PING_RATE);
         printf(" -t, --timeout=T     Set timeout to T seconds\n");
         printf("\n");
         printf("Parameters are parsed left-to-right in real-time.\n");
@@ -74,7 +76,7 @@ static void usage(char *progname)
 int main(int argc, char *argv[])
 {
         int flags;
-        unsigned int ping_rate = 1;
+        unsigned int ping_rate = DEFAULT_PING_RATE;
         int ret;
         int c;
 
@@ -107,6 +109,8 @@ int main(int argc, char *argv[])
                         break;
                 case 'p':
                         ping_rate = strtoul(optarg, NULL, 0);
+                        if (!ping_rate)
+                                ping_rate = DEFAULT_PING_RATE;
                         printf("Watchdog ping rate set to %u seconds.\n", ping_rate);
                         break;
                 case 't':