lkml.org 
[lkml]   [2021]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[RFC PATCH -tip 3/3] x86/kprobes,orc: Unwind optprobe trampoline correctly
Date
ORC Unwinder can not unwind if an optprobe trampoline is on the
stack because optprobe trampoline code has no ORC information.

This uses the ORC information on the template code of the
trampoline to adjust the sp register by ORC information and
extract the correct probed address from the optprobe trampoline
address.

Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
---
arch/x86/include/asm/kprobes.h | 6 ++++
arch/x86/kernel/kprobes/opt.c | 54 ++++++++++++++++++++++++++++++++++++++++
arch/x86/kernel/unwind_orc.c | 15 +++++++++--
3 files changed, 72 insertions(+), 3 deletions(-)

diff --git a/arch/x86/include/asm/kprobes.h b/arch/x86/include/asm/kprobes.h
index 71ea2eab43d5..9bbc45fcb3f1 100644
--- a/arch/x86/include/asm/kprobes.h
+++ b/arch/x86/include/asm/kprobes.h
@@ -119,9 +119,15 @@ extern int kprobe_exceptions_notify(struct notifier_block *self,
unsigned long val, void *data);
extern int kprobe_int3_handler(struct pt_regs *regs);

+unsigned long recover_optprobe_trampoline(unsigned long addr, unsigned long *sp);
#else

static inline int kprobe_debug_handler(struct pt_regs *regs) { return 0; }
+static inline unsigned long recover_optprobe_trampoline(unsigned long addr,
+ unsigned long *sp)
+{
+ return addr;
+}

#endif /* CONFIG_KPROBES */
#endif /* _ASM_X86_KPROBES_H */
diff --git a/arch/x86/kernel/kprobes/opt.c b/arch/x86/kernel/kprobes/opt.c
index 6d26e5cf2ba2..f91922ba4844 100644
--- a/arch/x86/kernel/kprobes/opt.c
+++ b/arch/x86/kernel/kprobes/opt.c
@@ -30,6 +30,9 @@
#include <asm/set_memory.h>
#include <asm/sections.h>
#include <asm/nospec-branch.h>
+#include <asm/orc_types.h>
+
+struct orc_entry *orc_find(unsigned long ip);

#include "common.h"

@@ -100,6 +103,21 @@ static void synthesize_set_arg1(kprobe_opcode_t *addr, unsigned long val)
*(unsigned long *)addr = val;
}

+/* Extract mov operand */
+static unsigned long extract_set_arg1(kprobe_opcode_t *addr)
+{
+#ifdef CONFIG_X86_64
+ if (addr[0] != 0x48 || addr[1] != 0xbf)
+ return 0;
+ addr += 2;
+#else
+ if (*addr != 0xb8)
+ return 0;
+ addr++;
+#endif
+ return *(unsigned long *)addr;
+}
+
static void
optimized_callback(struct optimized_kprobe *op, struct pt_regs *regs);

@@ -483,6 +501,42 @@ int arch_prepare_optimized_kprobe(struct optimized_kprobe *op,
goto out;
}

+#ifdef CONFIG_UNWINDER_ORC
+unsigned long recover_optprobe_trampoline(unsigned long addr, unsigned long *sp)
+{
+ unsigned long offset, entry, probe_addr;
+ struct optimized_kprobe *op;
+ struct orc_entry *orc;
+
+ entry = find_kprobe_optinsn_slot_entry(addr);
+ if (!entry)
+ return addr;
+
+ offset = addr - entry;
+
+ /* Decode arg1 and get the optprobe */
+ op = (void *)extract_set_arg1((void *)(entry + TMPL_MOVE_IDX));
+ if (!op)
+ return addr;
+
+ probe_addr = (unsigned long)op->kp.addr;
+
+ if (offset < TMPL_END_IDX) {
+ orc = orc_find((unsigned long)optprobe_template_func + offset);
+ if (!orc || orc->sp_reg != ORC_REG_SP)
+ return addr;
+ /*
+ * Since optprobe trampoline doesn't push caller on the stack,
+ * need to decrement 1 stack entry size
+ */
+ *sp += orc->sp_offset - sizeof(long);
+ return probe_addr;
+ } else {
+ return probe_addr + offset - TMPL_END_IDX;
+ }
+}
+#endif
+
/*
* Replace breakpoints (INT3) with relative jumps (JMP.d32).
* Caller must call with locking kprobe_mutex and text_mutex.
diff --git a/arch/x86/kernel/unwind_orc.c b/arch/x86/kernel/unwind_orc.c
index c70dfeea4552..9f685f9c2358 100644
--- a/arch/x86/kernel/unwind_orc.c
+++ b/arch/x86/kernel/unwind_orc.c
@@ -79,7 +79,7 @@ static struct orc_entry *orc_module_find(unsigned long ip)
#endif

#ifdef CONFIG_DYNAMIC_FTRACE
-static struct orc_entry *orc_find(unsigned long ip);
+struct orc_entry *orc_find(unsigned long ip);

/*
* Ftrace dynamic trampolines do not have orc entries of their own.
@@ -142,7 +142,7 @@ static struct orc_entry orc_fp_entry = {
.end = 0,
};

-static struct orc_entry *orc_find(unsigned long ip)
+struct orc_entry *orc_find(unsigned long ip)
{
static struct orc_entry *orc;

@@ -537,6 +537,7 @@ bool unwind_next_frame(struct unwind_state *state)

state->ip = unwind_recover_ret_addr(state, state->ip,
(unsigned long *)ip_p);
+ state->ip = recover_optprobe_trampoline(state->ip, &sp);
state->sp = sp;
state->regs = NULL;
state->prev_regs = NULL;
@@ -558,6 +559,14 @@ bool unwind_next_frame(struct unwind_state *state)
*/
state->ip = unwind_recover_kretprobe(state, state->ip,
(unsigned long *)(state->sp - sizeof(long)));
+
+ /*
+ * The optprobe trampoline has a unique stackframe. It has
+ * no caller (probed) address on the stack, Thus it has to
+ * decode the trampoline code and change the stack pointer
+ * for the next frame, but not change the pt_regs.
+ */
+ state->ip = recover_optprobe_trampoline(state->ip, &state->sp);
state->regs = (struct pt_regs *)sp;
state->prev_regs = NULL;
state->full_regs = true;
@@ -573,7 +582,7 @@ bool unwind_next_frame(struct unwind_state *state)
/* See UNWIND_HINT_TYPE_REGS case comment. */
state->ip = unwind_recover_kretprobe(state, state->ip,
(unsigned long *)(state->sp - sizeof(long)));
-
+ state->ip = recover_optprobe_trampoline(state->ip, &state->sp);
if (state->full_regs)
state->prev_regs = state->regs;
state->regs = (void *)sp - IRET_FRAME_OFFSET;
\
 
 \ /
  Last update: 2021-03-31 07:46    [W:0.089 / U:6.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site