diff mbox

[v2,3/3] arm64: mm: Restrictive printing for modules and PCI I/O in memory layout

Message ID 1460948988-7477-4-git-send-email-wangkefeng.wang@huawei.com
State New
Headers show

Commit Message

Kefeng Wang April 18, 2016, 3:09 a.m. UTC
The modules and PCI I/O informations in Virtual kernel memory layout
are inappropriate when corresponding config is not enabled, so don't
show them then.

Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>

---
 arch/arm64/mm/init.c | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.6.0.GIT


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Comments

Kefeng Wang April 19, 2016, 7:08 a.m. UTC | #1
On 2016/4/18 19:15, James Morse wrote:
> On 18/04/16 04:09, Kefeng Wang wrote:

>> The modules and PCI I/O informations in Virtual kernel memory layout

>> are inappropriate when corresponding config is not enabled, so don't

>> show them then.

> 

> I'm not sure about this, it looks like these areas are still reserved, so this

> is still useful debug information. mm/dump.c will still print any mappings in

> these ranges, its good to be consistent.


If the CONFIG is not enabled,  I think the areas will not be reserved and not mapped,
correct me if I am wrong. so it looks useless, but keeping them is ok for me.

BRs,
Kefeng

> 

> 

> Thanks,

> 

> James

> 



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff mbox

Patch

diff --git a/arch/arm64/mm/init.c b/arch/arm64/mm/init.c
index c96fe51..e06a445 100644
--- a/arch/arm64/mm/init.c
+++ b/arch/arm64/mm/init.c
@@ -367,8 +367,10 @@  void __init mem_init(void)
 	pr_cont("    kasan   : 0x%16lx - 0x%16lx   (%6ld GB)\n",
 		MLG(KASAN_SHADOW_START, KASAN_SHADOW_END));
 #endif
+#ifdef CONFIG_MODULES
 	pr_cont("    modules : 0x%16lx - 0x%16lx   (%6ld MB)\n",
 		MLM(MODULES_VADDR, MODULES_END));
+#endif
 	pr_cont("    vmalloc : 0x%16lx - 0x%16lx   (%6ld GB)\n",
 		MLG(VMALLOC_START, VMALLOC_END));
 	pr_cont("      .text : 0x%p" " - 0x%p" "   (%6ld KB)\n",
@@ -390,8 +392,10 @@  void __init mem_init(void)
 #endif
 	pr_cont("    fixed   : 0x%16lx - 0x%16lx   (%6ld KB)\n",
 		MLK(FIXADDR_START, FIXADDR_TOP));
+#ifdef CONFIG_PCI
 	pr_cont("    PCI I/O : 0x%16lx - 0x%16lx   (%6ld MB)\n",
 		MLM(PCI_IO_START, PCI_IO_END));
+#endif
 	pr_cont("    memory  : 0x%16lx - 0x%16lx   (%6ld MB)\n",
 		MLM(__phys_to_virt(memblock_start_of_DRAM()),
 		    (unsigned long)high_memory));