diff options
author | Jan Kiszka <jan.kiszka@siemens.com> | 2013-04-08 05:07:46 -0400 |
---|---|---|
committer | Gleb Natapov <gleb@redhat.com> | 2013-04-08 05:46:06 -0400 |
commit | a63cb56061239372fce2452dbedd35c95bd665aa (patch) | |
tree | 17740814ec573f67cf7191ef7fb1b7ccd0a2b0a4 /arch/x86/kvm/vmx.c | |
parent | 458f212e36b291067b74c0cac2bdcf8278817ee7 (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.c | 3 |
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) |