scsi: hisi_sas: remove unused variable hisi_sas_devices.running_req
authorXiang Chen <chenxiang66@hisilicon.com>
Wed, 7 Mar 2018 12:25:10 +0000 (20:25 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 13 Mar 2018 01:55:24 +0000 (21:55 -0400)
The structure element hisi_sas_devices.running_req to count how many
commands are active is in effect only ever written in the code, so remove
it.

Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/hisi_sas/hisi_sas.h
drivers/scsi/hisi_sas/hisi_sas_main.c
drivers/scsi/hisi_sas/hisi_sas_v1_hw.c

index e7fd287..d1153e8 100644 (file)
@@ -175,7 +175,6 @@ struct hisi_sas_device {
        struct hisi_sas_dq      *dq;
        struct list_head        list;
        u64 attached_phy;
-       atomic64_t running_req;
        enum sas_device_type    dev_type;
        int device_id;
        int sata_idx;
index 9ff8790..88ad8d4 100644 (file)
@@ -200,8 +200,6 @@ void hisi_sas_slot_task_free(struct hisi_hba *hisi_hba, struct sas_task *task,
 
        if (task) {
                struct device *dev = hisi_hba->dev;
-               struct domain_device *device = task->dev;
-               struct hisi_sas_device *sas_dev = device->lldd_dev;
 
                if (!task->lldd_task)
                        return;
@@ -213,9 +211,6 @@ void hisi_sas_slot_task_free(struct hisi_hba *hisi_hba, struct sas_task *task,
                                dma_unmap_sg(dev, task->scatter,
                                             task->num_scatter,
                                             task->data_dir);
-
-               if (sas_dev)
-                       atomic64_dec(&sas_dev->running_req);
        }
 
        if (slot->buf)
@@ -431,8 +426,6 @@ static int hisi_sas_task_prep(struct sas_task *task, struct hisi_sas_dq
        spin_unlock_irqrestore(&task->task_state_lock, flags);
 
        dq->slot_prep = slot;
-
-       atomic64_inc(&sas_dev->running_req);
        ++(*pass);
 
        return 0;
@@ -1517,8 +1510,6 @@ hisi_sas_internal_abort_task_exec(struct hisi_hba *hisi_hba, int device_id,
 
        dq->slot_prep = slot;
 
-       atomic64_inc(&sas_dev->running_req);
-
        /* send abort command to the chip */
        hisi_hba->hw->start_delivery(dq);
        spin_unlock_irqrestore(&dq->lock, flags_dq);
index 2eb8980..8dd0e6a 100644 (file)
@@ -1407,9 +1407,6 @@ static int slot_complete_v1_hw(struct hisi_hba *hisi_hba,
        }
 
 out:
-       if (sas_dev)
-               atomic64_dec(&sas_dev->running_req);
-
        hisi_sas_slot_task_free(hisi_hba, task, slot);
        sts = ts->stat;