diff options
author | Matthew Wilcox <matthew.r.wilcox@intel.com> | 2014-03-27 16:40:30 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-05-28 12:13:20 -0400 |
commit | c62e46de342d604f72347bd27dcf58b144ad8036 (patch) | |
tree | f70d8afbaacd2739e857ffdfd75c86ee03c48213 /drivers/scsi | |
parent | c1818f17b58e416ec19653be430e9e2925cec35c (diff) |
mpt3sas: Remove uses of serial_number
The mpt3sas_scsih_issue_tm() function does not use the 'serial_number'
argument passed to it. Removing it removes the last vestiges of the
scsi_cmnd's serial_number field from this driver.
Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
Reviewed-by: Nicholas Bellinger <nab@linux-iscsi.org>
Reviewed-by: Praveen Krishnamoorthy <Praveen.krishnamoorthy@lsi.com>
Acked-by: Sreekanth Reddy <Sreekanth.reddy@lsi.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/mpt3sas/mpt3sas_base.h | 2 | ||||
-rw-r--r-- | drivers/scsi/mpt3sas/mpt3sas_ctl.c | 2 | ||||
-rw-r--r-- | drivers/scsi/mpt3sas/mpt3sas_scsih.c | 15 |
3 files changed, 8 insertions, 11 deletions
diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.h b/drivers/scsi/mpt3sas/mpt3sas_base.h index 0ebf5d913c80..9b90a6fef706 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_base.h +++ b/drivers/scsi/mpt3sas/mpt3sas_base.h | |||
@@ -993,7 +993,7 @@ void mpt3sas_scsih_reset_handler(struct MPT3SAS_ADAPTER *ioc, int reset_phase); | |||
993 | 993 | ||
994 | int mpt3sas_scsih_issue_tm(struct MPT3SAS_ADAPTER *ioc, u16 handle, | 994 | int mpt3sas_scsih_issue_tm(struct MPT3SAS_ADAPTER *ioc, u16 handle, |
995 | uint channel, uint id, uint lun, u8 type, u16 smid_task, | 995 | uint channel, uint id, uint lun, u8 type, u16 smid_task, |
996 | ulong timeout, unsigned long serial_number, enum mutex_type m_type); | 996 | ulong timeout, enum mutex_type m_type); |
997 | void mpt3sas_scsih_set_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle); | 997 | void mpt3sas_scsih_set_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle); |
998 | void mpt3sas_scsih_clear_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle); | 998 | void mpt3sas_scsih_clear_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle); |
999 | void mpt3sas_expander_remove(struct MPT3SAS_ADAPTER *ioc, u64 sas_address); | 999 | void mpt3sas_expander_remove(struct MPT3SAS_ADAPTER *ioc, u64 sas_address); |
diff --git a/drivers/scsi/mpt3sas/mpt3sas_ctl.c b/drivers/scsi/mpt3sas/mpt3sas_ctl.c index 9b89de14a0a3..ba9cbe598a91 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_ctl.c +++ b/drivers/scsi/mpt3sas/mpt3sas_ctl.c | |||
@@ -980,7 +980,7 @@ _ctl_do_mpt_command(struct MPT3SAS_ADAPTER *ioc, struct mpt3_ioctl_command karg, | |||
980 | mpt3sas_scsih_issue_tm(ioc, | 980 | mpt3sas_scsih_issue_tm(ioc, |
981 | le16_to_cpu(mpi_request->FunctionDependent1), 0, 0, | 981 | le16_to_cpu(mpi_request->FunctionDependent1), 0, 0, |
982 | 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, 30, | 982 | 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, 30, |
983 | 0, TM_MUTEX_ON); | 983 | TM_MUTEX_ON); |
984 | } else | 984 | } else |
985 | mpt3sas_base_hard_reset_handler(ioc, CAN_SLEEP, | 985 | mpt3sas_base_hard_reset_handler(ioc, CAN_SLEEP, |
986 | FORCE_BIG_HAMMER); | 986 | FORCE_BIG_HAMMER); |
diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c index a961fe11b527..952f6e0d5fd1 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c +++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c | |||
@@ -2029,7 +2029,6 @@ mpt3sas_scsih_clear_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle) | |||
2029 | * @type: MPI2_SCSITASKMGMT_TASKTYPE__XXX (defined in mpi2_init.h) | 2029 | * @type: MPI2_SCSITASKMGMT_TASKTYPE__XXX (defined in mpi2_init.h) |
2030 | * @smid_task: smid assigned to the task | 2030 | * @smid_task: smid assigned to the task |
2031 | * @timeout: timeout in seconds | 2031 | * @timeout: timeout in seconds |
2032 | * @serial_number: the serial_number from scmd | ||
2033 | * @m_type: TM_MUTEX_ON or TM_MUTEX_OFF | 2032 | * @m_type: TM_MUTEX_ON or TM_MUTEX_OFF |
2034 | * Context: user | 2033 | * Context: user |
2035 | * | 2034 | * |
@@ -2042,7 +2041,7 @@ mpt3sas_scsih_clear_tm_flag(struct MPT3SAS_ADAPTER *ioc, u16 handle) | |||
2042 | int | 2041 | int |
2043 | mpt3sas_scsih_issue_tm(struct MPT3SAS_ADAPTER *ioc, u16 handle, uint channel, | 2042 | mpt3sas_scsih_issue_tm(struct MPT3SAS_ADAPTER *ioc, u16 handle, uint channel, |
2044 | uint id, uint lun, u8 type, u16 smid_task, ulong timeout, | 2043 | uint id, uint lun, u8 type, u16 smid_task, ulong timeout, |
2045 | unsigned long serial_number, enum mutex_type m_type) | 2044 | enum mutex_type m_type) |
2046 | { | 2045 | { |
2047 | Mpi2SCSITaskManagementRequest_t *mpi_request; | 2046 | Mpi2SCSITaskManagementRequest_t *mpi_request; |
2048 | Mpi2SCSITaskManagementReply_t *mpi_reply; | 2047 | Mpi2SCSITaskManagementReply_t *mpi_reply; |
@@ -2293,8 +2292,7 @@ _scsih_abort(struct scsi_cmnd *scmd) | |||
2293 | handle = sas_device_priv_data->sas_target->handle; | 2292 | handle = sas_device_priv_data->sas_target->handle; |
2294 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2293 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2295 | scmd->device->id, scmd->device->lun, | 2294 | scmd->device->id, scmd->device->lun, |
2296 | MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, | 2295 | MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, TM_MUTEX_ON); |
2297 | scmd->serial_number, TM_MUTEX_ON); | ||
2298 | 2296 | ||
2299 | out: | 2297 | out: |
2300 | sdev_printk(KERN_INFO, scmd->device, "task abort: %s scmd(%p)\n", | 2298 | sdev_printk(KERN_INFO, scmd->device, "task abort: %s scmd(%p)\n", |
@@ -2353,8 +2351,7 @@ _scsih_dev_reset(struct scsi_cmnd *scmd) | |||
2353 | 2351 | ||
2354 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2352 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2355 | scmd->device->id, scmd->device->lun, | 2353 | scmd->device->id, scmd->device->lun, |
2356 | MPI2_SCSITASKMGMT_TASKTYPE_LOGICAL_UNIT_RESET, 0, 30, 0, | 2354 | MPI2_SCSITASKMGMT_TASKTYPE_LOGICAL_UNIT_RESET, 0, 30, TM_MUTEX_ON); |
2357 | TM_MUTEX_ON); | ||
2358 | 2355 | ||
2359 | out: | 2356 | out: |
2360 | sdev_printk(KERN_INFO, scmd->device, "device reset: %s scmd(%p)\n", | 2357 | sdev_printk(KERN_INFO, scmd->device, "device reset: %s scmd(%p)\n", |
@@ -2414,7 +2411,7 @@ _scsih_target_reset(struct scsi_cmnd *scmd) | |||
2414 | 2411 | ||
2415 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2412 | r = mpt3sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2416 | scmd->device->id, 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, | 2413 | scmd->device->id, 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, |
2417 | 30, 0, TM_MUTEX_ON); | 2414 | 30, TM_MUTEX_ON); |
2418 | 2415 | ||
2419 | out: | 2416 | out: |
2420 | starget_printk(KERN_INFO, starget, "target reset: %s scmd(%p)\n", | 2417 | starget_printk(KERN_INFO, starget, "target reset: %s scmd(%p)\n", |
@@ -5425,7 +5422,7 @@ _scsih_sas_broadcast_primitive_event(struct MPT3SAS_ADAPTER *ioc, | |||
5425 | 5422 | ||
5426 | spin_unlock_irqrestore(&ioc->scsi_lookup_lock, flags); | 5423 | spin_unlock_irqrestore(&ioc->scsi_lookup_lock, flags); |
5427 | r = mpt3sas_scsih_issue_tm(ioc, handle, 0, 0, lun, | 5424 | r = mpt3sas_scsih_issue_tm(ioc, handle, 0, 0, lun, |
5428 | MPI2_SCSITASKMGMT_TASKTYPE_QUERY_TASK, smid, 30, 0, | 5425 | MPI2_SCSITASKMGMT_TASKTYPE_QUERY_TASK, smid, 30, |
5429 | TM_MUTEX_OFF); | 5426 | TM_MUTEX_OFF); |
5430 | if (r == FAILED) { | 5427 | if (r == FAILED) { |
5431 | sdev_printk(KERN_WARNING, sdev, | 5428 | sdev_printk(KERN_WARNING, sdev, |
@@ -5467,7 +5464,7 @@ _scsih_sas_broadcast_primitive_event(struct MPT3SAS_ADAPTER *ioc, | |||
5467 | 5464 | ||
5468 | r = mpt3sas_scsih_issue_tm(ioc, handle, sdev->channel, sdev->id, | 5465 | r = mpt3sas_scsih_issue_tm(ioc, handle, sdev->channel, sdev->id, |
5469 | sdev->lun, MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, | 5466 | sdev->lun, MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, |
5470 | scmd->serial_number, TM_MUTEX_OFF); | 5467 | TM_MUTEX_OFF); |
5471 | if (r == FAILED) { | 5468 | if (r == FAILED) { |
5472 | sdev_printk(KERN_WARNING, sdev, | 5469 | sdev_printk(KERN_WARNING, sdev, |
5473 | "mpt3sas_scsih_issue_tm: ABORT_TASK: FAILED : " | 5470 | "mpt3sas_scsih_issue_tm: ABORT_TASK: FAILED : " |