From patchwork Mon Nov 7 14:43:13 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Kacur X-Patchwork-Id: 16486 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2097203wru; Mon, 7 Nov 2022 06:53:31 -0800 (PST) X-Google-Smtp-Source: AMsMyM7BYnUUCPYtU6dTugJsKqx205qjTsv1kGuyx3yewllXq8CG/mRipzZIf0220QgVQp6C37ip X-Received: by 2002:a05:6402:538a:b0:457:b602:d5a6 with SMTP id ew10-20020a056402538a00b00457b602d5a6mr50901719edb.371.1667832811107; Mon, 07 Nov 2022 06:53:31 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667832811; cv=none; d=google.com; s=arc-20160816; b=HSlGFXlar4HBH6N5tZzMUY4uVY6gv1LBZzbCjYtJ0C9avNTr95JoYMF/nMoA2bGRpI L3nGN1mnmEVQrGNq9eqtYeX7xjswuFLb1BT25CB/AN39SA0f8YGZmQsYlr7cMUYUMa66 crMEuvEOQz9D1JW5rwqZ/ixgJxYbf62B75/54IWvk6qR+QVNxoKcoggCEy42ShwyIZe8 l/XplkH3Irf3EPXz7dQkQeuG5Mue2kD45+7MkJeuMvxuzJwox88kOKy9W4vRLdHpJnNt yLSDGz9eEhtoNXK1/AMBL2lVAoaC6ptaE6OmrcTt0n2RCGREB19DdVHD1YJVZBCvgGiK IwgA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:sender:dkim-signature; bh=gc232Q/sHxhB4Tw9GHxF3SEryN15BBvVfGsyv7GLejI=; b=DMrdo3PbATNihBQUeRLUT/fBu3xBfokbnY0HFN/nQgy8kbAh0Afo8Qy4Dk+8bpjdq9 mUmJ6OCNKE7TdYHPy6m0Te27POBs44kqcRfhqxaP21EMvEf06bV+Lm/ivWJQJlwSFwCu cOByadJ7ETsZM8aqm2sBQGcyP45K83vFeKfx81vW/nuTibFXrHVnAuBTOGPVmEDMs/Ou vBUgS5gYak+devGrTFhrcjGMJfonvCvjqxfr84ylXtK0YkKbwrkaLKzeTRixvrLkormx XLDCNDcL60YHsytM3JDuQjRLYLc3qkSIqo+B4Vpc7ae1YsywWgtdBrsjB3tvuOIPTZVn JCBg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=FzdNDEU8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id s5-20020a508dc5000000b00461f0d84ee1si8244182edh.228.2022.11.07.06.53.07; Mon, 07 Nov 2022 06:53:31 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=FzdNDEU8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232255AbiKGOns (ORCPT + 99 others); Mon, 7 Nov 2022 09:43:48 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57836 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231573AbiKGOno (ORCPT ); Mon, 7 Nov 2022 09:43:44 -0500 Received: from mail-qv1-xf34.google.com (mail-qv1-xf34.google.com [IPv6:2607:f8b0:4864:20::f34]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 991FB63AD; Mon, 7 Nov 2022 06:43:40 -0800 (PST) Received: by mail-qv1-xf34.google.com with SMTP id e15so8208523qvo.4; Mon, 07 Nov 2022 06:43:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:sender:from:to:cc:subject:date:message-id:reply-to; bh=gc232Q/sHxhB4Tw9GHxF3SEryN15BBvVfGsyv7GLejI=; b=FzdNDEU80Vfm+/Mn6taCLIfKXq0If84ZDF23eu2iKZG4DEt/BupujN2r0QeOFU9gzO Tb/9/nPia2P5z7dKXPBEA/Y5T9lqIzmZ1PpPN7Kzu6RWHyhfM594Iwh0EpXFmBI/lzZs jBEqWFdLHWA4UVjFnFYQln2xlvFYjyBoOr6WPxj/2bqNnVxGy2znTN77dlx5az9zJsFn 4YfyjTBDufIDyXIx2jf8+F43Nz9NGDe2YGzBmN4T7OPtPWwo1flz2I4FR8b1/1Xkt9Fe ZTq/R0ArvJON8rbcxtFOa59dNhU+VTznDQJws5FiUrKzOYAPoLn3cbwTX4MievZxJeyJ NdiA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:sender:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=gc232Q/sHxhB4Tw9GHxF3SEryN15BBvVfGsyv7GLejI=; b=Y2oOh/cZEyYAxSMryCG9LnHpSr3IFzx8GT2ZpS8K0PW4p4jNui1evWOZZoYbOzkqQG RXCCVxTTesx0nFvNre+wXO1Koj7LsHfbrec2OFGktmefz9gHyiX4KICPlgdYxXtNmmRB TUjZv67iWCNiIaUVJq283jL8GA4YOs9ILK5Dqh4jP4EB3SWYo+EbTgK/Gv53sN0htHCx dDfqomiOJZe6OSfSvJJ7EmT0kPT8qZgzlefxC7mJDHfsj2yqDfmGXOv7FsBY3Si9/yaF 9Vg3FnUKKeMMWmnD7xVqawPSZg5v3c9IkQR+41oHCxYHt6dwE8/fgxsZoysHpUpd7kF0 EpwQ== X-Gm-Message-State: ACrzQf0PkFg1LwdIpsEegbcbWWdpQkKjlZrY5ldri1pRdHUHUuIkePUh S5CAnAqzQAroDQz/EGJWDbcivVRvlPc= X-Received: by 2002:a05:6214:2346:b0:4c1:8906:8726 with SMTP id hu6-20020a056214234600b004c189068726mr16460233qvb.106.1667832219753; Mon, 07 Nov 2022 06:43:39 -0800 (PST) Received: from fionn.redhat.com (bras-base-rdwyon0600w-grc-08-184-147-142-10.dsl.bell.ca. [184.147.142.10]) by smtp.gmail.com with ESMTPSA id k3-20020a05620a414300b006f9f714cb6asm7055862qko.50.2022.11.07.06.43.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Nov 2022 06:43:38 -0800 (PST) Sender: John Kacur From: John Kacur To: Steven Rostedt , Daniel Bristot de Oliveria Cc: linux-trace-devel@vger.kernel.org, lkml , Clark Williams , Chang Yin , John Kacur Subject: [PATCH V2] rtla: Fix exit status when returning from calls to usage() Date: Mon, 7 Nov 2022 09:43:13 -0500 Message-Id: <20221107144313.22470-1-jkacur@redhat.com> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 X-Spam-Status: No, score=-1.5 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748849457725038748?= X-GMAIL-MSGID: =?utf-8?q?1748849457725038748?= rtla_usage(), osnoise_usage() and timerlat_usage() all exit with an error status. However when these are called from help, they should exit with a non-error status. Fix this by passing the exit status to the functions. Note, although we remove the subsequent call to exit after calling usage, we leave it in at the end of a function to suppress the compiler warning "control reaches end of a non-void function". Signed-off-by: John Kacur Acked-by: Daniel Bristot de Oliveira --- - Regenerated Patch against 6.1.0-rc4 - Added periods at the end of my sentences in the description. - Fixed the spelling of "suppress". --- tools/tracing/rtla/SAC/osnoise.c | 9 ++++----- tools/tracing/rtla/src/rtla.c | 12 +++++------- tools/tracing/rtla/src/timerlat.c | 9 ++++----- 3 files changed, 13 insertions(+), 17 deletions(-) diff --git a/tools/tracing/rtla/src/osnoise.c b/tools/tracing/rtla/src/osnoise.c index b8ec6c15bccb..4dee343909b1 100644 --- a/tools/tracing/rtla/src/osnoise.c +++ b/tools/tracing/rtla/src/osnoise.c @@ -903,7 +903,7 @@ struct osnoise_tool *osnoise_init_trace_tool(char *tracer) return NULL; } -static void osnoise_usage(void) +static void osnoise_usage(int err) { int i; @@ -923,7 +923,7 @@ static void osnoise_usage(void) for (i = 0; msg[i]; i++) fprintf(stderr, "%s\n", msg[i]); - exit(1); + exit(err); } int osnoise_main(int argc, char *argv[]) @@ -941,8 +941,7 @@ int osnoise_main(int argc, char *argv[]) } if ((strcmp(argv[1], "-h") == 0) || (strcmp(argv[1], "--help") == 0)) { - osnoise_usage(); - exit(0); + osnoise_usage(0); } else if (strncmp(argv[1], "-", 1) == 0) { /* the user skipped the tool, call the default one */ osnoise_top_main(argc, argv); @@ -956,6 +955,6 @@ int osnoise_main(int argc, char *argv[]) } usage: - osnoise_usage(); + osnoise_usage(1); exit(1); } diff --git a/tools/tracing/rtla/src/rtla.c b/tools/tracing/rtla/src/rtla.c index 09bd21b8af81..52e8f1825281 100644 --- a/tools/tracing/rtla/src/rtla.c +++ b/tools/tracing/rtla/src/rtla.c @@ -14,7 +14,7 @@ /* * rtla_usage - print rtla usage */ -static void rtla_usage(void) +static void rtla_usage(int err) { int i; @@ -33,7 +33,7 @@ static void rtla_usage(void) for (i = 0; msg[i]; i++) fprintf(stderr, "%s\n", msg[i]); - exit(1); + exit(err); } /* @@ -70,11 +70,9 @@ int main(int argc, char *argv[]) goto usage; if (strcmp(argv[1], "-h") == 0) { - rtla_usage(); - exit(0); + rtla_usage(0); } else if (strcmp(argv[1], "--help") == 0) { - rtla_usage(); - exit(0); + rtla_usage(0); } retval = run_command(argc, argv, 1); @@ -82,6 +80,6 @@ int main(int argc, char *argv[]) exit(0); usage: - rtla_usage(); + rtla_usage(1); exit(1); } diff --git a/tools/tracing/rtla/src/timerlat.c b/tools/tracing/rtla/src/timerlat.c index 97abbf494fee..21cdcc5c4a29 100644 --- a/tools/tracing/rtla/src/timerlat.c +++ b/tools/tracing/rtla/src/timerlat.c @@ -14,7 +14,7 @@ #include "timerlat.h" -static void timerlat_usage(void) +static void timerlat_usage(int err) { int i; @@ -34,7 +34,7 @@ static void timerlat_usage(void) for (i = 0; msg[i]; i++) fprintf(stderr, "%s\n", msg[i]); - exit(1); + exit(err); } int timerlat_main(int argc, char *argv[]) @@ -52,8 +52,7 @@ int timerlat_main(int argc, char *argv[]) } if ((strcmp(argv[1], "-h") == 0) || (strcmp(argv[1], "--help") == 0)) { - timerlat_usage(); - exit(0); + timerlat_usage(0); } else if (strncmp(argv[1], "-", 1) == 0) { /* the user skipped the tool, call the default one */ timerlat_top_main(argc, argv); @@ -67,6 +66,6 @@ int timerlat_main(int argc, char *argv[]) } usage: - timerlat_usage(); + timerlat_usage(1); exit(1); }