On 19/10/18 15:28, Wei Liu wrote: > The symbol will not be available when PV is disabled. > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > --- > v2: new > --- > xen/arch/x86/hvm/vmx/vmcs.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/xen/arch/x86/hvm/vmx/vmcs.c b/xen/arch/x86/hvm/vmx/vmcs.c > index d9747b4..282677a 100644 > --- a/xen/arch/x86/hvm/vmx/vmcs.c > +++ b/xen/arch/x86/hvm/vmx/vmcs.c > @@ -1159,9 +1159,16 @@ static int construct_vmcs(struct vcpu *v) > __vmwrite(HOST_CS_SELECTOR, __HYPERVISOR_CS); > __vmwrite(HOST_RIP, (unsigned long)vmx_asm_vmexit_handler); > > +#ifdef CONFIG_PV > /* Host SYSENTER CS:RIP. */ > __vmwrite(HOST_SYSENTER_CS, __HYPERVISOR_CS); > __vmwrite(HOST_SYSENTER_EIP, (unsigned long)sysenter_entry); > +#else > + /* > + * Should something be put here for debugging purpose? We never > + * set it up in the first place.
With MSR_SYSENTER_CS set to 0, SYSENTER/SYSEXIT instructions automatically #GP[0]. I don't think we need anything here in the !PV case. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
