diff options
author | Matthew Wilcox <matthew.r.wilcox@intel.com> | 2014-03-27 16:40:32 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-05-28 12:13:23 -0400 |
commit | 2e45c8b754f28e65dede5246a5e0848212c397e8 (patch) | |
tree | f84d2c6ebd230c09598a62924812613854e81b48 /drivers | |
parent | d8bfbd8d333f84f7c72a0e5f5979291ad8438dbb (diff) |
mpt2sas: Remove uses of serial_number
The mpt2sas_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')
-rw-r--r-- | drivers/scsi/mpt2sas/mpt2sas_base.h | 2 | ||||
-rw-r--r-- | drivers/scsi/mpt2sas/mpt2sas_ctl.c | 2 | ||||
-rw-r--r-- | drivers/scsi/mpt2sas/mpt2sas_scsih.c | 15 |
3 files changed, 8 insertions, 11 deletions
diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.h b/drivers/scsi/mpt2sas/mpt2sas_base.h index 1f2ac3a28621..fd3b998c75b1 100644 --- a/drivers/scsi/mpt2sas/mpt2sas_base.h +++ b/drivers/scsi/mpt2sas/mpt2sas_base.h | |||
@@ -1065,7 +1065,7 @@ void mpt2sas_scsih_event_callback(struct MPT2SAS_ADAPTER *ioc, u8 msix_index, | |||
1065 | u32 reply); | 1065 | u32 reply); |
1066 | int mpt2sas_scsih_issue_tm(struct MPT2SAS_ADAPTER *ioc, u16 handle, | 1066 | int mpt2sas_scsih_issue_tm(struct MPT2SAS_ADAPTER *ioc, u16 handle, |
1067 | uint channel, uint id, uint lun, u8 type, u16 smid_task, | 1067 | uint channel, uint id, uint lun, u8 type, u16 smid_task, |
1068 | ulong timeout, unsigned long serial_number, enum mutex_type m_type); | 1068 | ulong timeout, enum mutex_type m_type); |
1069 | void mpt2sas_scsih_set_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle); | 1069 | void mpt2sas_scsih_set_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle); |
1070 | void mpt2sas_scsih_clear_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle); | 1070 | void mpt2sas_scsih_clear_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle); |
1071 | void mpt2sas_expander_remove(struct MPT2SAS_ADAPTER *ioc, u64 sas_address); | 1071 | void mpt2sas_expander_remove(struct MPT2SAS_ADAPTER *ioc, u64 sas_address); |
diff --git a/drivers/scsi/mpt2sas/mpt2sas_ctl.c b/drivers/scsi/mpt2sas/mpt2sas_ctl.c index b7f887c9b0bf..62df8f9d4271 100644 --- a/drivers/scsi/mpt2sas/mpt2sas_ctl.c +++ b/drivers/scsi/mpt2sas/mpt2sas_ctl.c | |||
@@ -987,7 +987,7 @@ _ctl_do_mpt_command(struct MPT2SAS_ADAPTER *ioc, struct mpt2_ioctl_command karg, | |||
987 | mpt2sas_scsih_issue_tm(ioc, | 987 | mpt2sas_scsih_issue_tm(ioc, |
988 | le16_to_cpu(mpi_request->FunctionDependent1), 0, 0, | 988 | le16_to_cpu(mpi_request->FunctionDependent1), 0, 0, |
989 | 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, 10, | 989 | 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, 10, |
990 | 0, TM_MUTEX_ON); | 990 | TM_MUTEX_ON); |
991 | ioc->tm_cmds.status = MPT2_CMD_NOT_USED; | 991 | ioc->tm_cmds.status = MPT2_CMD_NOT_USED; |
992 | } else | 992 | } else |
993 | mpt2sas_base_hard_reset_handler(ioc, CAN_SLEEP, | 993 | mpt2sas_base_hard_reset_handler(ioc, CAN_SLEEP, |
diff --git a/drivers/scsi/mpt2sas/mpt2sas_scsih.c b/drivers/scsi/mpt2sas/mpt2sas_scsih.c index 6fd7d40b2c4d..e19dfde72066 100644 --- a/drivers/scsi/mpt2sas/mpt2sas_scsih.c +++ b/drivers/scsi/mpt2sas/mpt2sas_scsih.c | |||
@@ -2368,7 +2368,6 @@ mpt2sas_scsih_clear_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle) | |||
2368 | * @type: MPI2_SCSITASKMGMT_TASKTYPE__XXX (defined in mpi2_init.h) | 2368 | * @type: MPI2_SCSITASKMGMT_TASKTYPE__XXX (defined in mpi2_init.h) |
2369 | * @smid_task: smid assigned to the task | 2369 | * @smid_task: smid assigned to the task |
2370 | * @timeout: timeout in seconds | 2370 | * @timeout: timeout in seconds |
2371 | * @serial_number: the serial_number from scmd | ||
2372 | * @m_type: TM_MUTEX_ON or TM_MUTEX_OFF | 2371 | * @m_type: TM_MUTEX_ON or TM_MUTEX_OFF |
2373 | * Context: user | 2372 | * Context: user |
2374 | * | 2373 | * |
@@ -2381,7 +2380,7 @@ mpt2sas_scsih_clear_tm_flag(struct MPT2SAS_ADAPTER *ioc, u16 handle) | |||
2381 | int | 2380 | int |
2382 | mpt2sas_scsih_issue_tm(struct MPT2SAS_ADAPTER *ioc, u16 handle, uint channel, | 2381 | mpt2sas_scsih_issue_tm(struct MPT2SAS_ADAPTER *ioc, u16 handle, uint channel, |
2383 | uint id, uint lun, u8 type, u16 smid_task, ulong timeout, | 2382 | uint id, uint lun, u8 type, u16 smid_task, ulong timeout, |
2384 | unsigned long serial_number, enum mutex_type m_type) | 2383 | enum mutex_type m_type) |
2385 | { | 2384 | { |
2386 | Mpi2SCSITaskManagementRequest_t *mpi_request; | 2385 | Mpi2SCSITaskManagementRequest_t *mpi_request; |
2387 | Mpi2SCSITaskManagementReply_t *mpi_reply; | 2386 | Mpi2SCSITaskManagementReply_t *mpi_reply; |
@@ -2634,8 +2633,7 @@ _scsih_abort(struct scsi_cmnd *scmd) | |||
2634 | handle = sas_device_priv_data->sas_target->handle; | 2633 | handle = sas_device_priv_data->sas_target->handle; |
2635 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2634 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2636 | scmd->device->id, scmd->device->lun, | 2635 | scmd->device->id, scmd->device->lun, |
2637 | MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, | 2636 | MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, TM_MUTEX_ON); |
2638 | scmd->serial_number, TM_MUTEX_ON); | ||
2639 | 2637 | ||
2640 | out: | 2638 | out: |
2641 | sdev_printk(KERN_INFO, scmd->device, "task abort: %s scmd(%p)\n", | 2639 | sdev_printk(KERN_INFO, scmd->device, "task abort: %s scmd(%p)\n", |
@@ -2696,8 +2694,7 @@ _scsih_dev_reset(struct scsi_cmnd *scmd) | |||
2696 | 2694 | ||
2697 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2695 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2698 | scmd->device->id, scmd->device->lun, | 2696 | scmd->device->id, scmd->device->lun, |
2699 | MPI2_SCSITASKMGMT_TASKTYPE_LOGICAL_UNIT_RESET, 0, 30, 0, | 2697 | MPI2_SCSITASKMGMT_TASKTYPE_LOGICAL_UNIT_RESET, 0, 30, TM_MUTEX_ON); |
2700 | TM_MUTEX_ON); | ||
2701 | 2698 | ||
2702 | out: | 2699 | out: |
2703 | sdev_printk(KERN_INFO, scmd->device, "device reset: %s scmd(%p)\n", | 2700 | sdev_printk(KERN_INFO, scmd->device, "device reset: %s scmd(%p)\n", |
@@ -2757,7 +2754,7 @@ _scsih_target_reset(struct scsi_cmnd *scmd) | |||
2757 | 2754 | ||
2758 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, | 2755 | r = mpt2sas_scsih_issue_tm(ioc, handle, scmd->device->channel, |
2759 | scmd->device->id, 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, | 2756 | scmd->device->id, 0, MPI2_SCSITASKMGMT_TASKTYPE_TARGET_RESET, 0, |
2760 | 30, 0, TM_MUTEX_ON); | 2757 | 30, TM_MUTEX_ON); |
2761 | 2758 | ||
2762 | out: | 2759 | out: |
2763 | starget_printk(KERN_INFO, starget, "target reset: %s scmd(%p)\n", | 2760 | starget_printk(KERN_INFO, starget, "target reset: %s scmd(%p)\n", |
@@ -5880,7 +5877,7 @@ broadcast_aen_retry: | |||
5880 | 5877 | ||
5881 | spin_unlock_irqrestore(&ioc->scsi_lookup_lock, flags); | 5878 | spin_unlock_irqrestore(&ioc->scsi_lookup_lock, flags); |
5882 | r = mpt2sas_scsih_issue_tm(ioc, handle, 0, 0, lun, | 5879 | r = mpt2sas_scsih_issue_tm(ioc, handle, 0, 0, lun, |
5883 | MPI2_SCSITASKMGMT_TASKTYPE_QUERY_TASK, smid, 30, 0, | 5880 | MPI2_SCSITASKMGMT_TASKTYPE_QUERY_TASK, smid, 30, |
5884 | TM_MUTEX_OFF); | 5881 | TM_MUTEX_OFF); |
5885 | if (r == FAILED) { | 5882 | if (r == FAILED) { |
5886 | sdev_printk(KERN_WARNING, sdev, | 5883 | sdev_printk(KERN_WARNING, sdev, |
@@ -5922,7 +5919,7 @@ broadcast_aen_retry: | |||
5922 | 5919 | ||
5923 | r = mpt2sas_scsih_issue_tm(ioc, handle, sdev->channel, sdev->id, | 5920 | r = mpt2sas_scsih_issue_tm(ioc, handle, sdev->channel, sdev->id, |
5924 | sdev->lun, MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, | 5921 | sdev->lun, MPI2_SCSITASKMGMT_TASKTYPE_ABORT_TASK, smid, 30, |
5925 | scmd->serial_number, TM_MUTEX_OFF); | 5922 | TM_MUTEX_OFF); |
5926 | if (r == FAILED) { | 5923 | if (r == FAILED) { |
5927 | sdev_printk(KERN_WARNING, sdev, | 5924 | sdev_printk(KERN_WARNING, sdev, |
5928 | "mpt2sas_scsih_issue_tm: ABORT_TASK: FAILED : " | 5925 | "mpt2sas_scsih_issue_tm: ABORT_TASK: FAILED : " |