ERROR: trailing whitespace #30: FILE: arch/x86/net/bpf_jit_comp.c:3736: +^I^I$ CHECK: Blank lines aren't necessary after an open brace '{' #30: FILE: arch/x86/net/bpf_jit_comp.c:3736: if (!image || !prog->is_func || extra_pass) { + WARNING: line length of 113 exceeds 80 columns #32: FILE: arch/x86/net/bpf_jit_comp.c:3738: + struct bpf_term_patch_call_sites *patch_call_sites = prog->term_states->patch_call_sites; WARNING: line length of 84 exceeds 80 columns #33: FILE: arch/x86/net/bpf_jit_comp.c:3739: + for (int i = 0; i < patch_call_sites->call_sites_cnt; i++) { WARNING: Missing a blank line after declarations #33: FILE: arch/x86/net/bpf_jit_comp.c:3739: + struct bpf_term_patch_call_sites *patch_call_sites = prog->term_states->patch_call_sites; + for (int i = 0; i < patch_call_sites->call_sites_cnt; i++) { WARNING: line length of 104 exceeds 80 columns #34: FILE: arch/x86/net/bpf_jit_comp.c:3740: + struct call_aux_states *call_states = patch_call_sites->call_states + i; WARNING: line length of 98 exceeds 80 columns #35: FILE: arch/x86/net/bpf_jit_comp.c:3741: + call_states->jit_call_idx = addrs[call_states->call_bpf_insn_idx]; WARNING: Missing a blank line after declarations #35: FILE: arch/x86/net/bpf_jit_comp.c:3741: + struct call_aux_states *call_states = patch_call_sites->call_states + i; + call_states->jit_call_idx = addrs[call_states->call_bpf_insn_idx]; ERROR: trailing whitespace #38: FILE: arch/x86/net/bpf_jit_comp.c:3744: +^I^I$ WARNING: line length of 81 exceeds 80 columns #94: FILE: kernel/bpf/verifier.c:11323: + struct bpf_subprog_info *prev_info, *info = subprog_info(env, subprogno); WARNING: line length of 89 exceeds 80 columns #128: FILE: kernel/bpf/verifier.c:21129: + struct call_aux_states *call_states = term_states->patch_call_sites->call_states; WARNING: line length of 105 exceeds 80 columns #152: FILE: kernel/bpf/verifier.c:21616: + func[i]->aux->is_bpf_loop_cb_non_inline = env->subprog_info[i].is_bpf_loop_cb_non_inline; WARNING: line length of 84 exceeds 80 columns #157: FILE: kernel/bpf/verifier.c:21621: + func_call_states = vzalloc(sizeof(*func_call_states) * len); WARNING: Missing a blank line after declarations #157: FILE: kernel/bpf/verifier.c:21621: + struct call_aux_states *func_call_states; + func_call_states = vzalloc(sizeof(*func_call_states) * len); WARNING: line length of 112 exceeds 80 columns #160: FILE: kernel/bpf/verifier.c:21624: + for (int iter = 0; iter < prog->term_states->patch_call_sites->call_sites_cnt; iter++) { WARNING: line length of 124 exceeds 80 columns #161: FILE: kernel/bpf/verifier.c:21625: + struct call_aux_states call_states = prog->term_states->patch_call_sites->call_states[iter]; WARNING: line length of 82 exceeds 80 columns #162: FILE: kernel/bpf/verifier.c:21626: + if (call_states.call_bpf_insn_idx >= subprog_start WARNING: Missing a blank line after declarations #162: FILE: kernel/bpf/verifier.c:21626: + struct call_aux_states call_states = prog->term_states->patch_call_sites->call_states[iter]; + if (call_states.call_bpf_insn_idx >= subprog_start WARNING: line length of 97 exceeds 80 columns #163: FILE: kernel/bpf/verifier.c:21627: + && call_states.call_bpf_insn_idx < subprog_end) { CHECK: Logical continuations should be on the previous line #163: FILE: kernel/bpf/verifier.c:21627: + if (call_states.call_bpf_insn_idx >= subprog_start + && call_states.call_bpf_insn_idx < subprog_end) { WARNING: line length of 87 exceeds 80 columns #164: FILE: kernel/bpf/verifier.c:21628: + func_call_states[call_sites_cnt] = call_states; WARNING: line length of 108 exceeds 80 columns #165: FILE: kernel/bpf/verifier.c:21629: + func_call_states[call_sites_cnt].call_bpf_insn_idx -= subprog_start; WARNING: line length of 96 exceeds 80 columns #169: FILE: kernel/bpf/verifier.c:21633: + func[i]->term_states->patch_call_sites->call_sites_cnt = call_sites_cnt; WARNING: line length of 95 exceeds 80 columns #170: FILE: kernel/bpf/verifier.c:21634: + func[i]->term_states->patch_call_sites->call_states = func_call_states; WARNING: line length of 97 exceeds 80 columns #180: FILE: kernel/bpf/verifier.c:21925: + struct bpf_insn *insn_buf, int insn_idx, int *cnt, int *kfunc_btf_id) WARNING: line length of 100 exceeds 80 columns #244: FILE: kernel/bpf/verifier.c:22430: + ret = fixup_kfunc_call(env, insn, insn_buf, i + delta, &cnt, &kfunc_btf_id); WARNING: Missing a blank line after declarations #244: FILE: kernel/bpf/verifier.c:22430: + int kfunc_btf_id = 0; + ret = fixup_kfunc_call(env, insn, insn_buf, i + delta, &cnt, &kfunc_btf_id); WARNING: line length of 90 exceeds 80 columns #259: FILE: kernel/bpf/verifier.c:22445: + call_states[call_sites_cnt].call_bpf_insn_idx = i + delta; ERROR: trailing whitespace #272: FILE: kernel/bpf/verifier.c:22929: +^I^I^Icall_states[call_sites_cnt].call_bpf_insn_idx = i + delta;^I$ WARNING: line length of 88 exceeds 80 columns #272: FILE: kernel/bpf/verifier.c:22929: + call_states[call_sites_cnt].call_bpf_insn_idx = i + delta; WARNING: line length of 82 exceeds 80 columns #286: FILE: kernel/bpf/verifier.c:22956: + env->prog->term_states->patch_call_sites->call_sites_cnt = call_sites_cnt; WARNING: line length of 89 exceeds 80 columns #322: FILE: kernel/bpf/verifier.c:24665: + struct call_aux_states *call_states = term_states->patch_call_sites->call_states; total: 3 errors, 27 warnings, 2 checks, 299 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. NOTE: Whitespace errors detected. You may wish to use scripts/cleanpatch or scripts/cleanfile Commit d60fd1595561 ("bpf: Creating call sites table to stub instructions during runtime") 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.