MIPS: Detect bad _PFN_SHIFT values
authorPaul Burton <paul.burton@mips.com>
Wed, 18 Sep 2019 22:03:30 +0000 (22:03 +0000)
committerPaul Burton <paul.burton@mips.com>
Fri, 20 Sep 2019 21:55:07 +0000 (14:55 -0700)
2 recent commits have fixed issues where _PFN_SHIFT grew too large due
to the introduction of too many pgprot bits in our PTEs for some MIPS32
kernel configurations. Tracking down such issues can be tricky, so add a
BUILD_BUG_ON() to help.

Signed-off-by: Paul Burton <paul.burton@mips.com>
Cc: linux-mips@vger.kernel.org
arch/mips/mm/init.c

index 6fea3b54c961376265c7b726cf14ab122a13ba8d..090fa653dfa9e87532f8923a88008aee718907f8 100644 (file)
@@ -446,6 +446,12 @@ static inline void __init mem_init_free_highmem(void)
 
 void __init mem_init(void)
 {
+       /*
+        * When _PFN_SHIFT is greater than PAGE_SHIFT we won't have enough PTE
+        * bits to hold a full 32b physical address on MIPS32 systems.
+        */
+       BUILD_BUG_ON(IS_ENABLED(CONFIG_32BIT) && (_PFN_SHIFT > PAGE_SHIFT));
+
 #ifdef CONFIG_HIGHMEM
 #ifdef CONFIG_DISCONTIGMEM
 #error "CONFIG_HIGHMEM and CONFIG_DISCONTIGMEM dont work together yet"