Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-10 Thread Mel Gorman
On Wed, Oct 09, 2013 at 01:03:54PM +0200, Ingo Molnar wrote: > > * Mel Gorman wrote: > > > This series has roughly the same goals as previous versions despite the > > size. It reduces overhead of automatic balancing through scan rate reduction > > and the avoidance of TLB flushes. It selects a p

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Peter Zijlstra
On Wed, Oct 09, 2013 at 07:15:37PM +0200, Ingo Molnar wrote: > > It looks like -march=geode generates similar borkage to the > > -march=winchip2 like we found earlier today. > > > > Must be randconfig luck to only hit it now. > > Yes, very weird but such is life :-) > > Also note that this repro

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Wed, Oct 09, 2013 at 06:28:01PM +0200, Ingo Molnar wrote: > > > > Hm, so I'm seeing boot crashes with the config attached: > > > > INIT: version 2.86 booting > > BUG: unable to handle kernel BUG: unable to handle kernel paging > > requestpaging request at eaf1

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Peter Zijlstra
On Wed, Oct 09, 2013 at 07:09:34PM +0200, Ingo Molnar wrote: > > I started bisecting the crash, and the good news is that it's bisectable > and it's not the NUMA bits that are causing the crash. > > (the bad news is that I now face a boring, possibly very long bisection, > but hey ;-) Its the

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
I started bisecting the crash, and the good news is that it's bisectable and it's not the NUMA bits that are causing the crash. (the bad news is that I now face a boring, possibly very long bisection, but hey ;-) Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscrib

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Peter Zijlstra
On Wed, Oct 09, 2013 at 06:28:01PM +0200, Ingo Molnar wrote: > > Hm, so I'm seeing boot crashes with the config attached: > > INIT: version 2.86 booting > BUG: unable to handle kernel BUG: unable to handle kernel paging > requestpaging request at eaf10f40 > at eaf10f40 > IP:IP: [] task_

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
an interesting aspect is that this is a 32-bit UP kernel. Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ a

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Wed, Oct 09, 2013 at 01:03:54PM +0200, Ingo Molnar wrote: > > kernel/sched/fair.c:819:22: warning: 'task_h_load' declared 'static' but > > never defined [-Wunused-function] > > Not too pretty, but it avoids the warning: > > --- > --- a/kernel/sched/fair.c > +++ b

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Peter Zijlstra
On Wed, Oct 09, 2013 at 01:03:54PM +0200, Ingo Molnar wrote: > kernel/sched/fair.c:819:22: warning: 'task_h_load' declared 'static' but > never defined [-Wunused-function] Not too pretty, but it avoids the warning: --- --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -681,6 +681,8 @@ stat

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
* Ingo Molnar wrote: > mmzone.c: > > #if defined(CONFIG_NUMA_BALANCING) && !defined(LAST_CPUPID_IN_PAGE_FLAGS) > > Note the missing 'NOT_' in the latter line. I've changed it to: > > #if defined(CONFIG_NUMA_BALANCING) && defined(LAST_CPUPID_NOT_IN_PAGE_FLAGS) Actually, I think it should

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
* Ingo Molnar wrote: > 3) > > Plus in addition to PeterZ's build fix I noticed this new build warning on > i386 UP kernels: > > kernel/sched/fair.c:819:22: warning: 'task_h_load' declared 'static' but > never defined [-Wunused-function] > > Introduced here I think: > > sched/numa: Use

Re: [PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-09 Thread Ingo Molnar
* Mel Gorman wrote: > This series has roughly the same goals as previous versions despite the > size. It reduces overhead of automatic balancing through scan rate reduction > and the avoidance of TLB flushes. It selects a preferred node and moves tasks > towards their memory as well as moving me

[PATCH 0/63] Basic scheduler support for automatic NUMA balancing V9

2013-10-07 Thread Mel Gorman
This series has roughly the same goals as previous versions despite the size. It reduces overhead of automatic balancing through scan rate reduction and the avoidance of TLB flushes. It selects a preferred node and moves tasks towards their memory as well as moving memory toward their task. It hand