On Mon, 2006-03-06 at 00:26 -0800, Mike Anderson wrote: > Ran into a issue that on the first load I did not have the > attach_HostRAID=1 that I need for my model of card so when I went to do a > modprobe -r; modprobe I got an oops. The patch below helps with the > describe case, but does not fix the problem of not being able to unload if > devices are found. Yes, that's part of the refcounting problems. Eventually this will either become a transport class in its own right or an adjunct library to the sas transport class (in which case I'll make it inherit refcounting). In the meantime attach_HostRAID shouldn't be a parameter, so I'll just get rid of it as well. James diff --git a/drivers/scsi/aic94xx/aic94xx_init.c b/drivers/scsi/aic94xx/aic94xx_init.c index f0c5cf2..924075d 100644 --- a/drivers/scsi/aic94xx/aic94xx_init.c +++ b/drivers/scsi/aic94xx/aic94xx_init.c @@ -41,12 +41,6 @@ /* The format is "version.release.patchlevel" */ #define ASD_DRIVER_VERSION "1.0.2" -static int attach_HostRAID = 0; -module_param_named(attach_HostRAID, attach_HostRAID, int, S_IRUGO|S_IWUSR); -MODULE_PARM_DESC(attach_HostRAID, "\n" - "\tEnable(1) or disable(0) attaching to HostRAID enabled host adapters.\n" - "\tDefault: 0"); - static int use_msi = 0; module_param_named(use_msi, use_msi, int, S_IRUGO); MODULE_PARM_DESC(use_msi, "\n" @@ -541,13 +535,6 @@ static int __devinit asd_pci_probe(struc struct asd_ha_struct *asd_ha; int err; - if (dev->class == (PCI_CLASS_STORAGE_RAID << 8) && !attach_HostRAID) { - asd_printk("will not attach to HostRAID enabled device %s, " - "unless attach_HostRAID parameter is set\n", - pci_name(dev)); - return -ENODEV; - } - if (asd_id >= ARRAY_SIZE(asd_pcidev_data)) { asd_printk("wrong driver_data in PCI table\n"); return -ENODEV; - : 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