From patchwork Fri Jun 2 02:26:46 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dave Marchevsky X-Patchwork-Id: 13264675 X-Patchwork-Delegate: bpf@iogearbox.net Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 98F5817FA for ; Fri, 2 Jun 2023 02:27:13 +0000 (UTC) Received: from mx0b-00082601.pphosted.com (mx0b-00082601.pphosted.com [67.231.153.30]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D01DA192 for ; Thu, 1 Jun 2023 19:27:11 -0700 (PDT) Received: from pps.filterd (m0148460.ppops.net [127.0.0.1]) by mx0a-00082601.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 351NtcJI019682 for ; Thu, 1 Jun 2023 19:27:11 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fb.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=facebook; bh=XWrSlb+AG1qaHY7bnOaCSLx8bqGpMVGegcEPyDdpsRg=; b=a7QifC8aYykTy9yNFQrpH9NI8M2NtC/fuIUVTK1RoaJVqy34/iyALIFtV4I/7hcfLWW4 gfFGPM3rrdz8KLaych1QT/8h1QnYg2zMGoQMBrKM7YoJYbOlJj2hQSNe4ThllteZNm8n wcyMPNn8NdFg1pG0Gt/CKbJZZ6w1ZqE4XeI= Received: from maileast.thefacebook.com ([163.114.130.16]) by mx0a-00082601.pphosted.com (PPS) with ESMTPS id 3qxt1s6cwm-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Thu, 01 Jun 2023 19:27:10 -0700 Received: from twshared35445.38.frc1.facebook.com (2620:10d:c0a8:1c::1b) by mail.thefacebook.com (2620:10d:c0a8:82::d) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Thu, 1 Jun 2023 19:27:10 -0700 Received: by devbig077.ldc1.facebook.com (Postfix, from userid 158236) id 6D3001EF7C8EA; Thu, 1 Jun 2023 19:26:56 -0700 (PDT) From: Dave Marchevsky To: CC: Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Martin KaFai Lau , Kernel Team , Dave Marchevsky Subject: [PATCH v2 bpf-next 8/9] [DONOTAPPLY] selftests/bpf: Disable newly-added refcounted_kptr_races test Date: Thu, 1 Jun 2023 19:26:46 -0700 Message-ID: <20230602022647.1571784-9-davemarchevsky@fb.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230602022647.1571784-1-davemarchevsky@fb.com> References: <20230602022647.1571784-1-davemarchevsky@fb.com> Precedence: bulk X-Mailing-List: bpf@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-FB-Internal: Safe X-Proofpoint-ORIG-GUID: b6t6AMa5NFZkjCmnb6dCcsfb3Fuaw8_m X-Proofpoint-GUID: b6t6AMa5NFZkjCmnb6dCcsfb3Fuaw8_m X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.573,FMLib:17.11.176.26 definitions=2023-06-01_08,2023-05-31_03,2023-05-22_02 X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, RCVD_IN_DNSWL_LOW,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net X-Patchwork-Delegate: bpf@iogearbox.net The previous patch added a new test exercising a race condition which was fixed earlier in the series. Similarly to other tests in this file, the new test should not run while bpf_refcount_acquire is disabled as it requires that kfunc. Signed-off-by: Dave Marchevsky --- .../bpf/prog_tests/refcounted_kptr.c | 100 ------------------ 1 file changed, 100 deletions(-) diff --git a/tools/testing/selftests/bpf/prog_tests/refcounted_kptr.c b/tools/testing/selftests/bpf/prog_tests/refcounted_kptr.c index e7fcc1dd8864..6a53f304f3e4 100644 --- a/tools/testing/selftests/bpf/prog_tests/refcounted_kptr.c +++ b/tools/testing/selftests/bpf/prog_tests/refcounted_kptr.c @@ -18,103 +18,3 @@ void test_refcounted_kptr_fail(void) { RUN_TESTS(refcounted_kptr_fail); } - -static void force_cpu(pthread_t thread, int cpunum) -{ - cpu_set_t cpuset; - int err; - - CPU_ZERO(&cpuset); - CPU_SET(cpunum, &cpuset); - err = pthread_setaffinity_np(thread, sizeof(cpuset), &cpuset); - if (!ASSERT_OK(err, "pthread_setaffinity_np")) - return; -} - -struct refcounted_kptr *skel; - -static void *run_unstash_acq_ref(void *unused) -{ - LIBBPF_OPTS(bpf_test_run_opts, opts, - .data_in = &pkt_v4, - .data_size_in = sizeof(pkt_v4), - .repeat = 1, - ); - long ret, unstash_acq_ref_fd; - force_cpu(pthread_self(), 1); - - unstash_acq_ref_fd = bpf_program__fd(skel->progs.unstash_add_and_acquire_refcount); - - ret = bpf_prog_test_run_opts(unstash_acq_ref_fd, &opts); - ASSERT_EQ(opts.retval, 0, "unstash_add_and_acquire_refcount retval"); - ASSERT_EQ(skel->bss->ref_check_3, 2, "ref_check_3"); - ASSERT_EQ(skel->bss->ref_check_4, 1, "ref_check_4"); - ASSERT_EQ(skel->bss->ref_check_5, 0, "ref_check_5"); - pthread_exit((void *)ret); -} - -void test_refcounted_kptr_races(void) -{ - LIBBPF_OPTS(bpf_test_run_opts, opts, - .data_in = &pkt_v4, - .data_size_in = sizeof(pkt_v4), - .repeat = 1, - ); - int ref_acq_lock_fd, ref_acq_unlock_fd, rem_node_lock_fd; - int add_stash_fd, remove_tree_fd; - pthread_t thread_id; - int ret; - - force_cpu(pthread_self(), 0); - skel = refcounted_kptr__open_and_load(); - if (!ASSERT_OK_PTR(skel, "refcounted_kptr__open_and_load")) - return; - - add_stash_fd = bpf_program__fd(skel->progs.add_refcounted_node_to_tree_and_stash); - remove_tree_fd = bpf_program__fd(skel->progs.remove_refcounted_node_from_tree); - ref_acq_lock_fd = bpf_program__fd(skel->progs.unsafe_ref_acq_lock); - ref_acq_unlock_fd = bpf_program__fd(skel->progs.unsafe_ref_acq_unlock); - rem_node_lock_fd = bpf_program__fd(skel->progs.unsafe_rem_node_lock); - - ret = bpf_prog_test_run_opts(rem_node_lock_fd, &opts); - if (!ASSERT_OK(ret, "rem_node_lock")) - return; - - ret = bpf_prog_test_run_opts(ref_acq_lock_fd, &opts); - if (!ASSERT_OK(ret, "ref_acq_lock")) - return; - - ret = bpf_prog_test_run_opts(add_stash_fd, &opts); - if (!ASSERT_OK(ret, "add_stash")) - return; - if (!ASSERT_OK(opts.retval, "add_stash retval")) - return; - - ret = pthread_create(&thread_id, NULL, &run_unstash_acq_ref, NULL); - if (!ASSERT_OK(ret, "pthread_create")) - goto cleanup; - - force_cpu(thread_id, 1); - - /* This program will execute before unstash_acq_ref's refcount_acquire, then - * unstash_acq_ref can proceed after unsafe_unlock - */ - ret = bpf_prog_test_run_opts(remove_tree_fd, &opts); - if (!ASSERT_OK(ret, "remove_tree")) - goto cleanup; - - ret = bpf_prog_test_run_opts(ref_acq_unlock_fd, &opts); - if (!ASSERT_OK(ret, "ref_acq_unlock")) - goto cleanup; - - ret = pthread_join(thread_id, NULL); - if (!ASSERT_OK(ret, "pthread_join")) - goto cleanup; - - refcounted_kptr__destroy(skel); - return; -cleanup: - bpf_prog_test_run_opts(ref_acq_unlock_fd, &opts); - refcounted_kptr__destroy(skel); - return; -}