* [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES
@ 2014-01-28 18:34 Nishanth Aravamudan
2014-01-29 15:55 ` Christoph Lameter
2014-02-13 21:41 ` Nishanth Aravamudan
0 siblings, 2 replies; 5+ messages in thread
From: Nishanth Aravamudan @ 2014-01-28 18:34 UTC (permalink / raw)
To: Christoph Lameter
Cc: David Rientjes, Pekka Enberg, linux-mm, Paul Mackerras,
Anton Blanchard, Matt Mackall, Joonsoo Kim, linuxppc-dev,
Wanpeng Li
Anton Blanchard found an issue with an LPAR that had no memory in Node
0. Christoph Lameter recommended, as one possible solution, to use
numa_mem_id() for locality of the nearest memory node-wise. However,
numa_mem_id() [and the other related APIs] are only useful if
CONFIG_HAVE_MEMORYLESS_NODES is set. This is only the case for ia64
currently, but clearly we can have memoryless nodes on ppc64. Add the
Kconfig option and define it to be the same value as CONFIG_NUMA.
On the LPAR in question, which was very inefficiently using slabs, this
took the slab consumption at boot from roughly 7GB to roughly 4GB.
---
Ben, the only question I have wrt this change is if it's appropriate to
change it for all powerpc configs (that have NUMA on)?
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 25493a0..bb2d5fe 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -447,6 +447,9 @@ config NODES_SHIFT
default "4"
depends on NEED_MULTIPLE_NODES
+config HAVE_MEMORYLESS_NODES
+ def_bool NUMA
+
config ARCH_SELECT_MEMORY_MODEL
def_bool y
depends on PPC64
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES
2014-01-28 18:34 [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES Nishanth Aravamudan
@ 2014-01-29 15:55 ` Christoph Lameter
2014-02-13 21:41 ` Nishanth Aravamudan
1 sibling, 0 replies; 5+ messages in thread
From: Christoph Lameter @ 2014-01-29 15:55 UTC (permalink / raw)
To: Nishanth Aravamudan
Cc: David Rientjes, Pekka Enberg, linux-mm, Paul Mackerras,
Anton Blanchard, Matt Mackall, Joonsoo Kim, linuxppc-dev,
Wanpeng Li
On Tue, 28 Jan 2014, Nishanth Aravamudan wrote:
> Anton Blanchard found an issue with an LPAR that had no memory in Node
> 0. Christoph Lameter recommended, as one possible solution, to use
> numa_mem_id() for locality of the nearest memory node-wise. However,
> numa_mem_id() [and the other related APIs] are only useful if
> CONFIG_HAVE_MEMORYLESS_NODES is set. This is only the case for ia64
> currently, but clearly we can have memoryless nodes on ppc64. Add the
> Kconfig option and define it to be the same value as CONFIG_NUMA.
Well this is trivial but if you need encouragement:
Reviewed-by: Christoph Lameter <cl@linux.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES
2014-01-28 18:34 [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES Nishanth Aravamudan
2014-01-29 15:55 ` Christoph Lameter
@ 2014-02-13 21:41 ` Nishanth Aravamudan
2014-02-13 22:45 ` David Rientjes
1 sibling, 1 reply; 5+ messages in thread
From: Nishanth Aravamudan @ 2014-02-13 21:41 UTC (permalink / raw)
To: Christoph Lameter
Cc: David Rientjes, Pekka Enberg, linux-mm, Paul Mackerras,
Anton Blanchard, Matt Mackall, Joonsoo Kim, linuxppc-dev,
Wanpeng Li
On 28.01.2014 [10:34:57 -0800], Nishanth Aravamudan wrote:
> Anton Blanchard found an issue with an LPAR that had no memory in Node
> 0. Christoph Lameter recommended, as one possible solution, to use
> numa_mem_id() for locality of the nearest memory node-wise. However,
> numa_mem_id() [and the other related APIs] are only useful if
> CONFIG_HAVE_MEMORYLESS_NODES is set. This is only the case for ia64
> currently, but clearly we can have memoryless nodes on ppc64. Add the
> Kconfig option and define it to be the same value as CONFIG_NUMA.
>
> On the LPAR in question, which was very inefficiently using slabs, this
> took the slab consumption at boot from roughly 7GB to roughly 4GB.
Err, this should have been
Signed-off-by: Nishanth Aravamudan <nacc@linux.vnet.ibm.com>
!
Sorry about that Ben!
> ---
> Ben, the only question I have wrt this change is if it's appropriate to
> change it for all powerpc configs (that have NUMA on)?
>
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index 25493a0..bb2d5fe 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -447,6 +447,9 @@ config NODES_SHIFT
> default "4"
> depends on NEED_MULTIPLE_NODES
>
> +config HAVE_MEMORYLESS_NODES
> + def_bool NUMA
> +
> config ARCH_SELECT_MEMORY_MODEL
> def_bool y
> depends on PPC64
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES
2014-02-13 21:41 ` Nishanth Aravamudan
@ 2014-02-13 22:45 ` David Rientjes
2014-02-14 0:11 ` Nishanth Aravamudan
0 siblings, 1 reply; 5+ messages in thread
From: David Rientjes @ 2014-02-13 22:45 UTC (permalink / raw)
To: Nishanth Aravamudan
Cc: Pekka Enberg, linux-mm, Paul Mackerras, Anton Blanchard,
Matt Mackall, Joonsoo Kim, linuxppc-dev, Christoph Lameter,
Wanpeng Li
On Thu, 13 Feb 2014, Nishanth Aravamudan wrote:
> > Anton Blanchard found an issue with an LPAR that had no memory in Node
> > 0. Christoph Lameter recommended, as one possible solution, to use
> > numa_mem_id() for locality of the nearest memory node-wise. However,
> > numa_mem_id() [and the other related APIs] are only useful if
> > CONFIG_HAVE_MEMORYLESS_NODES is set. This is only the case for ia64
> > currently, but clearly we can have memoryless nodes on ppc64. Add the
> > Kconfig option and define it to be the same value as CONFIG_NUMA.
> >
> > On the LPAR in question, which was very inefficiently using slabs, this
> > took the slab consumption at boot from roughly 7GB to roughly 4GB.
>
> Err, this should have been
>
> Signed-off-by: Nishanth Aravamudan <nacc@linux.vnet.ibm.com>
>
> !
>
> Sorry about that Ben!
>
> > ---
> > Ben, the only question I have wrt this change is if it's appropriate to
> > change it for all powerpc configs (that have NUMA on)?
> >
I'm suspecting that Ben will request that the proper set_numa_mem() calls
are done for ppc init to make this actually do anything other than return
numa_mem_id() == numa_node_id().
> > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > index 25493a0..bb2d5fe 100644
> > --- a/arch/powerpc/Kconfig
> > +++ b/arch/powerpc/Kconfig
> > @@ -447,6 +447,9 @@ config NODES_SHIFT
> > default "4"
> > depends on NEED_MULTIPLE_NODES
> >
> > +config HAVE_MEMORYLESS_NODES
> > + def_bool NUMA
> > +
> > config ARCH_SELECT_MEMORY_MODEL
> > def_bool y
> > depends on PPC64
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES
2014-02-13 22:45 ` David Rientjes
@ 2014-02-14 0:11 ` Nishanth Aravamudan
0 siblings, 0 replies; 5+ messages in thread
From: Nishanth Aravamudan @ 2014-02-14 0:11 UTC (permalink / raw)
To: David Rientjes
Cc: Pekka Enberg, linux-mm, Paul Mackerras, Anton Blanchard,
Matt Mackall, Joonsoo Kim, linuxppc-dev, Christoph Lameter,
Wanpeng Li
On 13.02.2014 [14:45:49 -0800], David Rientjes wrote:
> On Thu, 13 Feb 2014, Nishanth Aravamudan wrote:
>
> > > Anton Blanchard found an issue with an LPAR that had no memory in Node
> > > 0. Christoph Lameter recommended, as one possible solution, to use
> > > numa_mem_id() for locality of the nearest memory node-wise. However,
> > > numa_mem_id() [and the other related APIs] are only useful if
> > > CONFIG_HAVE_MEMORYLESS_NODES is set. This is only the case for ia64
> > > currently, but clearly we can have memoryless nodes on ppc64. Add the
> > > Kconfig option and define it to be the same value as CONFIG_NUMA.
> > >
> > > On the LPAR in question, which was very inefficiently using slabs, this
> > > took the slab consumption at boot from roughly 7GB to roughly 4GB.
> >
> > Err, this should have been
> >
> > Signed-off-by: Nishanth Aravamudan <nacc@linux.vnet.ibm.com>
> >
> > !
> >
> > Sorry about that Ben!
> >
> > > ---
> > > Ben, the only question I have wrt this change is if it's appropriate to
> > > change it for all powerpc configs (that have NUMA on)?
> > >
>
> I'm suspecting that Ben will request that the proper set_numa_mem() calls
> are done for ppc init to make this actually do anything other than return
> numa_mem_id() == numa_node_id().
You're right, thanks for pointing this out. I could have sworn that in
my previous debugging I saw proper NUMA information, but perhaps it was
just correct based upon the system configuration.
> > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > > index 25493a0..bb2d5fe 100644
> > > --- a/arch/powerpc/Kconfig
> > > +++ b/arch/powerpc/Kconfig
> > > @@ -447,6 +447,9 @@ config NODES_SHIFT
> > > default "4"
> > > depends on NEED_MULTIPLE_NODES
> > >
> > > +config HAVE_MEMORYLESS_NODES
> > > + def_bool NUMA
> > > +
> > > config ARCH_SELECT_MEMORY_MODEL
> > > def_bool y
> > > depends on PPC64
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-02-14 0:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-28 18:34 [PATCH] powerpc: enable CONFIG_HAVE_MEMORYLESS_NODES Nishanth Aravamudan
2014-01-29 15:55 ` Christoph Lameter
2014-02-13 21:41 ` Nishanth Aravamudan
2014-02-13 22:45 ` David Rientjes
2014-02-14 0:11 ` Nishanth Aravamudan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).