From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tim Deegan Subject: Re: [PATCH 2 of 2 RFC] xen, pod: Only sweep in an emergency, and only for 4k pages Date: Thu, 14 Jun 2012 10:11:31 +0100 Message-ID: <20120614091131.GD82539@ocelot.phlegethon.org> References: <6e760eb25dac400f4873.1339155933@exile> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <6e760eb25dac400f4873.1339155933@exile> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org At 11:45 +0000 on 08 Jun (1339155933), George Dunlap wrote: > + if ( p2m->pod.count == 0 ) > + goto out_of_memory; > > /* Keep track of the highest gfn demand-populated by a guest fault */ > if ( gfn > p2m->pod.max_guest ) > p2m->pod.max_guest = gfn; > > - if ( p2m->pod.count == 0 ) > - goto out_of_memory; > - Is this code motion just noise? Since out_of_memory crasheds the guest, I assume so. Cheers, Tim.