Re: [PATCH 0 of 3] RFC Paging support for AMD NPT V2

[Hongkaixing <hongkaixing@huawei.com>]

> -----Original Message-----
> From: Andres Lagar-Cavilla [mailto:andres@lagarcavilla.org]
> Sent: Thursday, March 01, 2012 11:16 AM
> To: xen-devel@lists.xensource.com
> Cc: tim@xen.org; andres@gridcentric.ca; olaf@aepfle.de; adin@gridcentric.ca; wei.wang2@amd.com; hongkaixing@huawei.com
> Subject: [PATCH 0 of 3] RFC Paging support for AMD NPT V2
> 
> There has been some progress, but still no joy. Definitely not intended for
> inclusion at this point.
> 
> Tim, Wei, I added a Xen command line toggle to disable IOMMU and P2M table
> sharing.
> 
> Tim, I verified that changes to p2m-pt.c don't break shadow mode (64bit
> hypervisor and Win 7 guest).
> 
> Hongkaixing, I incorporated your suggestion in patch 2, so I should add your
> Signed-off-by eventually. Please review.

  I have checked the code, it looks like OK. But I don't have AMD machine to run it.
  We used almost the same code in a AMD server months before, and it worked fine. 
  The only difference is we have not changed the clipped invalid mfn to INVALID_MFN, just like EPT.
  

> 
> Olaf, I do not see errors when mapping pages prior to eviction. Let me know how
> it goes for you.
> 
> If I unleash xenpaging on a domain for a non-ambitious target (64 MiBs paged
> out for one minute), we sail by just fine. But if I kick things up a notch,
> still dying on an VMEXIT_SHUTDOWN (a.k.a. triple fault). Not sure at all what
> to do next.
> 
> Signed-off-by: Andres Lagar-Cavilla <andres@lagarcavilla.org>
> Signed-off-by: Adin Scannell <adin@scannell.ca>
> 
>  xen/drivers/passthrough/iommu.c |   2 +
>  xen/arch/x86/mm/p2m-pt.c        |  56 ++++++++++++++++++++++++++++------------
>  xen/arch/x86/mm/mem_event.c     |   7 +++-
>  xen/arch/x86/mm/mem_sharing.c   |   7 +++++
>  4 files changed, 53 insertions(+), 19 deletions(-)