niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

selftests/bpf: Make CO-RE reloc test impartial to test_progs flavor

Browse source 
test_core_reloc_kernel test captures its own process name and validates
it as part of the test. Given extra "flavors" of test_progs, this break
for anything by default test_progs binary. Fix the test to cut out
flavor part of the process name.

Fixes: ee2eb063d3 ("selftests/bpf: Add BPF_CORE_READ and BPF_CORE_READ_STR_INTO macro tests")
Signed-off-by: Andrii Nakryiko <andriin@fb.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20191016060051.2024182-3-andriin@fb.com
This commit is contained in:
Andrii Nakryiko 2019-10-15 23:00:46 -07:00 • committed by Alexei Starovoitov
parent 0b6e71c398
commit d25c5e2355
3 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tools/testing/selftests/bpf/prog_tests/core_reloc.c
View file

@ -211,8 +211,8 @@ static struct core_reloc_test_case test_cases[] = {
.input_len = 0, .input_len = 0,
.output = STRUCT_TO_CHAR_PTR(core_reloc_kernel_output) { .output = STRUCT_TO_CHAR_PTR(core_reloc_kernel_output) {
.valid = { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, }, .valid = { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, },
.comm = "test_progs\0\0\0\0\0", .comm = "test_progs",
.comm_len = 11, .comm_len = sizeof("test_progs"),
}, },
.output_len = sizeof(struct core_reloc_kernel_output), .output_len = sizeof(struct core_reloc_kernel_output),
}, },

2
tools/testing/selftests/bpf/progs/core_reloc_types.h
View file

@ -6,7 +6,7 @@
struct core_reloc_kernel_output { struct core_reloc_kernel_output {
int valid[10]; int valid[10];
char comm[16]; char comm[sizeof("test_progs")];
int comm_len; int comm_len;
}; };

3
tools/testing/selftests/bpf/progs/test_core_reloc_kernel.c
View file

@ -15,7 +15,8 @@ static volatile struct data {
struct core_reloc_kernel_output { struct core_reloc_kernel_output {
int valid[10]; int valid[10];
char comm[16]; /* we have test_progs[-flavor], so cut flavor part */
char comm[sizeof("test_progs")];
int comm_len; int comm_len;
}; };