diff options
| author | Kees Cook <[email protected]> | 2022-03-09 14:09:39 -0800 |
|---|---|---|
| committer | Thomas Bogendoerfer <[email protected]> | 2022-03-14 15:02:53 +0100 |
| commit | b847bd64ea9f484510e27065cb2bccc58d9b829b (patch) | |
| tree | 6ee0d94d6ce3516b2047ecf0bfcab2148afcc273 /arch/mips/kernel | |
| parent | 4d409ca3e5107a1b300d0f36550d61c5da4fd8a7 (diff) | |
MIPS: Only use current_stack_pointer on GCC
Unfortunately, Clang did not have support for "sp" as a global register
definition, and was crashing after the addition of current_stack_pointer.
This has been fixed in Clang 14, but earlier Clang versions need to
avoid this code, so add a versioned test and revert back to the
open-coded asm instances. Fixes Clang build error:
fatal error: error in backend: Invalid register name global variable
Fixes: 200ed341b864 ("mips: Implement "current_stack_pointer"")
Reported-by: Nathan Chancellor <[email protected]>
Reviewed-by: Nathan Chancellor <[email protected]>
Link: https://lore.kernel.org/lkml/[email protected]
Cc: Thomas Bogendoerfer <[email protected]>
Cc: Marc Zyngier <[email protected]>
Cc: Guenter Roeck <[email protected]>
Cc: Yanteng Si <[email protected]>
Cc: [email protected]
Signed-off-by: Kees Cook <[email protected]>
Reviewed-by: Nick Desaulniers <[email protected]>
Signed-off-by: Thomas Bogendoerfer <[email protected]>
Diffstat (limited to 'arch/mips/kernel')
| -rw-r--r-- | arch/mips/kernel/irq.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/mips/kernel/irq.c b/arch/mips/kernel/irq.c index fc313c49a417..5e11582fe308 100644 --- a/arch/mips/kernel/irq.c +++ b/arch/mips/kernel/irq.c @@ -75,8 +75,9 @@ void __init init_IRQ(void) #ifdef CONFIG_DEBUG_STACKOVERFLOW static inline void check_stack_overflow(void) { - unsigned long sp = current_stack_pointer; + unsigned long sp; + __asm__ __volatile__("move %0, $sp" : "=r" (sp)); sp &= THREAD_MASK; /* |