Re: [PATCH v5.10] scsi: pm80xx: Fix memory leak during rmmod

[Sasha Levin <sashal@kernel.org>]

[ Sasha's backport helper bot ]

Hi,

✅ All tests passed successfully. No issues detected.
No action required from the submitter.

The upstream commit SHA1 provided is correct: 51e6ed83bb4ade7c360551fa4ae55c4eacea354b

WARNING: Author mismatch between patch and upstream commit:
Backport author: Shivani Agarwal <shivani.agarwal@broadcom.com>
Commit author: Ajish Koshy <Ajish.Koshy@microchip.com>

Status in newer kernel trees:
6.15.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Present (exact SHA1)
6.1.y | Present (exact SHA1)
5.15.y | Present (different SHA1: 269a4311b15f)

Note: The patch differs from the upstream commit:
---
1:  51e6ed83bb4a ! 1:  1652f92390bb scsi: pm80xx: Fix memory leak during rmmod
    @@ Metadata
      ## Commit message ##
         scsi: pm80xx: Fix memory leak during rmmod
     
    +    [ Upstream commit 51e6ed83bb4ade7c360551fa4ae55c4eacea354b ]
    +
         Driver failed to release all memory allocated. This would lead to memory
         leak during driver removal.
     
    @@ Commit message
         Signed-off-by: Ajish Koshy <Ajish.Koshy@microchip.com>
         Signed-off-by: Viswas G <Viswas.G@microchip.com>
         Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
    +    Signed-off-by: Sasha Levin <sashal@kernel.org>
    +    [Shivani: Modified to apply on 5.10.y]
    +    Signed-off-by: Shivani Agarwal <shivani.agarwal@broadcom.com>
     
      ## drivers/scsi/pm8001/pm8001_init.c ##
     @@ drivers/scsi/pm8001/pm8001_init.c: pm8001_init_ccb_tag(struct pm8001_hba_info *pm8001_ha, struct Scsi_Host *shost,
    @@ drivers/scsi/pm8001/pm8001_init.c: pm8001_init_ccb_tag(struct pm8001_hba_info *p
      
      	/* Memory region for ccb_info*/
     +	pm8001_ha->ccb_count = ccb_count;
    - 	pm8001_ha->ccb_info =
    + 	pm8001_ha->ccb_info = (struct pm8001_ccb_info *)
      		kcalloc(ccb_count, sizeof(struct pm8001_ccb_info), GFP_KERNEL);
      	if (!pm8001_ha->ccb_info) {
     @@ drivers/scsi/pm8001/pm8001_init.c: static void pm8001_pci_remove(struct pci_dev *pdev)

---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| origin/linux-5.10.y       | Success     | Success    |