CHECK: Macro argument 'b' may be better as '(b)' to avoid precedence issues #38: FILE: tools/testing/selftests/drivers/net/hw/nk_forward.bpf.c:15: +#define v6_p64_equal(a, b) (a.s6_addr32[0] == b.s6_addr32[0] && \ + a.s6_addr32[1] == b.s6_addr32[1]) WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #41: FILE: tools/testing/selftests/drivers/net/hw/nk_forward.bpf.c:18: +volatile __u32 netkit_ifindex; WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst #42: FILE: tools/testing/selftests/drivers/net/hw/nk_forward.bpf.c:19: +volatile __u8 ipv6_prefix[16]; total: 0 errors, 2 warnings, 1 checks, 49 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 dbcbbf358416 ("selftests/net: Add bpf skb forwarding program") 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. total: 0 errors, 2 warnings, 1 checks, 49 lines checked