diff options
| author | Peter Zijlstra | 2009-02-11 14:27:17 +0100 |
|---|---|---|
| committer | Ingo Molnar | 2009-02-11 14:43:35 +0100 |
| commit | fc631c82e1734d718ff0832558f64c8f5d185f26 (patch) | |
| tree | 7bef1c5f3f2c3f454280026e1a528de526302827 /kernel/sched.c | |
| parent | sched: fix nohz load balancer on cpu offline (diff) | |
| download | kernel-qcow2-linux-fc631c82e1734d718ff0832558f64c8f5d185f26.tar.gz kernel-qcow2-linux-fc631c82e1734d718ff0832558f64c8f5d185f26.tar.xz kernel-qcow2-linux-fc631c82e1734d718ff0832558f64c8f5d185f26.zip | |
sched: revert recent sync wakeup changes
Intel reported a 10% regression (mysql+sysbench) on a 16-way machine
with these patches:
1596e29: sched: symmetric sync vs avg_overlap
d942fb6: sched: fix sync wakeups
Revert them.
Reported-by: "Zhang, Yanmin" <yanmin_zhang@linux.intel.com>
Bisected-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched.c')
| -rw-r--r-- | kernel/sched.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index e1fc67d0674c..f11c02b86c73 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -2266,16 +2266,6 @@ static int try_to_wake_up(struct task_struct *p, unsigned int state, int sync) if (!sched_feat(SYNC_WAKEUPS)) sync = 0; - if (!sync) { - if (current->se.avg_overlap < sysctl_sched_migration_cost && - p->se.avg_overlap < sysctl_sched_migration_cost) - sync = 1; - } else { - if (current->se.avg_overlap >= sysctl_sched_migration_cost || - p->se.avg_overlap >= sysctl_sched_migration_cost) - sync = 0; - } - #ifdef CONFIG_SMP if (sched_feat(LB_WAKEUP_UPDATE)) { struct sched_domain *sd; |