On Fri, Nov 14, 2025 at 6:39 PM Menglong Dong <[email protected]> wrote: > > > > How about bpf_trampoline_must_jmp() ? > > and drop if (!ret) fallback and BPF_TRAMP_F_JMPED bit. > > It doesn't look to be necessary. > > I think you are right. We can check if current trampoline is in "jmp" > mode with the "orig_flags" instead, and remove the > BPF_TRAMP_F_JMPED. That means that I need to pass the > "orig_flags" to > modify_fentry -> bpf_trampoline_update_fentry(bpf_text_poke).
Yep. Makes sense to me.
