WARNING: externs should be avoided in .c files #76: FILE: tools/testing/selftests/bpf/bench.c:544: +extern const struct bench bench_rb_prod; WARNING: line length of 86 exceeds 80 columns #143: FILE: tools/testing/selftests/bpf/benchs/bench_ringbufs.c:108: + fprintf(stderr, "overwite mode only works with benchmakr rb-prod!\n"); WARNING: line length of 83 exceeds 80 columns #153: FILE: tools/testing/selftests/bpf/benchs/bench_ringbufs.c:118: + fprintf(stderr, "back-to-back mode makes no sense for rb-prod!\n"); WARNING: line length of 85 exceeds 80 columns #163: FILE: tools/testing/selftests/bpf/benchs/bench_ringbufs.c:128: + fprintf(stderr, "benchmarks excluding rb-prod need one consumer!\n"); WARNING: line length of 100 exceeds 80 columns #288: FILE: tools/testing/selftests/bpf/benchs/run_bench_ringbufs.sh:54: + summarize "rb-prod nr_prod $b" "$($RUN_BENCH -p$b --rb-batch-cnt 50 --rb-overwrite rb-prod)" WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #298: FILE: tools/testing/selftests/bpf/progs/ringbuf_bench.c:17: +const volatile long bench_prod = 0; WARNING: Prefer __aligned(128) over __attribute__((aligned(128))) #302: FILE: tools/testing/selftests/bpf/progs/ringbuf_bench.c:21: +long hits __attribute__((aligned(128))) = 0; total: 0 errors, 7 warnings, 0 checks, 232 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 3c91416349cf ("selftests/bpf/benchs: Add producer and overwrite bench for ring buffer") 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, 7 warnings, 0 checks, 232 lines checked