Commit 3507e13f authored by Brian King's avatar Brian King Committed by James Bottomley
Browse files

[SCSI] ibmvscsi: Send adapter info before login 



The ibmvscsi driver currently sends the SRP Login before sending the Adapter
Info MAD, which can result in commands getting sent to the virtual adapter
before we are ready for them. This results in a slight window where the target
devices may not behave as expected. Change the order and close the window.

Signed-off-by: default avatarBrian King <brking@linux.vnet.ibm.com>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@HansenPartnership.com>
parent e1a5ce5b

drivers/scsi/ibmvscsi/ibmvscsi.c

+85 −85
Original line number Diff line number Diff line
@@ -785,6 +785,83 @@ static int ibmvscsi_queuecommand(struct scsi_cmnd *cmnd, 
/* ------------------------------------------------------------

 * Routines for driver initialization

 */



/**

 * login_rsp: - Handle response to SRP login request

 * @evt_struct:	srp_event_struct with the response

 *

 * Used as a "done" callback by when sending srp_login. Gets called

 * by ibmvscsi_handle_crq()

*/

static void login_rsp(struct srp_event_struct *evt_struct)

{

	struct ibmvscsi_host_data *hostdata = evt_struct->hostdata;

	switch (evt_struct->xfer_iu->srp.login_rsp.opcode) {

	case SRP_LOGIN_RSP:	/* it worked! */

		break;

	case SRP_LOGIN_REJ:	/* refused! */

		dev_info(hostdata->dev, "SRP_LOGIN_REJ reason %u\n",

			 evt_struct->xfer_iu->srp.login_rej.reason);

		/* Login failed.  */

		atomic_set(&hostdata->request_limit, -1);

		return;

	default:

		dev_err(hostdata->dev, "Invalid login response typecode 0x%02x!\n",

			evt_struct->xfer_iu->srp.login_rsp.opcode);

		/* Login failed.  */

		atomic_set(&hostdata->request_limit, -1);

		return;

	}



	dev_info(hostdata->dev, "SRP_LOGIN succeeded\n");



	/* Now we know what the real request-limit is.

	 * This value is set rather than added to request_limit because

	 * request_limit could have been set to -1 by this client.

	 */

	atomic_set(&hostdata->request_limit,

		   evt_struct->xfer_iu->srp.login_rsp.req_lim_delta);



	/* If we had any pending I/Os, kick them */

	scsi_unblock_requests(hostdata->host);

}



/**

 * send_srp_login: - Sends the srp login

 * @hostdata:	ibmvscsi_host_data of host

 *

 * Returns zero if successful.

*/

static int send_srp_login(struct ibmvscsi_host_data *hostdata)

{

	int rc;

	unsigned long flags;

	struct srp_login_req *login;

	struct srp_event_struct *evt_struct = get_event_struct(&hostdata->pool);



	BUG_ON(!evt_struct);

	init_event_struct(evt_struct, login_rsp,

			  VIOSRP_SRP_FORMAT, login_timeout);



	login = &evt_struct->iu.srp.login_req;

	memset(login, 0, sizeof(*login));

	login->opcode = SRP_LOGIN_REQ;

	login->req_it_iu_len = sizeof(union srp_iu);

	login->req_buf_fmt = SRP_BUF_FORMAT_DIRECT | SRP_BUF_FORMAT_INDIRECT;



	spin_lock_irqsave(hostdata->host->host_lock, flags);

	/* Start out with a request limit of 0, since this is negotiated in

	 * the login request we are just sending and login requests always

	 * get sent by the driver regardless of request_limit.

	 */

	atomic_set(&hostdata->request_limit, 0);



	rc = ibmvscsi_send_srp_event(evt_struct, hostdata, login_timeout * 2);

	spin_unlock_irqrestore(hostdata->host->host_lock, flags);

	dev_info(hostdata->dev, "sent SRP login\n");

	return rc;

};



/**

 * adapter_info_rsp: - Handle response to MAD adapter info request

 * @evt_struct:	srp_event_struct with the response
@@ -825,6 +902,8 @@ static void adapter_info_rsp(struct srp_event_struct *evt_struct) 
			hostdata->host->sg_tablesize = MAX_INDIRECT_BUFS;

		}

	}



	send_srp_login(hostdata);

}



/**
@@ -844,11 +923,7 @@ static void send_mad_adapter_info(struct ibmvscsi_host_data *hostdata) 
	dma_addr_t addr;



	evt_struct = get_event_struct(&hostdata->pool);

	if (!evt_struct) {

		dev_err(hostdata->dev,

			"couldn't allocate an event for ADAPTER_INFO_REQ!\n");

		return;

	}

	BUG_ON(!evt_struct);



	init_event_struct(evt_struct,

			  adapter_info_rsp,
@@ -886,89 +961,14 @@ static void send_mad_adapter_info(struct ibmvscsi_host_data *hostdata) 
};



/**

 * login_rsp: - Handle response to SRP login request

 * @evt_struct:	srp_event_struct with the response

 * init_adapter: Start virtual adapter initialization sequence

 *

 * Used as a "done" callback by when sending srp_login. Gets called

 * by ibmvscsi_handle_crq()

 */

static void login_rsp(struct srp_event_struct *evt_struct)

static void init_adapter(struct ibmvscsi_host_data *hostdata)

{

	struct ibmvscsi_host_data *hostdata = evt_struct->hostdata;

	switch (evt_struct->xfer_iu->srp.login_rsp.opcode) {

	case SRP_LOGIN_RSP:	/* it worked! */

		break;

	case SRP_LOGIN_REJ:	/* refused! */

		dev_info(hostdata->dev, "SRP_LOGIN_REJ reason %u\n",

			 evt_struct->xfer_iu->srp.login_rej.reason);

		/* Login failed.  */

		atomic_set(&hostdata->request_limit, -1);

		return;

	default:

		dev_err(hostdata->dev, "Invalid login response typecode 0x%02x!\n",

			evt_struct->xfer_iu->srp.login_rsp.opcode);

		/* Login failed.  */

		atomic_set(&hostdata->request_limit, -1);

		return;

	}



	dev_info(hostdata->dev, "SRP_LOGIN succeeded\n");



	/* Now we know what the real request-limit is.

	 * This value is set rather than added to request_limit because

	 * request_limit could have been set to -1 by this client.

	 */

	atomic_set(&hostdata->request_limit,

		   evt_struct->xfer_iu->srp.login_rsp.req_lim_delta);



	/* If we had any pending I/Os, kick them */

	scsi_unblock_requests(hostdata->host);



	send_mad_adapter_info(hostdata);

	return;

}



/**

 * send_srp_login: - Sends the srp login

 * @hostdata:	ibmvscsi_host_data of host

 * 

 * Returns zero if successful.

*/

static int send_srp_login(struct ibmvscsi_host_data *hostdata)

{

	int rc;

	unsigned long flags;

	struct srp_login_req *login;

	struct srp_event_struct *evt_struct = get_event_struct(&hostdata->pool);

	if (!evt_struct) {

		dev_err(hostdata->dev, "couldn't allocate an event for login req!\n");

		return FAILED;

	}



	init_event_struct(evt_struct,

			  login_rsp,

			  VIOSRP_SRP_FORMAT,

			  login_timeout);



	login = &evt_struct->iu.srp.login_req;

	memset(login, 0x00, sizeof(struct srp_login_req));

	login->opcode = SRP_LOGIN_REQ;

	login->req_it_iu_len = sizeof(union srp_iu);

	login->req_buf_fmt = SRP_BUF_FORMAT_DIRECT | SRP_BUF_FORMAT_INDIRECT;

	

	spin_lock_irqsave(hostdata->host->host_lock, flags);

	/* Start out with a request limit of 0, since this is negotiated in

	 * the login request we are just sending and login requests always

	 * get sent by the driver regardless of request_limit.

	 */

	atomic_set(&hostdata->request_limit, 0);



	rc = ibmvscsi_send_srp_event(evt_struct, hostdata, login_timeout * 2);

	spin_unlock_irqrestore(hostdata->host->host_lock, flags);

	dev_info(hostdata->dev, "sent SRP login\n");

	return rc;

};



/**

 * sync_completion: Signal that a synchronous command has completed

 * Note that after returning from this call, the evt_struct is freed.
@@ -1282,7 +1282,7 @@ void ibmvscsi_handle_crq(struct viosrp_crq *crq, 
			if ((rc = ibmvscsi_ops->send_crq(hostdata,

							 0xC002000000000000LL, 0)) == 0) {

				/* Now login */

				send_srp_login(hostdata);

				init_adapter(hostdata);

			} else {

				dev_err(hostdata->dev, "Unable to send init rsp. rc=%ld\n", rc);

			}
@@ -1292,7 +1292,7 @@ void ibmvscsi_handle_crq(struct viosrp_crq *crq, 
			dev_info(hostdata->dev, "partner initialization complete\n");



			/* Now login */

			send_srp_login(hostdata);

			init_adapter(hostdata);

			break;

		default:

			dev_err(hostdata->dev, "unknown crq message type: %d\n", crq->format);

CRA Git | Maintained and supported by SUSTech CRA and CCSE