Since the previous patch removed the only statement that sets
qla_tgt_cmd.released, remove the code that depends on that member
variable being set and the member variable itself.

Cc: Himanshu Madhani <hmadh...@marvell.com>
Cc: Giridhar Malavali <gmalav...@marvell.com>
Signed-off-by: Bart Van Assche <bvanass...@acm.org>
---
 drivers/scsi/qla2xxx/qla_target.h  |  1 -
 drivers/scsi/qla2xxx/tcm_qla2xxx.c | 15 +--------------
 2 files changed, 1 insertion(+), 15 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_target.h 
b/drivers/scsi/qla2xxx/qla_target.h
index 8fb197a4d740..262fc33fb473 100644
--- a/drivers/scsi/qla2xxx/qla_target.h
+++ b/drivers/scsi/qla2xxx/qla_target.h
@@ -889,7 +889,6 @@ struct qla_tgt_cmd {
        unsigned int term_exchg:1;
        unsigned int cmd_sent_to_fw:1;
        unsigned int cmd_in_wq:1;
-       unsigned int released:1;
 
        /*
         * This variable may be set from outside the LIO and I/O completion
diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c 
b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
index 4fcae1af2d63..f6797eed320a 100644
--- a/drivers/scsi/qla2xxx/tcm_qla2xxx.c
+++ b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
@@ -262,7 +262,6 @@ static void tcm_qla2xxx_free_mcmd(struct qla_tgt_mgmt_cmd 
*mcmd)
 static void tcm_qla2xxx_complete_free(struct work_struct *work)
 {
        struct qla_tgt_cmd *cmd = container_of(work, struct qla_tgt_cmd, work);
-       bool released = false;
        unsigned long flags;
 
        cmd->cmd_in_wq = 0;
@@ -273,14 +272,9 @@ static void tcm_qla2xxx_complete_free(struct work_struct 
*work)
        cmd->qpair->tgt_counters.qla_core_ret_sta_ctio++;
        cmd->trc_flags |= TRC_CMD_FREE;
        cmd->cmd_sent_to_fw = 0;
-       if (cmd->released)
-               released = true;
        spin_unlock_irqrestore(&cmd->cmd_lock, flags);
 
-       if (released)
-               qlt_free_cmd(cmd);
-       else
-               transport_generic_free_cmd(&cmd->se_cmd, 0);
+       transport_generic_free_cmd(&cmd->se_cmd, 0);
 }
 
 /*
@@ -490,13 +484,6 @@ static void tcm_qla2xxx_handle_data_work(struct 
work_struct *work)
 
        spin_lock_irqsave(&cmd->cmd_lock, flags);
        cmd->cmd_sent_to_fw = 0;
-
-       if (cmd->released) {
-               spin_unlock_irqrestore(&cmd->cmd_lock, flags);
-               qlt_free_cmd(cmd);
-               return;
-       }
-
        if (cmd->aborted) {
                spin_unlock_irqrestore(&cmd->cmd_lock, flags);
 
-- 
2.21.0.196.g041f5ea1cf98

Reply via email to