diff options
| author | Alan Cox <alan@redhat.com> | 2008-10-14 11:29:06 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-14 10:03:33 -0700 |
| commit | 4aba41ea8bdc1b475861f5e5c1649ab20251090c (patch) | |
| tree | 3a39612ac27c742d4999f1c411ee23a9eedde71f /drivers/serial | |
| parent | 7591103c08abade60aeddb432ed0686ddd0de1c6 (diff) | |
| download | op-kernel-dev-4aba41ea8bdc1b475861f5e5c1649ab20251090c.zip op-kernel-dev-4aba41ea8bdc1b475861f5e5c1649ab20251090c.tar.gz | |
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>
Diffstat (limited to 'drivers/serial')
| -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 d4104a3b..d3ca7d3 100644 --- a/drivers/serial/8250.c +++ b/drivers/serial/8250.c @@ -2969,6 +2969,9 @@ static int __init serial8250_init(void) "%d ports, IRQ sharing %sabled\n", nr_uarts, share_irqs ? "en" : "dis"); + for (i = 0; i < NR_IRQS; i++) + spin_lock_init(&irq_lists[i].lock); + #ifdef CONFIG_SPARC ret = sunserial_register_minors(&serial8250_reg, UART_NR); #else |
