lkml.org 
[lkml]   [2020]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] Perf: support to unwind userspace application stacks generated with thumb.
Date
Only push sp and lr in the stack for thumb mode. it will go
through the stack find sp and lr.

Change this to the more detailed description of the patch

Signed-off-by: Jiping Ma <jiping.ma2@windriver.com>
---
arch/arm64/kernel/perf_callchain.c | 36 +++++++++++++++++++++++++++---
1 file changed, 33 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/kernel/perf_callchain.c b/arch/arm64/kernel/perf_callchain.c
index bcafd7dcfe8b..97dde271c121 100644
--- a/arch/arm64/kernel/perf_callchain.c
+++ b/arch/arm64/kernel/perf_callchain.c
@@ -104,6 +104,30 @@ compat_user_backtrace(struct compat_frame_tail __user *tail,

return (struct compat_frame_tail __user *)compat_ptr(buftail.fp) - 1;
}
+
+void
+user_backtrace_thumb(struct perf_callchain_entry_ctx *entry,
+ struct pt_regs *regs)
+{
+ u32 sp;
+ u32 *sp_t;
+ /*
+ * Only push sp, lr to stack.
+ */
+ for (sp = regs->compat_sp; (sp < current->mm->start_stack) &&
+ (entry->nr < entry->max_stack); sp += 4) {
+ sp_t = (u32 *)(unsigned long)sp;
+ if ((*sp_t > regs->compat_sp) &&
+ (*sp_t < current->mm->start_stack)) {
+ if (*(sp_t + 1) < current->mm->end_code &&
+ *(sp_t + 1) > current->mm->start_code) {
+ perf_callchain_store(entry, *(sp_t + 1)-1);
+ sp += 4;
+ }
+ }
+ }
+}
+
#endif /* CONFIG_COMPAT */

void perf_callchain_user(struct perf_callchain_entry_ctx *entry,
@@ -132,9 +156,15 @@ void perf_callchain_user(struct perf_callchain_entry_ctx *entry,

tail = (struct compat_frame_tail __user *)regs->compat_fp - 1;

- while ((entry->nr < entry->max_stack) &&
- tail && !((unsigned long)tail & 0x3))
- tail = compat_user_backtrace(tail, entry);
+ if (((unsigned long)tail > current->mm->start_stack) ||
+ ((unsigned long)tail < regs->compat_sp) ||
+ regs->compat_sp == regs->compat_usr(7)) {
+ user_backtrace_thumb(entry, regs);
+ } else {
+ while ((entry->nr < entry->max_stack) &&
+ tail && !((unsigned long)tail & 0x3))
+ tail = compat_user_backtrace(tail, entry);
+ }
#endif
}
}
--
2.18.1
\
 
 \ /
  Last update: 2020-04-16 07:40    [W:0.333 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site