diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2008-04-19 15:31:26 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2008-04-21 01:59:51 -0400 |
commit | 259aae864ceeb2b34e7bafa1ce18d096a357fab2 (patch) | |
tree | 5adb390434df0d95497763177d729c5d6cc98d20 /kernel/hrtimer.c | |
parent | 833883d9ac4cfb31c1c4419335e68e6895a05b6b (diff) |
hrtimer: optimize the softirq time optimization
The previous optimization did not take the case into account where a
clock provides its own softirq_get_time() function.
Check for the availablitiy of the clock get time function first and
then check if we need to retrieve the time for both clocks via
hrtimer_softirq_gettime() to avoid a double evaluation of time in that
case as well.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel/hrtimer.c')
-rw-r--r-- | kernel/hrtimer.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c index 70d4adc74639..f78777abe769 100644 --- a/kernel/hrtimer.c +++ b/kernel/hrtimer.c | |||
@@ -1254,14 +1254,13 @@ void hrtimer_run_queues(void) | |||
1254 | if (!base->first) | 1254 | if (!base->first) |
1255 | continue; | 1255 | continue; |
1256 | 1256 | ||
1257 | if (gettime) { | 1257 | if (base->get_softirq_time) |
1258 | base->softirq_time = base->get_softirq_time(); | ||
1259 | else if (gettime) { | ||
1258 | hrtimer_get_softirq_time(cpu_base); | 1260 | hrtimer_get_softirq_time(cpu_base); |
1259 | gettime = 0; | 1261 | gettime = 0; |
1260 | } | 1262 | } |
1261 | 1263 | ||
1262 | if (base->get_softirq_time) | ||
1263 | base->softirq_time = base->get_softirq_time(); | ||
1264 | |||
1265 | spin_lock(&cpu_base->lock); | 1264 | spin_lock(&cpu_base->lock); |
1266 | 1265 | ||
1267 | while ((node = base->first)) { | 1266 | while ((node = base->first)) { |