From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: [PATCH 1 of 4] xen,pod: Cosmetic code motion Date: Wed, 27 Jun 2012 17:57:28 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: 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: xen-devel@lists.xensource.com Cc: george.dunlap@eu.citrix.com List-Id: xen-devel@lists.xenproject.org # HG changeset patch # User George Dunlap # Date 1340815810 -3600 # Node ID b4e1fec1c98f6cbad666a972f473854518c25500 # Parent b91ef972029ddaa110af6463171715ab9070c9d8 xen,pod: Cosmetic code motion No point in doing the assignment if we're just going to crash anyway. Signed-off-by: George Dunlap diff --git a/xen/arch/x86/mm/p2m-pod.c b/xen/arch/x86/mm/p2m-pod.c --- a/xen/arch/x86/mm/p2m-pod.c +++ b/xen/arch/x86/mm/p2m-pod.c @@ -1022,13 +1022,13 @@ p2m_pod_demand_populate(struct p2m_domai p2m_pod_emergency_sweep(p2m); } + 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; - /* Get a page f/ the cache. A NULL return value indicates that the * 2-meg range should be marked singleton PoD, and retried */ if ( (p = p2m_pod_cache_get(p2m, order)) == NULL )