alpha: Change do_timer() to xtime_update()
xtime_update() takes the xtime_lock itself. timer_interrupt() is only called on the boot cpu. See do_entInt(). So "state" in timer_interrupt does not require protection by xtime_lock. Signed-off-by: Torben Hohn <torbenh@gmx.de> Cc: Richard Henderson <rth@twiddle.net> Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru> Cc: Matt Turner <mattst88@gmail.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: johnstul@us.ibm.com Cc: hch@infradead.org Cc: yong.zhang0@gmail.com LKML-Reference: <20110127145915.23248.20919.stgit@localhost> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
This commit is contained in:
parent
f0af911a9d
commit
1340f3e0b2
|
@ -159,7 +159,7 @@ void read_persistent_clock(struct timespec *ts)
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* timer_interrupt() needs to keep up the real-time clock,
|
* timer_interrupt() needs to keep up the real-time clock,
|
||||||
* as well as call the "do_timer()" routine every clocktick
|
* as well as call the "xtime_update()" routine every clocktick
|
||||||
*/
|
*/
|
||||||
irqreturn_t timer_interrupt(int irq, void *dev)
|
irqreturn_t timer_interrupt(int irq, void *dev)
|
||||||
{
|
{
|
||||||
|
@ -172,8 +172,6 @@ irqreturn_t timer_interrupt(int irq, void *dev)
|
||||||
profile_tick(CPU_PROFILING);
|
profile_tick(CPU_PROFILING);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
write_seqlock(&xtime_lock);
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Calculate how many ticks have passed since the last update,
|
* Calculate how many ticks have passed since the last update,
|
||||||
* including any previous partial leftover. Save any resulting
|
* including any previous partial leftover. Save any resulting
|
||||||
|
@ -187,9 +185,7 @@ irqreturn_t timer_interrupt(int irq, void *dev)
|
||||||
nticks = delta >> FIX_SHIFT;
|
nticks = delta >> FIX_SHIFT;
|
||||||
|
|
||||||
if (nticks)
|
if (nticks)
|
||||||
do_timer(nticks);
|
xtime_update(nticks);
|
||||||
|
|
||||||
write_sequnlock(&xtime_lock);
|
|
||||||
|
|
||||||
if (test_irq_work_pending()) {
|
if (test_irq_work_pending()) {
|
||||||
clear_irq_work_pending();
|
clear_irq_work_pending();
|
||||||
|
|
Loading…
Reference in New Issue