Revision a6335fa11e08fc386740557b2a4c1b7ff34bc499 authored by Alexander Sverdlin on 02 July 2015, 15:16:01 UTC, committed by Ralf Baechle on 09 July 2015, 09:02:59 UTC
Commit f9a7febd leads to a fact that mapstart and therefore a page bitmap for bootmem allocator immediately follows initrd_end. This doesn't always work well on Octeon, where there are holes in PFN ranges (refer to 5b3b1688 and 4MB-aligned PFN allocation). Depending on the inird location it could happen, that mapstart would be in an area not allocated by plat_mem_setup() in arch/mips/cavium-octeon/setup.c, but in the alignment hole between initrd and the next PFN area. Later on this memory will be unconditionally made available to buddy allocator at the end of free_all_bootmem_core() (mm/bootmem.c). All of this results in Linux using the memory not designated for Linux in Octeon's plat_mem_setup(), which in turn means corruption of the memory used by another OS/baremetal code on the same SoC. It doesn't look to me as a problem of Octeon platform code, but rather as an inability of f9a7febd to deal correctly with the fragmented memory-mappings. Proposed fix moves the check for initrd address to the same calculation-loop in bootmem_init() (arch/mips/kernel/setup.c), which also accounts for kernel code location. This should result in mapstart located starting from the first PFN area after kernel code AND initrd. Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com> Cc: linux-mips@linux-mips.org Cc: David Daney <david.daney@cavium.com> Cc: Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com> Cc: Huacai Chen <chenhc@lemote.com> Cc: Andreas Herrmann <andreas.herrmann@caviumnetworks.com> Cc: Joe Perches <joe@perches.com> Cc: Steven J. Hill <Steven.Hill@imgtec.com> Cc: Yusuf Khan <yusuf.khan@nokia.com> Cc: Michael Kreuzer <michael.kreuzer@nokia.com> Cc: Aaro Koskinen <aaro.koskinen@iki.fi> Patchwork: https://patchwork.linux-mips.org/patch/10594/ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
1 parent 7928eb0
File | Mode | Size |
---|---|---|
Kconfig | -rw-r--r-- | 2.7 KB |
Makefile | -rw-r--r-- | 422 bytes |
addr.c | -rw-r--r-- | 3.8 KB |
addr.h | -rw-r--r-- | 615 bytes |
atm_misc.c | -rw-r--r-- | 2.6 KB |
atm_sysfs.c | -rw-r--r-- | 4.1 KB |
br2684.c | -rw-r--r-- | 23.2 KB |
clip.c | -rw-r--r-- | 22.3 KB |
common.c | -rw-r--r-- | 21.4 KB |
common.h | -rw-r--r-- | 1.6 KB |
ioctl.c | -rw-r--r-- | 9.2 KB |
lec.c | -rw-r--r-- | 60.1 KB |
lec.h | -rw-r--r-- | 5.1 KB |
lec_arpc.h | -rw-r--r-- | 2.9 KB |
mpc.c | -rw-r--r-- | 38.2 KB |
mpc.h | -rw-r--r-- | 1.9 KB |
mpoa_caches.c | -rw-r--r-- | 14.7 KB |
mpoa_caches.h | -rw-r--r-- | 3.0 KB |
mpoa_proc.c | -rw-r--r-- | 7.2 KB |
pppoatm.c | -rw-r--r-- | 15.6 KB |
proc.c | -rw-r--r-- | 11.5 KB |
protocols.h | -rw-r--r-- | 379 bytes |
pvc.c | -rw-r--r-- | 3.7 KB |
raw.c | -rw-r--r-- | 1.8 KB |
resources.c | -rw-r--r-- | 9.9 KB |
resources.h | -rw-r--r-- | 1019 bytes |
signaling.c | -rw-r--r-- | 6.2 KB |
signaling.h | -rw-r--r-- | 838 bytes |
svc.c | -rw-r--r-- | 16.0 KB |
Computing file changes ...