diff options
| author | Peng Hao <[email protected]> | 2022-02-22 18:40:54 +0800 |
|---|---|---|
| committer | Paolo Bonzini <[email protected]> | 2022-02-25 08:20:13 -0500 |
| commit | fbc2dfe53a222619b01f451f5bfc206738d53854 (patch) | |
| tree | 08b1c43746eab7ec31ec6953cd01485a4b4dc053 | |
| parent | c0f1eaeb9e628bf86bf50f11cb4a2b671528391e (diff) | |
KVM: nVMX: Make setup/unsetup under the same conditions
Make sure nested_vmx_hardware_setup/unsetup() are called in pairs under
the same conditions. Calling nested_vmx_hardware_unsetup() when nested
is false "works" right now because it only calls free_page() on zero-
initialized pointers, but it's possible that more code will be added to
nested_vmx_hardware_unsetup() in the future.
Reviewed-by: Sean Christopherson <[email protected]>
Signed-off-by: Peng Hao <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
| -rw-r--r-- | arch/x86/kvm/vmx/vmx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c index d8547144d3b7..e1b3743fb1b0 100644 --- a/arch/x86/kvm/vmx/vmx.c +++ b/arch/x86/kvm/vmx/vmx.c @@ -8049,7 +8049,7 @@ static __init int hardware_setup(void) vmx_set_cpu_caps(); r = alloc_kvm_area(); - if (r) + if (r && nested) nested_vmx_hardware_unsetup(); kvm_set_posted_intr_wakeup_handler(pi_wakeup_handler); |