diff options
author | Daniel Müller <deso@posteo.net> | 2023-05-26 02:22:48 +0300 |
---|---|---|
committer | Andrii Nakryiko <andrii@kernel.org> | 2023-05-26 02:51:05 +0300 |
commit | 321a64b328156fd43d3be589c24905a641c7995b (patch) | |
tree | 3b3e65cbb5e5b98a9e1a9352847cd81111cf710f /tools | |
parent | 4c857a719bf9c5ddbcf3bd92398632041bff50d0 (diff) | |
download | linux-321a64b328156fd43d3be589c24905a641c7995b.tar.xz |
selftests/bpf: Check whether to run selftest
The sockopt test invokes test__start_subtest and then unconditionally
asserts the success. That means that even if deny-listed, any test will
still run and potentially fail.
Evaluate the return value of test__start_subtest() to achieve the
desired behavior, as other tests do.
Signed-off-by: Daniel Müller <deso@posteo.net>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20230525232248.640465-1-deso@posteo.net
Diffstat (limited to 'tools')
-rw-r--r-- | tools/testing/selftests/bpf/prog_tests/sockopt.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/testing/selftests/bpf/prog_tests/sockopt.c b/tools/testing/selftests/bpf/prog_tests/sockopt.c index 33dd4532e642..9e6a5e3ed4de 100644 --- a/tools/testing/selftests/bpf/prog_tests/sockopt.c +++ b/tools/testing/selftests/bpf/prog_tests/sockopt.c @@ -1060,7 +1060,9 @@ void test_sockopt(void) return; for (i = 0; i < ARRAY_SIZE(tests); i++) { - test__start_subtest(tests[i].descr); + if (!test__start_subtest(tests[i].descr)) + continue; + ASSERT_OK(run_test(cgroup_fd, &tests[i]), tests[i].descr); } |