On Tue, Nov 05, 2019 at 12:26:29PM -0500, Steven Rostedt wrote: > > I'm guessing it will use kprobes (or optimized probes). I haven't had a > chance to look at your patches. People complained that kprobe and especially kretprobe is too slow and too unpredictable (not guarantees that prog will be executed and k*probe won't be missed). For bpf to attach to fentry/fexit none of k*probe stuff is needed. > I still think using the register_ftrace_direct() will be cleaner (as it > is built on top of code that's been in the kernel for a decade). > Perhaps we can make it work even without the full ftrace code. Yes and I still agree that long term using register_ftrace_direct() is cleaner. What I strongly disagree is that bpf developers need to wait for it to land. Especially when there is no direct dependency. It's nice to have both bpf trampoline and ftrace to be functional at the same time tracing the same kernel function. But for the time being running one or another is an acceptable limitation. Especially since the path to making it clean is already defined and agreed.
