Re: [PATCH v2 08/15] MIPS: memblock: Mark present sparsemem sections

[Matt Redfearn]

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.lan...@gmail.com>
---
  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();