Hi, Currently the test of BPF STRUCT_OPS depends on the specific bpf implementation (e.g, tcp_congestion_ops), but it can not cover all basic functionalities (e.g, return value handling), so introduce a dummy BPF STRUCT_OPS for test purpose. Instead of loading a userspace-implemeted bpf_dummy_ops map into kernel and calling the specific function by writing to sysfs provided by bpf_testmode.ko, only loading bpf_dummy_ops related prog into kernel and calling these prog by bpf_prog_test_run(). The latter is more flexible and has no dependency on extra kernel module. Now only the return value handling related test cases are supported, if more is needed, we can add afterwards. Comments are always welcome. Regards, Hou Changelog: * RFC: https://www.spinics.net/lists/bpf/msg46117.html Hou Tao (5): bpf: add dummy BPF STRUCT_OPS for test purpose bpf: factor out a helper to prepare trampoline for struct_ops prog bpf: do .test_run in dummy BPF STRUCT_OPS bpf: hook .test_run for struct_ops program selftests/bpf: test return value handling for struct_ops prog include/linux/bpf.h | 5 + include/linux/bpf_dummy_ops.h | 25 ++ kernel/bpf/bpf_struct_ops.c | 43 +++- kernel/bpf/bpf_struct_ops_types.h | 2 + net/bpf/Makefile | 3 + net/bpf/bpf_dummy_struct_ops.c | 220 ++++++++++++++++++ .../selftests/bpf/prog_tests/dummy_st_ops.c | 81 +++++++ .../selftests/bpf/progs/dummy_st_ops.c | 33 +++ 8 files changed, 403 insertions(+), 9 deletions(-) create mode 100644 include/linux/bpf_dummy_ops.h create mode 100644 net/bpf/bpf_dummy_struct_ops.c create mode 100644 tools/testing/selftests/bpf/prog_tests/dummy_st_ops.c create mode 100644 tools/testing/selftests/bpf/progs/dummy_st_ops.c -- 2.29.2
