[B][SRU][PATCH 1/2] selftests/bpf: Count tests skipped by unpriv
Po-Hsu Lin
po-hsu.lin at canonical.com
Tue Jul 5 08:28:16 UTC 2022
From: Joe Stringer <joe at wand.net.nz>
BugLink: https://bugs.launchpad.net/bugs/1980648
When priviliged tests are skipped due to user rights, count the number of
skipped tests so it's more obvious that the test did not check everything.
Signed-off-by: Joe Stringer <joe at wand.net.nz>
Signed-off-by: Daniel Borkmann <daniel at iogearbox.net>
(cherry picked from commit d0a0e4956f6c20754ef67db6dfb9746e85ecdcb5)
Signed-off-by: Po-Hsu Lin <po-hsu.lin at canonical.com>
---
tools/testing/selftests/bpf/test_verifier.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/bpf/test_verifier.c b/tools/testing/selftests/bpf/test_verifier.c
index 9ce1bad..c85a86c2 100644
--- a/tools/testing/selftests/bpf/test_verifier.c
+++ b/tools/testing/selftests/bpf/test_verifier.c
@@ -10198,7 +10198,7 @@ static int set_admin(bool admin)
static int do_test(bool unpriv, unsigned int from, unsigned int to)
{
- int i, passes = 0, errors = 0;
+ int i, passes = 0, errors = 0, skips = 0;
for (i = from; i < to; i++) {
struct bpf_test *test = &tests[i];
@@ -10215,13 +10215,17 @@ static int do_test(bool unpriv, unsigned int from, unsigned int to)
set_admin(true);
}
- if (!unpriv) {
+ if (unpriv) {
+ printf("#%d/p %s SKIP\n", i, test->descr);
+ skips++;
+ } else {
printf("#%d/p %s ", i, test->descr);
do_test_single(test, false, &passes, &errors);
}
}
- printf("Summary: %d PASSED, %d FAILED\n", passes, errors);
+ printf("Summary: %d PASSED, %d SKIPPED, %d FAILED\n", passes,
+ skips, errors);
return errors ? EXIT_FAILURE : EXIT_SUCCESS;
}
--
2.7.4
More information about the kernel-team
mailing list