Hengqi Chen 60f3caff14 LoongArch: BPF: Don't override subprog's return value ...

The verifier test `calls: div by 0 in subprog` triggers a panic at the
ld.bu instruction. The ld.bu insn is trying to load byte from memory
address returned by the subprog. The subprog actually set the correct
address at the a5 register (dedicated register for BPF return values).
But at commit 73c359d1d356 ("LoongArch: BPF: Sign-extend return values")
we also sign extended a5 to the a0 register (return value in LoongArch).
For function call insn, we later propagate the a0 register back to a5
register. This is right for native calls but wrong for bpf2bpf calls
which expect zero-extended return value in a5 register. So only move a0
to a5 for native calls (i.e. non-BPF_PSEUDO_CALL).

Cc: stable@vger.kernel.org
Fixes: 73c359d1d356 ("LoongArch: BPF: Sign-extend return values")
Signed-off-by: Hengqi Chen <hengqi.chen@gmail.com>
Signed-off-by: Huacai Chen <chenhuacai@loongson.cn>

2025-03-30 16:31:09 +08:00

..

bpf_jit.c

LoongArch: BPF: Don't override subprog's return value

2025-03-30 16:31:09 +08:00

bpf_jit.h

LoongArch: BPF: Fix off-by-one error in build_prologue()

2025-03-30 16:31:09 +08:00

Makefile

…