Thread (12 messages) 12 messages, 3 authors, 3d ago

Re: [RFC PATCH 2/4] tracing/probes: Compile all fetchargs into a single BPF program per event

From: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Date: 2026-07-01 22:40:11
Also in: bpf, linux-kselftest, lkml

On Wed, 01 Jul 2026 11:41:26 -0700
"Alexei Starovoitov" [off-list ref] wrote:
Nack.
I really don't like it.
There were days in the past when the kernel generating bpf directly was appealing.
These days are gone. Performance improvements for fetchargs is not a good reason
to add all this complexity and bypass verifier checks.
bpf insns should come from user space.
Thanks for your comment!
OK, I don't mind because this is a kind of investigation project. And some
people had asked me about the same idea, now I can tell them the result.

I'm satisfied with the current outcome, as this development process gave me
insight into the implementation of BPF and demonstrated the potential for
optimization via JIT. :)

And also, as noted in the cover letter, the current performance of fetcharg
is better than I thought, and is good enough for debugging. :)

BTW, I'm also interested in calling the verifier on this generated code.
Even it it is not merged, I think showing the correct way to implement it
will be useful in the future.

Thank you,

-- 
Masami Hiramatsu (Google) [off-list ref]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help