CHECK: Macro argument 'func' may be better as '(func)' to avoid precedence issues #101: FILE: include/net/smc.h:139: +#define smc_call_hsbpf(init_val, sk, func, ...) ({ \ + typeof(init_val) __ret = (init_val); \ + struct smc_hs_ctrl *ctrl; \ + rcu_read_lock(); \ + ctrl = READ_ONCE(sock_net(sk)->smc.hs_ctrl); \ + if (ctrl && ctrl->func) \ + __ret = ctrl->func(__VA_ARGS__); \ + rcu_read_unlock(); \ + __ret; \ +}) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #234: new file mode 100644 total: 0 errors, 1 warnings, 1 checks, 452 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 02a3e44a8ecc ("net/smc: bpf: Introduce generic hook for handshake flow") 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, 1 warnings, 1 checks, 452 lines checked