summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWill Deacon2011-07-20 15:18:46 +0200
committerWill Deacon2011-08-12 16:41:08 +0200
commitdfc40b24c0a37593724f3317cd485c73ee878c18 (patch)
tree03d500255c01936d8c2dbee8106faee13a37211f
parentARM: realview: ensure visibility of writes during reset (diff)
downloadkernel-qcow2-linux-dfc40b24c0a37593724f3317cd485c73ee878c18.tar.gz
kernel-qcow2-linux-dfc40b24c0a37593724f3317cd485c73ee878c18.tar.xz
kernel-qcow2-linux-dfc40b24c0a37593724f3317cd485c73ee878c18.zip
ARM: twd: register clockevents device before enabling PPI
The smp_twd clockevents driver currently enables the local timer PPI before the clockevents device is registered. This can lead to a kernel panic if a spurious timer interrupt is generated before registration has completed since the kernel will treat it as an IPI timer. This patch moves the clockevents device registration before the IRQ unmasking so that we can always handle timer interrupts once they can occur. Acked-by: Marc Zyngier <marc.zyngier@arm.com> Signed-off-by: Will Deacon <will.deacon@arm.com>
-rw-r--r--arch/arm/kernel/smp_twd.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm/kernel/smp_twd.c b/arch/arm/kernel/smp_twd.c
index 2c277d40cee6..01c186222f3b 100644
--- a/arch/arm/kernel/smp_twd.c
+++ b/arch/arm/kernel/smp_twd.c
@@ -137,8 +137,8 @@ void __cpuinit twd_timer_setup(struct clock_event_device *clk)
clk->max_delta_ns = clockevent_delta2ns(0xffffffff, clk);
clk->min_delta_ns = clockevent_delta2ns(0xf, clk);
+ clockevents_register_device(clk);
+
/* Make sure our local interrupt controller has this enabled */
gic_enable_ppi(clk->irq);
-
- clockevents_register_device(clk);
}