Skip to contentSkip to conversation

powerpc: sparsemem should be the default on powernv

PATCH In review Benjamin Herrenschmidt 2 participants 3 messages Mar 5, 2017 – Mar 6, 2017 PowerPC

lore
Share & tools
lore

Benjamin Herrenschmidt

author
#0 lore raw

PATCH 1/1 powerpc: sparsemem should be the default on powernv

This is especially true of POWER9 where memory between chips is discontiguous.

Signed-off-by Benjamin Herrenschmidt <[email protected]>

---

arch/powerpc/Kconfig +1 -1
index 4940917..c4b4c55 100644
@@ -580,7 +580,7 @@ config ARCH_SPARSEMEM_ENABLE
 
 config ARCH_SPARSEMEM_DEFAULT
 	def_bool y
-	depends on (SMP && PPC_PSERIES) || PPC_PS3
+	depends on (SMP && PPC_PSERIES) || PPC_PS3 || PPC_POWERNV
 
 config SYS_SUPPORTS_HUGETLBFS
 	bool
reviewer
#1 lore raw
Quoted 11 lines · Benjamin Herrenschmidt

This is especially true of POWER9 where memory between chips is discontiguous.

Signed-off-by Benjamin Herrenschmidt <[email protected]>

---

arch/powerpc/Kconfig +0 -0
index 4940917..c4b4c55 100644

arch/powerpc/Kconfig config ARCH_SPARSEMEM_ENABLE · line 583

Quoted 5 lines
arch/powerpc/Kconfig +1 -1
@@ -580,7 +580,7 @@ config ARCH_SPARSEMEM_ENABLE
 
 config ARCH_SPARSEMEM_DEFAULT
 	def_bool y
-	depends on (SMP && PPC_PSERIES) || PPC_PS3
+	depends on (SMP && PPC_PSERIES) || PPC_PS3 || PPC_POWERNV

Why don't we just make it default y for BOOK3S_64?

It's just the default after all, it can still be changed.

cheers

author
#2 lore raw
Quoted 13 lines · Michael Ellerman
Quoted 10 lines
arch/powerpc/Kconfig +0 -0
index 4940917..c4b4c55 100644
@@ -580,7 +580,7 @@ config ARCH_SPARSEMEM_ENABLE


  config ARCH_SPARSEMEM_DEFAULT
       def_bool y

-     depends on (SMP && PPC_PSERIES) || PPC_PS3
+     depends on (SMP && PPC_PSERIES) || PPC_PS3 || PPC_POWERNV

Why don't we just make it default y for BOOK3S_64?

Quoted 1 line

It's just the default after all, it can still be changed.

No objection.

Cheers,
Ben.