Daniel Henrique Barboza <dbarb...@ventanamicro.com> writes: > We're changing 'mmu' to true regardless of whether the profile is > being enabled or not, and at the same time we're changing satp_mode to > profile->enabled. > > This will promote a situation where we'll set mmu=on without a virtual > memory mode, which is a mistake. > > Only touch 'mmu' and satp_mode if the profile is being enabled. > > Suggested-by: Andrew Jones <ajo...@ventanamicro.com> > Fixes: 55398025e7 ("target/riscv: add satp_mode profile support") > Signed-off-by: Daniel Henrique Barboza <dbarb...@ventanamicro.com>
Reviewed-by: Björn Töpel <bj...@rivosinc.com>