benchmarks/cyclictest: fix comp. warning and optargs termination

zdebanos · raiden00pl · commit 9968fa064bb3 · 2025-09-14T10:37:27.000+02:00
Fix reqtimeout_timer may be uninitilized.

Signed-off-by: Stepan Pressl &lt;pressl.stepan@gmail.com&gt;
diff --git a/benchmarks/cyclictest/cyclictest.c b/benchmarks/cyclictest/cyclictest.c
@@ -148,6 +148,7 @@ static const struct option optargs[] =
   {"threads", optional_argument, 0, 't'},
   {"timer-device", optional_argument, 0, 'T'},
   {"policy", optional_argument, 0, 'y'},
+  {0, 0, 0, 0}
 };
 
 /****************************************************************************
@@ -525,7 +526,7 @@ static void *testthread(void *arg)
 
   while (running)
     {
-      /* This inicializes the stamp1.timeout field */
+      /* This initializes the stamp1.timeout field */
 
       if (config.meas_method == M_TIMER_API)
         {
@@ -925,7 +926,7 @@ int main(int argc, char *argv[])
         {
           reqtimeout_timer = config.interval;
         }
-      else if (config.wait_method == W_NANOSLEEP)
+      else
         {
           /* Multiply by 3 instead of 2, just to be sure */
 

Original file line number	Diff line number	Diff line change
`@@ -148,6 +148,7 @@ static const struct option optargs[] =`
`148`	`148`	`{"threads", optional_argument, 0, 't'},`
`149`	`149`	`{"timer-device", optional_argument, 0, 'T'},`
`150`	`150`	`{"policy", optional_argument, 0, 'y'},`
	`151`	`+ {0, 0, 0, 0}`
`151`	`152`	`};`
`152`	`153`
`153`	`154`	`/****************************************************************************`
`@@ -525,7 +526,7 @@ static void testthread(void arg)`
`525`	`526`
`526`	`527`	`while (running)`
`527`	`528`	`{`
`528`		`- /* This inicializes the stamp1.timeout field */`
	`529`	`+ /* This initializes the stamp1.timeout field */`
`529`	`530`
`530`	`531`	`if (config.meas_method == M_TIMER_API)`
`531`	`532`	`{`
`@@ -925,7 +926,7 @@ int main(int argc, char *argv[])`
`925`	`926`	`{`
`926`	`927`	`reqtimeout_timer = config.interval;`
`927`	`928`	`}`
`928`		`- else if (config.wait_method == W_NANOSLEEP)`
	`929`	`+ else`
`929`	`930`	`{`
`930`	`931`	`/* Multiply by 3 instead of 2, just to be sure */`
`931`	`932`