__iscsi_unbind_session() set session state to ISCSI_SESSION_UNBOUND, which
would overwrite the ISCSI_SESSION_FREE state.

Signed-off-by: Wenchao Hao <[email protected]>
---
 drivers/scsi/scsi_transport_iscsi.c | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/drivers/scsi/scsi_transport_iscsi.c 
b/drivers/scsi/scsi_transport_iscsi.c
index 97a9fee02efa..d8dd9279cea8 100644
--- a/drivers/scsi/scsi_transport_iscsi.c
+++ b/drivers/scsi/scsi_transport_iscsi.c
@@ -2173,6 +2173,22 @@ void iscsi_remove_session(struct iscsi_cls_session 
*session)
        if (!cancel_work_sync(&session->block_work))
                cancel_delayed_work_sync(&session->recovery_work);
        cancel_work_sync(&session->unblock_work);
+
+       scsi_target_unblock(&session->dev, SDEV_TRANSPORT_OFFLINE);
+       /*
+        * qla4xxx can perform it's own scans when it runs in kernel only
+        * mode. Make sure to flush those scans.
+        */
+       flush_work(&session->scan_work);
+
+       /*
+        * flush running unbind operations
+        * if unbind work did not queued, call __iscsi_unbind_session
+        * directly to perform target remove
+        */
+       if (!flush_work(&session->unbind_work))
+               __iscsi_unbind_session(&session->unbind_work);
+
        /*
         * If we are blocked let commands flow again. The lld or iscsi
         * layer should set up the queuecommand to fail commands.
@@ -2183,16 +2199,6 @@ void iscsi_remove_session(struct iscsi_cls_session 
*session)
        session->state = ISCSI_SESSION_FREE;
        spin_unlock_irqrestore(&session->lock, flags);
 
-       scsi_target_unblock(&session->dev, SDEV_TRANSPORT_OFFLINE);
-       /*
-        * qla4xxx can perform it's own scans when it runs in kernel only
-        * mode. Make sure to flush those scans.
-        */
-       flush_work(&session->scan_work);
-       /* flush running unbind operations */
-       flush_work(&session->unbind_work);
-       __iscsi_unbind_session(&session->unbind_work);
-
        /* hw iscsi may not have removed all connections from session */
        err = device_for_each_child(&session->dev, NULL,
                                    iscsi_iter_destroy_conn_fn);
-- 
2.32.0

-- 
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/open-iscsi/20220414014947.4168447-3-haowenchao%40huawei.com.

Reply via email to