riscv: Fix early ftrace nop patching
authorAlexandre Ghiti <alexghiti@rivosinc.com>
Thu, 23 May 2024 11:51:34 +0000 (13:51 +0200)
committerPalmer Dabbelt <palmer@rivosinc.com>
Thu, 23 May 2024 15:22:17 +0000 (08:22 -0700)
commit6ca445d8af0ed5950ebf899415fd6bfcd7d9d7a3
tree0927a03832641e507c08fbec6ed867cae878fc1c
parent46cad6cd9b10ab14acf20e0779998f88c6e44c4f
riscv: Fix early ftrace nop patching

Commit c97bf629963e ("riscv: Fix text patching when IPI are used")
converted ftrace_make_nop() to use patch_insn_write() which does not
emit any icache flush relying entirely on __ftrace_modify_code() to do
that.

But we missed that ftrace_make_nop() was called very early directly when
converting mcount calls into nops (actually on riscv it converts 2B nops
emitted by the compiler into 4B nops).

This caused crashes on multiple HW as reported by Conor and Björn since
the booting core could have half-patched instructions in its icache
which would trigger an illegal instruction trap: fix this by emitting a
local flush icache when early patching nops.

Fixes: c97bf629963e ("riscv: Fix text patching when IPI are used")
Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Reported-by: Conor Dooley <conor.dooley@microchip.com>
Tested-by: Conor Dooley <conor.dooley@microchip.com>
Reviewed-by: Björn Töpel <bjorn@rivosinc.com>
Tested-by: Björn Töpel <bjorn@rivosinc.com>
Link: https://lore.kernel.org/r/20240523115134.70380-1-alexghiti@rivosinc.com
Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
arch/riscv/include/asm/cacheflush.h
arch/riscv/kernel/ftrace.c