]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
selftests/bpf: Migrate fexit_noreturns case into tracing_failure test suite
authorKaFai Wan <kafai.wan@linux.dev>
Thu, 24 Jul 2025 15:14:54 +0000 (23:14 +0800)
committerAlexei Starovoitov <ast@kernel.org>
Tue, 29 Jul 2025 02:39:30 +0000 (19:39 -0700)
Delete fexit_noreturns.c files and migrate the cases into
tracing_failure.c files.

The result:

 $ tools/testing/selftests/bpf/test_progs -t tracing_failure/fexit_noreturns
 #467/4   tracing_failure/fexit_noreturns:OK
 #467     tracing_failure:OK
 Summary: 1/1 PASSED, 0 SKIPPED, 0 FAILED

Signed-off-by: KaFai Wan <kafai.wan@linux.dev>
Acked-by: Yonghong Song <yonghong.song@linux.dev>
Link: https://lore.kernel.org/r/20250724151454.499040-5-kafai.wan@linux.dev
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/prog_tests/fexit_noreturns.c [deleted file]
tools/testing/selftests/bpf/prog_tests/tracing_failure.c
tools/testing/selftests/bpf/progs/fexit_noreturns.c [deleted file]
tools/testing/selftests/bpf/progs/tracing_failure.c

diff --git a/tools/testing/selftests/bpf/prog_tests/fexit_noreturns.c b/tools/testing/selftests/bpf/prog_tests/fexit_noreturns.c
deleted file mode 100644 (file)
index 568d3aa..0000000
+++ /dev/null
@@ -1,9 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-
-#include <test_progs.h>
-#include "fexit_noreturns.skel.h"
-
-void test_fexit_noreturns(void)
-{
-       RUN_TESTS(fexit_noreturns);
-}
index 39b59276884a117dad5157ef0666e9cbd2bf7bbc..10e231965589e9ce2dd59494bd0dd587a9784c7f 100644 (file)
@@ -28,37 +28,54 @@ out:
        tracing_failure__destroy(skel);
 }
 
-static void test_tracing_deny(void)
+static void test_tracing_fail_prog(const char *prog_name, const char *exp_msg)
 {
        struct tracing_failure *skel;
+       struct bpf_program *prog;
        char log_buf[256];
-       int btf_id, err;
-
-       /* __rcu_read_lock depends on CONFIG_PREEMPT_RCU */
-       btf_id = libbpf_find_vmlinux_btf_id("__rcu_read_lock", BPF_TRACE_FENTRY);
-       if (btf_id <= 0) {
-               test__skip();
-               return;
-       }
+       int err;
 
        skel = tracing_failure__open();
        if (!ASSERT_OK_PTR(skel, "tracing_failure__open"))
                return;
 
-       bpf_program__set_autoload(skel->progs.tracing_deny, true);
-       bpf_program__set_log_buf(skel->progs.tracing_deny, log_buf, sizeof(log_buf));
+       prog = bpf_object__find_program_by_name(skel->obj, prog_name);
+       if (!ASSERT_OK_PTR(prog, "bpf_object__find_program_by_name"))
+               goto out;
+
+       bpf_program__set_autoload(prog, true);
+       bpf_program__set_log_buf(prog, log_buf, sizeof(log_buf));
 
        err = tracing_failure__load(skel);
        if (!ASSERT_ERR(err, "tracing_failure__load"))
                goto out;
 
-       ASSERT_HAS_SUBSTR(log_buf,
-                         "Attaching tracing programs to function '__rcu_read_lock' is rejected.",
-                         "log_buf");
+       ASSERT_HAS_SUBSTR(log_buf, exp_msg, "log_buf");
 out:
        tracing_failure__destroy(skel);
 }
 
+static void test_tracing_deny(void)
+{
+       int btf_id;
+
+       /* __rcu_read_lock depends on CONFIG_PREEMPT_RCU */
+       btf_id = libbpf_find_vmlinux_btf_id("__rcu_read_lock", BPF_TRACE_FENTRY);
+       if (btf_id <= 0) {
+               test__skip();
+               return;
+       }
+
+       test_tracing_fail_prog("tracing_deny",
+                              "Attaching tracing programs to function '__rcu_read_lock' is rejected.");
+}
+
+static void test_fexit_noreturns(void)
+{
+       test_tracing_fail_prog("fexit_noreturns",
+                              "Attaching fexit/fmod_ret to __noreturn function 'do_exit' is rejected.");
+}
+
 void test_tracing_failure(void)
 {
        if (test__start_subtest("bpf_spin_lock"))
@@ -67,4 +84,6 @@ void test_tracing_failure(void)
                test_bpf_spin_lock(false);
        if (test__start_subtest("tracing_deny"))
                test_tracing_deny();
+       if (test__start_subtest("fexit_noreturns"))
+               test_fexit_noreturns();
 }
diff --git a/tools/testing/selftests/bpf/progs/fexit_noreturns.c b/tools/testing/selftests/bpf/progs/fexit_noreturns.c
deleted file mode 100644 (file)
index b1c33d9..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-
-#include <linux/bpf.h>
-#include <bpf/bpf_helpers.h>
-#include <bpf/bpf_tracing.h>
-#include "bpf_misc.h"
-
-char _license[] SEC("license") = "GPL";
-
-SEC("fexit/do_exit")
-__failure __msg("Attaching fexit/fmod_ret to __noreturn function 'do_exit' is rejected.")
-int BPF_PROG(noreturns)
-{
-       return 0;
-}
index 58d2777014e18e9354aed92db5dad7fdd07cb2b7..65e485c4468c622e37f9e1028ce32333391f05c9 100644 (file)
@@ -24,3 +24,9 @@ int BPF_PROG(tracing_deny)
 {
        return 0;
 }
+
+SEC("?fexit/do_exit")
+int BPF_PROG(fexit_noreturns)
+{
+       return 0;
+}