From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH 8 of 9] x86/mm: Fix checks during foreign mapping of paged pages Date: Wed, 30 Nov 2011 13:46:50 +0100 Message-ID: <20111130124650.GA15723@aepfle.de> References: <3489152b3a560be744ab.1322598105@xdev.gridcentric.ca> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <3489152b3a560be744ab.1322598105@xdev.gridcentric.ca> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Andres Lagar-Cavilla Cc: xen-devel@lists.xensource.com, ian.campbell@citrix.com, andres@gridcentric.ca, tim@xen.org, keir.xen@gmail.com, JBeulich@suse.com, ian.jackson@citrix.com, adin@gridcentric.ca List-Id: xen-devel@lists.xenproject.org On Tue, Nov 29, Andres Lagar-Cavilla wrote: > Check that the valid mfn is the one we are mapping, not the > mfn of the page table of the foreign domain. > > Signed-off-by: Andres Lagar-Cavilla Acked-by: Olaf Hering > diff -r 5286ed662c1e -r 3489152b3a56 xen/arch/x86/mm.c > --- a/xen/arch/x86/mm.c > +++ b/xen/arch/x86/mm.c > @@ -3572,7 +3572,8 @@ int do_mmu_update( > rc = -ENOENT; > break; > } > - else if ( p2m_ram_paging_in_start == l1e_p2mt && !mfn_valid(mfn) ) > + else if ( p2m_ram_paging_in_start == l1e_p2mt && > + !mfn_valid(l1emfn) )