aboutsummaryrefslogtreecommitdiffstats
path: root/arch/x86/kvm/vmx.c
diff options
context:
space:
mode:
authorJan Kiszka <jan.kiszka@siemens.com>2013-04-08 05:07:46 -0400
committerGleb Natapov <gleb@redhat.com>2013-04-08 05:46:06 -0400
commita63cb56061239372fce2452dbedd35c95bd665aa (patch)
tree17740814ec573f67cf7191ef7fb1b7ccd0a2b0a4 /arch/x86/kvm/vmx.c
parent458f212e36b291067b74c0cac2bdcf8278817ee7 (diff)
KVM: VMX: Add missing braces to avoid redundant error check
The code was already properly aligned, now also add the braces to avoid that err is checked even if alloc_apic_access_page didn't run and change it. Found via Coccinelle by Fengguang Wu. Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com> Signed-off-by: Gleb Natapov <gleb@redhat.com>
Diffstat (limited to 'arch/x86/kvm/vmx.c')
-rw-r--r--arch/x86/kvm/vmx.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
index 1cf202cb47d5..669b80378731 100644
--- a/arch/x86/kvm/vmx.c
+++ b/arch/x86/kvm/vmx.c
@@ -6785,10 +6785,11 @@ static struct kvm_vcpu *vmx_create_vcpu(struct kvm *kvm, unsigned int id)
6785 put_cpu(); 6785 put_cpu();
6786 if (err) 6786 if (err)
6787 goto free_vmcs; 6787 goto free_vmcs;
6788 if (vm_need_virtualize_apic_accesses(kvm)) 6788 if (vm_need_virtualize_apic_accesses(kvm)) {
6789 err = alloc_apic_access_page(kvm); 6789 err = alloc_apic_access_page(kvm);
6790 if (err) 6790 if (err)
6791 goto free_vmcs; 6791 goto free_vmcs;
6792 }
6792 6793
6793 if (enable_ept) { 6794 if (enable_ept) {
6794 if (!kvm->arch.ept_identity_map_addr) 6795 if (!kvm->arch.ept_identity_map_addr)