On Wed, Aug 26, 2020 at 10:46:43PM +0900, Masami Hiramatsu wrote: > static __used __kprobes void *trampoline_handler(struct pt_regs *regs) > { > + return (void *)kretprobe_trampoline_handler(regs, > + (unsigned long)&kretprobe_trampoline, > + regs->ARM_fp); > } Does it make sense to have the generic code have a weak trampoline_handler() implemented like the above? It looks like a number of architectures have this trivial variant and it seems pointless to duplicate this.