Commit 27ce51df authored by Dan Williams's avatar Dan Williams

isci: fix hang after target reset

When aborting a task context we need to be sure that the hardware has acted on
this request (retrieved the task context) before invalidating the remote node
context.  In the case of the "dummy" task context and remote node we do not
have the full state machine that goes through the complete tc abort and rnc
invalidate states.  Instead we ensure the hardware has seen and acted on
Signed-off-by: default avatarJacek Danecki <Jacek.Danecki@intel.com>
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent d7628d05
...@@ -1673,12 +1673,6 @@ static void scic_sds_port_ready_substate_waiting_enter( ...@@ -1673,12 +1673,6 @@ static void scic_sds_port_ready_substate_waiting_enter(
scic_sds_port_suspend_port_task_scheduler(this_port); scic_sds_port_suspend_port_task_scheduler(this_port);
/* Kill the dummy task for this port if it has not yet posted
* the hardware will treat this as a NOP and just return abort
* complete.
*/
scic_sds_port_abort_dummy_request(this_port);
this_port->not_ready_reason = SCIC_PORT_NOT_READY_NO_ACTIVE_PHYS; this_port->not_ready_reason = SCIC_PORT_NOT_READY_NO_ACTIVE_PHYS;
if (this_port->active_phy_mask != 0) { if (this_port->active_phy_mask != 0) {
...@@ -1744,6 +1738,13 @@ static void scic_sds_port_ready_substate_operational_exit( ...@@ -1744,6 +1738,13 @@ static void scic_sds_port_ready_substate_operational_exit(
{ {
struct scic_sds_port *this_port = (struct scic_sds_port *)object; struct scic_sds_port *this_port = (struct scic_sds_port *)object;
/*
* Kill the dummy task for this port if it has not yet posted
* the hardware will treat this as a NOP and just return abort
* complete.
*/
scic_sds_port_abort_dummy_request(this_port);
isci_event_port_not_ready( isci_event_port_not_ready(
scic_sds_port_get_controller(this_port), scic_sds_port_get_controller(this_port),
this_port, this_port,
...@@ -2643,6 +2644,13 @@ void scic_sds_port_invalidate_dummy_remote_node(struct scic_sds_port *sci_port) ...@@ -2643,6 +2644,13 @@ void scic_sds_port_invalidate_dummy_remote_node(struct scic_sds_port *sci_port)
rnc->ssp.is_valid = false; rnc->ssp.is_valid = false;
/* ensure the preceding tc abort request has reached the
* controller and give it ample time to act before posting the rnc
* invalidate
*/
SMU_ISR_READ(scic); /* flush */
udelay(10);
command = SCU_CONTEXT_COMMAND_POST_RNC_INVALIDATE | command = SCU_CONTEXT_COMMAND_POST_RNC_INVALIDATE |
phys_index << SCU_CONTEXT_COMMAND_LOGICAL_PORT_SHIFT | rni; phys_index << SCU_CONTEXT_COMMAND_LOGICAL_PORT_SHIFT | rni;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment