Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-30 Thread Vincent Guittot
On 27 May 2016 at 22:38, Dietmar Eggemann wrote: > On 27/05/16 18:16, Vincent Guittot wrote: >> On 27 May 2016 at 17:48, Dietmar Eggemann wrote: >>> On 25/05/16 16:01, Vincent Guittot wrote: The cfs_rq->avg.last_update_time is initialize to 0 with the main effect that the 1st sched_enti

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-30 Thread Vincent Guittot
On 27 May 2016 at 22:38, Dietmar Eggemann wrote: > On 27/05/16 18:16, Vincent Guittot wrote: >> On 27 May 2016 at 17:48, Dietmar Eggemann wrote: >>> On 25/05/16 16:01, Vincent Guittot wrote: The cfs_rq->avg.last_update_time is initialize to 0 with the main effect that the 1st sched_enti

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-27 Thread Dietmar Eggemann
On 27/05/16 18:16, Vincent Guittot wrote: > On 27 May 2016 at 17:48, Dietmar Eggemann wrote: >> On 25/05/16 16:01, Vincent Guittot wrote: >>> The cfs_rq->avg.last_update_time is initialize to 0 with the main effect >>> that the 1st sched_entity that will be attached, will keep its >>> last_update_

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-27 Thread Vincent Guittot
On 27 May 2016 at 17:48, Dietmar Eggemann wrote: > On 25/05/16 16:01, Vincent Guittot wrote: >> The cfs_rq->avg.last_update_time is initialize to 0 with the main effect >> that the 1st sched_entity that will be attached, will keep its >> last_update_time set to 0 and will attached once again durin

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-27 Thread Dietmar Eggemann
On 25/05/16 16:01, Vincent Guittot wrote: > The cfs_rq->avg.last_update_time is initialize to 0 with the main effect > that the 1st sched_entity that will be attached, will keep its > last_update_time set to 0 and will attached once again during the > enqueue. > Initialize cfs_rq->avg.last_update_t

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-26 Thread Vincent Guittot
On 26 May 2016 at 02:40, Yuyang Du wrote: > On Thu, May 26, 2016 at 10:26:54AM +0200, Vincent Guittot wrote: >> On 26 May 2016 at 00:38, Yuyang Du wrote: >> > On Wed, May 25, 2016 at 05:01:11PM +0200, Vincent Guittot wrote: >> >> The cfs_rq->avg.last_update_time is initialize to 0 with the main e

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-26 Thread Yuyang Du
On Thu, May 26, 2016 at 10:26:54AM +0200, Vincent Guittot wrote: > On 26 May 2016 at 00:38, Yuyang Du wrote: > > On Wed, May 25, 2016 at 05:01:11PM +0200, Vincent Guittot wrote: > >> The cfs_rq->avg.last_update_time is initialize to 0 with the main effect > >> that the 1st sched_entity that will b

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-26 Thread Vincent Guittot
On 26 May 2016 at 00:38, Yuyang Du wrote: > On Wed, May 25, 2016 at 05:01:11PM +0200, Vincent Guittot wrote: >> The cfs_rq->avg.last_update_time is initialize to 0 with the main effect >> that the 1st sched_entity that will be attached, will keep its >> last_update_time set to 0 and will attached

Re: [PATCH v2] sched: fix first task of a task group is attached twice

2016-05-25 Thread Yuyang Du
On Wed, May 25, 2016 at 05:01:11PM +0200, Vincent Guittot wrote: > The cfs_rq->avg.last_update_time is initialize to 0 with the main effect > that the 1st sched_entity that will be attached, will keep its > last_update_time set to 0 and will attached once again during the > enqueue. > Initialize cf

[PATCH v2] sched: fix first task of a task group is attached twice

2016-05-25 Thread Vincent Guittot
The cfs_rq->avg.last_update_time is initialize to 0 with the main effect that the 1st sched_entity that will be attached, will keep its last_update_time set to 0 and will attached once again during the enqueue. Initialize cfs_rq->avg.last_update_time to 1 instead. Signed-off-by: Vincent Guittot -