diff options
author | Alan Cox <alan@redhat.com> | 2008-10-14 06:29:06 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-14 13:03:33 -0400 |
commit | 4aba41ea8bdc1b475861f5e5c1649ab20251090c (patch) | |
tree | 3a39612ac27c742d4999f1c411ee23a9eedde71f | |
parent | 7591103c08abade60aeddb432ed0686ddd0de1c6 (diff) |
8250: Fix lock warning (and possible crash)
Splitting the 8250 code back up to avoid a clash with the NR_IRQS removal
patch introduced a last minute bug. Put back the additional needed lines
for the old lock init
Signed-off-by: Alan Cox <alan@redhat.com>
[ Ingo also reports that this can cause a spontaneous reboot crash with
certain configs, and sends in an identical patch ]
Tested-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/serial/8250.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c index d4104a3bbe87..d3ca7d32abe0 100644 --- a/drivers/serial/8250.c +++ b/drivers/serial/8250.c | |||
@@ -2969,6 +2969,9 @@ static int __init serial8250_init(void) | |||
2969 | "%d ports, IRQ sharing %sabled\n", nr_uarts, | 2969 | "%d ports, IRQ sharing %sabled\n", nr_uarts, |
2970 | share_irqs ? "en" : "dis"); | 2970 | share_irqs ? "en" : "dis"); |
2971 | 2971 | ||
2972 | for (i = 0; i < NR_IRQS; i++) | ||
2973 | spin_lock_init(&irq_lists[i].lock); | ||
2974 | |||
2972 | #ifdef CONFIG_SPARC | 2975 | #ifdef CONFIG_SPARC |
2973 | ret = sunserial_register_minors(&serial8250_reg, UART_NR); | 2976 | ret = sunserial_register_minors(&serial8250_reg, UART_NR); |
2974 | #else | 2977 | #else |