On 2012-11-20 11:20, Jaegeuk Hanse wrote:

On 11/20/2012 10:43 AM, Jiang Liu wrote:

quoted

On 2012-11-20 10:13, Jaegeuk Hanse wrote:

quoted

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@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>