From mboxrd@z Thu Jan 1 00:00:00 1970 From: Judith Lebzelter Subject: Re: [PATCH] aacraid: Panics during init time reset (Was: [PATCH] aacraid: [Fastboot] Panics for AACRAID driver during 'insmod' for kexec test) Date: Mon, 2 Apr 2007 12:43:58 -0700 Message-ID: <20070402194358.GE2364@shell0.pdx.osdl.net> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.osdl.org ([65.172.181.24]:41860 "EHLO smtp.osdl.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965752AbXDBToD (ORCPT ); Mon, 2 Apr 2007 15:44:03 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: "Salyzyn, Mark" Cc: Judith Lebzelter , James Bottomley , linux-scsi@vger.kernel.org, Duane Cox On Mon, Apr 02, 2007 at 02:34:36PM -0400, Salyzyn, Mark wrote: > Duane discovered in the scsi-misc-2.6 code that the reset handler could > be called without the sync command handler set up resulting in a panic. > Judith discovered this issue within minutes and has recently reported > it. Here is a fix. Mark, I applied this patch and ran a kexec test again and I still got a panic: Loading aacraid.Adaptec aacraid driver (1.1-5[2437]-mh4)^M ko module^M ACPI: PCI Interrupt 0000:03:0e.0[A] -> Link [LNKC] -> GSI 3 (level, low) -> IRQ 3^M Unable to handle kernel NULL pointer dereference at 0000000000000000 RIP: ^M [<0000000000000000>]^M PGD 4791067 PUD 473c067 PMD 0 ^M Oops: 0010 [1] ^M CPU 0 ^M Modules linked in: aacraid^M Pid: 977, comm: insmod Not tainted 2.6.21-rc3-kdump #1^M RIP: 0010:[<0000000000000000>] [<0000000000000000>]^M RSP: 0000:ffff81000474dbf0 EFLAGS: 00010246^M RAX: ffffc20000010000 RBX: ffff810004fe4cd8 RCX: 000000005b540e96^M RDX: ffffc20000010000 RSI: ffff81000443cf40 RDI: ffff810004fe4cd8^M RBP: 00000000fffee138 R08: ffffffff81001c20 R09: ffffffff8143593e^M R10: ffff810004c537a0 R11: 0000000000000000 R12: ffff81000474dc7c^M R13: 0000000000000000 R14: 0000000000000000 R15: 0000000000000000^M FS: 000000000057b850(0063) GS:ffffffff814d6000(0000) knlGS:0000000000000000^M CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b^M CR2: 0000000000000000 CR3: 0000000004745000 CR4: 00000000000006e0^M Process insmod (pid: 977, threadinfo ffff81000474c000, task ffff81000443cf40)^M Stack: ffffffff88008e82 00003e00fc1f0000 0000000000000000 ffff810004fe4cd8^M ffff810004fe4800 0000000000000000 ffffffff8800a6dd 0000000000000032^M ffffffff88008c3b 0000000000000000 ffffffff00000000 ffff81000474dc7c^M Call Trace:^M [] :aacraid:rx_sync_cmd+0x15c/0x16a^M [] :aacraid:aac_rx_restart_adapter+0x7e/0x169^M [] :aacraid:_aac_rx_init+0x7b/0x2fc^M [] :aacraid:aac_probe_one+0x1a2/0x457^M [] pci_device_probe+0x4c/0x75^M [] really_probe+0xc4/0x148^M [] __driver_attach+0x6d/0xab^M [] __driver_attach+0x0/0xab^M [] __driver_attach+0x0/0xab^M [] bus_for_each_dev+0x43/0x6e^M [] bus_add_driver+0x6b/0x18d^M [] __pci_register_driver+0x72/0xa7^M [] :aacraid:aac_init+0x3a/0x75^M [] sys_init_module+0x1195/0x12e6^M [] system_call+0x7e/0x83^M ^M ^M Code: Bad RIP value.^M RIP [<0000000000000000>]^M RSP ^M CR2: 0000000000000000^M There is an extra line in the call trace for the 'rx_sync_cmd'. Judith > > IMHO, this needs to be applied immediately regardless of the status of > the kexec patch as this issue is present in the scsi-misc-2.6 driver for > all existing init-time recovery actions. This patch in principal would > not be different w/o the kexec patch. > > ObligatoryDisclaimer: Please accept my condolences regarding Outlook's > handling of patches. > > This attached patch is against current scsi-misc-2.6 > > Signed-off-by: Mark Salyzyn > > --- > > Sincerely -- Mark Salyzyn