From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx145.postini.com [74.125.245.145]) by kanga.kvack.org (Postfix) with SMTP id 0D35B6B0073 for ; Fri, 6 Jul 2012 11:59:29 -0400 (EDT) Received: by ghrr18 with SMTP id r18so10769490ghr.14 for ; Fri, 06 Jul 2012 08:59:29 -0700 (PDT) Date: Sat, 7 Jul 2012 00:59:20 +0900 From: Minchan Kim Subject: Re: [PATCH] mm: don't invoke __alloc_pages_direct_compact when order 0 Message-ID: <20120706155920.GA7721@barrios> References: <1341588521-17744-1-git-send-email-js1304@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1341588521-17744-1-git-send-email-js1304@gmail.com> Sender: owner-linux-mm@kvack.org List-ID: To: Joonsoo Kim Cc: akpm@linux-foundation.org, Pekka Enberg , Christoph Lameter , linux-kernel@vger.kernel.org, linux-mm@kvack.org Hi Joonsoo, On Sat, Jul 07, 2012 at 12:28:41AM +0900, Joonsoo Kim wrote: > __alloc_pages_direct_compact has many arguments so invoking it is very costly. It's already slow path so it's pointless for such optimization. > And in almost invoking case, order is 0, so return immediately. You can't make sure it. > > Let's not invoke it when order 0 Let's not ruin git blame. > > Signed-off-by: Joonsoo Kim > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index 6092f33..f4039aa 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -2056,7 +2056,10 @@ out: > } > > #ifdef CONFIG_COMPACTION > -/* Try memory compaction for high-order allocations before reclaim */ > +/* > + * Try memory compaction for high-order allocations before reclaim > + * Must be called with order > 0 > + */ > static struct page * > __alloc_pages_direct_compact(gfp_t gfp_mask, unsigned int order, > struct zonelist *zonelist, enum zone_type high_zoneidx, > @@ -2067,8 +2070,7 @@ __alloc_pages_direct_compact(gfp_t gfp_mask, unsigned int order, > { > struct page *page; > > - if (!order) > - return NULL; > + BUG_ON(!order); > > if (compaction_deferred(preferred_zone, order)) { > *deferred_compaction = true; > @@ -2363,15 +2365,17 @@ rebalance: > * Try direct compaction. The first pass is asynchronous. Subsequent > * attempts after direct reclaim are synchronous > */ > - page = __alloc_pages_direct_compact(gfp_mask, order, > - zonelist, high_zoneidx, > - nodemask, > - alloc_flags, preferred_zone, > - migratetype, sync_migration, > - &deferred_compaction, > - &did_some_progress); > - if (page) > - goto got_pg; > + if (unlikely(order)) { > + page = __alloc_pages_direct_compact(gfp_mask, order, > + zonelist, high_zoneidx, > + nodemask, > + alloc_flags, preferred_zone, > + migratetype, sync_migration, > + &deferred_compaction, > + &did_some_progress); > + if (page) > + goto got_pg; > + } > sync_migration = true; > > /* > @@ -2446,15 +2450,17 @@ rebalance: > * direct reclaim and reclaim/compaction depends on compaction > * being called after reclaim so call directly if necessary > */ > - page = __alloc_pages_direct_compact(gfp_mask, order, > - zonelist, high_zoneidx, > - nodemask, > - alloc_flags, preferred_zone, > - migratetype, sync_migration, > - &deferred_compaction, > - &did_some_progress); > - if (page) > - goto got_pg; > + if (unlikely(order)) { > + page = __alloc_pages_direct_compact(gfp_mask, order, > + zonelist, high_zoneidx, > + nodemask, > + alloc_flags, preferred_zone, > + migratetype, sync_migration, > + &deferred_compaction, > + &did_some_progress); > + if (page) > + goto got_pg; > + } > } > > nopage: > -- > 1.7.9.5 > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@kvack.org. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: email@kvack.org -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org