| Subject | [PATCH v2 11/20] frv: switch do_timer() to xtime_update() | From | Torben Hohn <> | Date | Thu, 27 Jan 2011 15:59:46 +0100 |
| |
__set_LEDS() does not need to be protected by the lock. its used unprotected in other places.
Cc: David Howells <dhowells@redhat.com> Signed-off-by: Torben Hohn <torbenh@gmx.de> --- arch/frv/kernel/time.c | 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/arch/frv/kernel/time.c b/arch/frv/kernel/time.c index 0ddbbae..dde2a8e 100644 --- a/arch/frv/kernel/time.c +++ b/arch/frv/kernel/time.c @@ -50,7 +50,7 @@ static struct irqaction timer_irq = { /* * 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 */ static irqreturn_t timer_interrupt(int irq, void *dummy) { @@ -61,10 +61,11 @@ static irqreturn_t timer_interrupt(int irq, void *dummy) * CPU. We need to avoid to SMP race with it. NOTE: we don't need * the irq version of write_lock because as just said we have irq * locally disabled. -arca + * + * xtime_update() does poper locking. */ - write_seqlock(&xtime_lock); - do_timer(1); + xtime_update(1); #ifdef CONFIG_HEARTBEAT static unsigned short n; @@ -72,8 +73,6 @@ static irqreturn_t timer_interrupt(int irq, void *dummy) __set_LEDS(n); #endif /* CONFIG_HEARTBEAT */ - write_sequnlock(&xtime_lock); - update_process_times(user_mode(get_irq_regs())); return IRQ_HANDLED;
|