diff mbox series

mpt3sas: Stop Error handler escalation when device removed

Message ID 20220816080801.13929-1-sreekanth.reddy@broadcom.com (mailing list archive)
State Accepted
Headers show
Series mpt3sas: Stop Error handler escalation when device removed | expand

Commit Message

Sreekanth Reddy Aug. 16, 2022, 8:08 a.m. UTC
If SCSI Error Handler is going on for timedout I/Os on a drive
and that corresponding drive is removed then stop escalating to
higher level of reset by returning the TUR with
"I_T NEXUS LOSS OCCURRED" sense key.

Signed-off-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
---
 drivers/scsi/mpt3sas/mpt3sas_scsih.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

Comments

Martin K. Petersen Sept. 1, 2022, 4:09 a.m. UTC | #1
Sreekanth,

> If SCSI Error Handler is going on for timedout I/Os on a drive and
> that corresponding drive is removed then stop escalating to higher
> level of reset by returning the TUR with "I_T NEXUS LOSS OCCURRED"
> sense key.

Applied to 6.1/scsi-staging, thanks!
diff mbox series

Patch

diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index e54747a..465603b 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -5156,6 +5156,19 @@  scsih_qcmd(struct Scsi_Host *shost, struct scsi_cmnd *scmd)
 
 	/* invalid device handle */
 	handle = sas_target_priv_data->handle;
+
+	/*
+	 * Avoid error handling escallation when device is disconnected
+	 */
+	if (handle == MPT3SAS_INVALID_DEVICE_HANDLE || sas_device_priv_data->block) {
+		if (scmd->device->host->shost_state == SHOST_RECOVERY &&
+		    scmd->cmnd[0] == TEST_UNIT_READY) {
+			scsi_build_sense(scmd, 0, UNIT_ATTENTION, 0x29, 0x07);
+			scsi_done(scmd);
+			return 0;
+		}
+	}
+
 	if (handle == MPT3SAS_INVALID_DEVICE_HANDLE) {
 		scmd->result = DID_NO_CONNECT << 16;
 		scsi_done(scmd);