From nobody Tue Jun 30 01:43:20 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AAC61C433F5 for ; Thu, 27 Jan 2022 22:11:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238341AbiA0WLX (ORCPT ); Thu, 27 Jan 2022 17:11:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45192 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229507AbiA0WLU (ORCPT ); Thu, 27 Jan 2022 17:11:20 -0500 Received: from mail-yb1-xb49.google.com (mail-yb1-xb49.google.com [IPv6:2607:f8b0:4864:20::b49]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A9433C061714 for ; Thu, 27 Jan 2022 14:11:20 -0800 (PST) Received: by mail-yb1-xb49.google.com with SMTP id 16-20020a250410000000b006191bfe8642so3868216ybe.19 for ; Thu, 27 Jan 2022 14:11:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=date:message-id:mime-version:subject:from:to:cc; bh=MHbMxovO6AcAN2vYhPcfy5isPlyEXuvf7h8ZW6A2Y+Y=; b=JCvw8MBgwkGImJIrpAHCQh0U5vi/BlQwmJPfV3rLDETbMfh6yfDO4M0JWJBqIFu7TJ TETjHmFWbv8fiQozxuzy1HUOPJsr3K5tWuj4R9UEgdiI50+VvKsvJ5MsKSagJExL9Fju 0PRFYzsN3Q0/gbZokFUXD2hxuaKKvIWacV5b2XthgTVM1FtGYJnfyl9t/0dsOoTGRkCQ dtbPluEjCNANUna5lmsz23rG9VD0w9KhYvGUfpCIbS/0bGlOrn8/4OC+aQzwJQK76QL9 YDS+dr4DylDWMaiYXN9mtpprN7MiRjc5TmMD/zb0AKoGoSgxbvTYzdtxuZMiYrj6er78 v4nA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:message-id:mime-version:subject:from:to:cc; bh=MHbMxovO6AcAN2vYhPcfy5isPlyEXuvf7h8ZW6A2Y+Y=; b=1SgLksyUV9m0LQvB+GqKx6bn3Dvy7sLS3ebGdRrWUUjgNy4XvKhdZa357EMBagPFXG V9MpMM1kaypozaDvCQFAF/lbAZ626D0nZWRtqZgrsv9anQPkjifG+DtmFd0AYt0c1miO qiSsJsb/Yv9osyLqz0Z3oM9J0JkFHsilnRLqsKflzU+HEFKa8gSSYNU8CUKIQf0Jwkf8 EVDHiNTHbdFXGO57/VxJOMwyvvLZ2jdooP94npdNVSUcw71I7OgpqpCnX667Q0yKjobm 9NxQG8oiJ5R6FKv89niDsQ1kAkgOgIAYncdws45PwcyXgcN44EcKfKTqtgB4Tl0WQHQ2 3URA== X-Gm-Message-State: AOAM530zJb9c7NrKCmF/Uh5Sf1nMFuVrY0xIg4dwURO4HEFQ/KCjy3yM wYWKPbw15Clf6G/53KhhjM6wuBK3MfqhGv1rn7Hl X-Google-Smtp-Source: ABdhPJwN3KVWegxyHHiIXTTmlstEucD6pQ1XLu2joNb+rGzqKl4AWrQa2pDPchrhP0iDqpgB5lXQxHdcXS1/oMS+vh8d X-Received: from ajr0.svl.corp.google.com ([2620:15c:2cd:203:c864:1e56:28bc:526b]) (user=axelrasmussen job=sendgmr) by 2002:a25:9988:: with SMTP id p8mr8106207ybo.128.1643321479925; Thu, 27 Jan 2022 14:11:19 -0800 (PST) Date: Thu, 27 Jan 2022 14:11:15 -0800 Message-Id: <20220127221115.3731388-1-axelrasmussen@google.com> Mime-Version: 1.0 X-Mailer: git-send-email 2.35.0.rc2.247.g8bbb082509-goog Subject: [PATCH] selftests: fixup build warnings in pidfd / clone3 tests From: Axel Rasmussen To: Christian Brauner , Shuah Khan , Nathan Chancellor , Nick Desaulniers Cc: "Zach O'Keefe" , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, llvm@lists.linux.dev, Axel Rasmussen Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" These are some trivial fixups, which were needed to build the tests with clang and -Werror. The following issues are fixed: - Remove various unused variables. - In child_poll_leader_exit_test, clang isn't smart enough to realize syscall(SYS_exit, 0) won't return, so it complains we never return from a non-void function. Add an extra exit(0) to appease it. - In test_pidfd_poll_leader_exit, ret may be branched on despite being uninitialized, if we have !use_waitpid. Initialize it to zero to get the right behavior in that case. Signed-off-by: Axel Rasmussen Acked-by: Christian Brauner --- tools/testing/selftests/clone3/clone3.c | 2 -- tools/testing/selftests/pidfd/pidfd_test.c | 6 +++--- tools/testing/selftests/pidfd/pidfd_wait.c | 5 ++--- 3 files changed, 5 insertions(+), 8 deletions(-) diff --git a/tools/testing/selftests/clone3/clone3.c b/tools/testing/selfte= sts/clone3/clone3.c index 076cf4325f78..cd4582129c7d 100644 --- a/tools/testing/selftests/clone3/clone3.c +++ b/tools/testing/selftests/clone3/clone3.c @@ -126,8 +126,6 @@ static void test_clone3(uint64_t flags, size_t size, in= t expected, =20 int main(int argc, char *argv[]) { - pid_t pid; - uid_t uid =3D getuid(); =20 ksft_print_header(); diff --git a/tools/testing/selftests/pidfd/pidfd_test.c b/tools/testing/sel= ftests/pidfd/pidfd_test.c index 529eb700ac26..9a2d64901d59 100644 --- a/tools/testing/selftests/pidfd/pidfd_test.c +++ b/tools/testing/selftests/pidfd/pidfd_test.c @@ -441,7 +441,6 @@ static void test_pidfd_poll_exec(int use_waitpid) { int pid, pidfd =3D 0; int status, ret; - pthread_t t1; time_t prog_start =3D time(NULL); const char *test_name =3D "pidfd_poll check for premature notification on= child thread exec"; =20 @@ -500,13 +499,14 @@ static int child_poll_leader_exit_test(void *args) */ *child_exit_secs =3D time(NULL); syscall(SYS_exit, 0); + /* Never reached, but appeases compiler thinking we should return. */ + exit(0); } =20 static void test_pidfd_poll_leader_exit(int use_waitpid) { int pid, pidfd =3D 0; - int status, ret; - time_t prog_start =3D time(NULL); + int status, ret =3D 0; const char *test_name =3D "pidfd_poll check for premature notification on= non-empty" "group leader exit"; =20 diff --git a/tools/testing/selftests/pidfd/pidfd_wait.c b/tools/testing/sel= ftests/pidfd/pidfd_wait.c index be2943f072f6..17999e082aa7 100644 --- a/tools/testing/selftests/pidfd/pidfd_wait.c +++ b/tools/testing/selftests/pidfd/pidfd_wait.c @@ -39,7 +39,7 @@ static int sys_waitid(int which, pid_t pid, siginfo_t *in= fo, int options, =20 TEST(wait_simple) { - int pidfd =3D -1, status =3D 0; + int pidfd =3D -1; pid_t parent_tid =3D -1; struct clone_args args =3D { .parent_tid =3D ptr_to_u64(&parent_tid), @@ -47,7 +47,6 @@ TEST(wait_simple) .flags =3D CLONE_PIDFD | CLONE_PARENT_SETTID, .exit_signal =3D SIGCHLD, }; - int ret; pid_t pid; siginfo_t info =3D { .si_signo =3D 0, @@ -88,7 +87,7 @@ TEST(wait_simple) =20 TEST(wait_states) { - int pidfd =3D -1, status =3D 0; + int pidfd =3D -1; pid_t parent_tid =3D -1; struct clone_args args =3D { .parent_tid =3D ptr_to_u64(&parent_tid), --=20 2.35.0.rc2.247.g8bbb082509-goog