path: root/kernel
diff options
authorYong Zhang <yong.zhang0@gmail.com>2011-01-24 00:59:17 +0000
committerJohn Rigby <john.rigby@linaro.org>2011-11-16 14:23:56 -0700
commitf6ccce4ecf270c831f5aba4c5796b6f38c083b4a (patch)
tree77b4e70b2aca8b4b9977ed8427e9fdb593c3a2a0 /kernel
parent16c659598189a8a549ae4bc207829257e4cd8384 (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')
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c
index 036311afeed..f36ea63a4c5 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -7999,7 +7999,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;