diff options
author | Mike Galbraith <efault@gmx.de> | 2010-03-11 11:16:43 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-03-11 12:32:53 -0500 |
commit | c6ee36c423c3ed1fb86bb3eabba9fc256a300d16 (patch) | |
tree | a7c98864e3a5c84fac57e8f0ce8777f83df81372 /kernel | |
parent | f2e74eeac03ffb779d64b66a643c5e598145a28b (diff) |
sched: Remove SYNC_WAKEUPS feature
Sync wakeups are critical functionality with a long history. Remove it, we don't
need the branch or icache footprint.
Signed-off-by: Mike Galbraith <efault@gmx.de>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <1268301817.6785.47.camel@marge.simson.net>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched.c | 3 | ||||
-rw-r--r-- | kernel/sched_features.h | 8 |
2 files changed, 0 insertions, 11 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 16559de4edea..cc6dc8caa380 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -2369,9 +2369,6 @@ static int try_to_wake_up(struct task_struct *p, unsigned int state, | |||
2369 | unsigned long flags; | 2369 | unsigned long flags; |
2370 | struct rq *rq; | 2370 | struct rq *rq; |
2371 | 2371 | ||
2372 | if (!sched_feat(SYNC_WAKEUPS)) | ||
2373 | wake_flags &= ~WF_SYNC; | ||
2374 | |||
2375 | this_cpu = get_cpu(); | 2372 | this_cpu = get_cpu(); |
2376 | 2373 | ||
2377 | smp_wmb(); | 2374 | smp_wmb(); |
diff --git a/kernel/sched_features.h b/kernel/sched_features.h index 1cb7c4701bf3..f54b6f9cc3dd 100644 --- a/kernel/sched_features.h +++ b/kernel/sched_features.h | |||
@@ -23,14 +23,6 @@ SCHED_FEAT(WAKEUP_PREEMPT, 1) | |||
23 | SCHED_FEAT(ASYM_GRAN, 1) | 23 | SCHED_FEAT(ASYM_GRAN, 1) |
24 | 24 | ||
25 | /* | 25 | /* |
26 | * Use the SYNC wakeup hint, pipes and the likes use this to indicate | ||
27 | * the remote end is likely to consume the data we just wrote, and | ||
28 | * therefore has cache benefit from being placed on the same cpu, see | ||
29 | * also AFFINE_WAKEUPS. | ||
30 | */ | ||
31 | SCHED_FEAT(SYNC_WAKEUPS, 1) | ||
32 | |||
33 | /* | ||
34 | * Based on load and program behaviour, see if it makes sense to place | 26 | * Based on load and program behaviour, see if it makes sense to place |
35 | * a newly woken task on the same cpu as the task that woke it -- | 27 | * a newly woken task on the same cpu as the task that woke it -- |
36 | * improve cache locality. Typically used with SYNC wakeups as | 28 | * improve cache locality. Typically used with SYNC wakeups as |