diff mbox

powerpc: sparsemem should be the default on powernv

Message ID 1488754796.2870.112.camel@kernel.crashing.org (mailing list archive)
State Superseded
Headers show

Commit Message

Benjamin Herrenschmidt March 5, 2017, 10:59 p.m. UTC
This is especially true of POWER9 where memory between chips is
discontiguous.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---

Comments

Michael Ellerman March 6, 2017, 9:41 a.m. UTC | #1
Benjamin Herrenschmidt <benh@kernel.crashing.org> writes:

> 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
Benjamin Herrenschmidt March 6, 2017, 9:41 p.m. UTC | #2
On Mon, 2017-03-06 at 20:41 +1100, Michael Ellerman wrote:
> > 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.
diff mbox

Patch

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