Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753463Ab1DZJtq (ORCPT ); Tue, 26 Apr 2011 05:49:46 -0400 Received: from fgwmail5.fujitsu.co.jp ([192.51.44.35]:50961 "EHLO fgwmail5.fujitsu.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753414Ab1DZJtm convert rfc822-to-8bit (ORCPT ); Tue, 26 Apr 2011 05:49:42 -0400 X-SecurityPolicyCheck-FJ: OK by FujitsuOutboundMailChecker v1.3.1 From: KOSAKI Motohiro To: Mel Gorman Subject: Re: [PATCH] mm: Check if any page in a pageblock is reserved before marking it MIGRATE_RESERVE Cc: kosaki.motohiro@jp.fujitsu.com, John Stultz , linux-kernel@vger.kernel.org, Arve Hj?nnev?g , Dave Hansen , Andrew Morton In-Reply-To: <20110426073410.GA4658@suse.de> References: <1303436043-26644-1-git-send-email-john.stultz@linaro.org> <20110426073410.GA4658@suse.de> Message-Id: <20110426185114.F3A4.A69D9226@jp.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-15" Content-Transfer-Encoding: 8BIT X-Mailer: Becky! ver. 2.56.05 [ja] Date: Tue, 26 Apr 2011 18:49:39 +0900 (JST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3733 Lines: 102 > On Thu, Apr 21, 2011 at 06:34:03PM -0700, John Stultz wrote: > > From: Arve Hj?nnev?g > > > > This fixes a problem where the first pageblock got marked MIGRATE_RESERVE even > > though it only had a few free pages. This in turn caused no contiguous memory > > to be reserved and frequent kswapd wakeups that emptied the caches to get more > > contiguous memory. > > > > CC: Dave Hansen > > CC: Mel Gorman > > CC: Andrew Morton > > Signed-off-by: Arve Hj?nnev?g > > Acked-by: Mel Gorman > > > > [This patch was submitted and acked a little over a year ago > > (see: http://lkml.org/lkml/2010/4/6/172 ), but never seemingly > > made it upstream. Resending for comments. -jstultz] > > > > Signed-off-by: John Stultz > > Whoops, should have spotted it slipped through. FWIW, I'm still happy > with my Ack being stuck onto it. Hehe, No. You acked another patch at last year and John taked up old one. Sigh. Look, correct one has pfn_valid_within(). http://lkml.org/lkml/2010/4/6/172 And, Minchan suggested to add more explanation to the description. Then, I think following is desiable one. Subject: [PATCH] mm: Check if any page in a pageblock is reserved before marking it MIGRATE_RESERVE From: Arve Hjonnevag This fixes a problem where the first pageblock got marked MIGRATE_RESERVE even though it only had a few free pages. eg, On current ARM port, The kernel starts at offset 0x8000 to leave room for boot parameters, and the memory is freed later. This in turn caused no contiguous memory to be reserved and frequent kswapd wakeups that emptied the caches to get more contiguous memory. Unfortunatelly, ARM need order-2 allocation for pgd (see arm/mm/pgd.c#pgd_alloc()). Therefore the issue is not minor nor easy avoidable. CC: Andrew Morton Signed-off-by: Arve Hjonnevag Acked-by: Mel Gorman Acked-by: Dave Hansen Signed-off-by: John Stultz Signed-off-by: KOSAKI Motohiro [added a few explanation] --- mm/page_alloc.c | 16 +++++++++++++++- 1 files changed, 15 insertions(+), 1 deletions(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 1d5c189..10d9fa7 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -3282,6 +3282,20 @@ static inline unsigned long wait_table_bits(unsigned long size) #define LONG_ALIGN(x) (((x)+(sizeof(long))-1)&~((sizeof(long))-1)) /* + * Check if a pageblock contains reserved pages + */ +static int pageblock_is_reserved(unsigned long start_pfn) +{ + unsigned long end_pfn = start_pfn + pageblock_nr_pages; + unsigned long pfn; + + for (pfn = start_pfn; pfn < end_pfn; pfn++) + if (!pfn_valid_within(pfn) || PageReserved(pfn_to_page(pfn))) + return 1; + return 0; +} + +/* * Mark a number of pageblocks as MIGRATE_RESERVE. The number * of blocks reserved is based on min_wmark_pages(zone). The memory within * the reserve will tend to store contiguous free pages. Setting min_free_kbytes @@ -3320,7 +3334,7 @@ static void setup_zone_migrate_reserve(struct zone *zone) continue; /* Blocks with reserved pages will never free, skip them. */ - if (PageReserved(page)) + if (pageblock_is_reserved(pfn)) continue; block_migratetype = get_pageblock_migratetype(page); -- 1.7.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/