| Message ID | 20240123090351.2207-1-yangtiezhu@loongson.cn (mailing list archive) |
|---|---|
| Headers | show |
| Series | Skip callback tests if jit is disabled in test_verifier | expand |
Hello: This series was applied to bpf/bpf-next.git (master) by Andrii Nakryiko <andrii@kernel.org>: On Tue, 23 Jan 2024 17:03:49 +0800 you wrote: > Thanks very much for the feedbacks from Eduard, John, Jiri, Daniel, > Hou Tao, Song Liu and Andrii. > > v7: > -- Add an explicit flag F_NEEDS_JIT_ENABLED for checking, > thanks Andrii. > > [...] Here is the summary with links: - [bpf-next,v7,1/2] selftests/bpf: Move is_jit_enabled() into testing_helpers https://git.kernel.org/bpf/bpf-next/c/15b4f88dcc0a - [bpf-next,v7,2/2] selftests/bpf: Skip callback tests if jit is disabled in test_verifier https://git.kernel.org/bpf/bpf-next/c/0b50478fd877 You are awesome, thank you!
Thanks very much for the feedbacks from Eduard, John, Jiri, Daniel, Hou Tao, Song Liu and Andrii. v7: -- Add an explicit flag F_NEEDS_JIT_ENABLED for checking, thanks Andrii. v6: -- Copy insn_is_pseudo_func() into testing_helpers, thanks Andrii. v5: -- Reuse is_ldimm64_insn() and insn_is_pseudo_func(), thanks Song Liu. v4: -- Move the not-allowed-checking into "if (expected_ret ...)" block, thanks Hou Tao. -- Do some small changes to avoid checkpatch warning about "line length exceeds 100 columns". v3: -- Rebase on the latest bpf-next tree. -- Address the review comments by Hou Tao, remove the second argument "0" of open(), check only once whether jit is disabled, check fd_prog, saved_errno and jit_disabled to skip. Tiezhu Yang (2): selftests/bpf: Move is_jit_enabled() into testing_helpers selftests/bpf: Skip callback tests if jit is disabled in test_verifier tools/testing/selftests/bpf/test_progs.c | 18 ------------------ tools/testing/selftests/bpf/test_verifier.c | 11 +++++++++++ tools/testing/selftests/bpf/testing_helpers.c | 18 ++++++++++++++++++ tools/testing/selftests/bpf/testing_helpers.h | 1 + .../selftests/bpf/verifier/bpf_loop_inline.c | 6 ++++++ 5 files changed, 36 insertions(+), 18 deletions(-)