diff options
| author | Vineet Gupta <vineet.gupta@linux.dev> | 2026-08-24 14:01:41 -0700 |
|---|---|---|
| committer | Daniel Borkmann <daniel@iogearbox.net> | 2026-08-26 18:29:36 +0200 |
| commit | 8928f5ffdb65f0f69babb006123d256bd774f2d3 (patch) | |
| tree | d1383a69a98b32c7496c81bb1429b5db785f0213 /scripts | |
| parent | b0eeb9e0f548ab4b555b588a8cbb09de7a690c47 (diff) | |
| download | linux-next-8928f5ffdb65f0f69babb006123d256bd774f2d3.tar.gz linux-next-8928f5ffdb65f0f69babb006123d256bd774f2d3.zip | |
selftests/bpf: Report placeholder tests as SKIP, not OK
Several verifier test files wrap their contents in a feature gate and provide
a one-line placeholder program in the #else arm, so the file still yields
something when the toolchain or JIT cannot support the tested instructions.
However the placeholder is annotated __success, so it reports as a pass,
whereas this is really a SKIP.
| #318/1 verifier_ldsx/cpuv4 is not supported by compiler or jit, ...:OK
| #318 verifier_ldsx:OK
This causes skipped coverage to be overlooked: see the next patch where
BPF_GCC builds miss a whole bunch of test due to clang only gate.
Add a __skip("reason") annotation for a compile-time gate, and use it for the
placeholders.
After this change, above tests reported as
| #318/1 verifier_ldsx/cpuv4 is not supported by compiler or jit, ...:SKIP
| #318 verifier_ldsx:SKIP
Signed-off-by: Vineet Gupta <vineet.gupta@linux.dev>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Eduard Zingerman <eddyz87@gmail.com>
Link: https://lore.kernel.org/bpf/20260824210142.3933580-3-vineet.gupta@linux.dev
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions
