target: Remove cmd->se_ordered_id (unused except debug log lines)

For every command, we set se_ordered_id by doing atomic_inc_return on
dev->dev_ordered_id for the corresponding device.  However, the only
places this value gets used are in pr_debug() calls, which doesn't
seem worth an extra atomic op per IO.

Signed-off-by: Roland Dreier <roland@purestorage.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>

3 files changed, 13 insertions, 29 deletions

diff --git a/drivers/target/target_core_device.c b/drivers/target/target_core_device.c
index 55f2cb2a9947..dcc424ac35d4 100644
--- a/drivers/target/target_core_device.c
+++ b/drivers/target/target_core_device.c
@@ -769,7 +769,6 @@ struct se_device *target_alloc_device(struct se_hba *hba, const char *name)
         spin_lock_init(&dev->se_tmr_lock);
         spin_lock_init(&dev->qf_cmd_lock);
         sema_init(&dev->caw_sem, 1);
-        atomic_set(&dev->dev_ordered_id, 0);
         INIT_LIST_HEAD(&dev->t10_wwn.t10_vpd_list);
         spin_lock_init(&dev->t10_wwn.t10_vpd_lock);
         INIT_LIST_HEAD(&dev->t10_pr.registration_list);
diff --git a/drivers/target/target_core_transport.c b/drivers/target/target_core_transport.c
index bd68727a6806..3f0b50082de4 100644
--- a/drivers/target/target_core_transport.c
+++ b/drivers/target/target_core_transport.c
@@ -1178,14 +1178,7 @@ transport_check_alloc_task_attr(struct se_cmd *cmd)
                         " emulation is not supported\n");
                 return TCM_INVALID_CDB_FIELD;
         }
-        /*
-         * Used to determine when ORDERED commands should go from
-         * Dormant to Active status.
-         */
-        cmd->se_ordered_id = atomic_inc_return(&dev->dev_ordered_id);
-        pr_debug("Allocated se_ordered_id: %u for Task Attr: 0x%02x on %s\n",
-                        cmd->se_ordered_id, cmd->sam_task_attr,
-                        dev->transport->name);
+
         return 0;
 }
 
@@ -1773,16 +1766,14 @@ static bool target_handle_task_attr(struct se_cmd *cmd)
          */
         switch (cmd->sam_task_attr) {
         case TCM_HEAD_TAG:
-                pr_debug("Added HEAD_OF_QUEUE for CDB: 0x%02x, "
-                         "se_ordered_id: %u\n",
-                         cmd->t_task_cdb[0], cmd->se_ordered_id);
+                pr_debug("Added HEAD_OF_QUEUE for CDB: 0x%02x\n",
+                         cmd->t_task_cdb[0]);
                 return false;
         case TCM_ORDERED_TAG:
                 atomic_inc_mb(&dev->dev_ordered_sync);
 
-                pr_debug("Added ORDERED for CDB: 0x%02x to ordered list, "
-                         " se_ordered_id: %u\n",
-                         cmd->t_task_cdb[0], cmd->se_ordered_id);
+                pr_debug("Added ORDERED for CDB: 0x%02x to ordered list\n",
+                         cmd->t_task_cdb[0]);
 
                 /*
                  * Execute an ORDERED command if no other older commands
@@ -1806,10 +1797,8 @@ static bool target_handle_task_attr(struct se_cmd *cmd)
         list_add_tail(&cmd->se_delayed_node, &dev->delayed_cmd_list);
         spin_unlock(&dev->delayed_cmd_lock);
 
-        pr_debug("Added CDB: 0x%02x Task Attr: 0x%02x to"
-                " delayed CMD list, se_ordered_id: %u\n",
-                cmd->t_task_cdb[0], cmd->sam_task_attr,
-                cmd->se_ordered_id);
+        pr_debug("Added CDB: 0x%02x Task Attr: 0x%02x to delayed CMD listn",
+                cmd->t_task_cdb[0], cmd->sam_task_attr);
         return true;
 }
 
@@ -1894,20 +1883,18 @@ static void transport_complete_task_attr(struct se_cmd *cmd)
         if (cmd->sam_task_attr == TCM_SIMPLE_TAG) {
                 atomic_dec_mb(&dev->simple_cmds);
                 dev->dev_cur_ordered_id++;
-                pr_debug("Incremented dev->dev_cur_ordered_id: %u for"
-                        " SIMPLE: %u\n", dev->dev_cur_ordered_id,
-                        cmd->se_ordered_id);
+                pr_debug("Incremented dev->dev_cur_ordered_id: %u for SIMPLE\n",
+                         dev->dev_cur_ordered_id);
         } else if (cmd->sam_task_attr == TCM_HEAD_TAG) {
                 dev->dev_cur_ordered_id++;
-                pr_debug("Incremented dev_cur_ordered_id: %u for"
-                        " HEAD_OF_QUEUE: %u\n", dev->dev_cur_ordered_id,
-                        cmd->se_ordered_id);
+                pr_debug("Incremented dev_cur_ordered_id: %u for HEAD_OF_QUEUE\n",
+                         dev->dev_cur_ordered_id);
         } else if (cmd->sam_task_attr == TCM_ORDERED_TAG) {
                 atomic_dec_mb(&dev->dev_ordered_sync);
 
                 dev->dev_cur_ordered_id++;
-                pr_debug("Incremented dev_cur_ordered_id: %u for ORDERED:"
-                        " %u\n", dev->dev_cur_ordered_id, cmd->se_ordered_id);
+                pr_debug("Incremented dev_cur_ordered_id: %u for ORDERED\n",
+                         dev->dev_cur_ordered_id);
         }
 
         target_restart_delayed_cmds(dev);
diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h
index 95e65bd31e05..3afd8dba54e8 100644
--- a/include/target/target_core_base.h
+++ b/include/target/target_core_base.h
@@ -454,7 +454,6 @@ struct se_cmd {
         unsigned                unknown_data_length:1;
         /* See se_cmd_flags_table */
         u32                     se_cmd_flags;
-        u32                     se_ordered_id;
         /* Total size in bytes associated with command */
         u32                     data_length;
         u32                     residual_count;
@@ -744,7 +743,6 @@ struct se_device {
         atomic_long_t           write_bytes;
         /* Active commands on this virtual SE device */
         atomic_t                simple_cmds;
-        atomic_t                dev_ordered_id;
         atomic_t                dev_ordered_sync;
         atomic_t                dev_qf_count;
         u32                     export_count;