diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2011-12-02 16:02:45 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-12-09 08:52:53 -0800 |
| commit | b01b383bbd04e9dcf7d9fe6ca3751b77ccdc533c (patch) | |
| tree | 011972826f7b83c5aafece5ef589e93a4d011508 | |
| parent | 4078977c46f627f553ed2d8ea047b9bf25dee48d (diff) | |
| download | lwn-b01b383bbd04e9dcf7d9fe6ca3751b77ccdc533c.tar.gz lwn-b01b383bbd04e9dcf7d9fe6ca3751b77ccdc533c.zip | |
clockevents: Set noop handler in clockevents_exchange_device()
commit de28f25e8244c7353abed8de0c7792f5f883588c upstream.
If a device is shutdown, then there might be a pending interrupt,
which will be processed after we reenable interrupts, which causes the
original handler to be run. If the old handler is the (broadcast)
periodic handler the shutdown state might hang the kernel completely.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | kernel/time/clockevents.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/time/clockevents.c b/kernel/time/clockevents.c index e4c699dfa4e8..13dfaaba4061 100644 --- a/kernel/time/clockevents.c +++ b/kernel/time/clockevents.c @@ -286,6 +286,7 @@ void clockevents_exchange_device(struct clock_event_device *old, * released list and do a notify add later. */ if (old) { + old->event_handler = clockevents_handle_noop; clockevents_set_mode(old, CLOCK_EVT_MODE_UNUSED); list_del(&old->list); list_add(&old->list, &clockevents_released); |