WARNING: Macros with flow control statements should be avoided #25: FILE: tools/testing/selftests/bpf/bpf_experimental.h:371: +#define __cond_break(expr) \ ({ __label__ l_break, l_continue; \ asm volatile goto("may_goto %l[l_break]" \ :::: l_break); \ goto l_continue; \ + l_break: expr; \ l_continue:; \ }) WARNING: labels should not be indented #31: FILE: tools/testing/selftests/bpf/bpf_experimental.h:376: + l_break: expr; \ WARNING: Argument 'expr' is not used in function-like macro #40: FILE: tools/testing/selftests/bpf/bpf_experimental.h:395: +#define __cond_break(expr) \ ({ __label__ l_break, l_continue; \ asm volatile goto("1:.byte 0xe5; \ WARNING: Macros with flow control statements should be avoided #40: FILE: tools/testing/selftests/bpf/bpf_experimental.h:395: +#define __cond_break(expr) \ ({ __label__ l_break, l_continue; \ asm volatile goto("1:.byte 0xe5; \ WARNING: Argument 'expr' is not used in function-like macro #58: FILE: tools/testing/selftests/bpf/bpf_experimental.h:421: +#define __cond_break(expr) \ ({ __label__ l_break, l_continue; \ asm volatile goto("1:.byte 0xe5; \ WARNING: Macros with flow control statements should be avoided #58: FILE: tools/testing/selftests/bpf/bpf_experimental.h:421: +#define __cond_break(expr) \ ({ __label__ l_break, l_continue; \ asm volatile goto("1:.byte 0xe5; \ total: 0 errors, 6 warnings, 0 checks, 54 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 ccf3ee927d64 ("selftests/bpf: Introduce cond_break_label") 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, 6 warnings, 0 checks, 54 lines checked