[PATCH] enable NUMA on s390x

[Dan Horák]

---
 baseconfig/s390x/CONFIG_NUMA | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/baseconfig/s390x/CONFIG_NUMA b/baseconfig/s390x/CONFIG_NUMA
index 7482c98..e480f0c 100644
--- a/baseconfig/s390x/CONFIG_NUMA
+++ b/baseconfig/s390x/CONFIG_NUMA
@@ -1 +1 @@
-# CONFIG_NUMA is not set
+CONFIG_NUMA=y

[Dan Horák]

Hi,

this is for Rawhide and related to
https://bugzilla.redhat.com/show_bug.cgi?id=1419064
so the user-space then will work.

Dan

On Fri, 10 Feb 2017 15:17:13 +0100
Dan Horák  wrote:
> baseconfig/s390x/CONFIG_NUMA | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/baseconfig/s390x/CONFIG_NUMA
> b/baseconfig/s390x/CONFIG_NUMA index 7482c98..e480f0c 100644
> --- a/baseconfig/s390x/CONFIG_NUMA
> +++ b/baseconfig/s390x/CONFIG_NUMA
> @@ -1 +1 @@
> -# CONFIG_NUMA is not set
> +CONFIG_NUMA=y
>
> 2.7.4
> _______________________________________________
> kernel mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Josh Boyer]

On Fri, Feb 10, 2017 at 9:19 AM, Dan Horák  wrote:
> Hi,
>
> this is for Rawhide and related to
> https://bugzilla.redhat.com/show_bug.cgi?id=1419064
> so the user-space then will work.

ACK.  Let's throw that in the patch itself next time so it lands in
the commit log :)

josh
> On Fri, 10 Feb 2017 15:17:13 +0100
> Dan Horák  wrote:
> > baseconfig/s390x/CONFIG_NUMA | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/baseconfig/s390x/CONFIG_NUMA
> > b/baseconfig/s390x/CONFIG_NUMA index 7482c98..e480f0c 100644
> > --- a/baseconfig/s390x/CONFIG_NUMA
> > +++ b/baseconfig/s390x/CONFIG_NUMA
> > @@ -1 +1 @@
> > -# CONFIG_NUMA is not set
> > +CONFIG_NUMA=y
> >
> > 2.7.4
> > _______________________________________________
> > kernel mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]
>
> kernel mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Justin Forbes]

On Fri, Feb 10, 2017 at 9:03 AM, Josh Boyer 
wrote:
> On Fri, Feb 10, 2017 at 9:19 AM, Dan Horák  wrote:
> > Hi,
> >
> > this is for Rawhide and related to
> > https://bugzilla.redhat.com/show_bug.cgi?id=1419064
> > so the user-space then will work.
>
> ACK.  Let's throw that in the patch itself next time so it lands in
> the commit log :)
>
> josh
> > On Fri, 10 Feb 2017 15:17:13 +0100
> > Dan Horák  wrote:
> > > baseconfig/s390x/CONFIG_NUMA | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/baseconfig/s390x/CONFIG_NUMA
> > > b/baseconfig/s390x/CONFIG_NUMA index 7482c98..e480f0c 100644
> > > --- a/baseconfig/s390x/CONFIG_NUMA
> > > +++ b/baseconfig/s390x/CONFIG_NUMA
> > > @@ -1 +1 @@
> > > -# CONFIG_NUMA is not set
> > > +CONFIG_NUMA=y

This will land in the build for today.

Thanks,
Justin

[Justin Forbes]

On Fri, Feb 10, 2017 at 2:40 PM, Justin Forbes  wrote:
> On Fri, Feb 10, 2017 at 9:03 AM, Josh Boyer 
> wrote:
> > On Fri, Feb 10, 2017 at 9:19 AM, Dan Horák  wrote:
> > > Hi,
> > >
> > > this is for Rawhide and related to
> > > https://bugzilla.redhat.com/show_bug.cgi?id=1419064
> > > so the user-space then will work.
> >
> > ACK.  Let's throw that in the patch itself next time so it lands in
> > the commit log :)
> >
> > josh
> > > On Fri, 10 Feb 2017 15:17:13 +0100
> > > Dan Horák  wrote:
> > > > baseconfig/s390x/CONFIG_NUMA | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/baseconfig/s390x/CONFIG_NUMA
> > > > b/baseconfig/s390x/CONFIG_NUMA index 7482c98..e480f0c 100644
> > > > --- a/baseconfig/s390x/CONFIG_NUMA
> > > > +++ b/baseconfig/s390x/CONFIG_NUMA
> > > > @@ -1 +1 @@
> > > > -# CONFIG_NUMA is not set
> > > > +CONFIG_NUMA=y
>
> This will land in the build for today.

This brought in some additional config items that were hidden with
NUMA off. I have made best attempts, but let me know if you would like
any of these to change:

CONFIG_EMU_SIZE=0x10000000
CONFIG_NODES_SHIFT=4
CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
CONFIG_NUMA_BALANCING=y
CONFIG_NUMA_EMU=y
CONFIG_NUMA=y

[Dan Horák]

> This brought in some additional config items that were hidden with
> NUMA off. I have made best attempts, but let me know if you would like
> any of these to change:
>
> CONFIG_EMU_SIZE=0x10000000
> CONFIG_NODES_SHIFT=4
> CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
> CONFIG_NUMA_BALANCING=y
> CONFIG_NUMA_EMU=y
> CONFIG_NUMA=y

I've asked IBM for a review of the config options and here is the reply:

"The config option CONFIG_NUMA_BALANCING_DEFAULT_ENABLED should be
*disabled *.
Automatic balancing does not make sense for FAKE NUMA and just produces
overhead by moving around memory.

The rest looks good to me."

Please update the config accordingly.

Dan

[Justin Forbes]

On Fri, Feb 17, 2017 at 11:37 AM, Dan Horák  wrote:
> > This brought in some additional config items that were hidden with
> > NUMA off. I have made best attempts, but let me know if you would like
> > any of these to change:
> >
> > CONFIG_EMU_SIZE=0x10000000
> > CONFIG_NODES_SHIFT=4
> > CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
> > CONFIG_NUMA_BALANCING=y
> > CONFIG_NUMA_EMU=y
> > CONFIG_NUMA=y
>
> I've asked IBM for a review of the config options and here is the reply:
>
> "The config option CONFIG_NUMA_BALANCING_DEFAULT_ENABLED should be
> *disabled *.
> Automatic balancing does not make sense for FAKE NUMA and just produces
> overhead by moving around memory.
>
> The rest looks good to me."
>
> Please update the config accordingly.

That sounds perfectly reasonable. I have updated the config for today's
build.

Thanks,
Justin