From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save Date: Fri, 19 Feb 2016 17:18:46 +0000 Message-ID: <56C74E76.5090008@citrix.com> References: <1455563063-17012-1-git-send-email-tlengyel@novetta.com> <56C30C5802000078000D28E0@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7683246983729882753==" Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aWohp-0004rE-TH for xen-devel@lists.xenproject.org; Fri, 19 Feb 2016 17:19:02 +0000 In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Lengyel, Tamas" , Jan Beulich Cc: "xen-devel@lists.xenproject.org" , Kevin Tian , Keir Fraser , Jun Nakajima List-Id: xen-devel@lists.xenproject.org --===============7683246983729882753== Content-Type: multipart/alternative; boundary="------------020502080801040401030703" --------------020502080801040401030703 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit On 19/02/16 17:06, Lengyel, Tamas wrote: > > > On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich > wrote: > > >>> On 16.02.16 at 07:58, > 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? I specifically suggested the use of vmx_save_dr() to make all debug state consistent. I don't see much purpose in being able to introspect just %dr7. If any debug related activities are going on, all debug registers are relevant. Is this not the case? ~Andrew --------------020502080801040401030703 Content-Type: text/html; charset="windows-1252" Content-Length: 3472 Content-Transfer-Encoding: quoted-printable
On 19/02/16 17:06, Lengyel, Tamas wrote:


On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>> On 16.02.16 at 07:58, <kevin.tian@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)
>>=A0 =A0 =A0 __vmread(GUEST_SYSENTER_CS, &c->sysenter_cs);
>>=A0 =A0 =A0 __vmread(GUEST_SYSENTER_ESP, &c->sysenter_esp);
>>=A0 =A0 =A0 __vmread(GUEST_SYSENTER_EIP, &c->sysenter_eip);
>> +=A0 =A0 __vmread(GUEST_DR7, &c->dr7);
>>
>
> Hi, Tamas, I didn't see the open closed around "v !=3D current", if
> I'm not missing some mails... Have you confirmed with Jan that
> he is OK with it=3F

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=3F

I specifically suggested the use of vmx_save_dr() to make all debug state consistent.

I don't see much purpose in being able to introspect just %dr7.=A0 If any debug related activities are going on, all debug registers are relevant.

Is this not the case=3F

~Andrew
--------------020502080801040401030703-- --===============7683246983729882753== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============7683246983729882753==--