Re: [GIT PULL] bpf for v6.13-rc4

From: Linus Torvalds
Date: Sat Dec 21 2024 - 14:24:24 EST


On Fri, 20 Dec 2024 at 16:21, Daniel Borkmann <daniel@xxxxxxxxxxxxx> wrote:
>
> - Fix inlining of bpf_get_smp_processor_id helper for !CONFIG_SMP
> systems (Andrea Righi)

LOL.

However, it strikes me that this only handles the x86-64 case.

The other cases (arm64, RISC-V) may not have the pcpu_hot crash, but
they still generate silly code to load off the thread pointer. Does
that even exist (or get initialized) in UP?

End result: I think you should have done the UP case separately and
outside the CONFIG_X86_64.. And why do this only for the
"verifier_inlines_helper_call()" case rather than just do it
unconditionally?

Anyway, I obviously pulled this, but it does seem silly.

Linus