tools/rtla: Replace timerlat_top_usage("...") with fatal("...")

A long time ago, when the usage help was short, it was a favor
to the user to show it on error. Now that the usage help has
become very long, it is too noisy to dump the complete help text
for each typo after the error message itself.

Replace timerlat_top_usage("...\n") with fatal("...") on errors.

Remove the already unused 'usage' argument from timerlat_top_usage().

Signed-off-by: Costa Shulyupin <costa.shul@redhat.com>
Reviewed-by: Tomas Glozar <tglozar@redhat.com>
Link: https://lore.kernel.org/r/20251011082738.173670-3-costa.shul@redhat.com
Signed-off-by: Tomas Glozar <tglozar@redhat.com>

diff --git a/tools/tracing/rtla/src/timerlat_top.c b/tools/tracing/rtla/src/timerlat_top.c
index 9664b8af727e6d828b73515d7e3f732c6754e3d0..67a5b6ab78a6f220a0f692e882ea76f113661467 100644 (file)
--- a/tools/tracing/rtla/src/timerlat_top.c
+++ b/tools/tracing/rtla/src/timerlat_top.c
@@ -474,7 +474,7 @@ timerlat_print_stats(struct osnoise_tool *top)
 /*
  * timerlat_top_usage - prints timerlat top usage message
  */
-static void timerlat_top_usage(char *usage)
+static void timerlat_top_usage(void)
 {
        int i;
 
@@ -522,18 +522,12 @@ static void timerlat_top_usage(char *usage)
                NULL,
        };
 
-       if (usage)
-               fprintf(stderr, "%s\n", usage);
-
        fprintf(stderr, "rtla timerlat top: a per-cpu summary of the timer latency (version %s)\n",
                        VERSION);
 
        for (i = 0; msg[i]; i++)
                fprintf(stderr, "%s\n", msg[i]);
 
-       if (usage)
-               exit(EXIT_FAILURE);
-
        exit(EXIT_SUCCESS);
 }
 
@@ -643,7 +637,7 @@ static struct common_params
                case 'c':
                        retval = parse_cpu_set(optarg, &params->common.monitored_cpus);
                        if (retval)
-                               timerlat_top_usage("\nInvalid -c cpu list\n");
+                               fatal("Invalid -c cpu list");
                        params->common.cpus = optarg;
                        break;
                case 'C':
@@ -662,7 +656,7 @@ static struct common_params
                case 'd':
                        params->common.duration = parse_seconds_duration(optarg);
                        if (!params->common.duration)
-                               timerlat_top_usage("Invalid -d duration\n");
+                               fatal("Invalid -d duration");
                        break;
                case 'e':
                        tevent = trace_event_alloc(optarg);
@@ -675,7 +669,7 @@ static struct common_params
                        break;
                case 'h':
                case '?':
-                       timerlat_top_usage(NULL);
+                       timerlat_top_usage();
                        break;
                case 'H':
                        params->common.hk_cpus = 1;
@@ -695,12 +689,12 @@ static struct common_params
                case 'p':
                        params->timerlat_period_us = get_llong_from_str(optarg);
                        if (params->timerlat_period_us > 1000000)
-                               timerlat_top_usage("Period longer than 1 s\n");
+                               fatal("Period longer than 1 s");
                        break;
                case 'P':
                        retval = parse_prio(optarg, &params->common.sched_param);
                        if (retval == -1)
-                               timerlat_top_usage("Invalid -P priority");
+                               fatal("Invalid -P priority");
                        params->common.set_sched = 1;
                        break;
                case 'q':
@@ -735,7 +729,7 @@ static struct common_params
                                if (retval)
                                        fatal("Error adding trigger %s", optarg);
                        } else {
-                               timerlat_top_usage("--trigger requires a previous -e\n");
+                               fatal("--trigger requires a previous -e");
                        }
                        break;
                case '1': /* filter */
@@ -744,7 +738,7 @@ static struct common_params
                                if (retval)
                                        fatal("Error adding filter %s", optarg);
                        } else {
-                               timerlat_top_usage("--filter requires a previous -e\n");
+                               fatal("--filter requires a previous -e");
                        }
                        break;
                case '2': /* dma-latency */
@@ -780,7 +774,7 @@ static struct common_params
                                fatal("Invalid action %s", optarg);
                        break;
                default:
-                       timerlat_top_usage("Invalid option");
+                       fatal("Invalid option");
                }
        }
 
@@ -797,10 +791,10 @@ static struct common_params
                params->no_aa = 1;
 
        if (params->no_aa && params->common.aa_only)
-               timerlat_top_usage("--no-aa and --aa-only are mutually exclusive!");
+               fatal("--no-aa and --aa-only are mutually exclusive!");
 
        if (params->common.kernel_workload && params->common.user_workload)
-               timerlat_top_usage("--kernel-threads and --user-threads are mutually exclusive!");
+               fatal("--kernel-threads and --user-threads are mutually exclusive!");
 
        /*
         * If auto-analysis or trace output is enabled, switch from BPF mode to