> diff --git a/arch/riscv/net/Makefile b/arch/riscv/net/Makefile
> index 9a1e5f0a94e5..6458d4d51990 100644
> --- a/arch/riscv/net/Makefile
> +++ b/arch/riscv/net/Makefile

[ ... ]

> diff --git a/arch/riscv/net/bpf_jit_comp64.c b/arch/riscv/net/bpf_jit_comp64.c
> index 2f1109dbf105..ca9d0e811aea 100644
> --- a/arch/riscv/net/bpf_jit_comp64.c
> +++ b/arch/riscv/net/bpf_jit_comp64.c
> @@ -1834,7 +1834,13 @@ int bpf_jit_emit_insn(const struct bpf_insn *insn, 
> struct rv_jit_context *ctx,
>               if (ret)
>                       return ret;
>
> -             if (insn->src_reg != BPF_PSEUDO_CALL)
> +             /*
> +              * arch_bpf_timed_may_goto() is emitted by the verifier and
> +              * returns its result in BPF_REG_AX instead of BPF_REG_0, so
> +              * skip the normal "move return register into R0".
> +              */
> +             if (insn->src_reg != BPF_PSEUDO_CALL &&
> +                 addr != (u64)arch_bpf_timed_may_goto)
                    ^^^^

This isn't a bug, but could this fit on one line? The 100-character line
limit would allow the full condition without the break.

[ ... ]


---
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/29858413299

Reply via email to