WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #99: FILE: tools/testing/selftests/bpf/network_helpers.c:685: + volatile bool done; WARNING: line length of 85 exceeds 80 columns #170: FILE: tools/testing/selftests/bpf/network_helpers.c:756: + printf("%-7s %-3s IPv4 %s > %s: ICMP, length %d, type %d, code %d\n", WARNING: line length of 87 exceeds 80 columns #175: FILE: tools/testing/selftests/bpf/network_helpers.c:761: + printf("%-7s %-3s IPv6 %s > %s: ICMPv6, length %d, type %d, code %d\n", WARNING: else is not generally useful after a break or return #179: FILE: tools/testing/selftests/bpf/network_helpers.c:765: + return; + } else { WARNING: line length of 91 exceeds 80 columns #321: FILE: tools/testing/selftests/bpf/network_helpers.c:907: + snprintf(_ifname, sizeof(_ifname), "unknown(%d)", ifindex); WARNING: line length of 100 exceeds 80 columns #388: FILE: tools/testing/selftests/bpf/network_helpers.c:974: +static void encode_test_name(char *buf, size_t len, const char *test_name, const char *subtest_name) WARNING: line length of 84 exceeds 80 columns #415: FILE: tools/testing/selftests/bpf/network_helpers.c:1001: +struct tmonitor_ctx *traffic_monitor_start(const char *netns, const char *test_name, WARNING: line length of 88 exceeds 80 columns #436: FILE: tools/testing/selftests/bpf/network_helpers.c:1022: + encode_test_name(test_name_buf, sizeof(test_name_buf), test_name, subtest_name); WARNING: line length of 84 exceeds 80 columns #542: FILE: tools/testing/selftests/bpf/network_helpers.h:142: +struct tmonitor_ctx *traffic_monitor_start(const char *netns, const char *test_name, WARNING: line length of 98 exceeds 80 columns #546: FILE: tools/testing/selftests/bpf/network_helpers.h:146: +static inline struct tmonitor_ctx *traffic_monitor_start(const char *netns, const char *test_name, WARNING: line length of 82 exceeds 80 columns #547: FILE: tools/testing/selftests/bpf/network_helpers.h:147: + const char *subtest_name) total: 0 errors, 11 warnings, 0 checks, 506 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 8d47be66d958 ("selftests/bpf: Add traffic monitor functions.") 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, 11 warnings, 0 checks, 506 lines checked