Re: 2.6.16-rc3 qlogic panic ?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 16 Feb 2006, Badari Pulavarty wrote:

> I am not sure if its already reported. I get following panic
> while using qla2200 on 2.6.16-rc3.
>

The qlogicfc driver attaches to 2100 and 2200 devices:

> Unable to handle kernel NULL pointer dereference at 0000000000000000
> RIP:
> <ffffffff88042690>{:qla2xxx:qla2x00_mem_free+648}
> PGD 1bddac067 PUD 1be314067 PMD 0
> Oops: 0000 [1] SMP
> CPU 0
> Modules linked in: thermal processor fan button battery ac parport_pc lp
> parport ipv6 sg qlogicfc qla2200 qla2300 qla2xxx ohci_hcd hw_random
> usbcore dm_mod

Choose either qlogicfc or qla2xxx to manage the 2200, not both.

> Pid: 4601, comm: modprobe Tainted: GF     2.6.16-rc3 #1
> RIP: 0010:[<ffffffff88042690>]
> <ffffffff88042690>{:qla2xxx:qla2x00_mem_free+648}RSP:

The crash is occuring due to PCI resources being held by qlogicfc and
qla2xxx bailing-out into it's cleanup routines.

The attached patch should handle the cleanup issues during resource
contention -- I'll workup a larger one for 2.6.17 to address the
clean-up-everything-in-one-function mess.

Thanks,
AV

---

diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
index 9f91f1a..df9c434 100644
--- a/drivers/scsi/qla2xxx/qla_os.c
+++ b/drivers/scsi/qla2xxx/qla_os.c
@@ -1311,8 +1311,11 @@ int qla2x00_probe_one(struct pci_dev *pd
 
 	/* Configure PCI I/O space */
 	ret = qla2x00_iospace_config(ha);
-	if (ret)
-		goto probe_failed;
+	if (ret) {
+		scsi_host_put(host);
+		pci_release_regions(pdev);
+		goto probe_disable_device;
+	}
 
 	qla_printk(KERN_INFO, ha,
 	    "Found an ISP%04X, irq %d, iobase 0x%p\n", pdev->device, pdev->irq,
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux