[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/2] hw/riscv: Make CPU config error handling generous (vi
From: |
Alistair Francis |
Subject: |
Re: [PATCH v2 1/2] hw/riscv: Make CPU config error handling generous (virt/spike) |
Date: |
Tue, 17 May 2022 10:36:06 +1000 |
On Sat, May 14, 2022 at 4:29 PM Tsukasa OI <research_trasio@irq.a4lg.com> wrote:
>
> If specified CPU configuration is not valid, not just it prints error
> message, it aborts and generates core dumps (depends on the operating
> system). This kind of error handling should be used only when a serious
> runtime error occurs.
>
> This commit makes error handling on CPU configuration more generous on
> virt/spike machines. It now just prints error message and quits (without
> coredumps and aborts).
>
> Signed-off-by: Tsukasa OI <research_trasio@irq.a4lg.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> hw/riscv/spike.c | 2 +-
> hw/riscv/virt.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/riscv/spike.c b/hw/riscv/spike.c
> index 068ba3493e..e41b6aa9f0 100644
> --- a/hw/riscv/spike.c
> +++ b/hw/riscv/spike.c
> @@ -230,7 +230,7 @@ static void spike_board_init(MachineState *machine)
> base_hartid, &error_abort);
> object_property_set_int(OBJECT(&s->soc[i]), "num-harts",
> hart_count, &error_abort);
> - sysbus_realize(SYS_BUS_DEVICE(&s->soc[i]), &error_abort);
> + sysbus_realize(SYS_BUS_DEVICE(&s->soc[i]), &error_fatal);
>
> /* Core Local Interruptor (timer and IPI) for each socket */
> riscv_aclint_swi_create(
> diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> index 3326f4db96..244d6408b5 100644
> --- a/hw/riscv/virt.c
> +++ b/hw/riscv/virt.c
> @@ -1351,7 +1351,7 @@ static void virt_machine_init(MachineState *machine)
> base_hartid, &error_abort);
> object_property_set_int(OBJECT(&s->soc[i]), "num-harts",
> hart_count, &error_abort);
> - sysbus_realize(SYS_BUS_DEVICE(&s->soc[i]), &error_abort);
> + sysbus_realize(SYS_BUS_DEVICE(&s->soc[i]), &error_fatal);
>
> if (!kvm_enabled()) {
> if (s->have_aclint) {
> --
> 2.34.1
>