target: Remove useless if statement

We do the same thing no matter which way the test goes, so just remove
the test and do what we're going to do.

The debug messages printed the wrong value of CMD_T_ACTIVE and don't
seem particularly useful, remove them too.

Signed-off-by: Roland Dreier <roland@purestorage.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>

diff --git a/drivers/target/target_core_tmr.c b/drivers/target/target_core_tmr.c
index c6e0293ffdb0460340ccd33c33e40fae9e61cc5d..d0b4dd95b91e96628999a362b409543caee5578f 100644 (file)
--- a/drivers/target/target_core_tmr.c
+++ b/drivers/target/target_core_tmr.c
@@ -331,18 +331,6 @@ static void core_tmr_drain_state_list(
 
                fe_count = atomic_read(&cmd->t_fe_count);
 
-               if (!(cmd->transport_state & CMD_T_ACTIVE)) {
-                       pr_debug("LUN_RESET: got CMD_T_ACTIVE for"
-                               " cdb: %p, t_fe_count: %d dev: %p\n", cmd,
-                               fe_count, dev);
-                       cmd->transport_state |= CMD_T_ABORTED;
-                       spin_unlock_irqrestore(&cmd->t_state_lock, flags);
-
-                       core_tmr_handle_tas_abort(tmr_nacl, cmd, tas, fe_count);
-                       continue;
-               }
-               pr_debug("LUN_RESET: Got !CMD_T_ACTIVE for cdb: %p,"
-                       " t_fe_count: %d dev: %p\n", cmd, fe_count, dev);
                cmd->transport_state |= CMD_T_ABORTED;
                spin_unlock_irqrestore(&cmd->t_state_lock, flags);