On Thu, Sep 14, 2017 at 09:25:22PM +0200, Greg Kurz wrote: > On a server-class ppc host, this capability depends on the KVM type, > ie, HV or PR. If both KVM are present in the kernel, we will always > get the HV specific value, even if we explicitely requested PR on > the command line. > > This can have an impact if we're using hugepages or a balloon device. > > Since we've already created the VM at the time any user calls > kvm_has_sync_mmu(), switching to kvm_vm_check_extension() is > enough to fix any potential issue. > > It is okay for the other archs that also implement KVM_CAP_SYNC_MMU, > ie, mips, s390, x86 and arm, because they don't depend on the VM being > created or not. > > Signed-off-by: Greg Kurz <gr...@kaod.org>
Reviewed-by: David Gibson <da...@gibson.dropbear.id.au> > --- > accel/kvm/kvm-all.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c > index f85553a85194..323c567cfb68 100644 > --- a/accel/kvm/kvm-all.c > +++ b/accel/kvm/kvm-all.c > @@ -2234,7 +2234,7 @@ int kvm_device_access(int fd, int group, uint64_t attr, > /* Return 1 on success, 0 on failure */ > int kvm_has_sync_mmu(void) > { > - return kvm_check_extension(kvm_state, KVM_CAP_SYNC_MMU); > + return kvm_vm_check_extension(kvm_state, KVM_CAP_SYNC_MMU); > } > > int kvm_has_vcpu_events(void) > -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature