clockevents: check broadcast device not tick device
authorThomas Gleixner <tglx@linutronix.de>
Mon, 22 Sep 2008 17:02:25 +0000 (19:02 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 23 Sep 2008 09:38:53 +0000 (11:38 +0200)
commit302745699c1b675b5d2a1af87271de10e4d96b6a
tree4efc6c93c038e14021fa10b1eb358695f45f2130
parent49d670fb8dd62d3ed4e3ed2513538ea65b051aed
clockevents: check broadcast device not tick device

Impact: Possible hang on CPU online observed on AMD C1E machines.

The broadcast setup code looks at the mode of the tick device to
determine whether it needs to be shut down or setup. This is wrong
when the broadcast mode is set to one shot already. This can happen
when a CPU is brought online as it goes through the periodic setup
first.

The problem went unnoticed as sane systems do not call into that code
before the switch to one shot for the clock event device happens.
The AMD C1E idle routine switches over immediately and thereby shuts
down the just setup device before the first interrupt happens.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/time/tick-broadcast.c