Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-12 Thread Michael Ellerman
On Thu, 2013-12-12 at 08:11 +0100, Philippe Bergheaud wrote: > Michael Ellerman wrote: > > On Wed, 2013-12-11 at 11:30 +0100, Philippe Bergheaud wrote: > > > >>Shouldn't we define a new macro HMT_DEFAULT, to identify explicitely where > >>the default priority is required? > > > > > > That might

Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-11 Thread Philippe Bergheaud
Michael Ellerman wrote: On Wed, 2013-12-11 at 11:30 +0100, Philippe Bergheaud wrote: Benjamin Herrenschmidt wrote: On Wed, 2013-12-11 at 17:29 +1100, Michael Ellerman wrote: It would be nice if you could make an assertion about what the state of HMT handling should be once your patch is a

Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-11 Thread Michael Ellerman
On Wed, 2013-12-11 at 11:30 +0100, Philippe Bergheaud wrote: > Benjamin Herrenschmidt wrote: > > On Wed, 2013-12-11 at 17:29 +1100, Michael Ellerman wrote: > > > > > >>It would be nice if you could make an assertion about what the state of HMT > >>handling should be once your patch is applied. >

Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-11 Thread Philippe Bergheaud
Benjamin Herrenschmidt wrote: On Wed, 2013-12-11 at 17:29 +1100, Michael Ellerman wrote: It would be nice if you could make an assertion about what the state of HMT handling should be once your patch is applied. I think it's: * The kernel should use HMT_MEDIUM_LOW as it's "default" priority

Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-11 Thread Benjamin Herrenschmidt
On Wed, 2013-12-11 at 17:29 +1100, Michael Ellerman wrote: > It would be nice if you could make an assertion about what the state of HMT > handling should be once your patch is applied. > > I think it's: > > * The kernel should use HMT_MEDIUM_LOW as it's "default" priority > * The kernel shoul

Re: [PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-10 Thread Michael Ellerman
On Tue, 2013-12-10 at 08:39 +0100, Philippe Bergheaud wrote: > All the important PThread locking occurs in GLIBC libpthread.so > > For scaling to large core counts we need to stay out of the kernel and > scheduler as much as possible which implies increasing the spin time in user > mode. For POW

[PATCH] powerpc: set default kernel thread priority to medium-low

2013-12-09 Thread Philippe Bergheaud
All the important PThread locking occurs in GLIBC libpthread.so For scaling to large core counts we need to stay out of the kernel and scheduler as much as possible which implies increasing the spin time in user mode. For POWER implementations with SMT this implies that user mode needs to manag