[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] nestedhvm: xentrace support


  • To: Christoph Egger <Christoph.Egger@xxxxxxx>
  • From: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
  • Date: Mon, 18 Apr 2011 11:22:15 +0100
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir@xxxxxxx>
  • Delivery-date: Mon, 18 Apr 2011 03:23:09 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; b=gwbgRZc9MsTfYXQ1QQJj9nlWNEEOKvmhe2Bvcln+SkkQGA3aAQSkluvpeuQ6AC6/rF d8F5EjOuICvySeXpBWWZt0XTchAZOWJLGthOmBX9ZujM+APBfKGnsav37PsVS1gVCjZg 9SfOqTIZ/IdkHju/mviSxsCOb9+wC1/WdXC2g=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

On Thu, Apr 14, 2011 at 3:36 PM, Christoph Egger
<Christoph.Egger@xxxxxxx> wrote:
>> Finally, George may have a
>> reason for not including RIP in VMENTRY records (e.g., not that useful
>> compared with the extra space taken up in trace records). So you'd need to
>> submit a new patch and get an Ack from George (who is currently on
>> holiday).

Yeah, the general principle is just to not log duplicate information,
or information which can be inferred.  VMENTRY logs happen a *lot*,
and so a small change has a non-negligible impact on trace size.  To
this point, I haven't needed the RIP on VMENTRY, since having it on
VMEXIT is generally sufficient.

> Ok. If he has one I would like to know it.
> For nested virtualization it is at least good debugging information to see
> if the l2 guest loops somewhere or not.

Can't you get this information from looking at the RIP on the
corresponding VMEXIT?

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.