WARNING: 'unitialized' may be misspelled - perhaps 'uninitialized'? #30: The logic to detect initialized and unitialized irq flag slots, marking ^^^^^^^^^^^ WARNING: 'unsinged' may be misspelled - perhaps 'unsigned'? #107: FILE: kernel/bpf/helpers.c:3060: +/* Keep unsinged long in prototype so that kfunc is usable when emitted to ^^^^^^^^ WARNING: line length of 84 exceeds 80 columns #161: FILE: kernel/bpf/verifier.c:663: +static int irq_flag_get_spi(struct bpf_verifier_env *env, struct bpf_reg_state *reg) CHECK: Alignment should match open parenthesis #177: FILE: kernel/bpf/verifier.c:1167: +static int mark_stack_slot_irq_flag(struct bpf_verifier_env *env, + struct bpf_kfunc_call_arg_meta *meta, WARNING: line length of 94 exceeds 80 columns #208: FILE: kernel/bpf/verifier.c:1198: +static int unmark_stack_slot_irq_flag(struct bpf_verifier_env *env, struct bpf_reg_state *reg) WARNING: line length of 85 exceeds 80 columns #231: FILE: kernel/bpf/verifier.c:1221: + /* see unmark_stack_slots_dynptr() for why we need to set REG_LIVE_WRITTEN */ WARNING: line length of 97 exceeds 80 columns #241: FILE: kernel/bpf/verifier.c:1231: +static bool is_irq_flag_reg_valid_uninit(struct bpf_verifier_env *env, struct bpf_reg_state *reg) WARNING: line length of 94 exceeds 80 columns #265: FILE: kernel/bpf/verifier.c:1255: +static int is_irq_flag_reg_valid_init(struct bpf_verifier_env *env, struct bpf_reg_state *reg) WARNING: line length of 86 exceeds 80 columns #376: FILE: kernel/bpf/verifier.c:3420: +static int mark_irq_flag_read(struct bpf_verifier_env *env, struct bpf_reg_state *reg) WARNING: line length of 98 exceeds 80 columns #394: FILE: kernel/bpf/verifier.c:10189: + verbose(env, "global function calls are not allowed with IRQs disabled,\n" WARNING: line length of 96 exceeds 80 columns #407: FILE: kernel/bpf/verifier.c:10724: + verbose(env, "%s cannot be used inside bpf_local_irq_save-ed region\n", prefix); WARNING: line length of 87 exceeds 80 columns #420: FILE: kernel/bpf/verifier.c:10934: + verbose(env, "sleepable helper %s#%d in IRQ-disabled region\n", WARNING: line length of 84 exceeds 80 columns #426: FILE: kernel/bpf/verifier.c:10940: + env->insn_aux_data[insn_idx].storage_get_func_atomic = true; WARNING: line length of 85 exceeds 80 columns #436: FILE: kernel/bpf/verifier.c:11504: +static bool is_kfunc_arg_irq_flag(const struct btf *btf, const struct btf_param *arg) CHECK: Alignment should match open parenthesis #485: FILE: kernel/bpf/verifier.c:11961: +static int process_irq_flag(struct bpf_verifier_env *env, int regno, + struct bpf_kfunc_call_arg_meta *meta) WARNING: line length of 83 exceeds 80 columns #493: FILE: kernel/bpf/verifier.c:11969: + } else if (meta->func_id == special_kfunc_list[KF_bpf_local_irq_restore]) { WARNING: line length of 83 exceeds 80 columns #496: FILE: kernel/bpf/verifier.c:11972: + verbose(env, "verifier internal error: unknown irq flags kfunc\n"); WARNING: line length of 91 exceeds 80 columns #502: FILE: kernel/bpf/verifier.c:11978: + verbose(env, "expected uninitialized irq flag as arg#%d\n", regno); WARNING: line length of 106 exceeds 80 columns #506: FILE: kernel/bpf/verifier.c:11982: + err = check_mem_access(env, env->insn_idx, regno, 0, BPF_DW, BPF_WRITE, -1, false, false); WARNING: line length of 92 exceeds 80 columns #516: FILE: kernel/bpf/verifier.c:11992: + verbose(env, "expected an initialized irq flag as arg#%d\n", regno); CHECK: Please don't use multiple blank lines #531: FILE: kernel/bpf/verifier.c:12007: + + WARNING: line length of 98 exceeds 80 columns #549: FILE: kernel/bpf/verifier.c:12894: + verbose(env, "arg#%d doesn't point to an irq flag on stack\n", i); WARNING: line length of 100 exceeds 80 columns #564: FILE: kernel/bpf/verifier.c:13082: + verbose(env, "kernel func %s is sleepable within IRQ-disabled region\n", func_name); WARNING: line length of 88 exceeds 80 columns #578: FILE: kernel/bpf/verifier.c:18022: + if (!check_ids(old_reg->ref_obj_id, cur_reg->ref_obj_id, idmap)) WARNING: line length of 88 exceeds 80 columns #599: FILE: kernel/bpf/verifier.c:18065: + if (!check_ids(old->res[i].prev_id, cur->res[i].prev_id, idmap)) total: 0 errors, 22 warnings, 3 checks, 511 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 3bebf7359439 ("bpf: Introduce support for bpf_local_irq_{save,restore}") 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, 22 warnings, 3 checks, 511 lines checked