CHECK: spaces preferred around that '*' (ctx:VxV) #76: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:19: +#define MAX_BPFTOOL_OUTPUT_LEN (64*1024) ^ CHECK: Alignment should match open parenthesis #118: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:61: + ret = snprintf(cmd, MAX_BPFTOOL_CMD_LEN, "prog load %s %s", + test->bpf_prog, test->bpffs_path); CHECK: Alignment should match open parenthesis #131: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:74: + ret = get_bpftool_command_output(cmd, output, + MAX_BPFTOOL_OUTPUT_LEN); CHECK: Alignment should match open parenthesis #134: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:77: + ret = check_metadata(output, test->expected_output, + ARRAY_SIZE(test->expected_output)); CHECK: Alignment should match open parenthesis #147: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:90: + ret = check_metadata(output, test->expected_output_json, + ARRAY_SIZE(test->expected_output_json)); CHECK: Please use a blank line after function/struct/union/enum declarations #187: FILE: tools/testing/selftests/bpf/prog_tests/bpftool_metadata.c:130: +}; +static const int tests_count = ARRAY_SIZE(tests); total: 0 errors, 0 warnings, 6 checks, 151 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 630264090b5d ("selftests/bpf: convert test_bpftool_metadata.sh into test_progs framework") 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, 0 warnings, 6 checks, 151 lines checked