diff options
| author | Steven Rostedt <[email protected]> | 2010-12-06 17:10:31 -0500 |
|---|---|---|
| committer | Steven Rostedt <[email protected]> | 2010-12-13 19:56:43 -0500 |
| commit | e69c634190dc724ef2d845ace8d783031d3e492e (patch) | |
| tree | 03c16d8f6f3ec24408adbe9042bc42546725141a /kernel/rtmutex-debug.c | |
| parent | 63f01241176d7cbc976385aec32f0a209b0bc36a (diff) | |
sched: Remove unlikely() from ttwu_post_activation
The unlikely() used in ttwu_post_activation() tests if the rq->idle_stamp
is set. But since this is for a wakeup, and wakeups happen when tasks
block on IO, and blocking tasks on IO may put the system into idle,
this can actually be a common occurence.
Running the annotated branch profiler on an average desktop running
firefox, evolution, xchat and distcc, the report shows:
correct incorrect % Function File Line
------- --------- - -------- ---- ----
34884862 146110926 80 ttwu_post_activation sched.c 2309
80% of the time, this unlikely is incorrect. Best not to assume what the
result is, and just remove the branch annotation.
Cc: Peter Zijlstra <[email protected]>
Signed-off-by: Steven Rostedt <[email protected]>
Diffstat (limited to 'kernel/rtmutex-debug.c')
0 files changed, 0 insertions, 0 deletions