diff options
author | Yong Zhang <yong.zhang0@gmail.com> | 2011-01-24 00:59:17 +0000 |
---|---|---|
committer | John Rigby <john.rigby@linaro.org> | 2011-09-23 08:48:08 -0600 |
commit | 9d3ca062117f8587aeafa32a183845c117736941 (patch) | |
tree | 88bec1df15a505f3741004477216b845dfc556db /kernel | |
parent | b47391c140c05dcb911bc73a44a1f90f3aea34d0 (diff) |
sched: tg->se->load should be initialised to tg->shares
Michael reported that when enable autogroup on UP, system
responsiveness becomes very bad.
Because in init_tg_cfs_entry() we initialise se->load
to 0 instead of tg->shares, in the end we have 0-weight
sched entity on rq, then lead to misbehavior.
Reported-by: Michael Witten <mfwitten@gmail.com>
Reported-by: Christian Kujau <christian@nerdbynature.de>
Signed-off-by: Yong Zhang <yong.zhang0@gmail.com>
Signed-off-by: Andy Whitcroft <apw@canonical.com>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index fde6ff90352..b0004c1f100 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -7972,7 +7972,7 @@ static void init_tg_cfs_entry(struct task_group *tg, struct cfs_rq *cfs_rq, se->cfs_rq = parent->my_q; se->my_q = cfs_rq; - update_load_set(&se->load, 0); + update_load_set(&se->load, tg->shares); se->parent = parent; } #endif |