On Thu, Jul 11, 2013 at 05:06:04PM +0800, Arthur Chunqi Li wrote: > When L2 exits to L1, segment infomations of L1 are not set correctly. > According to Intel SDM 27.5.2(Loading Host Segment and Descriptor > Table Registers), segment base/limit/access right of L1 should be > set to some designed value when L2 exits to L1. This patch fixes > this. > Does this fix your test case? > Signed-off-by: Arthur Chunqi Li <yzt356@xxxxxxxxx> > --- > arch/x86/kvm/vmx.c | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c > index 1200e4e..3cf702b 100644 > --- a/arch/x86/kvm/vmx.c > +++ b/arch/x86/kvm/vmx.c > @@ -8028,6 +8028,32 @@ static void load_vmcs12_host_state(struct kvm_vcpu *vcpu, > vmcs_write16(GUEST_GS_SELECTOR, vmcs12->host_gs_selector); > vmcs_write16(GUEST_TR_SELECTOR, vmcs12->host_tr_selector); > > + /* Set L1 segment info according to Intel SDM > + 27.5.2 Loading Host Segment and Descriptor-Table Registers */ > + vmcs_writel(GUEST_CS_BASE, 0x0); > + vmcs_writel(GUEST_SS_BASE, 0x0); > + vmcs_writel(GUEST_DS_BASE, 0x0); > + vmcs_writel(GUEST_ES_BASE, 0x0); > + vmcs_write32(GUEST_CS_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_SS_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_DS_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_ES_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_FS_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_GS_LIMIT, 0xFFFFFFFF); > + vmcs_write32(GUEST_TR_LIMIT, 0x67); > + vmcs_write32(GUEST_ES_AR_BYTES, 0xc093); > + if (vmcs12->vm_exit_controls & VM_EXIT_HOST_ADDR_SPACE_SIZE) > + vmcs_write32(GUEST_CS_AR_BYTES, 0xa09b); > + else > + vmcs_write32(GUEST_CS_AR_BYTES, 0xc09b); > + vmcs_write32(GUEST_SS_AR_BYTES, 0xc093); > + vmcs_write32(GUEST_DS_AR_BYTES, 0xc093); > + vmcs_write32(GUEST_FS_AR_BYTES, 0xc093); > + vmcs_write32(GUEST_GS_AR_BYTES, 0xc093); > + vmcs_write32(GUEST_TR_AR_BYTES, 0x8b); Please use AR_ defines for AR_BYTES fields (AR_S_MASK etc). > + vmcs_writel(GUEST_RFLAGS, 0x2); > + > + > if (vmcs12->vm_exit_controls & VM_EXIT_LOAD_IA32_PAT) > vmcs_write64(GUEST_IA32_PAT, vmcs12->host_ia32_pat); > if (vmcs12->vm_exit_controls & VM_EXIT_LOAD_IA32_PERF_GLOBAL_CTRL) > -- > 1.7.9.5 -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
