WARNING: line length of 82 exceeds 80 columns #58: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:18: +static void __test_run(struct bpf_program *prog, void *ctx_in, size_t ctx_size_in) CHECK: Alignment should match open parenthesis #61: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:21: + LIBBPF_OPTS(bpf_test_run_opts, topts, + .ctx_in = ctx_in, WARNING: line length of 86 exceeds 80 columns #107: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:67: +/* validate that for two loads of the same jump table libbpf generates only one map */ WARNING: line length of 86 exceeds 80 columns #145: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:105: + if (!ASSERT_EQ(seen, false, "more than one INSN_ARRAY map")) { WARNING: line length of 91 exceeds 80 columns #174: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:134: + check_simple(skel, skel->progs.one_switch_non_zero_sec_off, in[i], out[i]); ERROR: space prohibited before that ',' (ctx:WxW) #200: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:160: + __u64 out[] = {12, 15, 7 , 15, 12, 15, 15}; ^ WARNING: line length of 81 exceeds 80 columns #204: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:164: + check_simple(skel, skel->progs.one_jump_two_maps, in[i], out[i]); WARNING: line length of 82 exceeds 80 columns #214: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:174: + check_simple(skel, skel->progs.use_static_global1, in[i], out[i]); WARNING: line length of 82 exceeds 80 columns #216: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:176: + check_simple(skel, skel->progs.use_static_global2, in[i], out[i]); WARNING: line length of 85 exceeds 80 columns #226: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:186: + check_simple(skel, skel->progs.use_nonstatic_global1, in[i], out[i]); WARNING: line length of 85 exceeds 80 columns #229: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:189: + check_simple(skel, skel->progs.use_nonstatic_global2, in[i], out[i]); WARNING: line length of 92 exceeds 80 columns #240: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:200: + check_simple_fentry(skel, skel->progs.simple_test_other_sec, in[i], out[i]); WARNING: line length of 98 exceeds 80 columns #251: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:211: + check_simple_fentry(skel, skel->progs.use_static_global_other_sec, in[i], out[i]); WARNING: line length of 101 exceeds 80 columns #262: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:222: + check_simple_fentry(skel, skel->progs.use_nonstatic_global_other_sec, in[i], out[i]); WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__)) #522: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:204: +int big_jump_table(struct simple_ctx *ctx __attribute__((unused))) WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__)) #561: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:243: +int one_jump_two_maps(struct simple_ctx *ctx __attribute__((unused))) WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #568: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:250: + volatile int ret = 0; WARNING: labels should not be indented #578: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:260: + l1: ret += 1; WARNING: labels should not be indented #579: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:261: + l2: ret += 3; WARNING: labels should not be indented #580: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:262: + l3: ret += 5; WARNING: labels should not be indented #581: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:263: + l4: ret += 7; WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__)) #588: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:270: +int one_map_two_jumps(struct simple_ctx *ctx __attribute__((unused))) WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #594: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:276: + volatile int ret = 0; WARNING: labels should not be indented #602: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:284: + l1: ret += 3; WARNING: labels should not be indented #603: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:285: + l2: ret += 5; WARNING: labels should not be indented #604: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:286: + l3: ret += 7; WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #611: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:293: +static __noinline int f0(volatile struct simple_ctx *ctx __arg_ctx) WARNING: Macros with flow control statements should be avoided #733: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:415: +#define SKIP_TEST(TEST_NAME) \ + SEC("syscall") int TEST_NAME(void *ctx) \ + { \ + return 0; \ + } total: 1 errors, 26 warnings, 1 checks, 719 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 aced2dd45c9d ("selftests/bpf: add C-level selftests for indirect jumps") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES 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.