diff options
| author |   Lorenz Bauer <lmb@cloudflare.com> | 2021-03-03 10:18:16 +0000 |
|---|---|---|
| committer |   Alexei Starovoitov <ast@kernel.org> | 2021-03-04 19:11:30 -0800 |
| commit | b4f894633fa14d7d46ba7676f950b90a401504bb (patch) | |
| tree | 84687bd50bbab7707d0fbace6e3b848e58ae031b /tools/testing/selftests/bpf/test_verifier.c | |
| parent | selftests: bpf: Check that PROG_TEST_RUN repeats as requested (diff) | |
| download | wireguard-linux-b4f894633fa14d7d46ba7676f950b90a401504bb.tar.xz wireguard-linux-b4f894633fa14d7d46ba7676f950b90a401504bb.zip | |
selftests: bpf: Don't run sk_lookup in verifier tests
sk_lookup doesn't allow setting data_in for bpf_prog_run. This doesn't
play well with the verifier tests, since they always set a 64 byte
input buffer. Allow not running verifier tests by setting
bpf_test.runs to a negative value and don't run the ctx access case
for sk_lookup. We have dedicated ctx access tests so skipping here
doesn't reduce coverage.
Signed-off-by: Lorenz Bauer <lmb@cloudflare.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20210303101816.36774-6-lmb@cloudflare.com
Diffstat (limited to 'tools/testing/selftests/bpf/test_verifier.c')
| -rw-r--r-- | tools/testing/selftests/bpf/test_verifier.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/bpf/test_verifier.c b/tools/testing/selftests/bpf/test_verifier.c index 58b5a349d3ba..1512092e1e68 100644 --- a/tools/testing/selftests/bpf/test_verifier.c +++ b/tools/testing/selftests/bpf/test_verifier.c @@ -105,7 +105,7 @@ struct bpf_test { enum bpf_prog_type prog_type; uint8_t flags; void (*fill_helper)(struct bpf_test *self); - uint8_t runs; + int runs; #define bpf_testdata_struct_t \ struct { \ uint32_t retval, retval_unpriv; \ @@ -1165,7 +1165,7 @@ static void do_test_single(struct bpf_test *test, bool unpriv, run_errs = 0; run_successes = 0; - if (!alignment_prevented_execution && fd_prog >= 0) { + if (!alignment_prevented_execution && fd_prog >= 0 && test->runs >= 0) { uint32_t expected_val; int i; |