From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx205.postini.com [74.125.245.205]) by kanga.kvack.org (Postfix) with SMTP id E5A9D6B006C for ; Mon, 19 Nov 2012 21:44:20 -0500 (EST) Message-ID: <50AAEE61.2090504@huawei.com> Date: Tue, 20 Nov 2012 10:43:45 +0800 From: Jiang Liu MIME-Version: 1.0 Subject: Re: [RFT PATCH v1 0/5] fix up inaccurate zone->present_pages References: <20121115112454.e582a033.akpm@linux-foundation.org> <1353254850-27336-1-git-send-email-jiang.liu@huawei.com> <50AAE72E.3090101@gmail.com> In-Reply-To: <50AAE72E.3090101@gmail.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Jaegeuk Hanse Cc: Jiang Liu , Andrew Morton , Wen Congyang , David Rientjes , Maciej Rutecki , Chris Clayton , "Rafael J . Wysocki" , Mel Gorman , Minchan Kim , KAMEZAWA Hiroyuki , Michal Hocko , Jianguo Wu , linux-mm@kvack.org, linux-kernel@vger.kernel.org On 2012-11-20 10:13, Jaegeuk Hanse wrote: > On 11/19/2012 12:07 AM, Jiang Liu wrote: >> The commit 7f1290f2f2a4 ("mm: fix-up zone present pages") tries to >> resolve an issue caused by inaccurate zone->present_pages, but that >> fix is incomplete and causes regresions with HIGHMEM. And it has been >> reverted by commit >> 5576646 revert "mm: fix-up zone present pages" >> >> This is a following-up patchset for the issue above. It introduces a >> new field named "managed_pages" to struct zone, which counts pages >> managed by the buddy system from the zone. And zone->present_pages >> is used to count pages existing in the zone, which is >> spanned_pages - absent_pages. >> >> But that way, zone->present_pages will be kept in consistence with >> pgdat->node_present_pages, which is sum of zone->present_pages. >> >> This patchset has only been tested on x86_64 with nobootmem.c. So need >> help to test this patchset on machines: >> 1) use bootmem.c > > If only x86_32 use bootmem.c instead of nobootmem.c? How could I confirm it? Hi Jaegeuk, Thanks for review this patch set. Currently x86/x86_64/Sparc have been converted to use nobootmem.c, and other Arches still use bootmem.c. So need to test it on other Arches, such as ARM etc. Yesterday we have tested it patchset on an Itanium platform, so bootmem.c should work as expected too. Thanks! Gerry -- 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