Skip to contentSkip to conversation

hrtimers: Remove unnecessary clock events device programming

RFC PATCH New Ashwin Chaugule 1 participant 1 message Aug 27, 2009

lore
Share & tools
lore
#0 lore raw

cover hrtimers: Remove unnecessary clock events device programming

hrtimer_force_reprogram() was being called quite often unnecessarily. The following patch caches the hrtimer that causes the cpu_base->expires_next value to change. This way, we force the reprogram only when remove_hrtimer() finds that its going to remove an hrtimer which was about expire next. IOW, we forgo reprogramming the clock events device, if the current timer to be removed isn't at the root of the expires tree.

Also removed the hrtimer_is_queued check from remove_hrtimer and made appropriate changes to all callers. That check seemed redundant.

Results:

Shows ~60% reduction in the following calls to program clock events device.
- tick_nohz_restart_sched_tick (tick_sched_timer)
- hrtimer_start (tick_sched_timer)

I haven't traced all the callers yet, this was from a hack to timer_stats. If you guys have a better way to test this, I'll give that a go.

Tested on v2.6.29 on an ARM based embedded device.

Cheers,
Ashwin