mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-17 07:02:03 -06:00
target/riscv: remove warnings about Smdbltrp/Smrnmi being disabled
As raised by Richard Henderson, these warnings are displayed in user only as well. Since they aren't really useful for the end-user, remove them and add a "TODO" note in the leading comments. Signed-off-by: Clément Léger <cleger@rivosinc.com> Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com> Message-ID: <20250213145640.117275-1-cleger@rivosinc.com> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
This commit is contained in:
parent
abe9b81ee4
commit
cb0c476026
1 changed files with 3 additions and 5 deletions
|
@ -1458,22 +1458,20 @@ static void riscv_init_max_cpu_extensions(Object *obj)
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* ext_smrnmi requires OpenSBI changes that our current
|
* TODO: ext_smrnmi requires OpenSBI changes that our current
|
||||||
* image does not have. Disable it for now.
|
* image does not have. Disable it for now.
|
||||||
*/
|
*/
|
||||||
if (cpu->cfg.ext_smrnmi) {
|
if (cpu->cfg.ext_smrnmi) {
|
||||||
isa_ext_update_enabled(cpu, CPU_CFG_OFFSET(ext_smrnmi), false);
|
isa_ext_update_enabled(cpu, CPU_CFG_OFFSET(ext_smrnmi), false);
|
||||||
qemu_log("Smrnmi is disabled in the 'max' type CPU\n");
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* ext_smdbltrp requires the firmware to clear MSTATUS.MDT on startup to
|
* TODO: ext_smdbltrp requires the firmware to clear MSTATUS.MDT on startup
|
||||||
* avoid generating a double trap. OpenSBI does not currently support it,
|
* to avoid generating a double trap. OpenSBI does not currently support it,
|
||||||
* disable it for now.
|
* disable it for now.
|
||||||
*/
|
*/
|
||||||
if (cpu->cfg.ext_smdbltrp) {
|
if (cpu->cfg.ext_smdbltrp) {
|
||||||
isa_ext_update_enabled(cpu, CPU_CFG_OFFSET(ext_smdbltrp), false);
|
isa_ext_update_enabled(cpu, CPU_CFG_OFFSET(ext_smdbltrp), false);
|
||||||
qemu_log("Smdbltrp is disabled in the 'max' type CPU\n");
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue