Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-10 Thread Vincent Guittot
On 10 April 2013 09:26, Peter Zijlstra wrote: > On Tue, 2013-04-09 at 11:06 +0200, Vincent Guittot wrote: >> >> +void idle_enter(struct rq *this_rq) >> >> +{ >> >> + update_rq_runnable_avg(this_rq, 1); >> >> +} > >> >> +void idle_exit(struct rq *this_rq) >> >> +{ >> >> +

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-10 Thread Peter Zijlstra
On Tue, 2013-04-09 at 11:06 +0200, Vincent Guittot wrote: > >> +void idle_enter(struct rq *this_rq) > >> +{ > >> + update_rq_runnable_avg(this_rq, 1); > >> +} > >> +void idle_exit(struct rq *this_rq) > >> +{ > >> + update_rq_runnable_avg(this_rq, 0); > >> +} > > > > These seem like

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-10 Thread Peter Zijlstra
On Tue, 2013-04-09 at 11:06 +0200, Vincent Guittot wrote: +void idle_enter(struct rq *this_rq) +{ + update_rq_runnable_avg(this_rq, 1); +} +void idle_exit(struct rq *this_rq) +{ + update_rq_runnable_avg(this_rq, 0); +} These seem like fairly unfortunate names to

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-10 Thread Vincent Guittot
On 10 April 2013 09:26, Peter Zijlstra pet...@infradead.org wrote: On Tue, 2013-04-09 at 11:06 +0200, Vincent Guittot wrote: +void idle_enter(struct rq *this_rq) +{ + update_rq_runnable_avg(this_rq, 1); +} +void idle_exit(struct rq *this_rq) +{ +

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 15:16, Steven Rostedt wrote: > On Tue, 2013-04-09 at 14:18 +0200, Vincent Guittot wrote: >> On 9 April 2013 10:55, Peter Zijlstra wrote: >> > On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: >> >> Changes since V2: >> >> - remove useless definition for UP platform >> >>

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Steven Rostedt
On Tue, 2013-04-09 at 14:18 +0200, Vincent Guittot wrote: > On 9 April 2013 10:55, Peter Zijlstra wrote: > > On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: > >> Changes since V2: > >> - remove useless definition for UP platform > >> - rebased on top of Steven Rostedt's patches : > >>

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 10:55, Peter Zijlstra wrote: > On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: >> Changes since V2: >> - remove useless definition for UP platform >> - rebased on top of Steven Rostedt's patches : >> https://lkml.org/lkml/2013/2/12/558 > > So what's the status of those

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 10:50, Peter Zijlstra wrote: > On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: >> Signed-off-by: Vincent Guittot >> --- >> kernel/sched/fair.c | 23 +-- >> kernel/sched/idle_task.c | 10 ++ >> kernel/sched/sched.h | 12

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Peter Zijlstra
On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: > Changes since V2: > - remove useless definition for UP platform > - rebased on top of Steven Rostedt's patches : > https://lkml.org/lkml/2013/2/12/558 So what's the status of those patches? I still worry about the extra context switch

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Peter Zijlstra
On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: > Signed-off-by: Vincent Guittot > --- > kernel/sched/fair.c | 23 +-- > kernel/sched/idle_task.c | 10 ++ > kernel/sched/sched.h | 12 > 3 files changed, 43 insertions(+), 2

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Peter Zijlstra
On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Signed-off-by: Vincent Guittot vincent.guit...@linaro.org --- kernel/sched/fair.c | 23 +-- kernel/sched/idle_task.c | 10 ++ kernel/sched/sched.h | 12 3 files changed, 43

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Peter Zijlstra
On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Changes since V2: - remove useless definition for UP platform - rebased on top of Steven Rostedt's patches : https://lkml.org/lkml/2013/2/12/558 So what's the status of those patches? I still worry about the extra context switch

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 10:50, Peter Zijlstra pet...@infradead.org wrote: On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Signed-off-by: Vincent Guittot vincent.guit...@linaro.org --- kernel/sched/fair.c | 23 +-- kernel/sched/idle_task.c | 10 ++

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 10:55, Peter Zijlstra pet...@infradead.org wrote: On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Changes since V2: - remove useless definition for UP platform - rebased on top of Steven Rostedt's patches : https://lkml.org/lkml/2013/2/12/558 So what's the status

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Steven Rostedt
On Tue, 2013-04-09 at 14:18 +0200, Vincent Guittot wrote: On 9 April 2013 10:55, Peter Zijlstra pet...@infradead.org wrote: On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Changes since V2: - remove useless definition for UP platform - rebased on top of Steven Rostedt's patches

Re: [PATCH v4] sched: fix wrong rq's runnable_avg update with rt tasks

2013-04-09 Thread Vincent Guittot
On 9 April 2013 15:16, Steven Rostedt rost...@goodmis.org wrote: On Tue, 2013-04-09 at 14:18 +0200, Vincent Guittot wrote: On 9 April 2013 10:55, Peter Zijlstra pet...@infradead.org wrote: On Thu, 2013-04-04 at 16:15 +0200, Vincent Guittot wrote: Changes since V2: - remove useless