On Fri, 02 Dec 2005, Michael Reed wrote: > Andrew Vasquez wrote: > > > >> From: Michael Reed [mailto:mdr@xxxxxxx] > >> > > > > Sidenote: I'm on the east-coast until hopefully tonight -- won't > > have a chance to look at debugging this for a couple of days... > > > >> I've been testing with the qla2300 driver with 2.6.14.3 and 2.6.15-rc4. > >> I've observed two sets of error messages which are not present with > >> 2.6.14.3. > >> > >> First, the qla2300 driver is generating soft lockups. > > > > Have a backtrace? > > In the previously attached dmesg file, extracted below. Is the driver polling > for mailbox interrupt? Yes during init-time, the driver polls for mailbox completions, in this case it's for a PLOGI completion... ... > qla2300 0002:01:04.0: Topology - (F_Port), Host Loop address 0xffff > BUG: soft lockup detected on CPU#0! > Modules linked in: ... > [<a00000010000bce0>] ia64_leave_kernel+0x0/0x290 > sp=e00000b07bb27920 bsp=e00000b07bb21558 > [<a000000100580380>] qla2x00_mailbox_command+0x860/0xc40 > sp=e00000b07bb27af0 bsp=e00000b07bb21498 > [<a000000100581070>] qla2x00_login_fabric+0x150/0x260 > sp=e00000b07bb27b40 bsp=e00000b07bb21440 > [<a000000100575580>] qla2x00_fabric_login+0xc0/0x3a0 > sp=e00000b07bb27ba0 bsp=e00000b07bb21388 I'm wondering if something like the following is more appropriate. --- Slight variant from a patch submitted by Jeff Layton <jlayton@xxxxxxxxxx>. diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c index 9746cd1..3de8fee 100644 --- a/drivers/scsi/qla2xxx/qla_mbx.c +++ b/drivers/scsi/qla2xxx/qla_mbx.c @@ -196,7 +196,9 @@ qla2x00_mailbox_command(scsi_qla_host_t /* Check for pending interrupts. */ qla2x00_poll(ha); - udelay(10); /* v4.27 */ + if (command != MBC_LOAD_RISC_RAM_EXTENDED && + !ha->flags.mbox_int) + msleep(10); } /* while */ } - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
