sched/rt: Code cleanup, remove a redundant function call
authorShan Hai <haishan.bai@gmail.com>
Tue, 29 Nov 2011 03:03:56 +0000 (11:03 +0800)
committerIngo Molnar <mingo@elte.hu>
Tue, 6 Dec 2011 08:06:28 +0000 (09:06 +0100)
The second call to sched_rt_period() is redundant, because the value of the
rt_runtime was already read and it was protected by the ->rt_runtime_lock.

Signed-off-by: Shan Hai <haishan.bai@gmail.com>
Reviewed-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1322535836-13590-2-git-send-email-haishan.bai@gmail.com
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/sched/rt.c

index 58a48444e5c931dfabb8123470e4447ef592dff9..3640ebbb466b0bb82d2894ec8223c96bfe502b74 100644 (file)
@@ -846,7 +846,7 @@ static int sched_rt_runtime_exceeded(struct rt_rq *rt_rq)
        if (rt_rq->rt_throttled)
                return rt_rq_throttled(rt_rq);
 
-       if (sched_rt_runtime(rt_rq) >= sched_rt_period(rt_rq))
+       if (runtime >= sched_rt_period(rt_rq))
                return 0;
 
        balance_runtime(rt_rq);