On Tue, 22 Mar 2022 21:46:29 +0900 Masami Hiramatsu <mhiramat@xxxxxxxxxx> wrote: > > > Indeed. I would like to replace the trampoline code of kretprobe with > > > rethook, eventually. There is no reason why we keep the clone. > > > (But I need more arch maintainers help for that, there are too many > > > archs implemented kretprobes) > > > > CONFIG_KPROBE_ON_RETHOOK - and then implement archs one by one? > > Sounds good! Maybe we will see different data structure fields > which depends on that config, but those are internal fields, so > user will not access it. Which is basically what I do for ftrace. Which is why we have all these: select HAVE_DYNAMIC_FTRACE select HAVE_DYNAMIC_FTRACE_WITH_REGS select HAVE_DYNAMIC_FTRACE_WITH_ARGS select HAVE_DYNAMIC_FTRACE_WITH_DIRECT_CALLS select HAVE_SAMPLE_FTRACE_DIRECT select HAVE_SAMPLE_FTRACE_DIRECT_MULTI in the architecture Kconfigs. -- Steve