Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-30 Thread Yuyang Du
Hi Vincent, On Fri, May 27, 2016 at 03:37:11PM +0200, Vincent Guittot wrote: > On 26 May 2016 at 21:44, Yuyang Du wrote: > > Hi Vincent, > > > > On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: > >> On 26 May 2016 at 03:14, Yuyang Du

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-30 Thread Yuyang Du
Hi Vincent, On Fri, May 27, 2016 at 03:37:11PM +0200, Vincent Guittot wrote: > On 26 May 2016 at 21:44, Yuyang Du wrote: > > Hi Vincent, > > > > On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: > >> On 26 May 2016 at 03:14, Yuyang Du wrote: > >> > Vincent reported that the first

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-27 Thread Vincent Guittot
On 26 May 2016 at 21:44, Yuyang Du wrote: > Hi Vincent, > > On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: >> On 26 May 2016 at 03:14, Yuyang Du wrote: >> > Vincent reported that the first task to a new task group's cfs_rq will >> > be

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-27 Thread Vincent Guittot
On 26 May 2016 at 21:44, Yuyang Du wrote: > Hi Vincent, > > On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: >> On 26 May 2016 at 03:14, Yuyang Du wrote: >> > Vincent reported that the first task to a new task group's cfs_rq will >> > be attached in attach_task_cfs_rq() and once

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Yuyang Du
Hi Vincent, On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: > On 26 May 2016 at 03:14, Yuyang Du wrote: > > Vincent reported that the first task to a new task group's cfs_rq will > > be attached in attach_task_cfs_rq() and once more when it is enqueued > >

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Yuyang Du
Hi Vincent, On Thu, May 26, 2016 at 01:50:56PM +0200, Vincent Guittot wrote: > On 26 May 2016 at 03:14, Yuyang Du wrote: > > Vincent reported that the first task to a new task group's cfs_rq will > > be attached in attach_task_cfs_rq() and once more when it is enqueued > > (see

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Vincent Guittot
On 26 May 2016 at 03:14, Yuyang Du wrote: > Vincent reported that the first task to a new task group's cfs_rq will > be attached in attach_task_cfs_rq() and once more when it is enqueued > (see https://lkml.org/lkml/2016/5/25/388). > > Actually, it is worse,

Re: [PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Vincent Guittot
On 26 May 2016 at 03:14, Yuyang Du wrote: > Vincent reported that the first task to a new task group's cfs_rq will > be attached in attach_task_cfs_rq() and once more when it is enqueued > (see https://lkml.org/lkml/2016/5/25/388). > > Actually, it is worse, attach_task_cfs_rq() is called for new

[PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Yuyang Du
Vincent reported that the first task to a new task group's cfs_rq will be attached in attach_task_cfs_rq() and once more when it is enqueued (see https://lkml.org/lkml/2016/5/25/388). Actually, it is worse, attach_task_cfs_rq() is called for new task even way before

[PATCH 2/2] sched/fair: Skip detach and attach load avgs for new group task

2016-05-26 Thread Yuyang Du
Vincent reported that the first task to a new task group's cfs_rq will be attached in attach_task_cfs_rq() and once more when it is enqueued (see https://lkml.org/lkml/2016/5/25/388). Actually, it is worse, attach_task_cfs_rq() is called for new task even way before