| From 46b4b428b523d1e1276c23969bbdde4ad654e1e5 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Thu, 12 Jan 2023 04:05:57 -0500 |
| Subject: riscv: ftrace: Fixup panic by disabling preemption |
| |
| From: Andy Chiu <andy.chiu@sifive.com> |
| |
| [ Upstream commit 8547649981e6631328cd64f583667501ae385531 ] |
| |
| In RISCV, we must use an AUIPC + JALR pair to encode an immediate, |
| forming a jump that jumps to an address over 4K. This may cause errors |
| if we want to enable kernel preemption and remove dependency from |
| patching code with stop_machine(). For example, if a task was switched |
| out on auipc. And, if we changed the ftrace function before it was |
| switched back, then it would jump to an address that has updated 11:0 |
| bits mixing with previous XLEN:12 part. |
| |
| p: patched area performed by dynamic ftrace |
| ftrace_prologue: |
| p| REG_S ra, -SZREG(sp) |
| p| auipc ra, 0x? ------------> preempted |
| ... |
| change ftrace function |
| ... |
| p| jalr -?(ra) <------------- switched back |
| p| REG_L ra, -SZREG(sp) |
| func: |
| xxx |
| ret |
| |
| Fixes: afc76b8b8011 ("riscv: Using PATCHABLE_FUNCTION_ENTRY instead of MCOUNT") |
| Signed-off-by: Andy Chiu <andy.chiu@sifive.com> |
| Signed-off-by: Guo Ren <guoren@kernel.org> |
| Link: https://lore.kernel.org/r/20230112090603.1295340-2-guoren@kernel.org |
| Cc: stable@vger.kernel.org |
| Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| arch/riscv/Kconfig | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig |
| index ae11d5647f9d4..06b9b2f60b9fb 100644 |
| --- a/arch/riscv/Kconfig |
| +++ b/arch/riscv/Kconfig |
| @@ -278,7 +278,7 @@ config ARCH_RV64I |
| select HAVE_DYNAMIC_FTRACE_WITH_REGS if HAVE_DYNAMIC_FTRACE |
| select HAVE_FTRACE_MCOUNT_RECORD if !XIP_KERNEL |
| select HAVE_FUNCTION_GRAPH_TRACER |
| - select HAVE_FUNCTION_TRACER if !XIP_KERNEL |
| + select HAVE_FUNCTION_TRACER if !XIP_KERNEL && !PREEMPTION |
| select SWIOTLB if MMU |
| |
| endchoice |
| -- |
| 2.39.2 |
| |