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:01:34 +0100 Message-ID: <20120329150134.GH72859@ocelot.phlegethon.org> References: <20120329110357.GE72859@ocelot.phlegethon.org> <7e6f4eefa902b1d9bbfc918e29c867d7.squirrel@webmail.lagarcavilla.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <7e6f4eefa902b1d9bbfc918e29c867d7.squirrel@webmail.lagarcavilla.org> 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 At 07:46 -0700 on 29 Mar (1333007195), Andres Lagar-Cavilla wrote: > > At 11:43 -0400 on 27 Mar (1332848606), Andres Lagar-Cavilla wrote: > >> This is a partial repost of the series first sent on Mar 22nd. > >> > >> It currently contains two patches > >> - Teach paging types to p2m-pt. > >> - Add interlock for iommu and mem paging/sharing. > >> > >> The first patch is rebased to apply on top of > >> http://lists.xen.org/archives/html/xen-devel/2012-03/msg01982.html > >> and addresses some feedbackfrom Tim Deegan. > >> > >> The second patch prevents a passthrough device from being assigned to a > >> domain > >> if mem paging or sharing have been turned on. And viceversa. This is a > >> missing > >> check that is necessary for both Intel and AMD processors. > >> > >> Signed-off-by: Andres Lagar-Cavilla > > > > Applied, thanks. > > Great, thanks. > > The question now is what to do for AMD+paging in Xen 4.2. It works > partially, there are still VMEXIT_shutdown hvm crashes happening with pv > drivers. > > I see two options > - leave as is, disabled > - enable it and tag it experimental (printk, documentation, etc) I'm happy to mark it 'experimental' and put appropriate words in the release notes about the PV driver problem. Tim.