| From b5f319360371087d52070d8f3fc7789e80ce69a6 Mon Sep 17 00:00:00 2001 |
| From: John Kacur <jkacur@redhat.com> |
| Date: Fri, 2 Feb 2024 19:16:07 -0500 |
| Subject: tools/rtla: Exit with EXIT_SUCCESS when help is invoked |
| |
| From: John Kacur <jkacur@redhat.com> |
| |
| commit b5f319360371087d52070d8f3fc7789e80ce69a6 upstream. |
| |
| Fix rtla so that the following commands exit with 0 when help is invoked |
| |
| rtla osnoise top -h |
| rtla osnoise hist -h |
| rtla timerlat top -h |
| rtla timerlat hist -h |
| |
| Link: https://lore.kernel.org/linux-trace-devel/20240203001607.69703-1-jkacur@redhat.com |
| |
| Cc: stable@vger.kernel.org |
| Fixes: 1eeb6328e8b3 ("rtla/timerlat: Add timerlat hist mode") |
| Signed-off-by: John Kacur <jkacur@redhat.com> |
| Signed-off-by: Daniel Bristot de Oliveira <bristot@kernel.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| tools/tracing/rtla/src/osnoise_hist.c | 6 +++++- |
| tools/tracing/rtla/src/osnoise_top.c | 6 +++++- |
| tools/tracing/rtla/src/timerlat_hist.c | 6 +++++- |
| tools/tracing/rtla/src/timerlat_top.c | 6 +++++- |
| 4 files changed, 20 insertions(+), 4 deletions(-) |
| |
| --- a/tools/tracing/rtla/src/osnoise_hist.c |
| +++ b/tools/tracing/rtla/src/osnoise_hist.c |
| @@ -480,7 +480,11 @@ static void osnoise_hist_usage(char *usa |
| |
| for (i = 0; msg[i]; i++) |
| fprintf(stderr, "%s\n", msg[i]); |
| - exit(1); |
| + |
| + if (usage) |
| + exit(EXIT_FAILURE); |
| + |
| + exit(EXIT_SUCCESS); |
| } |
| |
| /* |
| --- a/tools/tracing/rtla/src/osnoise_top.c |
| +++ b/tools/tracing/rtla/src/osnoise_top.c |
| @@ -331,7 +331,11 @@ static void osnoise_top_usage(struct osn |
| |
| for (i = 0; msg[i]; i++) |
| fprintf(stderr, "%s\n", msg[i]); |
| - exit(1); |
| + |
| + if (usage) |
| + exit(EXIT_FAILURE); |
| + |
| + exit(EXIT_SUCCESS); |
| } |
| |
| /* |
| --- a/tools/tracing/rtla/src/timerlat_hist.c |
| +++ b/tools/tracing/rtla/src/timerlat_hist.c |
| @@ -546,7 +546,11 @@ static void timerlat_hist_usage(char *us |
| |
| for (i = 0; msg[i]; i++) |
| fprintf(stderr, "%s\n", msg[i]); |
| - exit(1); |
| + |
| + if (usage) |
| + exit(EXIT_FAILURE); |
| + |
| + exit(EXIT_SUCCESS); |
| } |
| |
| /* |
| --- a/tools/tracing/rtla/src/timerlat_top.c |
| +++ b/tools/tracing/rtla/src/timerlat_top.c |
| @@ -375,7 +375,11 @@ static void timerlat_top_usage(char *usa |
| |
| for (i = 0; msg[i]; i++) |
| fprintf(stderr, "%s\n", msg[i]); |
| - exit(1); |
| + |
| + if (usage) |
| + exit(EXIT_FAILURE); |
| + |
| + exit(EXIT_SUCCESS); |
| } |
| |
| /* |