]> www.infradead.org Git - users/griffoul/linux.git/commitdiff
riscv, bpf: use lw when reading int cpu in bpf_get_smp_processor_id
authorRadim Krčmář <rkrcmar@ventanamicro.com>
Tue, 12 Aug 2025 09:02:56 +0000 (11:02 +0200)
committerPaul Walmsley <pjw@kernel.org>
Fri, 5 Sep 2025 21:31:20 +0000 (15:31 -0600)
emit_ld is wrong, because thread_info.cpu is 32-bit, not xlen-bit wide.
The struct currently has a hole after cpu, so little endian accesses
seemed fine.

Fixes: 2ddec2c80b44 ("riscv, bpf: inline bpf_get_smp_processor_id()")
Cc: stable@vger.kernel.org
Signed-off-by: Radim Krčmář <rkrcmar@ventanamicro.com>
Reviewed-by: Pu Lehui <pulehui@huawei.com>
Link: https://lore.kernel.org/r/20250812090256.757273-4-rkrcmar@ventanamicro.com
Signed-off-by: Paul Walmsley <pjw@kernel.org>
arch/riscv/net/bpf_jit_comp64.c

index 6e1554d89681ff97ffd036974838a8c2dfef9320..9883a55d61b5b9b62e3e74afce7d76bf95eb59a6 100644 (file)
@@ -1763,7 +1763,7 @@ int bpf_jit_emit_insn(const struct bpf_insn *insn, struct rv_jit_context *ctx,
                 */
                if (insn->src_reg == 0 && insn->imm == BPF_FUNC_get_smp_processor_id) {
                        /* Load current CPU number in R0 */
-                       emit_ld(bpf_to_rv_reg(BPF_REG_0, ctx), offsetof(struct thread_info, cpu),
+                       emit_lw(bpf_to_rv_reg(BPF_REG_0, ctx), offsetof(struct thread_info, cpu),
                                RV_REG_TP, ctx);
                        break;
                }