RE: [PATCH 4/4 v3] KVM: VMX: VMXON/VMXOFF usage changes.

2010-05-11 Thread Xu, Dongxiao
Avi Kivity wrote: > On 05/11/2010 12:38 PM, Xu, Dongxiao wrote: >> >>> I'm not sure why this is needed. vmx_cpuid_update() is called from >>> a vcpu ioctl which should have called vcpu_load() before. >>> >>> Apart from that, everything looks good for merging. >>> >> Vcpu_load() and vcpu_put() i

Re: [PATCH 4/4 v3] KVM: VMX: VMXON/VMXOFF usage changes.

2010-05-11 Thread Avi Kivity
On 05/11/2010 12:38 PM, Xu, Dongxiao wrote: I'm not sure why this is needed. vmx_cpuid_update() is called from a vcpu ioctl which should have called vcpu_load() before. Apart from that, everything looks good for merging. Vcpu_load() and vcpu_put() is not called in that ioctl. I will ad

RE: [PATCH 4/4 v3] KVM: VMX: VMXON/VMXOFF usage changes.

2010-05-11 Thread Xu, Dongxiao
Avi Kivity wrote: > On 05/07/2010 05:43 AM, Xu, Dongxiao wrote: >> From: Dongxiao Xu >> >> SDM suggests VMXON should be called before VMPTRLD, and VMXOFF >> should be called after doing VMCLEAR. >> >> Therefore in vmm coexistence case, we should firstly call VMXON >> before any VMCS operation, an

Re: [PATCH 4/4 v3] KVM: VMX: VMXON/VMXOFF usage changes.

2010-05-11 Thread Avi Kivity
On 05/07/2010 05:43 AM, Xu, Dongxiao wrote: From: Dongxiao Xu SDM suggests VMXON should be called before VMPTRLD, and VMXOFF should be called after doing VMCLEAR. Therefore in vmm coexistence case, we should firstly call VMXON before any VMCS operation, and then call VMXOFF after the operation

[PATCH 4/4 v3] KVM: VMX: VMXON/VMXOFF usage changes.

2010-05-06 Thread Xu, Dongxiao
From: Dongxiao Xu SDM suggests VMXON should be called before VMPTRLD, and VMXOFF should be called after doing VMCLEAR. Therefore in vmm coexistence case, we should firstly call VMXON before any VMCS operation, and then call VMXOFF after the operation is done. Signed-off-by: Dongxiao Xu --- ar