diff options
author | Steven Rostedt <srostedt@redhat.com> | 2010-12-06 14:48:10 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2010-12-13 19:56:40 -0500 |
commit | 63f01241176d7cbc976385aec32f0a209b0bc36a (patch) | |
tree | f09c81454139ff3364b69f185d52cd82ffa203a1 /kernel/sched_rt.c | |
parent | 8e54a2c036d8c47195f094af1628834f4c55844a (diff) | |
download | linux-63f01241176d7cbc976385aec32f0a209b0bc36a.tar.gz linux-63f01241176d7cbc976385aec32f0a209b0bc36a.tar.bz2 linux-63f01241176d7cbc976385aec32f0a209b0bc36a.zip |
sched: Remove unlikely() from rt_policy() in sched.c
The rt_policy() has an unlikely() that the policy it is checking is
of RT priority (SCHED_FIFO or SCHED_RR).
According to the annotate branch profiler it is incorrect most of the time:
correct incorrect % Function File Line
------- --------- - -------- ---- ----
36667 654674 94 rt_policy sched.c 126
This makes sense because the rt_policy() is used by the sched_set_scheduler()
and nice(). Although users may use sys_nice a bit, all RT users use
the sched_set_scheduler() to set their RT priority, including kernel
threads.
The above numbers were from a normal desktop computer running
firefox, evolution, xchat and was part of a distcc compile farm.
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/sched_rt.c')
0 files changed, 0 insertions, 0 deletions