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.

