diff options
author | Yong Zhang <yong.zhang0@gmail.com> | 2011-01-24 00:59:17 +0000 |
---|---|---|
committer | John Rigby <john.rigby@linaro.org> | 2011-10-17 02:34:17 -0600 |
commit | 4979f9cd16932c2548de8bcada5077e37b625e0a (patch) | |
tree | 0c72b2279f7d28d0cceda2fe70fb199cb36f36f2 /kernel | |
parent | cf6b541dc2688eef89d57e8fb8a6967a42b36218 (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 8b3736076ac..d2161bb0d9f 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -7983,7 +7983,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 |