CHECK: No space is necessary after a cast #27: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:79: +extern bool bpf_session_is_return(void) __ksym; CHECK: extern prototypes should be avoided in .h files #27: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:79: +extern bool bpf_session_is_return(void) __ksym; WARNING: Missing a blank line after declarations #48: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:333: + struct kprobe_multi_session *skel = NULL; + LIBBPF_OPTS(bpf_kprobe_multi_opts, opts); WARNING: line length of 82 exceeds 80 columns #77: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:362: + ASSERT_EQ(skel->bss->kretprobe_test1_result, 0, "kretprobe_test1_result"); WARNING: line length of 82 exceeds 80 columns #78: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:363: + ASSERT_EQ(skel->bss->kretprobe_test2_result, 1, "kretprobe_test2_result"); WARNING: line length of 82 exceeds 80 columns #79: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:364: + ASSERT_EQ(skel->bss->kretprobe_test3_result, 0, "kretprobe_test3_result"); WARNING: line length of 82 exceeds 80 columns #80: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:365: + ASSERT_EQ(skel->bss->kretprobe_test4_result, 1, "kretprobe_test4_result"); WARNING: line length of 82 exceeds 80 columns #81: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:366: + ASSERT_EQ(skel->bss->kretprobe_test5_result, 0, "kretprobe_test5_result"); WARNING: line length of 82 exceeds 80 columns #82: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:367: + ASSERT_EQ(skel->bss->kretprobe_test6_result, 1, "kretprobe_test6_result"); WARNING: line length of 82 exceeds 80 columns #83: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:368: + ASSERT_EQ(skel->bss->kretprobe_test7_result, 0, "kretprobe_test7_result"); WARNING: line length of 82 exceeds 80 columns #84: FILE: tools/testing/selftests/bpf/prog_tests/kprobe_multi_test.c:369: + ASSERT_EQ(skel->bss->kretprobe_test8_result, 1, "kretprobe_test8_result"); WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #102: new file mode 100644 WARNING: externs should be avoided in .c files #116: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:10: +extern const void bpf_fentry_test1 __ksym; WARNING: externs should be avoided in .c files #117: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:11: +extern const void bpf_fentry_test2 __ksym; WARNING: externs should be avoided in .c files #118: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:12: +extern const void bpf_fentry_test3 __ksym; WARNING: externs should be avoided in .c files #119: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:13: +extern const void bpf_fentry_test4 __ksym; WARNING: externs should be avoided in .c files #120: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:14: +extern const void bpf_fentry_test5 __ksym; WARNING: externs should be avoided in .c files #121: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:15: +extern const void bpf_fentry_test6 __ksym; WARNING: externs should be avoided in .c files #122: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:16: +extern const void bpf_fentry_test7 __ksym; WARNING: externs should be avoided in .c files #123: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:17: +extern const void bpf_fentry_test8 __ksym; CHECK: Macro argument '__addr' may be better as '(__addr)' to avoid precedence issues #152: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:46: +#define SET(__var, __addr) ({ \ + if ((const void *) addr == __addr) \ + __var = 1; \ +}) CHECK: No space is necessary after a cast #153: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:47: + if ((const void *) addr == __addr) \ CHECK: No space is necessary after a cast #183: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:77: + if (((const void *) addr == &bpf_fentry_test1) || CHECK: No space is necessary after a cast #184: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:78: + ((const void *) addr == &bpf_fentry_test3) || CHECK: No space is necessary after a cast #185: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:79: + ((const void *) addr == &bpf_fentry_test5) || CHECK: No space is necessary after a cast #186: FILE: tools/testing/selftests/bpf/progs/kprobe_multi_session.c:80: + ((const void *) addr == &bpf_fentry_test7)) total: 0 errors, 18 warnings, 8 checks, 171 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit 3fe67ec8b3e6 ("selftests/bpf: Add kprobe multi session test") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 18 warnings, 8 checks, 171 lines checked