Moving the initialization code from rdac_activate to rdac_bus_attach which is more efficient. We don't have to collect all these information during every activate.
Signed-off-by: Babu Moger <babu.moger@xxxxxxx>
Reviewed-by: Vijay Chauhan <vijay.chauhan@xxxxxxx>
Reviewed-by: Bob Stankey <Robert.stankey@xxxxxxx>
---
--- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig 2009-08-27 18:30:24.000000000 -0500
+++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c 2009-08-27 18:31:31.000000000 -0500
@@ -555,21 +555,6 @@ static int rdac_activate(struct scsi_dev
if (err != SCSI_DH_OK)
goto done;
- if (!h->ctlr) {
- err = initialize_controller(sdev, h);
- if (err != SCSI_DH_OK)
- goto done;
-
- err = get_array_name(sdev, h);
- if (err != SCSI_DH_OK)
- goto done;
- }
-
- if (h->ctlr->use_ms10 == -1) {
- err = set_mode_select(sdev, h);
- if (err != SCSI_DH_OK)
- goto done;
- }
if (h->lun_state == RDAC_LUN_UNOWNED)
err = send_mode_select(sdev, h);
done:
@@ -712,6 +697,18 @@ static int rdac_bus_attach(struct scsi_d
if (err != SCSI_DH_OK)
goto failed;
+ err = initialize_controller(sdev, h);
+ if (err != SCSI_DH_OK)
+ goto failed;
+
+ err = get_array_name(sdev, h);
+ if (err != SCSI_DH_OK)
+ goto failed;
+
+ err = set_mode_select(sdev, h);
+ if (err != SCSI_DH_OK)
+ goto failed;
+
if (!try_module_get(THIS_MODULE))
goto failed;
--
To unsubscribe from this list: 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
