[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: Keir Fraser <keir.xen@xxxxxxxxx>
  • Date: Thu, 14 Apr 2011 15:49:51 +0100
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 14 Apr 2011 07:51:23 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=user-agent:date:subject:from:to:cc:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=n7Nou2izhG+OlRWIqQ8x0varYs90cxaRjunrCXPHYcG5DkD7zehTkH8PAMF18OxwDO nzPzvZMr3siFWefM/UdGABIrV14NhMgwUyszXgLM6F5kZJUeX5ZKWauRflATzVJqzNKU MdwwH5MKo1aJBE4cPVrX8uLvew4FIAxN2cqRU=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Acv6szVFpOnw74wE5ECr5QuMErIN5Q==
  • Thread-topic: [Xen-devel] [PATCH] nestedhvm: xentrace support

On 14/04/2011 15:36, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

>> I also discarded the change to the VMENTRY record to add the guest EIP.
>> Firstly, it was incomplete, because it did not handle 64-bit RIP. Secondly,
>> it does not logically belong in this patch. 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).
> 
> 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.

He might at least agree to it in the L2 case only. Anyhow, that's his call
not mine.

 -- Keir



_______________________________________________
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®.