Thanks test bot catch the issue. On Wed, May 13, 2020 at 03:44:18PM +0800, kernel test robot wrote:
Greeting,
FYI, we noticed the following commit (built with gcc-7):
commit: 77bb53cb094828a31cd3c5b402899810f63073c1 ("selftests/bpf: Fix perf_buffer test on systems w/ offline CPUs") https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable-rc.git linux-5.4.y
The author for this commit is Andrii(cc'd).
Mine is f1c3656c6d9c ("selftests/bpf: Skip perf hw events test if the setup disabled it")
prog_tests/stacktrace_build_id_nmi.c:55:3: error: label ‘cleanup’ used but not defined goto cleanup; ^~~~
Hi Greg, we are missing a depend commit dde53c1b763b ("selftests/bpf: Convert few more selftest to skeletons").
So either we need backport this patch, or if you like, we can also fix it by changing 'goto cleanup;' to 'goto close_prog;'. So which one do you prefer?
prog_tests/perf_buffer.c: In function ‘test_perf_buffer’: prog_tests/perf_buffer.c:39:8: warning: implicit declaration of function ‘parse_cpu_mask_file’ [-Wimplicit-function-declaration] err = parse_cpu_mask_file("/sys/devices/system/cpu/online", ^~~~~~~~~~~~~~~~~~~
I guess, this is due to the header file path changed. Hi Andrii, what do you think?
Thanks Hangbin