File 5b34b8fe-VMX-defer-vmx_vmcs_exit-as-long-as-possible.patch of Package xen.8389
# Commit f30e3cf34042846e391e3f8361fc6a76d181a7ee
# Date 2018-06-28 11:31:26 +0100
# Author Andrew Cooper <andrew.cooper3@citrix.com>
# Committer Andrew Cooper <andrew.cooper3@citrix.com>
x86/vmx: Defer vmx_vmcs_exit() as long as possible in construct_vmcs()
paging_update_paging_modes() and vmx_vlapic_msr_changed() both operate on the
VMCS being constructed. Avoid dropping and re-acquiring the reference
multiple times.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Kevin Tian <kevin.tian@intel.com>
--- a/xen/arch/x86/hvm/vmx/vmcs.c
+++ b/xen/arch/x86/hvm/vmx/vmcs.c
@@ -999,6 +999,7 @@ static int construct_vmcs(struct vcpu *v
unsigned long sysenter_eip;
u32 vmexit_ctl = vmx_vmexit_control;
u32 vmentry_ctl = vmx_vmentry_control;
+ int rc = 0;
vmx_vmcs_enter(v);
@@ -1086,8 +1087,8 @@ static int construct_vmcs(struct vcpu *v
if ( msr_bitmap == NULL )
{
- vmx_vmcs_exit(v);
- return -ENOMEM;
+ rc = -ENOMEM;
+ goto out;
}
memset(msr_bitmap, ~0, PAGE_SIZE);
@@ -1271,14 +1272,15 @@ static int construct_vmcs(struct vcpu *v
if ( cpu_has_vmx_tsc_scaling )
__vmwrite(TSC_MULTIPLIER, d->arch.hvm_domain.tsc_scaling_ratio);
- vmx_vmcs_exit(v);
-
/* will update HOST & GUEST_CR3 as reqd */
paging_update_paging_modes(v);
vmx_vlapic_msr_changed(v);
- return 0;
+ out:
+ vmx_vmcs_exit(v);
+
+ return rc;
}
static int vmx_msr_entry_key_cmp(const void *key, const void *elt)