[PATCH] powerpc: sparsemem should be the default on powernv

Subsystems: linux for powerpc (32-bit and 64-bit), the rest

STALE3451d

3 messages, 2 authors, 2017-03-06 · open the first message on its own page

[PATCH] powerpc: sparsemem should be the default on powernv

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2017-03-05 23:00:08

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

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 4940917..c4b4c55 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -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

Re: [PATCH] powerpc: sparsemem should be the default on powernv

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2017-03-06 09:42:00

Benjamin Herrenschmidt [off-list ref] writes:
quoted hunk
This is especially true of POWER9 where memory between chips is
discontiguous.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 4940917..c4b4c55 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -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

Re: [PATCH] powerpc: sparsemem should be the default on powernv

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2017-03-06 21:41:38

On Mon, 2017-03-06 at 20:41 +1100, Michael Ellerman wrote:
quoted
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 4940917..c4b4c55 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -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.
No objection.

Cheers,
Ben.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help