On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich <jbeul...@suse.com> wrote:
> >>> On 16.02.16 at 07:58, <kevin.t...@intel.com> wrote: > >> --- a/xen/arch/x86/hvm/vmx/vmx.c > >> +++ b/xen/arch/x86/hvm/vmx/vmx.c > >> @@ -490,6 +490,7 @@ static void vmx_vmcs_save(struct vcpu *v, struct > hvm_hw_cpu > >> *c) > >> __vmread(GUEST_SYSENTER_CS, &c->sysenter_cs); > >> __vmread(GUEST_SYSENTER_ESP, &c->sysenter_esp); > >> __vmread(GUEST_SYSENTER_EIP, &c->sysenter_eip); > >> + __vmread(GUEST_DR7, &c->dr7); > >> > > > > Hi, Tamas, I didn't see the open closed around "v != current", if > > I'm not missing some mails... Have you confirmed with Jan that > > he is OK with it? > > We didn't really settle on this yet. I'm not heavily opposed to it > remaining unconditional for now, but as said in the other replay > my fear is that this might later lead to further additions which > may then also be of no interest to the main (save/migration) > user of this code. > Andrew, any comment if this is OK from your perspective? Thanks, Tamas
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
