Commit d7cfb60c5cf904ecf1e0ae23ec178175b86f0d4a

Authored by Mark McLoughlin
Committed by Ingo Molnar
1 parent d82f0b0f6f

hrtimer: remove hrtimer_clock_base::get_softirq_time()

Peter Zijlstra noticed this 8 months ago and I just noticed
it again.

hrtimer_clock_base::get_softirq_time() is currently unused
in the entire tree. In fact, looking at the logs, it appears
as if it was never used. Remove it.

Signed-off-by: Mark McLoughlin <markmc@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

Showing 2 changed files with 1 additions and 5 deletions Side-by-side Diff

include/linux/hrtimer.h
... ... @@ -145,7 +145,6 @@
145 145 * @first: pointer to the timer node which expires first
146 146 * @resolution: the resolution of the clock, in nanoseconds
147 147 * @get_time: function to retrieve the current time of the clock
148   - * @get_softirq_time: function to retrieve the current time from the softirq
149 148 * @softirq_time: the time when running the hrtimer queue in the softirq
150 149 * @offset: offset of this clock to the monotonic base
151 150 * @reprogram: function to reprogram the timer event
... ... @@ -157,7 +156,6 @@
157 156 struct rb_node *first;
158 157 ktime_t resolution;
159 158 ktime_t (*get_time)(void);
160   - ktime_t (*get_softirq_time)(void);
161 159 ktime_t softirq_time;
162 160 #ifdef CONFIG_HIGH_RES_TIMERS
163 161 ktime_t offset;
... ... @@ -1401,9 +1401,7 @@
1401 1401 if (!base->first)
1402 1402 continue;
1403 1403  
1404   - if (base->get_softirq_time)
1405   - base->softirq_time = base->get_softirq_time();
1406   - else if (gettime) {
  1404 + if (gettime) {
1407 1405 hrtimer_get_softirq_time(cpu_base);
1408 1406 gettime = 0;
1409 1407 }