> diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
> index 2abc79dbf281..f65eff28ecd2 100644
> --- a/kernel/bpf/verifier.c
> +++ b/kernel/bpf/verifier.c

[ ... ]

> @@ -12135,7 +12146,8 @@ static int check_kfunc_args(struct bpf_verifier_env 
> *env, struct bpf_kfunc_call_
>
>               if (regno == meta->release_regno)
>                       arg_type |= OBJ_RELEASE;
> -             ret = check_func_arg_reg_off(env, reg, argno, arg_type);
> +             ret = __check_func_arg_reg_off(env, reg, argno, arg_type,
> +                                            btf_id_fixed_off_ok);

This isn't a bug, but does this call need to be split across two lines?
The line length limit is 100 characters, and it looks like it would fit
on a single line:

                ret = __check_func_arg_reg_off(env, reg, argno, arg_type, 
btf_id_fixed_off_ok);

>               if (ret < 0)
>                       return ret;


---
AI reviewed your patch. Please fix the bug or email reply why it's not a bug.
See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md

CI run summary: https://github.com/kernel-patches/bpf/actions/runs/28007209948

Reply via email to