target: make the se_task task_state_active a normal bool

There is no need to make task_state_active an atomic_t given that it is
always set under the execute_task_lock so we can make it a simple bool.
Also rename it to t_state_active to be closer to the list it guards,
and make sure all checks before the list addion/removal actually happen
under execute_task_lock.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>

1 files changed, 1 insertions, 1 deletions

diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h
index e2473e6770b3..311f5fc607e6 100644
--- a/include/target/target_core_base.h
+++ b/include/target/target_core_base.h
@@ -491,10 +491,10 @@ struct se_task {
         u16                     task_flags;
         u8                      task_scsi_status;
         enum dma_data_direction task_data_direction;
-        atomic_t                task_state_active;
         struct list_head        t_list;
         struct list_head        t_execute_list;
         struct list_head        t_state_list;
+        bool                    t_state_active;
         struct completion       task_stop_comp;
 } ____cacheline_aligned;