From: Punit Agrawal <punit1.agrawal@xxxxxxxxxxxxx>
The timespec structure "stop" is initialised whether it is used or not
as the compiler is not smart enough to figure out that it's use is
always guarded by the "duration" variable. As a result, "stop" needs
to be initialised whether it's used or not to avoid a compiler
warning.
Replace the duplicate memset statements by initializing "stop" using
structure initialiser.
Signed-off-by: Punit Agrawal <punit1.agrawal@xxxxxxxxxxxxx>
Signed-off-by: John Kacur <jkacur@xxxxxxxxxx>
---
src/cyclictest/cyclictest.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
index 5758f88a24d0..73c5be972391 100644
--- a/src/cyclictest/cyclictest.c
+++ b/src/cyclictest/cyclictest.c
@@ -507,7 +507,7 @@ static void *timerthread(void *param)
struct sigevent sigev;
sigset_t sigset;
timer_t timer;
- struct timespec now, next, interval, stop;
+ struct timespec now, next, interval, stop = { 0 };
struct itimerval itimer;
struct itimerspec tspec;
struct thread_stat *stat = par->stats;
@@ -516,8 +516,6 @@ static void *timerthread(void *param)
pthread_t thread;
unsigned long smi_now, smi_old = 0;
- memset(&stop, 0, sizeof(stop));
-
/* if we're running in numa mode, set our memory node */
if (par->node != -1)
rt_numa_set_numa_run_on_node(par->node, par->cpu);
@@ -598,7 +596,6 @@ static void *timerthread(void *param)
tsnorm(&next);
if (duration) {
- memset(&stop, 0, sizeof(stop)); /* grrr */
stop = now;
stop.tv_sec += duration;
}
--
2.31.1
