Re: [PATCH v4 1/6] sched/fair: Fix reschedule which is generated on throttled cfs_rq

2014-10-24 Thread Kirill Tkhai
В Пт, 24/10/2014 в 17:32 +0800, Wanpeng Li пишет: > Hi Kirill, > 10/24/14, 2:01 PM, Kirill Tkhai: > > Hi, Wanpeng, > > > > the commit commentary confuses, I'm agree. Really it's just a cleanup. > > > > On Пт, 2014-10-24 at 07:27 +0800, Wanpeng Li wrote: > >> Hi Kirill, > >> 8/6/14, 4:06 PM, Kirill

Re: [PATCH v4 1/6] sched/fair: Fix reschedule which is generated on throttled cfs_rq

2014-10-24 Thread Wanpeng Li
Hi Kirill, 10/24/14, 2:01 PM, Kirill Tkhai: Hi, Wanpeng, the commit commentary confuses, I'm agree. Really it's just a cleanup. On Пт, 2014-10-24 at 07:27 +0800, Wanpeng Li wrote: Hi Kirill, 8/6/14, 4:06 PM, Kirill Tkhai: (sched_entity::on_rq == 1) does not guarantee the task is pickable; cha

Re: [PATCH v4 1/6] sched/fair: Fix reschedule which is generated on throttled cfs_rq

2014-10-23 Thread Kirill Tkhai
Hi, Wanpeng, the commit commentary confuses, I'm agree. Really it's just a cleanup. On Пт, 2014-10-24 at 07:27 +0800, Wanpeng Li wrote: > Hi Kirill, > 8/6/14, 4:06 PM, Kirill Tkhai: > > (sched_entity::on_rq == 1) does not guarantee the task is pickable; > > changes on throttled cfs_rq must not le

Re: [PATCH v4 1/6] sched/fair: Fix reschedule which is generated on throttled cfs_rq

2014-10-23 Thread Wanpeng Li
Hi Kirill, 8/6/14, 4:06 PM, Kirill Tkhai: (sched_entity::on_rq == 1) does not guarantee the task is pickable; changes on throttled cfs_rq must not lead to reschedule. Why (sched_entity::on_rq == 1) doesn't guarantee the task is pickable since entity will be dequeued during throttling cfs_rq?

[PATCH v4 1/6] sched/fair: Fix reschedule which is generated on throttled cfs_rq

2014-08-06 Thread Kirill Tkhai
(sched_entity::on_rq == 1) does not guarantee the task is pickable; changes on throttled cfs_rq must not lead to reschedule. Check for task_struct::on_rq instead. Signed-off-by: Kirill Tkhai --- kernel/sched/fair.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ke