[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/8] ppc/spapr: Allow FWNMI on TCG
From: |
Greg Kurz |
Subject: |
Re: [PATCH v2 5/8] ppc/spapr: Allow FWNMI on TCG |
Date: |
Mon, 16 Mar 2020 19:01:03 +0100 |
On Tue, 17 Mar 2020 00:26:10 +1000
Nicholas Piggin <address@hidden> wrote:
> There should no longer be a reason to prevent TCG providing FWNMI.
> System Reset interrupts are generated to the guest with nmi monitor
> command and H_SIGNAL_SYS_RESET. Machine Checks can not be injected
> currently, but this could be implemented with the mce monitor cmd
> similarly to i386.
>
> Signed-off-by: Nicholas Piggin <address@hidden>
> ---
> hw/ppc/spapr_caps.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/hw/ppc/spapr_caps.c b/hw/ppc/spapr_caps.c
> index f626d769a0..679ae7959f 100644
> --- a/hw/ppc/spapr_caps.c
> +++ b/hw/ppc/spapr_caps.c
> @@ -516,10 +516,7 @@ static void cap_fwnmi_apply(SpaprMachineState *spapr,
> uint8_t val,
> return; /* Disabled by default */
> }
>
> - if (tcg_enabled()) {
> - warn_report("Firmware Assisted Non-Maskable Interrupts(FWNMI) not "
> - "supported in TCG");
With this warning removed, we can now drop the "cap-fwnmi=off" setting
in qtest, but this can be done as a followup.
Reviewed-by: Greg Kurz <address@hidden>
> - } else if (kvm_enabled()) {
> + if (kvm_enabled()) {
> if (kvmppc_set_fwnmi() < 0) {
> error_setg(errp, "Firmware Assisted Non-Maskable
> Interrupts(FWNMI) "
> "not supported by KVM");
[PATCH v2 6/8] target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector, Nicholas Piggin, 2020/03/16
Re: [PATCH v2 6/8] target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector, David Gibson, 2020/03/16
[PATCH v2 7/8] ppc/spapr: Implement FWNMI System Reset delivery, Nicholas Piggin, 2020/03/16