]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
scsi: libsas: Add struct sas_tmf_task
authorJohn Garry <john.garry@huawei.com>
Thu, 17 Feb 2022 15:42:35 +0000 (23:42 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 2 May 2025 05:44:07 +0000 (07:44 +0200)
[ Upstream commit bbfe82cdbaf84e6622ceb6f3447c8c4bb7dde7ab ]

Some of the LLDDs which use libsas have their own definition of a struct
to hold TMF info, so add a common struct for libsas.

Also add an interim force phy id field for hisi_sas driver, which will be
removed once the STP "TMF" code is factored out.

Even though some LLDDs (pm8001) use a u32 for the tag, u16 will be adequate,
as that named driver only uses tags in range [0, 1024).

Link: https://lore.kernel.org/r/1645112566-115804-8-git-send-email-john.garry@huawei.com
Tested-by: Yihang Li <liyihang6@hisilicon.com>
Tested-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Stable-dep-of: 8aa580cd9284 ("scsi: hisi_sas: Enable force phy when SATA disk directly connected")
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/scsi/hisi_sas/hisi_sas.h
drivers/scsi/hisi_sas/hisi_sas_main.c
drivers/scsi/hisi_sas/hisi_sas_v1_hw.c
drivers/scsi/hisi_sas/hisi_sas_v2_hw.c
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
drivers/scsi/mvsas/mv_defs.h
drivers/scsi/mvsas/mv_sas.c
drivers/scsi/pm8001/pm8001_hwi.c
drivers/scsi/pm8001/pm8001_sas.c
drivers/scsi/pm8001/pm8001_sas.h
include/scsi/libsas.h

index 39da9f1645135a402e9c11f903f653b66a3e341b..081b4e22a502f9e8b11f6276fdade2794409ba6f 100644 (file)
@@ -234,13 +234,6 @@ struct hisi_sas_device {
        spinlock_t lock; /* For protecting slots */
 };
 
-struct hisi_sas_tmf_task {
-       int force_phy;
-       int phy_id;
-       u8 tmf;
-       u16 tag_of_task_to_be_managed;
-};
-
 struct hisi_sas_slot {
        struct list_head entry;
        struct list_head delivery;
@@ -259,7 +252,7 @@ struct hisi_sas_slot {
        dma_addr_t cmd_hdr_dma;
        struct timer_list internal_abort_timer;
        bool is_internal;
-       struct hisi_sas_tmf_task *tmf;
+       struct sas_tmf_task *tmf;
        /* Do not reorder/change members after here */
        void    *buf;
        dma_addr_t buf_dma;
index bb95153bd22f08629483eb96bacccb542336055b..0ca80e00835cbf110c9d9574b7c8d7718e3ad365 100644 (file)
@@ -11,7 +11,7 @@
        ((!dev) || (dev->dev_type == SAS_PHY_UNUSED))
 
 static int hisi_sas_debug_issue_ssp_tmf(struct domain_device *device,
-                               u8 *lun, struct hisi_sas_tmf_task *tmf);
+                               u8 *lun, struct sas_tmf_task *tmf);
 static int
 hisi_sas_internal_task_abort(struct hisi_hba *hisi_hba,
                             struct domain_device *device,
@@ -477,7 +477,7 @@ void hisi_sas_task_deliver(struct hisi_hba *hisi_hba,
 }
 
 static int hisi_sas_task_exec(struct sas_task *task, gfp_t gfp_flags,
-                             struct hisi_sas_tmf_task *tmf)
+                             struct sas_tmf_task *tmf)
 {
        int n_elem = 0, n_elem_dif = 0, n_elem_req = 0;
        struct domain_device *device = task->dev;
@@ -691,7 +691,7 @@ static int hisi_sas_init_device(struct domain_device *device)
 {
        int rc = TMF_RESP_FUNC_COMPLETE;
        struct scsi_lun lun;
-       struct hisi_sas_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int retry = HISI_SAS_DISK_RECOVER_CNT;
        struct hisi_hba *hisi_hba = dev_to_hisi_hba(device);
        struct device *dev = hisi_hba->dev;
@@ -1195,7 +1195,7 @@ static void hisi_sas_tmf_timedout(struct timer_list *t)
 #define INTERNAL_ABORT_TIMEOUT         (6 * HZ)
 static int hisi_sas_exec_internal_tmf_task(struct domain_device *device,
                                           void *parameter, u32 para_len,
-                                          struct hisi_sas_tmf_task *tmf)
+                                          struct sas_tmf_task *tmf)
 {
        struct hisi_sas_device *sas_dev = device->lldd_dev;
        struct hisi_hba *hisi_hba = sas_dev->hisi_hba;
@@ -1330,7 +1330,7 @@ static int hisi_sas_softreset_ata_disk(struct domain_device *device)
        struct hisi_hba *hisi_hba = dev_to_hisi_hba(device);
        struct device *dev = hisi_hba->dev;
        int s = sizeof(struct host_to_dev_fis);
-       struct hisi_sas_tmf_task tmf = {};
+       struct sas_tmf_task tmf = {};
 
        ata_for_each_link(link, ap, EDGE) {
                int pmp = sata_srst_pmp(link);
@@ -1364,7 +1364,7 @@ static int hisi_sas_softreset_ata_disk(struct domain_device *device)
 }
 
 static int hisi_sas_debug_issue_ssp_tmf(struct domain_device *device,
-                               u8 *lun, struct hisi_sas_tmf_task *tmf)
+                               u8 *lun, struct sas_tmf_task *tmf)
 {
        struct sas_ssp_task ssp_task;
 
@@ -1469,7 +1469,7 @@ static void hisi_sas_send_ata_reset_each_phy(struct hisi_hba *hisi_hba,
                                             struct asd_sas_port *sas_port,
                                             struct domain_device *device)
 {
-       struct hisi_sas_tmf_task tmf_task = { .force_phy = 1 };
+       struct sas_tmf_task tmf_task = { .force_phy = 1 };
        struct ata_port *ap = device->sata_dev.ap;
        struct device *dev = hisi_hba->dev;
        int s = sizeof(struct host_to_dev_fis);
@@ -1624,7 +1624,7 @@ static int hisi_sas_controller_reset(struct hisi_hba *hisi_hba)
 static int hisi_sas_abort_task(struct sas_task *task)
 {
        struct scsi_lun lun;
-       struct hisi_sas_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        struct domain_device *device = task->dev;
        struct hisi_sas_device *sas_dev = device->lldd_dev;
        struct hisi_hba *hisi_hba;
@@ -1733,7 +1733,7 @@ static int hisi_sas_abort_task_set(struct domain_device *device, u8 *lun)
 {
        struct hisi_hba *hisi_hba = dev_to_hisi_hba(device);
        struct device *dev = hisi_hba->dev;
-       struct hisi_sas_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int rc;
 
        rc = hisi_sas_internal_task_abort(hisi_hba, device,
@@ -1868,7 +1868,7 @@ static int hisi_sas_lu_reset(struct domain_device *device, u8 *lun)
                        hisi_sas_release_task(hisi_hba, device);
                sas_put_local_phy(phy);
        } else {
-               struct hisi_sas_tmf_task tmf_task = { .tmf =  TMF_LU_RESET };
+               struct sas_tmf_task tmf_task = { .tmf =  TMF_LU_RESET };
 
                rc = hisi_sas_debug_issue_ssp_tmf(device, lun, &tmf_task);
                if (rc == TMF_RESP_FUNC_COMPLETE)
@@ -1926,7 +1926,7 @@ static int hisi_sas_clear_nexus_ha(struct sas_ha_struct *sas_ha)
 static int hisi_sas_query_task(struct sas_task *task)
 {
        struct scsi_lun lun;
-       struct hisi_sas_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int rc = TMF_RESP_FUNC_FAILED;
 
        if (task->lldd_task && task->task_proto & SAS_PROTOCOL_SSP) {
index 862f4e8b7eb58e0907f932b256f7ec07b39bfcaa..76756dd318d75d513a4cc2151ba08b5c12458fc7 100644 (file)
@@ -958,7 +958,7 @@ static void prep_ssp_v1_hw(struct hisi_hba *hisi_hba,
        struct hisi_sas_port *port = slot->port;
        struct sas_ssp_task *ssp_task = &task->ssp_task;
        struct scsi_cmnd *scsi_cmnd = ssp_task->cmd;
-       struct hisi_sas_tmf_task *tmf = slot->tmf;
+       struct sas_tmf_task *tmf = slot->tmf;
        int has_data = 0, priority = !!tmf;
        u8 *buf_cmd, fburst = 0;
        u32 dw1, dw2;
index a6d89a1495461923afb21ee453ff0a7c67227868..9a8009fc3f20685b6f7f30809385bb351cf84a71 100644 (file)
@@ -1742,7 +1742,7 @@ static void prep_ssp_v2_hw(struct hisi_hba *hisi_hba,
        struct hisi_sas_port *port = slot->port;
        struct sas_ssp_task *ssp_task = &task->ssp_task;
        struct scsi_cmnd *scsi_cmnd = ssp_task->cmd;
-       struct hisi_sas_tmf_task *tmf = slot->tmf;
+       struct sas_tmf_task *tmf = slot->tmf;
        int has_data = 0, priority = !!tmf;
        u8 *buf_cmd;
        u32 dw1 = 0, dw2 = 0;
@@ -2499,7 +2499,7 @@ static void prep_ata_v2_hw(struct hisi_hba *hisi_hba,
        struct hisi_sas_cmd_hdr *hdr = slot->cmd_hdr;
        struct asd_sas_port *sas_port = device->port;
        struct hisi_sas_port *port = to_hisi_sas_port(sas_port);
-       struct hisi_sas_tmf_task *tmf = slot->tmf;
+       struct sas_tmf_task *tmf = slot->tmf;
        u8 *buf_cmd;
        int has_data = 0, hdr_tag = 0;
        u32 dw0, dw1 = 0, dw2 = 0;
index 3f3d768548c576610f4d729e4def14900cdc609f..2fce1c0a54635d60b613e3f1b96c22a4d07fc649 100644 (file)
@@ -1223,7 +1223,7 @@ static void prep_ssp_v3_hw(struct hisi_hba *hisi_hba,
        struct hisi_sas_port *port = slot->port;
        struct sas_ssp_task *ssp_task = &task->ssp_task;
        struct scsi_cmnd *scsi_cmnd = ssp_task->cmd;
-       struct hisi_sas_tmf_task *tmf = slot->tmf;
+       struct sas_tmf_task *tmf = slot->tmf;
        int has_data = 0, priority = !!tmf;
        unsigned char prot_op;
        u8 *buf_cmd;
index 199ab49aa047f8f58a9b8dc4577ec3bf818682bd..7123a2efbf5836b8610539564882434c85bcc792 100644 (file)
@@ -486,9 +486,4 @@ enum datapres_field {
        SENSE_DATA      = 2,
 };
 
-/* define task management IU */
-struct mvs_tmf_task{
-       u8 tmf;
-       u16 tag_of_task_to_be_managed;
-};
 #endif
index 107f1993932bf257ee37f5b011cca4f0dddf364f..04d3710c683fe7f59930149cac6bdf5e55156a76 100644 (file)
@@ -551,7 +551,7 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
 
 static int mvs_task_prep_ssp(struct mvs_info *mvi,
                             struct mvs_task_exec_info *tei, int is_tmf,
-                            struct mvs_tmf_task *tmf)
+                            struct sas_tmf_task *tmf)
 {
        struct sas_task *task = tei->task;
        struct mvs_cmd_hdr *hdr = tei->hdr;
@@ -691,7 +691,7 @@ static int mvs_task_prep_ssp(struct mvs_info *mvi,
 
 #define        DEV_IS_GONE(mvi_dev)    ((!mvi_dev || (mvi_dev->dev_type == SAS_PHY_UNUSED)))
 static int mvs_task_prep(struct sas_task *task, struct mvs_info *mvi, int is_tmf,
-                               struct mvs_tmf_task *tmf, int *pass)
+                               struct sas_tmf_task *tmf, int *pass)
 {
        struct domain_device *dev = task->dev;
        struct mvs_device *mvi_dev = dev->lldd_dev;
@@ -837,7 +837,7 @@ prep_out:
 
 static int mvs_task_exec(struct sas_task *task, gfp_t gfp_flags,
                                struct completion *completion, int is_tmf,
-                               struct mvs_tmf_task *tmf)
+                               struct sas_tmf_task *tmf)
 {
        struct mvs_info *mvi = NULL;
        u32 rc = 0;
@@ -1275,7 +1275,7 @@ static void mvs_tmf_timedout(struct timer_list *t)
 
 #define MVS_TASK_TIMEOUT 20
 static int mvs_exec_internal_tmf_task(struct domain_device *dev,
-                       void *parameter, u32 para_len, struct mvs_tmf_task *tmf)
+                       void *parameter, u32 para_len, struct sas_tmf_task *tmf)
 {
        int res, retry;
        struct sas_task *task = NULL;
@@ -1350,7 +1350,7 @@ ex_err:
 }
 
 static int mvs_debug_issue_ssp_tmf(struct domain_device *dev,
-                               u8 *lun, struct mvs_tmf_task *tmf)
+                               u8 *lun, struct sas_tmf_task *tmf)
 {
        struct sas_ssp_task ssp_task;
        if (!(dev->tproto & SAS_PROTOCOL_SSP))
@@ -1382,7 +1382,7 @@ int mvs_lu_reset(struct domain_device *dev, u8 *lun)
 {
        unsigned long flags;
        int rc = TMF_RESP_FUNC_FAILED;
-       struct mvs_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        struct mvs_device * mvi_dev = dev->lldd_dev;
        struct mvs_info *mvi = mvi_dev->mvi_info;
 
@@ -1426,7 +1426,7 @@ int mvs_query_task(struct sas_task *task)
 {
        u32 tag;
        struct scsi_lun lun;
-       struct mvs_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int rc = TMF_RESP_FUNC_FAILED;
 
        if (task->lldd_task && task->task_proto & SAS_PROTOCOL_SSP) {
@@ -1463,7 +1463,7 @@ int mvs_query_task(struct sas_task *task)
 int mvs_abort_task(struct sas_task *task)
 {
        struct scsi_lun lun;
-       struct mvs_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        struct domain_device *dev = task->dev;
        struct mvs_device *mvi_dev = (struct mvs_device *)dev->lldd_dev;
        struct mvs_info *mvi;
@@ -1540,7 +1540,7 @@ out:
 int mvs_abort_task_set(struct domain_device *dev, u8 *lun)
 {
        int rc;
-       struct mvs_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
 
        tmf_task.tmf = TMF_ABORT_TASK_SET;
        rc = mvs_debug_issue_ssp_tmf(dev, lun, &tmf_task);
@@ -1551,7 +1551,7 @@ int mvs_abort_task_set(struct domain_device *dev, u8 *lun)
 int mvs_clear_task_set(struct domain_device *dev, u8 *lun)
 {
        int rc = TMF_RESP_FUNC_FAILED;
-       struct mvs_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
 
        tmf_task.tmf = TMF_CLEAR_TASK_SET;
        rc = mvs_debug_issue_ssp_tmf(dev, lun, &tmf_task);
index 352705e023c8309752156659dd794209f40c26f1..537b762a2b8343605d8f42c2f5f41208e18c6a9f 100644 (file)
@@ -4624,7 +4624,7 @@ int pm8001_chip_abort_task(struct pm8001_hba_info *pm8001_ha,
  * @tmf: task management function.
  */
 int pm8001_chip_ssp_tm_req(struct pm8001_hba_info *pm8001_ha,
-       struct pm8001_ccb_info *ccb, struct pm8001_tmf_task *tmf)
+       struct pm8001_ccb_info *ccb, struct sas_tmf_task *tmf)
 {
        struct sas_task *task = ccb->task;
        struct domain_device *dev = task->dev;
@@ -4636,7 +4636,7 @@ int pm8001_chip_ssp_tm_req(struct pm8001_hba_info *pm8001_ha,
 
        memset(&sspTMCmd, 0, sizeof(sspTMCmd));
        sspTMCmd.device_id = cpu_to_le32(pm8001_dev->device_id);
-       sspTMCmd.relate_tag = cpu_to_le32(tmf->tag_of_task_to_be_managed);
+       sspTMCmd.relate_tag = cpu_to_le32((u32)tmf->tag_of_task_to_be_managed);
        sspTMCmd.tmf = cpu_to_le32(tmf->tmf);
        memcpy(sspTMCmd.lun, task->ssp_task.LUN, 8);
        sspTMCmd.tag = cpu_to_le32(ccb->ccb_tag);
index 93fe1bec25e211572e0ecae97a0849492ac4f3e6..2c003c17fe6aa8284a0b50bc179a541675806cf0 100644 (file)
@@ -335,7 +335,7 @@ static int pm8001_task_prep_ata(struct pm8001_hba_info *pm8001_ha,
   * @tmf: the task management IU
   */
 static int pm8001_task_prep_ssp_tm(struct pm8001_hba_info *pm8001_ha,
-       struct pm8001_ccb_info *ccb, struct pm8001_tmf_task *tmf)
+       struct pm8001_ccb_info *ccb, struct sas_tmf_task *tmf)
 {
        return PM8001_CHIP_DISP->ssp_tm_req(pm8001_ha, ccb, tmf);
 }
@@ -378,7 +378,7 @@ static int sas_find_local_port_id(struct domain_device *dev)
   * @tmf: the task management IU
   */
 static int pm8001_task_exec(struct sas_task *task,
-       gfp_t gfp_flags, int is_tmf, struct pm8001_tmf_task *tmf)
+       gfp_t gfp_flags, int is_tmf, struct sas_tmf_task *tmf)
 {
        struct domain_device *dev = task->dev;
        struct pm8001_hba_info *pm8001_ha;
@@ -715,7 +715,7 @@ static void pm8001_tmf_timedout(struct timer_list *t)
   * this function, note it is also with the task execute interface.
   */
 static int pm8001_exec_internal_tmf_task(struct domain_device *dev,
-       void *parameter, u32 para_len, struct pm8001_tmf_task *tmf)
+       void *parameter, u32 para_len, struct sas_tmf_task *tmf)
 {
        int res, retry;
        struct sas_task *task = NULL;
@@ -906,7 +906,7 @@ void pm8001_dev_gone(struct domain_device *dev)
 }
 
 static int pm8001_issue_ssp_tmf(struct domain_device *dev,
-       u8 *lun, struct pm8001_tmf_task *tmf)
+       u8 *lun, struct sas_tmf_task *tmf)
 {
        struct sas_ssp_task ssp_task;
        if (!(dev->tproto & SAS_PROTOCOL_SSP))
@@ -1108,7 +1108,7 @@ out:
 int pm8001_lu_reset(struct domain_device *dev, u8 *lun)
 {
        int rc = TMF_RESP_FUNC_FAILED;
-       struct pm8001_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        struct pm8001_device *pm8001_dev = dev->lldd_dev;
        struct pm8001_hba_info *pm8001_ha = pm8001_find_ha_by_dev(dev);
        DECLARE_COMPLETION_ONSTACK(completion_setstate);
@@ -1137,7 +1137,7 @@ int pm8001_query_task(struct sas_task *task)
 {
        u32 tag = 0xdeadbeef;
        struct scsi_lun lun;
-       struct pm8001_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int rc = TMF_RESP_FUNC_FAILED;
        if (unlikely(!task || !task->lldd_task || !task->dev))
                return rc;
@@ -1186,7 +1186,7 @@ int pm8001_abort_task(struct sas_task *task)
        struct pm8001_hba_info *pm8001_ha;
        struct scsi_lun lun;
        struct pm8001_device *pm8001_dev;
-       struct pm8001_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        int rc = TMF_RESP_FUNC_FAILED, ret;
        u32 phy_id;
        struct sas_task_slow slow_task;
@@ -1335,7 +1335,7 @@ out:
 
 int pm8001_abort_task_set(struct domain_device *dev, u8 *lun)
 {
-       struct pm8001_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
 
        tmf_task.tmf = TMF_ABORT_TASK_SET;
        return pm8001_issue_ssp_tmf(dev, lun, &tmf_task);
@@ -1343,7 +1343,7 @@ int pm8001_abort_task_set(struct domain_device *dev, u8 *lun)
 
 int pm8001_clear_task_set(struct domain_device *dev, u8 *lun)
 {
-       struct pm8001_tmf_task tmf_task;
+       struct sas_tmf_task tmf_task;
        struct pm8001_device *pm8001_dev = dev->lldd_dev;
        struct pm8001_hba_info *pm8001_ha = pm8001_find_ha_by_dev(dev);
 
index 52c62a29df18a2e7e1f5877d290a999ce716788c..75864b47921aa92a9d4f45fb08655aaf4051ed4e 100644 (file)
@@ -99,11 +99,7 @@ extern const struct pm8001_dispatch pm8001_80xx_dispatch;
 struct pm8001_hba_info;
 struct pm8001_ccb_info;
 struct pm8001_device;
-/* define task management IU */
-struct pm8001_tmf_task {
-       u8      tmf;
-       u32     tag_of_task_to_be_managed;
-};
+
 struct pm8001_ioctl_payload {
        u32     signature;
        u16     major_function;
@@ -203,7 +199,7 @@ struct pm8001_dispatch {
                struct pm8001_device *pm8001_dev, u8 flag, u32 task_tag,
                u32 cmd_tag);
        int (*ssp_tm_req)(struct pm8001_hba_info *pm8001_ha,
-               struct pm8001_ccb_info *ccb, struct pm8001_tmf_task *tmf);
+               struct pm8001_ccb_info *ccb, struct sas_tmf_task *tmf);
        int (*get_nvmd_req)(struct pm8001_hba_info *pm8001_ha, void *payload);
        int (*set_nvmd_req)(struct pm8001_hba_info *pm8001_ha, void *payload);
        int (*fw_flash_update_req)(struct pm8001_hba_info *pm8001_ha,
@@ -682,7 +678,7 @@ int pm8001_chip_set_nvmd_req(struct pm8001_hba_info *pm8001_ha, void *payload);
 int pm8001_chip_get_nvmd_req(struct pm8001_hba_info *pm8001_ha, void *payload);
 int pm8001_chip_ssp_tm_req(struct pm8001_hba_info *pm8001_ha,
                                struct pm8001_ccb_info *ccb,
-                               struct pm8001_tmf_task *tmf);
+                               struct sas_tmf_task *tmf);
 int pm8001_chip_abort_task(struct pm8001_hba_info *pm8001_ha,
                                struct pm8001_device *pm8001_dev,
                                u8 flag, u32 task_tag, u32 cmd_tag);
index 22c54f21206d4431d41ac2a60b61b90e8578a410..306005b3b60f3c99570a3340942632cebd6b46d2 100644 (file)
@@ -576,6 +576,15 @@ struct sas_ssp_task {
        struct scsi_cmnd *cmd;
 };
 
+struct sas_tmf_task {
+       u8 tmf;
+       u16 tag_of_task_to_be_managed;
+
+       /* Temp */
+       int force_phy;
+       int phy_id;
+};
+
 struct sas_task {
        struct domain_device *dev;