From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 3/6] hugetlb: introduce alloc_nodemask_of_node Date: Thu, 10 Sep 2009 16:36:41 -0700 Message-ID: <20090910163641.9ebaa601.akpm@linux-foundation.org> References: <20090909163127.12963.612.sendpatchset@localhost.localdomain> <20090909163146.12963.79545.sendpatchset@localhost.localdomain> <20090910160541.9f902126.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-numa-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: David Rientjes Cc: lee.schermerhorn@hp.com, linux-mm@kvack.org, linux-numa@vger.kernel.org, mel@csn.ul.ie, randy.dunlap@oracle.com, nacc@us.ibm.com, agl@us.ibm.com, apw@canonical.com, eric.whitney@hp.com On Thu, 10 Sep 2009 16:17:22 -0700 (PDT) David Rientjes wrote: > On Thu, 10 Sep 2009, Andrew Morton wrote: > > > alloc_nodemask_of_node() has no callers, so I can think of a good fix > > for these problems. If it _did_ have a caller then I might ask "can't > > we fix this by moving alloc_nodemask_of_node() into the .c file". But > > it doesn't so I can't. > > > > It gets a caller in patch 5 of the series in set_max_huge_pages(). ooh, there it is. So alloc_nodemask_of_node() could be moved into mm/hugetlb.c. > My early criticism of both alloc_nodemask_of_node() and > alloc_nodemask_of_mempolicy() was that for small CONFIG_NODES_SHIFT (say, > 6 or less, which covers all defconfigs except ia64), it is perfectly > reasonable to allocate 64 bytes on the stack in the caller. Spose so. But this stuff is only called when userspace reconfigures via sysfs, so it'll be low bandwidth (one sincerely hopes).