Hi Serge, On 02/02/18 03:54, Serge Semin wrote:
If sparsemem is activated all sections with present pages must be accordingly marked after memblock is fully initialized. Signed-off-by: Serge Semin <fancer.lancer@xxxxxxxxx> --- arch/mips/kernel/setup.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/arch/mips/kernel/setup.c b/arch/mips/kernel/setup.c index b2a5b89ae6b2..54302319ce1c 100644 --- a/arch/mips/kernel/setup.c +++ b/arch/mips/kernel/setup.c @@ -837,6 +837,11 @@ static void __init arch_mem_init(char **cmdline_p) crashk_res.end - crashk_res.start + 1); #endif device_tree_init(); +#ifdef CONFIG_SPARSEMEM + for_each_memblock(memory, reg) + memory_present(0, memblock_region_memory_base_pfn(reg), + memblock_region_memory_end_pfn(reg)); +#endif /* CONFIG_SPARSEMEM */
Existing code calls memory_present without CONFIG_SPARSEMEM, is it really conditional on SPARSEMEM?
Thanks, Matt
sparse_init(); plat_swiotlb_setup();
