Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6f15667e authored by Roland Dreier's avatar Roland Dreier Committed by Nicholas Bellinger
Browse files

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: default avatarRoland Dreier <roland@purestorage.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent 0dfa1c5d

drivers/target/target_core_tmr.c

+0 −12
Original line number Original line Diff line number Diff line
@@ -331,18 +331,6 @@ static void core_tmr_drain_state_list( 




		fe_count = atomic_read(&cmd->t_fe_count);

		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;

		cmd->transport_state |= CMD_T_ABORTED;

		spin_unlock_irqrestore(&cmd->t_state_lock, flags);

		spin_unlock_irqrestore(&cmd->t_state_lock, flags);