mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2025-01-19 12:24:34 +08:00
rtla: Tools main loop cleanup
I probably started using "do {} while();", but changed all but osnoise_top to "while(){};" leaving the ; behind. Cleanup the main loop code, making all tools use "while() {}" Changcheng Deng reported this problem, as reported by coccicheck: Fix the following coccicheck review: ./tools/tracing/rtla/src/timerlat_hist.c: 800: 2-3: Unneeded semicolon ./tools/tracing/rtla/src/osnoise_hist.c: 776: 2-3: Unneeded semicolon ./tools/tracing/rtla/src/timerlat_top.c: 596: 2-3: Unneeded semicolon Link: https://lkml.kernel.org/r/3c1642110aa87c396f5da4a037dabc72dbb9c601.1646247211.git.bristot@kernel.org Cc: Daniel Bristot de Oliveira <bristot@kernel.org> Cc: Clark Williams <williams@redhat.com> Cc: Juri Lelli <juri.lelli@redhat.com> Cc: Jonathan Corbet <corbet@lwn.net> Reported-by: Changcheng Deng <deng.changcheng@zte.com.cn> Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Daniel Bristot de Oliveira <bristot@kernel.org> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
This commit is contained in:
parent
7d0dc9576d
commit
75016ca3ac
@ -850,7 +850,7 @@ int osnoise_hist_main(int argc, char *argv[])
|
||||
|
||||
if (trace_is_off(&tool->trace, &record->trace))
|
||||
break;
|
||||
};
|
||||
}
|
||||
|
||||
osnoise_read_trace_hist(tool);
|
||||
|
||||
|
@ -612,7 +612,7 @@ int osnoise_top_main(int argc, char **argv)
|
||||
tool->start_time = time(NULL);
|
||||
osnoise_top_set_signals(params);
|
||||
|
||||
do {
|
||||
while (!stop_tracing) {
|
||||
sleep(params->sleep_time);
|
||||
|
||||
retval = tracefs_iterate_raw_events(trace->tep,
|
||||
@ -632,7 +632,7 @@ int osnoise_top_main(int argc, char **argv)
|
||||
if (trace_is_off(&tool->trace, &record->trace))
|
||||
break;
|
||||
|
||||
} while (!stop_tracing);
|
||||
}
|
||||
|
||||
osnoise_print_stats(params, tool);
|
||||
|
||||
|
@ -885,7 +885,7 @@ int timerlat_hist_main(int argc, char *argv[])
|
||||
|
||||
if (trace_is_off(&tool->trace, &record->trace))
|
||||
break;
|
||||
};
|
||||
}
|
||||
|
||||
timerlat_print_stats(params, tool);
|
||||
|
||||
|
@ -680,7 +680,7 @@ int timerlat_top_main(int argc, char *argv[])
|
||||
if (trace_is_off(&top->trace, &record->trace))
|
||||
break;
|
||||
|
||||
};
|
||||
}
|
||||
|
||||
timerlat_print_stats(params, top);
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user