]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
scsi: pm8001: Fix pm8001_mpi_task_abort_resp()
authorDamien Le Moal <damien.lemoal@opensource.wdc.com>
Sun, 20 Feb 2022 03:17:57 +0000 (12:17 +0900)
committerStefan Bader <stefan.bader@canonical.com>
Fri, 20 May 2022 12:40:44 +0000 (14:40 +0200)
commite5e2d634a50eb1ff95cc91e950d8f4bd7fc1d3eb
tree638319eea1d6d938ca870b517fe2059cce06a170
parent1e00ed011065774dd50708c12689a8df067a2183
scsi: pm8001: Fix pm8001_mpi_task_abort_resp()

BugLink: https://bugs.launchpad.net/bugs/1969107
[ Upstream commit 7e6b7e740addcea450041b5be8e42f0a4ceece0f ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
(cherry picked from commit ece79aaec8fcd261b56b9331907f9961b794e723)
Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>
drivers/scsi/pm8001/pm8001_hwi.c