Re: [RFC] aic94xx: attaching to the sas transport class

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

 



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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux