On 2012-11-20 11:20, Jaegeuk Hanse wrote: > On 11/20/2012 10:43 AM, Jiang Liu wrote: >> On 2012-11-20 10:13, Jaegeuk Hanse wrote: >>> On 11/19/2012 12:07 AM, Jiang Liu wrote: >> Hi Jaegeuk, >> Thanks for review this patch set. >> Currently x86/x86_64/Sparc have been converted to use nobootmem.c, >> and other Arches still use bootmem.c. So need to test it on other Arches, >> such as ARM etc. Yesterday we have tested it patchset on an Itanium platform, >> so bootmem.c should work as expected too. > > Hi Jiang, > > If there are any codes changed in x86/x86_64 to meet nobootmem.c logic? I mean if remove > config NO_BOOTMEM > def_bool y > in arch/x86/Kconfig, whether x86/x86_64 can take advantage of bootmem.c or not. There are code change in x86/x86_64 arch directory to convert from bootmem.c to nobootmem.c, so you can't simply comment out NO_BOOTMEM Kconfig item. There are differences in APIs for bootmem.c and nobootmem.c. For example, free_low_memory_core_early() is only provided by nobootmem.c and init_bootmem_node() is only provided by bootmem.c. Thanks! -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>