clocksource: Resolve cpu hotplug dead lock with TSC unstable
authorThomas Gleixner <tglx@linutronix.de>
Fri, 28 Aug 2009 18:25:24 +0000 (20:25 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Fri, 28 Aug 2009 18:25:24 +0000 (20:25 +0200)
commit7285dd7fd375763bfb8ab1ac9cf3f1206f503c16
tree42f809ab3616cc3d93d655acccfc2d54e9f6d0e4
parent90cba64a5f672a239f43ec5cb9a11b806887331e
clocksource: Resolve cpu hotplug dead lock with TSC unstable

Martin Schwidefsky analyzed it:
To register a clocksource the clocksource_mutex is acquired and if
necessary timekeeping_notify is called to install the clocksource as
the timekeeper clock. timekeeping_notify uses stop_machine which needs
to take cpu_add_remove_lock mutex.
Starting a new cpu is done with the cpu_add_remove_lock mutex held.
native_cpu_up checks the tsc of the new cpu and if the tsc is no good
clocksource_change_rating is called. Which needs the clocksource_mutex
and the deadlock is complete.

The solution is to replace the TSC via the clocksource watchdog
mechanism. Mark the TSC as unstable and schedule the watchdog work so
it gets removed in the watchdog thread context.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
LKML-Reference: <new-submission>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: John Stultz <johnstul@us.ibm.com>
arch/x86/kernel/tsc.c
include/linux/clocksource.h
kernel/time/clocksource.c