[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.1108251356220.18747@chino.kir.corp.google.com>
Date: Thu, 25 Aug 2011 13:57:16 -0700 (PDT)
From: David Rientjes <rientjes@...gle.com>
To: Cong Wang <amwang@...hat.com>
cc: Randy Dunlap <rdunlap@...otime.net>,
Andrew Morton <akpm@...ux-foundation.org>,
Stephen Rothwell <sfr@...b.auug.org.au>, gregkh@...e.de,
linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
linux-mm@...ck.org
Subject: Re: [Patch] numa: introduce CONFIG_NUMA_SYSFS for
drivers/base/node.c
On Thu, 25 Aug 2011, Cong Wang wrote:
> > No, it doesn't work, CONFIG_HUGETLBFS can be enabled with CONFIG_NUMA=y
> > and CONFIG_SYSFS=n and that uses data structures from drivers/base/node.c
> > which doesn't get compiled with this patch.
>
> So, you mean with CONFIG_NUMA=y && CONFIG_SYSFS=n && CONFIG_HUGETLBFS=y
> we still get compile error?
>
> Which data structures are used by hugetlbfs?
>
node_states[], which is revealed at link time if you tried to compile your
patch. It's obvious that we don't want to add per-node hugetlbfs
attributes to sysfs directories if sysfs is disabled, so you need to
modify the hugetlbfs code as well.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists