public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>, Jiri Slaby <jirislaby@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	Linus Torvalds <torvalds@linux-foundation.org>
Subject: [patch] drivers/char/esp.c: fix bootup lockup (was: Re: Linux 2.6.25-rc4)
Date: Fri, 7 Mar 2008 11:08:06 +0100	[thread overview]
Message-ID: <20080307100806.GA4350@elte.hu> (raw)
In-Reply-To: <alpine.LFD.1.00.0803042044460.12253@woody.linux-foundation.org>


randconfig testing found a bootup lockup in drivers/char/esp.c - find 
the fix below. Not sure why it became more prominent in 2.6.25-rc4, the 
bug seems rather old and i've been doing allyesconfig bootups for ages 
with CONFIG_ESP enabled.

------------->
Subject: drivers/char/esp.c: fix bootup lockup
From: Ingo Molnar <mingo@elte.hu>
Date: Fri Mar 07 10:47:43 CET 2008

fix this bootup lockup:

 PM: Adding info for No Bus:ttyP63
 ttyP32 at 0x0240 (irq = 0) is an ESP primary port
 BUG: spinlock lockup on CPU#0, swapper/1, f56dd004
 Pid: 1, comm: swapper Not tainted 2.6.25-rc4-sched-devel.git-x86-latest.git #402 [<c03ac6f4>] _raw_spin_lock+0x134/0x140
  [<c08649be>] _spin_lock_irqsave+0x5e/0x80
  [<c0b9fbfe>] ? espserial_init+0x2be/0x6e0
  [<c0b9fbfe>] espserial_init+0x2be/0x6e0
  [<c0b877a3>] kernel_init+0x83/0x260
  [<c0b9f940>] ? espserial_init+0x0/0x6e0
  [<c010416a>] ? restore_nocheck_notrace+0x0/0xe
  [<c0b87720>] ? kernel_init+0x0/0x260
  [<c0b87720>] ? kernel_init+0x0/0x260
  [<c0104507>] kernel_thread_helper+0x7/0x10
  =======================

kzalloc() is not the way to initialize spinlocks anymore.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 drivers/char/esp.c |    1 +
 1 file changed, 1 insertion(+)

Index: linux/drivers/char/esp.c
===================================================================
--- linux.orig/drivers/char/esp.c
+++ linux/drivers/char/esp.c
@@ -2484,6 +2484,7 @@ static int __init espserial_init(void)
 			return 0;
 		}
 
+		spin_lock_init(&info->lock);
 		/* rx_trigger, tx_trigger are needed by autoconfig */
 		info->config.rx_trigger = rx_trigger;
 		info->config.tx_trigger = tx_trigger;

  parent reply	other threads:[~2008-03-07 10:08 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-05  5:03 Linux 2.6.25-rc4 Linus Torvalds
2008-03-05  8:09 ` FUJITA Tomonori
2008-03-05 16:46   ` Grant Grundler
2008-03-06  9:00 ` Ingo Molnar
2008-03-06 12:59   ` Jens Axboe
2008-03-06 13:06     ` Ingo Molnar
2008-03-06 13:12       ` Jens Axboe
2008-03-07  8:53       ` Ingo Molnar
2008-03-07  8:57         ` Jens Axboe
2008-03-07  9:02           ` Ingo Molnar
2008-03-07  9:59             ` Paul Mackerras
2008-03-07 15:20           ` Valdis.Kletnieks
2008-03-08 23:36         ` Pavel Machek
2008-03-09 11:59           ` Ingo Molnar
2008-03-09 12:55           ` Andi Kleen
2008-03-10 10:10             ` Pavel Machek
2008-03-10 11:52               ` Andi Kleen
2008-03-06 13:38     ` Bartlomiej Zolnierkiewicz
2008-03-06 13:33       ` Ingo Molnar
2008-03-06 14:06         ` Bartlomiej Zolnierkiewicz
2008-03-06 13:55           ` Jens Axboe
2008-03-06 21:17             ` Anders Eriksson
2008-03-07  8:48               ` Jens Axboe
2008-03-07 22:04                 ` Anders Eriksson
2008-03-08 20:22                   ` Linus Torvalds
2008-03-08 21:05                     ` Anders Eriksson
2008-03-10  8:55                       ` Anders Eriksson
2008-03-10 12:36                         ` Bartlomiej Zolnierkiewicz
2008-03-10 13:10                           ` Rafael J. Wysocki
2008-03-10 14:04                             ` Bartlomiej Zolnierkiewicz
2008-03-16 14:01                               ` Anders Eriksson
2008-03-16 14:29                                 ` Bartlomiej Zolnierkiewicz
2008-03-16 14:29                                   ` Anders Eriksson
2008-03-16 15:14                                     ` Bartlomiej Zolnierkiewicz
2008-03-16 16:56                                 ` Linus Torvalds
2008-03-16 17:13                                   ` Linus Torvalds
2008-03-16 18:18                                     ` Anders Eriksson
2008-03-16 18:07                                   ` Bartlomiej Zolnierkiewicz
2008-03-16 18:13                                     ` Linus Torvalds
2008-03-16 18:36                                       ` Bartlomiej Zolnierkiewicz
2008-03-16 19:08                                         ` Anders Eriksson
2008-03-16 18:56                                       ` Alan Cox
2008-03-16 19:39                                         ` Linus Torvalds
2008-03-16 20:31                                           ` Alan Cox
2008-03-16 21:06                                             ` Linus Torvalds
2008-03-21 15:03                                             ` Mark Lord
2008-03-21 14:49                                               ` Alan Cox
2008-03-16 19:54                                       ` Bartlomiej Zolnierkiewicz
2008-03-16 22:59                                         ` Anders Eriksson
2008-03-16 23:27                                           ` Linus Torvalds
2008-03-17 21:09                                             ` Anders Eriksson
2008-03-17 22:52                                               ` Linus Torvalds
2008-03-18  0:18                                                 ` Anders Eriksson
2008-03-18 13:03                                                   ` Bartlomiej Zolnierkiewicz
2008-03-18 13:32                                                     ` Anders Eriksson
2008-03-18 14:48                                                       ` Bartlomiej Zolnierkiewicz
2008-03-18 15:10                                                         ` Anders Eriksson
2008-03-18 15:41                                                       ` Linus Torvalds
2008-03-18 16:30                                                         ` Anders Eriksson
2008-03-18 16:47                                                           ` Linus Torvalds
2008-03-18 21:02                                                             ` Anders Eriksson
2008-03-19  1:21                                                         ` Bartlomiej Zolnierkiewicz
2008-03-19  1:28                                                           ` Linus Torvalds
2008-03-19  3:24                                                             ` Bartlomiej Zolnierkiewicz
2008-03-19  3:28                                                               ` Linus Torvalds
2008-03-19  3:56                                                                 ` Linus Torvalds
2008-03-19  4:03                                                                 ` Bartlomiej Zolnierkiewicz
2008-03-19  4:48                                                                   ` Linus Torvalds
2008-03-19 11:14                                                                     ` Bartlomiej Zolnierkiewicz
2008-03-16 18:23                                     ` Anders Eriksson
2008-03-16 18:26                                     ` Bartlomiej Zolnierkiewicz
2008-03-16 18:25                                       ` Anders Eriksson
2008-03-17  7:23                                       ` Jens Axboe
2008-03-16 18:44                                   ` Alan Cox
2008-03-10 13:19                           ` Anders Eriksson
2008-03-10 13:56                             ` Bartlomiej Zolnierkiewicz
2008-03-16 18:59     ` Andrey Borzenkov
2008-03-07 10:08 ` Ingo Molnar [this message]
2008-03-09 13:41   ` [patch] drivers/char/esp.c: fix bootup lockup Jiri Slaby
2008-03-09 22:49     ` Rafael J. Wysocki
2008-03-09 23:04       ` Jiri Slaby

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080307100806.GA4350@elte.hu \
    --to=mingo@elte.hu \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=jirislaby@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rjw@sisk.pl \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox