From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Date: Mon, 05 Jul 2021 08:17:26 +0000 Subject: Re: [PATCH -tip v8 10/13] x86/kprobes: Push a fake return address at kretprobe_trampoline Message-Id: List-Id: References: <162399992186.506599.8457763707951687195.stgit@devnote2> <162400001661.506599.5153975410607447958.stgit@devnote2> In-Reply-To: <162400001661.506599.5153975410607447958.stgit@devnote2> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Masami Hiramatsu Cc: Steven Rostedt , Josh Poimboeuf , X86 ML , Daniel Xu , linux-kernel@vger.kernel.org, bpf@vger.kernel.org, kuba@kernel.org, mingo@redhat.com, ast@kernel.org, Thomas Gleixner , Borislav Petkov , Peter Zijlstra , kernel-team@fb.com, yhs@fb.com, linux-ia64@vger.kernel.org, Abhishek Sagar , Andrii Nakryiko * Masami Hiramatsu wrote: > + /* Replace fake return address with real one. */ > + *frame_pointer = kretprobe_trampoline_handler(regs, frame_pointer); > + /* > + * Move flags to sp so that kretprobe_trapmoline can return > + * right after popf. What is a trapmoline? Also, in the x86 code we capitalize register and instruction names so that they are more distinctive and easier to read in the flow of English text. Thanks, Ingo