Skip to content

Commit 4088024

Browse files
fancertsbogend
authored andcommitted
Revert "mips: Manually call fdt_init_reserved_mem() method"
This reverts commit 3751cbd. Originally the patch was created to fix the reserved-memory DT-node parsing failure on the early stages of the platform memory initialization. That happened due to the two early memory allocators utilization that time: bootmem and memblock. At first the platform-specific memory mapping array was initialized. Then the early_init_fdt_scan_reserved_mem() was called, which couldn't fully parse the "reserved-memory" DT-node since neither memblock nor bootmem allocators hadn't been initialized at that stage, so the fdt_init_reserved_mem() method failed on the memory allocation calls. Only after that the platform-specific memory mapping were used to create proper bootmem and memblock structures and let the early memory allocations work. That's why we had to call the fdt_init_reserved_mem() method one more time to retry the initialization of the features like CMA. The necessity to have that fix was disappeared after the full memblock support had been added to the MIPS kernel and all plat_mem_setup() had been fixed to add the memory regions right into the memblock memory pool. Let's revert that patch then especially after having Paul reported that the second fdt_init_reserved_mem() call causes the reserved memory pool being created twice bigger than implied. Fixes: a94e4f2 ("MIPS: init: Drop boot_mem_map") Reported-by: Paul Cercueil <[email protected]> Signed-off-by: Serge Semin <[email protected]> Signed-off-by: Thomas Bogendoerfer <[email protected]>
1 parent 9f0781b commit 4088024

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

arch/mips/kernel/setup.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
#include <linux/dma-map-ops.h>
2828
#include <linux/decompress/generic.h>
2929
#include <linux/of_fdt.h>
30-
#include <linux/of_reserved_mem.h>
3130
#include <linux/dmi.h>
3231

3332
#include <asm/addrspace.h>
@@ -689,8 +688,6 @@ static void __init arch_mem_init(char **cmdline_p)
689688
memblock_reserve(__pa_symbol(&__nosave_begin),
690689
__pa_symbol(&__nosave_end) - __pa_symbol(&__nosave_begin));
691690

692-
fdt_init_reserved_mem();
693-
694691
early_memtest(PFN_PHYS(ARCH_PFN_OFFSET), PFN_PHYS(max_low_pfn));
695692
}
696693

0 commit comments

Comments
 (0)