Commit 8acf1b50 authored by Hannes Reinecke's avatar Hannes Reinecke Committed by Martin K. Petersen
Browse files

scsi: libfc: Return LS_RJT_BUSY for PRLI in status PLOGI 



Occasionally it might happen that we receive a PRLI while we're still
waiting for our PLOGI response. In that case we should return
'busy' LS status instead of 'plogi required' LS status.

Signed-off-by: default avatarHannes Reinecke <hare@suse.com>
Reviewed-by: default avatarBart Van Assche <bart.vanassche@sandisk.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 386b97b4

drivers/scsi/libfc/fc_rport.c

+17 −0
Original line number Diff line number Diff line
@@ -1705,6 +1705,15 @@ static void fc_rport_recv_els_req(struct fc_lport *lport, struct fc_frame *fp) 
	case RPORT_ST_READY:

	case RPORT_ST_ADISC:

		break;

	case RPORT_ST_PLOGI:

		if (fc_frame_payload_op(fp) == ELS_PRLI) {

			FC_RPORT_DBG(rdata, "Reject ELS PRLI "

				     "while in state %s\n",

				     fc_rport_state(rdata));

			mutex_unlock(&rdata->rp_mutex);

			kref_put(&rdata->kref, lport->tt.rport_destroy);

			goto busy;

		}

	default:

		FC_RPORT_DBG(rdata,

			     "Reject ELS 0x%02x while in state %s\n",
@@ -1752,6 +1761,14 @@ static void fc_rport_recv_els_req(struct fc_lport *lport, struct fc_frame *fp) 
	els_data.explan = ELS_EXPL_PLOGI_REQD;

	lport->tt.seq_els_rsp_send(fp, ELS_LS_RJT, &els_data);

	fc_frame_free(fp);

	return;



busy:

	els_data.reason = ELS_RJT_BUSY;

	els_data.explan = ELS_EXPL_NONE;

	lport->tt.seq_els_rsp_send(fp, ELS_LS_RJT, &els_data);

	fc_frame_free(fp);

	return;

}



/**