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

Commit f01690d3 authored by nagalakshmi.nandigama@lsi.com's avatar nagalakshmi.nandigama@lsi.com Committed by James Bottomley
Browse files

[SCSI] mpt2sas: Rearrange the the code so that the completion queues are... 


[SCSI] mpt2sas: Rearrange the the code so that the completion queues are initialized prior to sending the request to controller firmware

Signed-off-by: default avatarNagalakshmi Nandigama <nagalakshmi.nandigama@lsi.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Parallels.com>
parent 23edb6e7

drivers/scsi/mpt2sas/mpt2sas_base.c

+3 −3
Original line number Diff line number Diff line
@@ -3199,8 +3199,8 @@ mpt2sas_base_sas_iounit_control(struct MPT2SAS_ADAPTER *ioc, 
	if (mpi_request->Operation == MPI2_SAS_OP_PHY_HARD_RESET ||

	    mpi_request->Operation == MPI2_SAS_OP_PHY_LINK_RESET)

		ioc->ioc_link_reset_in_progress = 1;

	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->base_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->base_cmds.done,

	    msecs_to_jiffies(10000));

	if ((mpi_request->Operation == MPI2_SAS_OP_PHY_HARD_RESET ||
@@ -3301,8 +3301,8 @@ mpt2sas_base_scsi_enclosure_processor(struct MPT2SAS_ADAPTER *ioc, 
	request = mpt2sas_base_get_msg_frame(ioc, smid);

	ioc->base_cmds.smid = smid;

	memcpy(request, mpi_request, sizeof(Mpi2SepReply_t));

	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->base_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->base_cmds.done,

	    msecs_to_jiffies(10000));

	if (!(ioc->base_cmds.status & MPT2_CMD_COMPLETE)) {
@@ -3809,8 +3809,8 @@ _base_event_notification(struct MPT2SAS_ADAPTER *ioc, int sleep_flag) 
	for (i = 0; i < MPI2_EVENT_NOTIFY_EVENTMASK_WORDS; i++)

		mpi_request->EventMasks[i] =

		    cpu_to_le32(ioc->event_masks[i]);

	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->base_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->base_cmds.done, 30*HZ);

	if (!(ioc->base_cmds.status & MPT2_CMD_COMPLETE)) {

		printk(MPT2SAS_ERR_FMT "%s: timeout\n",

drivers/scsi/mpt2sas/mpt2sas_ctl.c

+4 −4
Original line number Diff line number Diff line
@@ -818,6 +818,7 @@ _ctl_do_mpt_command(struct MPT2SAS_ADAPTER *ioc, 
	_ctl_display_some_debug(ioc, smid, "ctl_request", NULL);

#endif



	init_completion(&ioc->ctl_cmds.done);

	switch (mpi_request->Function) {

	case MPI2_FUNCTION_SCSI_IO_REQUEST:

	case MPI2_FUNCTION_RAID_SCSI_IO_PASSTHROUGH:
@@ -903,7 +904,6 @@ _ctl_do_mpt_command(struct MPT2SAS_ADAPTER *ioc, 
		timeout = MPT2_IOCTL_DEFAULT_TIMEOUT;

	else

		timeout = karg.timeout;

	init_completion(&ioc->ctl_cmds.done);

	timeleft = wait_for_completion_timeout(&ioc->ctl_cmds.done,

	    timeout*HZ);

	if (mpi_request->Function == MPI2_FUNCTION_SCSI_TASK_MGMT) {
@@ -1477,8 +1477,8 @@ _ctl_diag_register_2(struct MPT2SAS_ADAPTER *ioc, 
		mpi_request->ProductSpecific[i] =

			cpu_to_le32(ioc->product_specific[buffer_type][i]);



	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->ctl_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->ctl_cmds.done,

	    MPT2_IOCTL_DEFAULT_TIMEOUT*HZ);
@@ -1821,8 +1821,8 @@ _ctl_send_release(struct MPT2SAS_ADAPTER *ioc, u8 buffer_type, u8 *issue_reset) 
	mpi_request->VF_ID = 0; /* TODO */

	mpi_request->VP_ID = 0;



	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->ctl_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->ctl_cmds.done,

	    MPT2_IOCTL_DEFAULT_TIMEOUT*HZ);
@@ -2095,8 +2095,8 @@ _ctl_diag_read_buffer(void __user *arg, enum block_state state) 
	mpi_request->VF_ID = 0; /* TODO */

	mpi_request->VP_ID = 0;



	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->ctl_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->ctl_cmds.done,

	    MPT2_IOCTL_DEFAULT_TIMEOUT*HZ);

drivers/scsi/mpt2sas/mpt2sas_transport.c

+5 −4
Original line number Diff line number Diff line
@@ -398,8 +398,8 @@ _transport_expander_report_manufacture(struct MPT2SAS_ADAPTER *ioc, 
	dtransportprintk(ioc, printk(MPT2SAS_INFO_FMT "report_manufacture - "

	    "send to sas_addr(0x%016llx)\n", ioc->name,

	    (unsigned long long)sas_address));

	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->transport_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->transport_cmds.done,

	    10*HZ);
@@ -1184,8 +1184,8 @@ _transport_get_expander_phy_error_log(struct MPT2SAS_ADAPTER *ioc, 
	dtransportprintk(ioc, printk(MPT2SAS_INFO_FMT "phy_error_log - "

	    "send to sas_addr(0x%016llx), phy(%d)\n", ioc->name,

	    (unsigned long long)phy->identify.sas_address, phy->number));

	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->transport_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->transport_cmds.done,

	    10*HZ);
@@ -1509,8 +1509,9 @@ _transport_expander_phy_control(struct MPT2SAS_ADAPTER *ioc, 
	    "send to sas_addr(0x%016llx), phy(%d), opcode(%d)\n", ioc->name,

	    (unsigned long long)phy->identify.sas_address, phy->number,

	    phy_operation));

	mpt2sas_base_put_smid_default(ioc, smid);



	init_completion(&ioc->transport_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->transport_cmds.done,

	    10*HZ);
@@ -1949,8 +1950,8 @@ _transport_smp_handler(struct Scsi_Host *shost, struct sas_rphy *rphy, 
	dtransportprintk(ioc, printk(MPT2SAS_INFO_FMT "%s - "

	    "sending smp request\n", ioc->name, __func__));



	mpt2sas_base_put_smid_default(ioc, smid);

	init_completion(&ioc->transport_cmds.done);

	mpt2sas_base_put_smid_default(ioc, smid);

	timeleft = wait_for_completion_timeout(&ioc->transport_cmds.done,

	    10*HZ);