Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-13 Thread Greg Kroah-Hartman
On Sat, Oct 13, 2012 at 11:39:37AM +0800, 陈华才 wrote: > So I still think that "sched: Add missing call to calc_load_exit_idle()" > should be reverted in 3.5 branch... 3.5 is now end-of-life, with no more releases, everyone should have moved off of it to 3.6 now, so there's nothing to do here

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-13 Thread Greg Kroah-Hartman
On Sat, Oct 13, 2012 at 11:39:37AM +0800, 陈华才 wrote: So I still think that sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch... 3.5 is now end-of-life, with no more releases, everyone should have moved off of it to 3.6 now, so there's nothing to do here anymore.

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-12 Thread 陈华才
So I still think that "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch... > On 10/06/2012 01:23 AM, Peter Zijlstra wrote: >> On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: >>> Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-12 Thread 陈华才
So I still think that sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch... On 10/06/2012 01:23 AM, Peter Zijlstra wrote: On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote:

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-08 Thread Charles Wang
On 10/06/2012 01:23 AM, Peter Zijlstra wrote: On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-08 Thread Charles Wang
On 10/05/2012 09:39 AM, Jonathan Nieder wrote: Peter Zijlstra wrote: I can't find wtf went wrong either, the initial patch 5167e8d5417bf5c contains both hunks, but in that case the fixup 749c8814f0 doesn't make sense, not can I find anything in merge commits using: git log -S

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-08 Thread Charles Wang
On 10/05/2012 09:39 AM, Jonathan Nieder wrote: Peter Zijlstra wrote: I can't find wtf went wrong either, the initial patch 5167e8d5417bf5c contains both hunks, but in that case the fixup 749c8814f0 doesn't make sense, not can I find anything in merge commits using: git log -S

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-08 Thread Charles Wang
On 10/06/2012 01:23 AM, Peter Zijlstra wrote: On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-05 Thread Peter Zijlstra
On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: > Peter Zijlstra wrote: > > On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: > > >> I'm puzzled as well. Any ideas if I should do anything here or not? > > > > So I think the current v3.5.5 code is fine. > > Now I'm puzzled.

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-05 Thread Jonathan Nieder
Peter Zijlstra wrote: > On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: >> I'm puzzled as well. Any ideas if I should do anything here or not? > > So I think the current v3.5.5 code is fine. Now I'm puzzled. You wrote: | However, since we don't restart the tick, we won't be

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-05 Thread Greg Kroah-Hartman
On Fri, Oct 05, 2012 at 12:21:06PM +0200, Peter Zijlstra wrote: > On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: > > I'm puzzled as well. Any ideas if I should do anything here or not? > > So I think the current v3.5.5 code is fine. I'm just not smart enough to > figure out how 3.6

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-05 Thread Peter Zijlstra
On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: > I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5 code is fine. I'm just not smart enough to figure out how 3.6 got fuzzed, this git thing is confusing as hell. -- To unsubscribe from

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-05 Thread Peter Zijlstra
On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5 code is fine. I'm just not smart enough to figure out how 3.6 got fuzzed, this git thing is confusing as hell. -- To unsubscribe from

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-05 Thread Greg Kroah-Hartman
On Fri, Oct 05, 2012 at 12:21:06PM +0200, Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5 code is fine. I'm just not smart enough to figure out how 3.6 got

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-05 Thread Jonathan Nieder
Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5 code is fine. Now I'm puzzled. You wrote: | However, since we don't restart the tick, we won't be sampling

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-05 Thread Peter Zijlstra
On Fri, 2012-10-05 at 10:10 -0700, Jonathan Nieder wrote: Peter Zijlstra wrote: On Thu, 2012-10-04 at 15:27 -0700, Greg Kroah-Hartman wrote: I'm puzzled as well. Any ideas if I should do anything here or not? So I think the current v3.5.5 code is fine. Now I'm puzzled. You wrote:

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-04 Thread Jonathan Nieder
Peter Zijlstra wrote: > I can't find wtf went wrong either, the initial patch 5167e8d5417bf5c > contains both hunks, but in that case the fixup 749c8814f0 doesn't make > sense, not can I find anything in merge commits using: > > git log -S calc_load_exit_idle kernel/time/tick-sched.c git log

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-04 Thread 陈华才
My opinion: The original patch "sched/nohz: Rewrite and fix load-avg computation -- again" is designed for 3.5-branch and calc_load_exit_idle() is called directly in tick_nohz_idle_exit(). So, the patch can be fully applied in 3.5 and doesn't need to fix (Add the missing call), but not fully

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-04 Thread Greg Kroah-Hartman
On Thu, Oct 04, 2012 at 08:31:59PM +0200, Peter Zijlstra wrote: > On Thu, 2012-10-04 at 10:46 -0700, Greg Kroah-Hartman wrote: > > On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: > > > Hi, Greg > > > > > > I found that Linux-3.5.5 accept this commit "sched: Add missing call > > > to

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-04 Thread Peter Zijlstra
On Thu, 2012-10-04 at 10:46 -0700, Greg Kroah-Hartman wrote: > On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: > > Hi, Greg > > > > I found that Linux-3.5.5 accept this commit "sched: Add missing call > > to calc_load_exit_idle()" but I think this isn't needed. Because > >

Re: Seems like "sched: Add missing call to calc_load_exit_idle()" should be reverted in 3.5 branch

2012-10-04 Thread Greg Kroah-Hartman
On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: > Hi, Greg > > I found that Linux-3.5.5 accept this commit "sched: Add missing call > to calc_load_exit_idle()" but I think this isn't needed. Because > "5167e8d5417b sched/nohz: Rewrite and fix load-avg computation -- > again not fully

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-04 Thread Greg Kroah-Hartman
On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: Hi, Greg I found that Linux-3.5.5 accept this commit sched: Add missing call to calc_load_exit_idle() but I think this isn't needed. Because 5167e8d5417b sched/nohz: Rewrite and fix load-avg computation -- again not fully applied

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-04 Thread Peter Zijlstra
On Thu, 2012-10-04 at 10:46 -0700, Greg Kroah-Hartman wrote: On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: Hi, Greg I found that Linux-3.5.5 accept this commit sched: Add missing call to calc_load_exit_idle() but I think this isn't needed. Because 5167e8d5417b

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-04 Thread Greg Kroah-Hartman
On Thu, Oct 04, 2012 at 08:31:59PM +0200, Peter Zijlstra wrote: On Thu, 2012-10-04 at 10:46 -0700, Greg Kroah-Hartman wrote: On Thu, Oct 04, 2012 at 12:11:01PM +0800, Huacai Chen wrote: Hi, Greg I found that Linux-3.5.5 accept this commit sched: Add missing call to

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-04 Thread 陈华才
My opinion: The original patch sched/nohz: Rewrite and fix load-avg computation -- again is designed for 3.5-branch and calc_load_exit_idle() is called directly in tick_nohz_idle_exit(). So, the patch can be fully applied in 3.5 and doesn't need to fix (Add the missing call), but not fully applied

Re: Seems like sched: Add missing call to calc_load_exit_idle() should be reverted in 3.5 branch

2012-10-04 Thread Jonathan Nieder
Peter Zijlstra wrote: I can't find wtf went wrong either, the initial patch 5167e8d5417bf5c contains both hunks, but in that case the fixup 749c8814f0 doesn't make sense, not can I find anything in merge commits using: git log -S calc_load_exit_idle kernel/time/tick-sched.c git log -m -p