Re: Re:Re: [PATCH] scsi: hpsa: fix DMA mapping leak on IOACCEL2 reset path

[<Don.Brace@microchip.com>]

________________________________________
From: haoxiang_li2024 <haoxiang_li2024@163.com>
Sent: Wednesday, June 24, 2026 1:10 AM
To: Don Brace - C33706 <Don.Brace@microchip.com>
Cc: James.Bottomley@HansenPartnership.com <James.Bottomley@HansenPartnership.com>; martin.petersen@oracle.com <martin.petersen@oracle.com>; david.carroll@microsemi.com <david.carroll@microsemi.com>; justin.lindley@microsemi.com <justin.lindley@microsemi.com>; scott.teel@microsemi.com <scott.teel@microsemi.com>; storagedev <storagedev@microchip.com>; linux-scsi@vger.kernel.org <linux-scsi@vger.kernel.org>; linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; stable@vger.kernel.org <stable@vger.kernel.org>
Subject: Re:Re: [PATCH] scsi: hpsa: fix DMA mapping leak on IOACCEL2 reset path
 
EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe

On 2026-06-23 02:26:42，Don.Brace@microchip.com wrote：

>Fixes: c5dfd106414f ("scsi: hpsa: correct device resets")
>Cc: stable@vger.kernel.org
>Signed-off-by: Haoxiang Li <haoxiang_li2024@163.com>
>
>Acked-by: Don Brace <don.brace@microchip.com
>Thanks for your patch. Can fix potential performance issues with devices undergoing resets.
>What about another patch for when call to hpsa_map_ioaccel2_sg_chain_block() fails?
>

>

Thanks for your review! I checked the hpsa_map_ioaccel2_sg_chain_block() failure path. It
already decrements ioaccel_cmds_out and calls  scsi_dma_unmap(cmd). I think this patch
is enough?

Yes, good enough.
Thanks for checking.
>---
> drivers/scsi/hpsa.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
>index a1b116cd4723..8edad1830abe 100644
>--- a/drivers/scsi/hpsa.c
>+++ b/drivers/scsi/hpsa.c
>@@ -5017,6 +5017,10 @@ static int hpsa_scsi_ioaccel2_queue_command(struct ctlr_info *h,
>
>        if (phys_disk->in_reset) {
>                cmd->result = DID_RESET << 16;
>+               atomic_dec(&phys_disk->ioaccel_cmds_out);
>+               scsi_dma_unmap(cmd);
>+               if (use_sg > h->ioaccel_maxsg)
>+                       hpsa_unmap_ioaccel2_sg_chain_block(h, cp);
>                return -1;
>        }
>
>--
>2.25.1

Thanks,
Haoxiang