Skip to content

Commit ebc9cb0

Browse files
guoren83palmer-dabbelt
authored andcommitted
riscv: stack: Fixup independent softirq stack for CONFIG_FRAME_POINTER=n
The independent softirq stack uses s0 to save & restore sp, but s0 would be corrupted when CONFIG_FRAME_POINTER=n. So add s0 in the clobber list to fix the problem. Fixes: dd69d07 ("riscv: stack: Support HAVE_SOFTIRQ_ON_OWN_STACK") Cc: stable@vger.kernel.org Reported-by: Zhangjin Wu <falcon@tinylab.org> Signed-off-by: Guo Ren <guoren@linux.alibaba.com> Signed-off-by: Guo Ren <guoren@kernel.org> Tested-by: Drew Fustini <dfustini@baylibre.com> Link: https://lore.kernel.org/r/20230716001506.3506041-3-guoren@kernel.org Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
1 parent 8d0be64 commit ebc9cb0

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

arch/riscv/kernel/irq.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,9 @@ void do_softirq_own_stack(void)
8484
: [sp] "r" (sp)
8585
: "a0", "a1", "a2", "a3", "a4", "a5", "a6", "a7",
8686
"t0", "t1", "t2", "t3", "t4", "t5", "t6",
87+
#ifndef CONFIG_FRAME_POINTER
88+
"s0",
89+
#endif
8790
"memory");
8891
} else
8992
#endif

0 commit comments

Comments
 (0)