path: root/arch/arm64/include/asm/irq.h
diff options
authorArd Biesheuvel <ard.biesheuvel@linaro.org>2017-07-22 12:48:34 +0100
committerMark Rutland <mark.rutland@arm.com>2017-08-08 16:28:26 +0100
commitc7365330753c55a061db0a1837a27fd5e44b1408 (patch)
tree496297bba1f31b647063b6d2c2440eebe4884521 /arch/arm64/include/asm/irq.h
parentarm64: unwind: avoid percpu indirection for irq stack (diff)
arm64: unwind: disregard frame.sp when validating frame pointer
Currently, when unwinding the call stack, we validate the frame pointer of each frame against frame.sp, whose value is not clearly defined, and which makes it more difficult to link stack frames together across different stacks. It is far better to simply check whether the frame pointer itself points into a valid stack. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Signed-off-by: Mark Rutland <mark.rutland@arm.com> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: James Morse <james.morse@arm.com> Cc: Will Deacon <will.deacon@arm.com>
Diffstat (limited to '')
1 files changed, 9 insertions, 1 deletions
diff --git a/arch/arm64/include/asm/irq.h b/arch/arm64/include/asm/irq.h
index 6d6f85e4923e..8155e486ce48 100644
--- a/arch/arm64/include/asm/irq.h
+++ b/arch/arm64/include/asm/irq.h
@@ -7,6 +7,7 @@
#ifndef __ASSEMBLER__
#include <linux/percpu.h>
+#include <linux/sched/task_stack.h>
#include <asm-generic/irq.h>
#include <asm/thread_info.h>
@@ -49,12 +50,19 @@ static inline int nr_legacy_irqs(void)
static inline bool on_irq_stack(unsigned long sp)
- /* variable names the same as kernel/stacktrace.c */
unsigned long low = (unsigned long)raw_cpu_ptr(irq_stack);
unsigned long high = low + IRQ_STACK_START_SP;
return (low <= sp && sp <= high);
+static inline bool on_task_stack(struct task_struct *tsk, unsigned long sp)
+ unsigned long low = (unsigned long)task_stack_page(tsk);
+ unsigned long high = low + THREAD_SIZE;
+ return (low <= sp && sp < high);
#endif /* !__ASSEMBLER__ */