Messages in this thread | | | Date | Thu, 23 Jun 2016 14:52:51 -0500 | From | Reza Arbab <> | Subject | Re: [PATCH] powerpc/mm: update arch_{add,remove}_memory() for radix |
| |
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
| |