| Message ID | 20231211110331.359534-1-songshuaishuai@tinylab.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | a7565f4d068b2e60f95c3223c3167c40b8fe83ae |
| Headers | show |
| Series | [--fixes] riscv: Remove SHADOW_OVERFLOW_STACK_SIZE macro | expand |
Hi, On Mon, Dec 11, 2023 at 3:04 AM Song Shuai <songshuaishuai@tinylab.org> wrote: > > The commit be97d0db5f44 ("riscv: VMAP_STACK overflow > detection thread-safe") got rid of `shadow_stack`, > so SHADOW_OVERFLOW_STACK_SIZE should be removed too. > > Fixes: be97d0db5f44 ("riscv: VMAP_STACK overflow detection thread-safe") > Signed-off-by: Song Shuai <songshuaishuai@tinylab.org> > --- > arch/riscv/include/asm/thread_info.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/riscv/include/asm/thread_info.h b/arch/riscv/include/asm/thread_info.h > index 574779900bfb..4856697c5f25 100644 > --- a/arch/riscv/include/asm/thread_info.h > +++ b/arch/riscv/include/asm/thread_info.h > @@ -28,7 +28,6 @@ > > #define THREAD_SHIFT (PAGE_SHIFT + THREAD_SIZE_ORDER) > #define OVERFLOW_STACK_SIZE SZ_4K > -#define SHADOW_OVERFLOW_STACK_SIZE (1024) > > #define IRQ_STACK_SIZE THREAD_SIZE Ah, good catch. Thanks for the patch! Reviewed-by: Sami Tolvanen <samitolvanen@google.com> Sami
Hello: This patch was applied to riscv/linux.git (for-next) by Palmer Dabbelt <palmer@rivosinc.com>: On Mon, 11 Dec 2023 19:03:31 +0800 you wrote: > The commit be97d0db5f44 ("riscv: VMAP_STACK overflow > detection thread-safe") got rid of `shadow_stack`, > so SHADOW_OVERFLOW_STACK_SIZE should be removed too. > > Fixes: be97d0db5f44 ("riscv: VMAP_STACK overflow detection thread-safe") > Signed-off-by: Song Shuai <songshuaishuai@tinylab.org> > > [...] Here is the summary with links: - [--fixes] riscv: Remove SHADOW_OVERFLOW_STACK_SIZE macro https://git.kernel.org/riscv/c/a7565f4d068b You are awesome, thank you!
diff --git a/arch/riscv/include/asm/thread_info.h b/arch/riscv/include/asm/thread_info.h index 574779900bfb..4856697c5f25 100644 --- a/arch/riscv/include/asm/thread_info.h +++ b/arch/riscv/include/asm/thread_info.h @@ -28,7 +28,6 @@ #define THREAD_SHIFT (PAGE_SHIFT + THREAD_SIZE_ORDER) #define OVERFLOW_STACK_SIZE SZ_4K -#define SHADOW_OVERFLOW_STACK_SIZE (1024) #define IRQ_STACK_SIZE THREAD_SIZE
The commit be97d0db5f44 ("riscv: VMAP_STACK overflow detection thread-safe") got rid of `shadow_stack`, so SHADOW_OVERFLOW_STACK_SIZE should be removed too. Fixes: be97d0db5f44 ("riscv: VMAP_STACK overflow detection thread-safe") Signed-off-by: Song Shuai <songshuaishuai@tinylab.org> --- arch/riscv/include/asm/thread_info.h | 1 - 1 file changed, 1 deletion(-)