isci: fix hang after target reset
authorDan Williams <dan.j.williams@intel.com>
Thu, 3 Mar 2011 00:45:18 +0000 (16:45 -0800)
committerDan Williams <dan.j.williams@intel.com>
Sun, 3 Jul 2011 10:55:29 +0000 (03:55 -0700)
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: Jacek Danecki <Jacek.Danecki@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/scsi/isci/core/scic_sds_port.c

index 410c9a104bc090463088c50dd0e1ead0cb8eeb9a..a41fe426ac26cfa04d93046cbb59605c37568605 100644 (file)
@@ -1673,12 +1673,6 @@ static void scic_sds_port_ready_substate_waiting_enter(
 
        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;
 
        if (this_port->active_phy_mask != 0) {
@@ -1744,6 +1738,13 @@ static void scic_sds_port_ready_substate_operational_exit(
 {
        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(
                scic_sds_port_get_controller(this_port),
                this_port,
@@ -2643,6 +2644,13 @@ void scic_sds_port_invalidate_dummy_remote_node(struct scic_sds_port *sci_port)
 
        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 |
                  phys_index << SCU_CONTEXT_COMMAND_LOGICAL_PORT_SHIFT | rni;