On Thu, Jun 23, 2016 at 02:37:39PM -0500, Reza Arbab wrote:
Could it be that the functions just need to be renamed hash__create_mapping()/radix__create_mapping() and moved out of #ifdef SPARSEMEM_VMEMMAP?

Or vice-versa, maybe the callers should have been wrapped in the first place:

arch_add_memory() {
        ...
        if (IS_ENABLED(CONFIG_SPARSEMEM_VMEMMAP))
                vmemmap_create_mapping()
        ...
}

arch_remove_memory() {
        ...
        if (IS_ENABLED(CONFIG_SPARSEMEM_VMEMMAP))
                vmemmap_remove_mapping()
        ...
}

--
Reza Arbab

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to