WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #36: new file mode 100644 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, ERROR: space prohibited before that ',' (ctx:WxW) #108: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:68: + __u64 out4[] = {12, 15, 7 , 15, 12, 15, 15}; ^ ERROR: trailing statements should be on next line #119: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:79: + if (0) for (i = 0; i < ARRAY_SIZE(in); i++) ERROR: trailing statements should be on next line #122: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:82: + if (0) for (i = 0; i < ARRAY_SIZE(in); i++) WARNING: line length of 83 exceeds 80 columns #123: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:83: + check_simple(skel, skel->progs.one_jump_two_maps, in4[i], out4[i]); WARNING: line length of 82 exceeds 80 columns #126: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:86: + check_simple(skel, skel->progs.use_static_global1, in[i], out[i]); WARNING: line length of 82 exceeds 80 columns #129: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:89: + check_simple(skel, skel->progs.use_static_global2, in[i], out[i]); WARNING: line length of 85 exceeds 80 columns #132: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:92: + check_simple(skel, skel->progs.use_nonstatic_global1, in[i], out[i]); WARNING: line length of 85 exceeds 80 columns #135: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:95: + check_simple(skel, skel->progs.use_nonstatic_global2, in[i], out[i]); WARNING: line length of 92 exceeds 80 columns #139: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:99: + check_simple_fentry(skel, skel->progs.simple_test_other_sec, in[i], out[i]); WARNING: line length of 98 exceeds 80 columns #143: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:103: + check_simple_fentry(skel, skel->progs.use_static_global_other_sec, in[i], out[i]); WARNING: line length of 101 exceeds 80 columns #147: FILE: tools/testing/selftests/bpf/prog_tests/bpf_gotox.c:107: + 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__)) #368: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:190: +int big_jump_table(struct simple_ctx *ctx __attribute__((unused))) WARNING: Consider removing the code enclosed by this #if 0 and its #endif #370: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:192: +#if 0 WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__)) #411: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:233: +int one_jump_two_maps(struct simple_ctx *ctx __attribute__((unused))) WARNING: Consider removing the code enclosed by this #if 0 and its #endif #413: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:235: +#if 0 WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #419: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:241: + volatile int ret = 0; WARNING: labels should not be indented #429: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:251: + l1: ret += 1; WARNING: labels should not be indented #430: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:252: + l2: ret += 3; WARNING: labels should not be indented #431: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:253: + l3: ret += 5; WARNING: labels should not be indented #432: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:254: + l4: ret += 7; WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #442: FILE: tools/testing/selftests/bpf/progs/bpf_gotox.c:264: +static __noinline int f0(volatile struct simple_ctx *ctx __arg_ctx) total: 3 errors, 20 warnings, 1 checks, 526 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 1305e99b6dee ("selftests/bpf: add 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 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.