]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commit - kernel/time/timer.c
timer: Implement TIMER_IRQSAFE
authorTejun Heo <tj@kernel.org>
Wed, 8 Aug 2012 18:10:28 +0000 (11:10 -0700)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 21 Aug 2012 14:28:31 +0000 (16:28 +0200)
commitc5f66e99b7cb091e3d51ae8e8156892e8feb7fa3
treeb73b64ceabad685abd9a2bc24f0258d9dbfae367
parentfc683995a6c4e604d62ab9a488ac2c1ba94fa868
timer: Implement TIMER_IRQSAFE

Timer internals are protected with irq-safe locks but timer execution
isn't, so a timer being dequeued for execution and its execution
aren't atomic against IRQs.  This makes it impossible to wait for its
completion from IRQ handlers and difficult to shoot down a timer from
IRQ handlers.

This issue caused some issues for delayed_work interface.  Because
there's no way to reliably shoot down delayed_work->timer from IRQ
handlers, __cancel_delayed_work() can't share the logic to steal the
target delayed_work with cancel_delayed_work_sync(), and can only
steal delayed_works which are on queued on timer.  Similarly, the
pending mod_delayed_work() can't be used from IRQ handlers.

This patch adds a new timer flag TIMER_IRQSAFE, which makes the timer
to be executed without enabling IRQ after dequeueing such that its
dequeueing and execution are atomic against IRQ handlers.

This makes it safe to wait for the timer's completion from IRQ
handlers, for example, using del_timer_sync().  It can never be
executing on the local CPU and if executing on other CPUs it won't be
interrupted until done.

This will enable simplifying delayed_work cancel/mod interface.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: torvalds@linux-foundation.org
Cc: peterz@infradead.org
Link: http://lkml.kernel.org/r/1344449428-24962-5-git-send-email-tj@kernel.org
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
include/linux/timer.h
kernel/timer.c