diff options
| author | Greg Kurz <groug@kaod.org> | 2017-09-21 18:00:53 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2017-10-02 14:38:06 +0200 |
| commit | 62dd4edaaf859b60f74a51f2a526d4d3d85d0248 (patch) | |
| tree | 253f8f5e5d2e3b5d40100a31944c40d0ede62107 /accel/stubs | |
| parent | d4083f50e021f52addd02c9f83465937f8486556 (diff) | |
| download | qemu-62dd4edaaf859b60f74a51f2a526d4d3d85d0248.zip qemu-62dd4edaaf859b60f74a51f2a526d4d3d85d0248.tar.gz qemu-62dd4edaaf859b60f74a51f2a526d4d3d85d0248.tar.bz2 | |
kvm: check KVM_CAP_SYNC_MMU with kvm_vm_check_extension()
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.
While here, let's cache the state of this extension in a bool variable,
since it has several users in the code, as suggested by Thomas Huth.
Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <150600965332.30533.14702405809647835716.stgit@bahia.lan>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'accel/stubs')
| -rw-r--r-- | accel/stubs/kvm-stub.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/accel/stubs/kvm-stub.c b/accel/stubs/kvm-stub.c index 3965c52..c964af3 100644 --- a/accel/stubs/kvm-stub.c +++ b/accel/stubs/kvm-stub.c @@ -64,9 +64,9 @@ int kvm_cpu_exec(CPUState *cpu) abort(); } -int kvm_has_sync_mmu(void) +bool kvm_has_sync_mmu(void) { - return 0; + return false; } int kvm_has_many_ioeventfds(void) |