From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tim Deegan Subject: Re: [PATCH 0 of 2] Support for Paging/Sharing on AMD Date: Thu, 29 Mar 2012 16:42:43 +0100 Message-ID: <20120329154243.GI72859@ocelot.phlegethon.org> References: <20120329110357.GE72859@ocelot.phlegethon.org> <7e6f4eefa902b1d9bbfc918e29c867d7.squirrel@webmail.lagarcavilla.org> <20120329150134.GH72859@ocelot.phlegethon.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline 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: Andres Lagar-Cavilla Cc: olaf@aepfle.de, keir@xen.org, andres@gridcentric.ca, xen-devel@lists.xen.org, wei.wang2@amd.com, jbeulich@suse.com, adin@gridcentric.ca List-Id: xen-devel@lists.xenproject.org Hi, > diff -r 5a7e18e57c72 -r 228f7aebb0ff xen/arch/x86/mm/mem_event.c > --- a/xen/arch/x86/mm/mem_event.c > +++ b/xen/arch/x86/mm/mem_event.c > @@ -563,8 +563,11 @@ int mem_event_domctl(struct domain *d, x > if ( !hap_enabled(d) ) > break; > > - /* Currently only EPT is supported */ > - if ( boot_cpu_data.x86_vendor != X86_VENDOR_INTEL ) > + /* Currently EPT or AMD with no iommu/hap page table sharing are > + * supported. Further, AMD considered experimental atm. */ > + if ( !((boot_cpu_data.x86_vendor == X86_VENDOR_INTEL) || > + ((boot_cpu_data.x86_vendor == X86_VENDOR_AMD) && > + !iommu_use_hap_pt(d))) ) I think we can just drop the test for Intelness rather than adding one for AMDness. hap_enabled() above is good enough to avoid Cyrix &c. :) Also, I think the cset I just checked in to interlock IOMMU vs sharing/paging should be enough that we don't need the !iommu_use_hap_pt(d) test either (or is there some concern about the pagetable layout?) That is, can we just drop this whole test? The right place to mark it experimental is in the user docs and example config files. Cheers, Tim.