Re: [RFC PATCH 03/14] arm: kprobes: Use generic kretprobe trampoline handler

From: peterz
Date: Wed Aug 26 2020 - 10:09:24 EST


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.