* [PATCH] xen/arm64: fix stack dump in show_trace
@ 2013-06-04 10:54 Ian Campbell
2013-06-13 17:28 ` Ian Campbell
0 siblings, 1 reply; 2+ messages in thread
From: Ian Campbell @ 2013-06-04 10:54 UTC (permalink / raw)
To: xen-devel; +Cc: julien.grall, tim, Ian Campbell, stefano.stabellini
On aarch64 the frame pointer points to the next frame pointer and the return
address is the previous stack slot (so below on the downward growing stack,
therefore above in memory):
|<RETURN ADDR> ^addresses grow up
FP -> |<NEXT FP> |
| |
v |
stack grows down.
This is contrary to aarch32 where the frame pointer points to the return
address and the next frame pointer is the next stack slot (so above on the
downward growing stack, below in memory):
FP -> |<RETURN ADDR> ^addresses grow up
|<NEXT FP> |
| |
v |
stack grows down.
In addition print out LR as part of the trace, since it may contain the
penultimate return address e.g. if the ultimate function is a leaf function.
Lastly nuke some unnecessary braces.
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
xen/arch/arm/traps.c | 69 +++++++++++++++++++++++++++++++------------------
1 files changed, 44 insertions(+), 25 deletions(-)
diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c
index 83a7fbc..398d209 100644
--- a/xen/arch/arm/traps.c
+++ b/xen/arch/arm/traps.c
@@ -519,7 +519,41 @@ static void show_guest_stack(struct cpu_user_regs *regs)
}
#define STACK_BEFORE_EXCEPTION(regs) ((register_t*)(regs)->sp)
-
+#ifdef CONFIG_ARM_32
+/* Frame pointer points to the return address:
+ * (largest address)
+ * | cpu_info
+ * | [...] |
+ * | return addr <-----------------, |
+ * | fp --------------------------------+----'
+ * | [...] |
+ * | return addr <------------, |
+ * | fp ---------------------------+----'
+ * | [...] |
+ * | return addr <- regs->fp |
+ * | fp ---------------------------'
+ * |
+ * v (smallest address, sp)
+ */
+#define STACK_FRAME_BASE(fp) ((register_t*)(fp) - 1)
+#else
+/* Frame pointer points to the next frame:
+ * (largest address)
+ * | cpu_info
+ * | [...] |
+ * | return addr |
+ * | fp <-------------------------------, >--'
+ * | [...] |
+ * | return addr |
+ * | fp <--------------------------, >--'
+ * | [...] |
+ * | return addr <- regs->fp |
+ * | fp ---------------------------'
+ * |
+ * v (smallest address, sp)
+ */
+#define STACK_FRAME_BASE(fp) ((register_t*)(fp))
+#endif
static void show_trace(struct cpu_user_regs *regs)
{
register_t *frame, next, addr, low, high;
@@ -527,29 +561,15 @@ static void show_trace(struct cpu_user_regs *regs)
printk("Xen call trace:\n ");
printk("[<%p>]", _p(regs->pc));
- print_symbol(" %s\n ", regs->pc);
+ print_symbol(" %s (PC)\n ", regs->pc);
+ printk("[<%p>]", _p(regs->lr));
+ print_symbol(" %s (LR)\n ", regs->lr);
/* Bounds for range of valid frame pointer. */
- low = (register_t)(STACK_BEFORE_EXCEPTION(regs)/* - 2*/);
+ low = (register_t)(STACK_BEFORE_EXCEPTION(regs));
high = (low & ~(STACK_SIZE - 1)) +
(STACK_SIZE - sizeof(struct cpu_info));
- /* Frame:
- * (largest address)
- * | cpu_info
- * | [...] |
- * | return addr <-----------------, |
- * | fp --------------------------------+----'
- * | [...] |
- * | return addr <------------, |
- * | fp ---------------------------+----'
- * | [...] |
- * | return addr <- regs->fp |
- * | fp ---------------------------'
- * |
- * v (smallest address, sp)
- */
-
/* The initial frame pointer. */
next = regs->fp;
@@ -557,12 +577,11 @@ static void show_trace(struct cpu_user_regs *regs)
{
if ( (next < low) || (next >= high) )
break;
- {
- /* Ordinary stack frame. */
- frame = (register_t *)next;
- next = frame[-1];
- addr = frame[0];
- }
+
+ /* Ordinary stack frame. */
+ frame = STACK_FRAME_BASE(next);
+ next = frame[0];
+ addr = frame[1];
printk("[<%p>]", _p(addr));
print_symbol(" %s\n ", addr);
--
1.7.2.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] xen/arm64: fix stack dump in show_trace
2013-06-04 10:54 [PATCH] xen/arm64: fix stack dump in show_trace Ian Campbell
@ 2013-06-13 17:28 ` Ian Campbell
0 siblings, 0 replies; 2+ messages in thread
From: Ian Campbell @ 2013-06-13 17:28 UTC (permalink / raw)
To: xen-devel; +Cc: julien.grall, tim, stefano.stabellini
On Tue, 2013-06-04 at 11:54 +0100, Ian Campbell wrote:
> On aarch64 the frame pointer points to the next frame pointer and the return
> address is the previous stack slot (so below on the downward growing stack,
> therefore above in memory):
>
> |<RETURN ADDR> ^addresses grow up
> FP -> |<NEXT FP> |
> | |
> v |
> stack grows down.
>
> This is contrary to aarch32 where the frame pointer points to the return
> address and the next frame pointer is the next stack slot (so above on the
> downward growing stack, below in memory):
>
> FP -> |<RETURN ADDR> ^addresses grow up
> |<NEXT FP> |
> | |
> v |
> stack grows down.
>
> In addition print out LR as part of the trace, since it may contain the
> penultimate return address e.g. if the ultimate function is a leaf function.
>
> Lastly nuke some unnecessary braces.
>
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Stefano Acked this IRL so I have applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-13 17:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-04 10:54 [PATCH] xen/arm64: fix stack dump in show_trace Ian Campbell
2013-06-13 17:28 ` Ian Campbell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).