Messages in this thread | | | Date | Wed, 24 Aug 2011 20:50:44 -0700 | From | Randy Dunlap <> | Subject | Re: [Patch] numa: introduce CONFIG_NUMA_SYSFS for drivers/base/node.c |
| |
On Thu, 25 Aug 2011 11:31:45 +0800 Cong Wang wrote:
> 于 2011年08月25日 10:14, Randy Dunlap 写道: > > On Wed, 24 Aug 2011 11:34:45 +0800 Cong Wang wrote: > > > >> Hi, Andrew, > >> > >> Do you think my patch below is better? > > > > Hi, > > > > This causes build errors for me because node.o is not being built: > > > > arch/x86/built-in.o: In function `topology_init': > > topology.c:(.init.text+0x3668): undefined reference to `register_one_node' > > drivers/built-in.o: In function `unregister_cpu': > > (.text+0x7aecc): undefined reference to `unregister_cpu_under_node' > > drivers/built-in.o: In function `register_cpu': > > (.cpuinit.text+0xc1): undefined reference to `register_cpu_under_node' > > Ah, this is because I missed the part in include/linux/node.h. :) > > Below is the updated version. > > Thanks for testing!
Yes, that works after changing #ifdef defined(...) to #if defined(...)
Acked-by: Randy Dunlap <rdunlap@xenotime.net>
--- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- 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/
| |