Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934350Ab3DPPE3 (ORCPT ); Tue, 16 Apr 2013 11:04:29 -0400 Received: from mail-pa0-f54.google.com ([209.85.220.54]:37741 "EHLO mail-pa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757296Ab3DPPE1 (ORCPT ); Tue, 16 Apr 2013 11:04:27 -0400 Message-ID: <516D6870.1010303@gmail.com> Date: Tue, 16 Apr 2013 23:04:16 +0800 From: Jiang Liu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130308 Thunderbird/17.0.4 MIME-Version: 1.0 To: Paul Mackerras CC: Andrew Morton , Yinghai Lu , Jiang Liu , David Rientjes , Wen Congyang , Mel Gorman , Minchan Kim , KAMEZAWA Hiroyuki , Michal Hocko , David Howells , Mark Salter , Jianguo Wu , linux-mm@kvack.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH v1 00/19] kill free_all_bootmem() and clean up VALID_PAGE() References: <1365867399-21323-1-git-send-email-jiang.liu@huawei.com> <20130415045614.GB7494@iris.ozlabs.ibm.com> In-Reply-To: <20130415045614.GB7494@iris.ozlabs.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1682 Lines: 45 On 04/15/2013 12:56 PM, Paul Mackerras wrote: > On Sat, Apr 13, 2013 at 11:36:20PM +0800, Jiang Liu wrote: >> Commit 600cc5b7f6 "mm: Kill NO_BOOTMEM version free_all_bootmem_node()" >> has kill free_all_bootmem_node() for NO_BOOTMEM. >> >> Currently the usage pattern for free_all_bootmem_node() is like: >> for_each_online_pgdat(pgdat) >> free_all_bootmem_node(pgdat); >> >> It's equivalent to free_all_bootmem(), so this patchset goes one >> step further to kill free_all_bootmem_node() for BOOTMEM too. >> >> This patchset also tries to clean up code and comments related to >> VALID_PAGE() because it has been removed from kernel long time ago. >> >> Patch 1-11: >> Kill free_all_bootmem_node() >> Patch 12-16: >> Clean up code and comments related to VALID_PAGE() >> Patch 17: >> Fix a minor build warning for m68k >> Patch 18: >> merge Alpha's mem_init() for UMA and NUMA. >> Patch 19: >> call register_page_bootmem_info_node() from mm core > > How does this not break bisection? Will a kernel still boot with > patches 1-11 applied but not patch 19? As far as I can see such a > kernel would have no memory available to it after mem_init() time > and would therefore crash early in boot. Hi Paul, Thanks for review! Patch 1-11 replace free_all_bootmem_node() with free_all_bootmem(), so all normal pages will be freed into the buddy system as before. And patch 1-11 are independent with patch 19. Gerry > > Paul. > -- 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/