Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ff454b01 authored by Chad Dupuis's avatar Chad Dupuis Committed by James Bottomley
Browse files

[SCSI] qla2xxx: Group CS_RESET return status with other link level event statuses.



Currently when we receive a CS_RESET as a response for a SCSI command the
driver will return DID_TRANSPORT_DISRUPTED back to the SCSI mid-layer.  There
are certain circumstances where this could cause the mid-layer to exhaust all of
its retries if the FC port goes away for a short time.  This will result in
commands being prematurly failed.  Moving the CS_RESET return code to be
grouped with other link level events will cause the FC transport layer to block
that target's queue thus preventing the premature exhaustion of retries.

Signed-off-by: default avatarChad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: default avatarMadhuranath Iyengar <Madhu.Iyengar@qlogic.com>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@suse.de>
parent 4d97cc53
Loading
Loading
Loading
Loading
+2 −4
Original line number Original line Diff line number Diff line
@@ -1757,6 +1757,8 @@ qla2x00_status_entry(scsi_qla_host_t *vha, struct rsp_que *rsp, void *pkt)
	case CS_INCOMPLETE:
	case CS_INCOMPLETE:
	case CS_PORT_UNAVAILABLE:
	case CS_PORT_UNAVAILABLE:
	case CS_TIMEOUT:
	case CS_TIMEOUT:
	case CS_RESET:

		/*
		/*
		 * We are going to have the fc class block the rport
		 * We are going to have the fc class block the rport
		 * while we try to recover so instruct the mid layer
		 * while we try to recover so instruct the mid layer
@@ -1781,10 +1783,6 @@ qla2x00_status_entry(scsi_qla_host_t *vha, struct rsp_que *rsp, void *pkt)
			qla2x00_mark_device_lost(fcport->vha, fcport, 1, 1);
			qla2x00_mark_device_lost(fcport->vha, fcport, 1, 1);
		break;
		break;


	case CS_RESET:
		cp->result = DID_TRANSPORT_DISRUPTED << 16;
		break;

	case CS_ABORTED:
	case CS_ABORTED:
		cp->result = DID_RESET << 16;
		cp->result = DID_RESET << 16;
		break;
		break;