From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-17.4 required=3.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT, USER_IN_DEF_DKIM_WL autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E9D42C3A59E for ; Wed, 21 Aug 2019 23:44:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id BB9F820679 for ; Wed, 21 Aug 2019 23:44:43 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="LbxaqkLH" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730789AbfHUXon (ORCPT ); Wed, 21 Aug 2019 19:44:43 -0400 Received: from mail-yw1-f73.google.com ([209.85.161.73]:36859 "EHLO mail-yw1-f73.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729326AbfHUXol (ORCPT ); Wed, 21 Aug 2019 19:44:41 -0400 Received: by mail-yw1-f73.google.com with SMTP id q7so2739490ywg.3 for ; Wed, 21 Aug 2019 16:44:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:in-reply-to:message-id:mime-version:references:subject:from:to :cc; bh=OP734iGIdmeaq8Cdc0GPz42aZMNpU2Up9/1+UftyNgQ=; b=LbxaqkLHGbJugPDYwuKR9fu89dpPy522kBl/I0ch4YsedpyoZd7Pf2NeWXhCzQsjlv BRvbpEpwvl1Sxc6R03eUSa6Qz+6Y6auLl2YP756cj4M+Ab7x7c9Ita+uH1gqRbOR4Kwk nosP/EukzF8n+Vs+UPUXaziMyHoNof41tAZe3ALvqMmSxLVJU3zXxPErVq+cP/SW9l75 KEjEpYHqbjqUBkRD4I98HW8IkJNQWq3liO2bYIDApReezijz/MRDon5+f4gTrBpWTdmJ jegwaGP+hGGqTkkVAvSyHbwOagDwYPevlQCh/X6oaPbTYrFu1B/Sf76V27noGF00SCVw vwmA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=OP734iGIdmeaq8Cdc0GPz42aZMNpU2Up9/1+UftyNgQ=; b=p7j4TgrBH/YF+U5jk1H50Sc4dbxTjyOkxabN5Le5WuCnzS3vGrIcutBGwL5kzDLyGL niVLBXw48oi8t8Xnt8sHI6nijJVC5XUbDsek5G6TIDY2XbFyRwiXW3z+qe+ydB2uEco+ 4bwJAdYJOWQItJtU8WCqxkXTwiKb10wjllLWRJPMjrY58+R67PjTsAI0CErEcvr3Erg5 AHihByEMdACrUxRWBgFKkayyMLjCDO0dPjYPmgXlZo2gMjnCB3hgpfGrmZGHMt/0y5pT hyohR4fWGClVq0GOGKlqnPqaZQAcASVbBIKWqn7OqzTP3TDf6xEqo9j1R0UJm9LY6wb+ vL7g== X-Gm-Message-State: APjAAAUzP0W+LyUL7F/Iab6ReOtt3hpD5nfh62f/lm3jecS8QvglDfHn x9/vo1pL7lRUJn/Z3y1ucEAPJRnMN2UQCPjG9lPmdJr1ZhuUAvZOB9AgUdHt0ziUHDFH8usVEEO rgk0DkmGi/306n/7HnIhQZ6WwFUVg0Z5/qleVq4Mb+jwVynAaTIGlJw== X-Google-Smtp-Source: APXvYqyyDZNKv08ZF/AULz4FqWYHrnc6NTi7ed8q5KViDJ9G4WWhPA6GX1ztweyby2VhXWiumI5gaSA= X-Received: by 2002:a81:2915:: with SMTP id p21mr27806200ywp.152.1566431080547; Wed, 21 Aug 2019 16:44:40 -0700 (PDT) Date: Wed, 21 Aug 2019 16:44:27 -0700 In-Reply-To: <20190821234427.179886-1-sdf@google.com> Message-Id: <20190821234427.179886-5-sdf@google.com> Mime-Version: 1.0 References: <20190821234427.179886-1-sdf@google.com> X-Mailer: git-send-email 2.23.0.187.g17f5b7556c-goog Subject: [PATCH bpf-next v3 4/4] selftests/bpf: test_progs: remove unused ret From: Stanislav Fomichev To: netdev@vger.kernel.org, bpf@vger.kernel.org Cc: davem@davemloft.net, ast@kernel.org, daniel@iogearbox.net, Stanislav Fomichev Content-Type: text/plain; charset="UTF-8" Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org send_signal test returns static codes from the subtests which nobody looks at, let's rely on the CHECK macros instead. Signed-off-by: Stanislav Fomichev --- .../selftests/bpf/prog_tests/send_signal.c | 42 +++++++++---------- 1 file changed, 19 insertions(+), 23 deletions(-) diff --git a/tools/testing/selftests/bpf/prog_tests/send_signal.c b/tools/testing/selftests/bpf/prog_tests/send_signal.c index 40c2c5efdd3e..b607112c64e7 100644 --- a/tools/testing/selftests/bpf/prog_tests/send_signal.c +++ b/tools/testing/selftests/bpf/prog_tests/send_signal.c @@ -8,7 +8,7 @@ static void sigusr1_handler(int signum) sigusr1_received++; } -static int test_send_signal_common(struct perf_event_attr *attr, +static void test_send_signal_common(struct perf_event_attr *attr, int prog_type, const char *test_name) { @@ -23,13 +23,13 @@ static int test_send_signal_common(struct perf_event_attr *attr, if (CHECK(pipe(pipe_c2p), test_name, "pipe pipe_c2p error: %s\n", strerror(errno))) - goto no_fork_done; + return; if (CHECK(pipe(pipe_p2c), test_name, "pipe pipe_p2c error: %s\n", strerror(errno))) { close(pipe_c2p[0]); close(pipe_c2p[1]); - goto no_fork_done; + return; } pid = fork(); @@ -38,7 +38,7 @@ static int test_send_signal_common(struct perf_event_attr *attr, close(pipe_c2p[1]); close(pipe_p2c[0]); close(pipe_p2c[1]); - goto no_fork_done; + return; } if (pid == 0) { @@ -125,7 +125,7 @@ static int test_send_signal_common(struct perf_event_attr *attr, goto disable_pmu; } - err = CHECK(buf[0] != '2', test_name, "incorrect result\n"); + CHECK(buf[0] != '2', test_name, "incorrect result\n"); /* notify child safe to exit */ write(pipe_p2c[1], buf, 1); @@ -138,11 +138,9 @@ static int test_send_signal_common(struct perf_event_attr *attr, close(pipe_c2p[0]); close(pipe_p2c[1]); wait(NULL); -no_fork_done: - return err; } -static int test_send_signal_tracepoint(void) +static void test_send_signal_tracepoint(void) { const char *id_path = "/sys/kernel/debug/tracing/events/syscalls/sys_enter_nanosleep/id"; struct perf_event_attr attr = { @@ -159,21 +157,21 @@ static int test_send_signal_tracepoint(void) if (CHECK(efd < 0, "tracepoint", "open syscalls/sys_enter_nanosleep/id failure: %s\n", strerror(errno))) - return -1; + return; bytes = read(efd, buf, sizeof(buf)); close(efd); if (CHECK(bytes <= 0 || bytes >= sizeof(buf), "tracepoint", "read syscalls/sys_enter_nanosleep/id failure: %s\n", strerror(errno))) - return -1; + return; attr.config = strtol(buf, NULL, 0); - return test_send_signal_common(&attr, BPF_PROG_TYPE_TRACEPOINT, "tracepoint"); + test_send_signal_common(&attr, BPF_PROG_TYPE_TRACEPOINT, "tracepoint"); } -static int test_send_signal_perf(void) +static void test_send_signal_perf(void) { struct perf_event_attr attr = { .sample_period = 1, @@ -181,11 +179,11 @@ static int test_send_signal_perf(void) .config = PERF_COUNT_SW_CPU_CLOCK, }; - return test_send_signal_common(&attr, BPF_PROG_TYPE_PERF_EVENT, - "perf_sw_event"); + test_send_signal_common(&attr, BPF_PROG_TYPE_PERF_EVENT, + "perf_sw_event"); } -static int test_send_signal_nmi(void) +static void test_send_signal_nmi(void) { struct perf_event_attr attr = { .sample_freq = 50, @@ -205,25 +203,23 @@ static int test_send_signal_nmi(void) printf("%s:SKIP:no PERF_COUNT_HW_CPU_CYCLES\n", __func__); test__skip(); - return 0; + return; } /* Let the test fail with a more informative message */ } else { close(pmu_fd); } - return test_send_signal_common(&attr, BPF_PROG_TYPE_PERF_EVENT, - "perf_hw_event"); + test_send_signal_common(&attr, BPF_PROG_TYPE_PERF_EVENT, + "perf_hw_event"); } void test_send_signal(void) { - int ret = 0; - if (test__start_subtest("send_signal_tracepoint")) - ret |= test_send_signal_tracepoint(); + test_send_signal_tracepoint(); if (test__start_subtest("send_signal_perf")) - ret |= test_send_signal_perf(); + test_send_signal_perf(); if (test__start_subtest("send_signal_nmi")) - ret |= test_send_signal_nmi(); + test_send_signal_nmi(); } -- 2.23.0.187.g17f5b7556c-goog