From patchwork Wed Apr 20 02:48:53 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: James Bottomley X-Patchwork-Id: 720521 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p3K2mvv3014968 for ; Wed, 20 Apr 2011 02:48:57 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752098Ab1DTCs5 (ORCPT ); Tue, 19 Apr 2011 22:48:57 -0400 Received: from bedivere.hansenpartnership.com ([66.63.167.143]:59798 "EHLO bedivere.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751737Ab1DTCs4 (ORCPT ); Tue, 19 Apr 2011 22:48:56 -0400 Received: from localhost (localhost [127.0.0.1]) by bedivere.hansenpartnership.com (Postfix) with ESMTP id 3292D8EE0F5; Tue, 19 Apr 2011 19:48:56 -0700 (PDT) Received: from bedivere.hansenpartnership.com ([127.0.0.1]) by localhost (bedivere.hansenpartnership.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id OmS0sU3SwNfz; Tue, 19 Apr 2011 19:48:56 -0700 (PDT) Received: from [192.168.2.10] (dagonet.hansenpartnership.com [76.243.235.53]) by bedivere.hansenpartnership.com (Postfix) with ESMTPSA id 40D438EE01D; Tue, 19 Apr 2011 19:48:55 -0700 (PDT) Subject: Re: [PATCH v3] mm: make expand_downwards symmetrical to expand_upwards From: James Bottomley To: KOSAKI Motohiro Cc: Christoph Lameter , Pekka Enberg , Michal Hocko , Andrew Morton , Hugh Dickins , linux-mm@kvack.org, LKML , linux-parisc@vger.kernel.org, David Rientjes In-Reply-To: <20110420102314.4604.A69D9226@jp.fujitsu.com> References: <1303249716.11237.26.camel@mulgrave.site> <20110420102314.4604.A69D9226@jp.fujitsu.com> Date: Tue, 19 Apr 2011 21:48:53 -0500 Message-ID: <1303267733.11237.42.camel@mulgrave.site> Mime-Version: 1.0 X-Mailer: Evolution 2.32.1 Sender: linux-parisc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-parisc@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Wed, 20 Apr 2011 02:48:58 +0000 (UTC) On Wed, 2011-04-20 at 10:23 +0900, KOSAKI Motohiro wrote: > > On Tue, 19 Apr 2011, James Bottomley wrote: > > > > > > Which part of me telling you that you will break lots of other things in > > > > the core kernel dont you get? > > > > > > I get that you tell me this ... however, the systems that, according to > > > you, should be failing to get to boot prompt do, in fact, manage it. > > > > If you dont use certain subsystems then it may work. Also do you run with > > debuggin on. > > > > The following patch is I think what would be needed to fix it. > > I'm worry about this patch. A lot of mm code assume !NUMA systems > only have node 0. Not only SLUB. > > I'm not sure why this unfortunate mismatch occur. but I think DISCONTIG > hacks makes less sense. Can we consider parisc turn NUMA on instead? Well, you mean a patch like this? It won't build ... obviously we need some more machinery CC arch/parisc/kernel/asm-offsets.s In file included from include/linux/sched.h:78, from arch/parisc/kernel/asm-offsets.c:31: include/linux/topology.h:212:2: error: #error Please define an appropriate SD_NODE_INIT in include/asm/topology.h!!! In file included from include/linux/sched.h:78, from arch/parisc/kernel/asm-offsets.c:31: include/linux/topology.h: In function 'numa_node_id': include/linux/topology.h:255: error: implicit declaration of function 'cpu_to_node' James --- -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/parisc/Kconfig b/arch/parisc/Kconfig index 69ff049..ffe4058 100644 --- a/arch/parisc/Kconfig +++ b/arch/parisc/Kconfig @@ -261,6 +261,9 @@ config HPUX bool "Support for HP-UX binaries" depends on !64BIT +config NUMA + def_bool n + config NR_CPUS int "Maximum number of CPUs (2-32)" range 2 32 diff --git a/mm/Kconfig b/mm/Kconfig index e9c0c61..17a1474 100644 --- a/mm/Kconfig +++ b/mm/Kconfig @@ -64,6 +64,7 @@ endchoice config DISCONTIGMEM def_bool y depends on (!SELECT_MEMORY_MODEL && ARCH_DISCONTIGMEM_ENABLE) || DISCONTIGMEM_MANUAL + select NUMA config SPARSEMEM def_bool y