diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2008-05-13 17:44:11 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-05-29 05:25:14 -0400 |
commit | 3f33a7ce9567ded582af1ab71f9802165fe12f09 (patch) | |
tree | 04304757dc84bc36400f306677d3452635cf9f74 | |
parent | f9305d4a0968201b2818dbed0dc8cb0d4ee7aeb3 (diff) |
sched: unite unlikely pairs in rt_policy() and schedule_debug()
Removes obfuscation and may improve assembly.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/sched.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 4aac8aa16037..97017356669a 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -136,7 +136,7 @@ static inline void sg_inc_cpu_power(struct sched_group *sg, u32 val) | |||
136 | 136 | ||
137 | static inline int rt_policy(int policy) | 137 | static inline int rt_policy(int policy) |
138 | { | 138 | { |
139 | if (unlikely(policy == SCHED_FIFO) || unlikely(policy == SCHED_RR)) | 139 | if (unlikely(policy == SCHED_FIFO || policy == SCHED_RR)) |
140 | return 1; | 140 | return 1; |
141 | return 0; | 141 | return 0; |
142 | } | 142 | } |
@@ -4433,7 +4433,7 @@ static inline void schedule_debug(struct task_struct *prev) | |||
4433 | * schedule() atomically, we ignore that path for now. | 4433 | * schedule() atomically, we ignore that path for now. |
4434 | * Otherwise, whine if we are scheduling when we should not be. | 4434 | * Otherwise, whine if we are scheduling when we should not be. |
4435 | */ | 4435 | */ |
4436 | if (unlikely(in_atomic_preempt_off()) && unlikely(!prev->exit_state)) | 4436 | if (unlikely(in_atomic_preempt_off() && !prev->exit_state)) |
4437 | __schedule_bug(prev); | 4437 | __schedule_bug(prev); |
4438 | 4438 | ||
4439 | profile_hit(SCHED_PROFILING, __builtin_return_address(0)); | 4439 | profile_hit(SCHED_PROFILING, __builtin_return_address(0)); |