Re: [Xen-devel] [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 16.02.16 at 07:58, <kevin.tian@xxxxxxxxx> 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.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel