diff options
| author | Nicholas Piggin | 2020-03-25 15:29:03 +0100 |
|---|---|---|
| committer | David Gibson | 2020-04-07 00:55:10 +0200 |
| commit | ec010c00665ba1e78e6b3df104f923c4ea68504a (patch) | |
| tree | fd21867008d11d44d80364a2bf6359049d3a7e47 /hw/ppc/spapr_caps.c | |
| parent | vfio/spapr: Fix page size calculation (diff) | |
| download | qemu-ec010c00665ba1e78e6b3df104f923c4ea68504a.tar.gz qemu-ec010c00665ba1e78e6b3df104f923c4ea68504a.tar.xz qemu-ec010c00665ba1e78e6b3df104f923c4ea68504a.zip | |
ppc/spapr: KVM FWNMI should not be enabled until guest requests it
The KVM FWNMI capability should be enabled with the "ibm,nmi-register"
rtas call. Although MCEs from KVM will be delivered as architected
interrupts to the guest before "ibm,nmi-register" is called, KVM has
different behaviour depending on whether the guest has enabled FWNMI
(it attempts to do more recovery on behalf of a non-FWNMI guest).
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Message-Id: <20200325142906.221248-2-npiggin@gmail.com>
Reviewed-by: Greg Kurz <groug@kaod.org>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'hw/ppc/spapr_caps.c')
| -rw-r--r-- | hw/ppc/spapr_caps.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/hw/ppc/spapr_caps.c b/hw/ppc/spapr_caps.c index 679ae7959f..eb54f94227 100644 --- a/hw/ppc/spapr_caps.c +++ b/hw/ppc/spapr_caps.c @@ -517,9 +517,10 @@ static void cap_fwnmi_apply(SpaprMachineState *spapr, uint8_t val, } if (kvm_enabled()) { - if (kvmppc_set_fwnmi() < 0) { - error_setg(errp, "Firmware Assisted Non-Maskable Interrupts(FWNMI) " - "not supported by KVM"); + if (!kvmppc_get_fwnmi()) { + error_setg(errp, +"Firmware Assisted Non-Maskable Interrupts(FWNMI) not supported by KVM."); + error_append_hint(errp, "Try appending -machine cap-fwnmi=off\n"); } } } |