KVM: PPC: Book3S HV: XIVE: Set kvm->arch.xive when VPs are allocated
authorGreg Kurz <groug@kaod.org>
Fri, 27 Sep 2019 11:53:38 +0000 (13:53 +0200)
committerPaul Mackerras <paulus@ozlabs.org>
Tue, 22 Oct 2019 05:29:01 +0000 (16:29 +1100)
commite7d71c943040c23f2fd042033d319f56e84f845b
tree180a7725bd68112891d836c8d73fa0a6ece49b03
parentf41c4989c8de1fa70aafe950abaf80c56a8b8712
KVM: PPC: Book3S HV: XIVE: Set kvm->arch.xive when VPs are allocated

If we cannot allocate the XIVE VPs in OPAL, the creation of a XIVE or
XICS-on-XIVE device is aborted as expected, but we leave kvm->arch.xive
set forever since the release method isn't called in this case. Any
subsequent tentative to create a XIVE or XICS-on-XIVE for this VM will
thus always fail (DoS). This is a problem for QEMU since it destroys
and re-creates these devices when the VM is reset: the VM would be
restricted to using the much slower emulated XIVE or XICS forever.

As an alternative to adding rollback, do not assign kvm->arch.xive before
making sure the XIVE VPs are allocated in OPAL.

Cc: stable@vger.kernel.org # v5.2
Fixes: 5422e95103cf ("KVM: PPC: Book3S HV: XIVE: Replace the 'destroy' method by a 'release' method")
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Signed-off-by: Paul Mackerras <paulus@ozlabs.org>
arch/powerpc/kvm/book3s_xive.c
arch/powerpc/kvm/book3s_xive_native.c