diff options
author | Sebastien Dugue <sebastien.dugue@bull.net> | 2008-12-01 08:09:07 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-12-30 01:37:19 -0500 |
commit | 5762ba1873b0bb9faa631aaa02f533c2b9837f82 (patch) | |
tree | 3ea61b03a12ec16620f8527c4c83cb90d80cf6cc /kernel/hrtimer.c | |
parent | 0ce74d9296c971b2355c26984ad0bc538e34dd6c (diff) |
hrtimers: allow the hot-unplugging of all cpus
Impact: fix CPU hotplug hang on Power6 testbox
On architectures that support offlining all cpus (at least powerpc/pseries),
hot-unpluging the tick_do_timer_cpu can result in a system hang.
This comes from the fact that if the cpu going down happens to be the
cpu doing the tick, then as the tick_do_timer_cpu handover happens after the
cpu is dead (via the CPU_DEAD notification), we're left without ticks,
jiffies are frozen and any task relying on timers (msleep, ...) is stuck.
That's particularly the case for the cpu looping in __cpu_die() waiting
for the dying cpu to be dead.
This patch addresses this by having the tick_do_timer_cpu handover happen
earlier during the CPU_DYING notification. For this, a new clockevent
notification type is introduced (CLOCK_EVT_NOTIFY_CPU_DYING) which is triggered
in hrtimer_cpu_notify().
Signed-off-by: Sebastien Dugue <sebastien.dugue@bull.net>
Cc: <stable@kernel.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/hrtimer.c')
-rw-r--r-- | kernel/hrtimer.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c index c2a69b89ac61..61cb933395ba 100644 --- a/kernel/hrtimer.c +++ b/kernel/hrtimer.c | |||
@@ -1609,6 +1609,10 @@ static int __cpuinit hrtimer_cpu_notify(struct notifier_block *self, | |||
1609 | break; | 1609 | break; |
1610 | 1610 | ||
1611 | #ifdef CONFIG_HOTPLUG_CPU | 1611 | #ifdef CONFIG_HOTPLUG_CPU |
1612 | case CPU_DYING: | ||
1613 | case CPU_DYING_FROZEN: | ||
1614 | clockevents_notify(CLOCK_EVT_NOTIFY_CPU_DYING, &scpu); | ||
1615 | break; | ||
1612 | case CPU_DEAD: | 1616 | case CPU_DEAD: |
1613 | case CPU_DEAD_FROZEN: | 1617 | case CPU_DEAD_FROZEN: |
1614 | { | 1618 | { |