| Message ID | 20240904120812.1798715-2-andrew.jones@linux.dev (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [kvm-unit-tests] riscv: Fix smp_boot_secondary | expand |
diff --git a/riscv/cstart.S b/riscv/cstart.S index a9ac72df4dd2..8f26999759da 100644 --- a/riscv/cstart.S +++ b/riscv/cstart.S @@ -130,9 +130,9 @@ secondary_entry: mv a0, sp call secondary_cinit addi sp, sp, SECONDARY_DATA_SIZE - jr a0 - la a0, do_idle - jr a0 + jalr ra, a0 + call do_idle + j . /* unreachable */ /* * Save context to address in a0.
The last few instructions of secondary_entry had the right concept, but were the totally wrong implementation. Without setting ra, then, when the boot function doesn't stay in an infinite loop, like do_idle() would, we'd go off into the weeds when trying to return from it. Make sure we set ra to come back to where we can then call do_idle() instead. The bug was found by inspection since nobody is calling smp_boot_secondary() with anything other than do_idle() at this time. Fixes: 9c92b28e6b7b ("riscv: Add SMP support") Signed-off-by: Andrew Jones <andrew.jones@linux.dev> --- riscv/cstart.S | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)