scsi: pm8001: Fix pm8001_mpi_task_abort_resp()

The call to pm8001_ccb_task_free() at the end of
pm8001_mpi_task_abort_resp() already frees the ccb tag. So when the device
NCQ_ABORT_ALL_FLAG is set, the tag should not be freed again.  Also change
the hardcoded 0xBFFFFFFF value to ~NCQ_ABORT_ALL_FLAG as it ought to be.

Link: https://lore.kernel.org/r/20220220031810.738362-19-damien.lemoal@opensource.wdc.com
Reviewed-by: Jack Wang <jinpu.wang@ionos.com>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c
index cfeefdf26f5d4a1a9c6b9818e4af01746188d7ca..be57b9ae5486fd2deeb7c8dff144d7687a6fc2c6 100644 (file)
--- a/drivers/scsi/pm8001/pm8001_hwi.c
+++ b/drivers/scsi/pm8001/pm8001_hwi.c
@@ -3700,12 +3700,11 @@ int pm8001_mpi_task_abort_resp(struct pm8001_hba_info *pm8001_ha, void *piomb)
        mb();
 
        if (pm8001_dev->id & NCQ_ABORT_ALL_FLAG) {
-               pm8001_tag_free(pm8001_ha, tag);
                sas_free_task(t);
-               /* clear the flag */
-               pm8001_dev->id &= 0xBFFFFFFF;
-       } else
+               pm8001_dev->id &= ~NCQ_ABORT_ALL_FLAG;
+       } else {
                t->task_done(t);
+       }
 
        return 0;
 }