diff options
author | Stephen M. Cameron <scameron@beardog.cce.hp.com> | 2012-01-19 15:00:42 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-19 09:08:55 -0500 |
commit | 55e14e764df5e24bedf93220f1da167af8300c2f (patch) | |
tree | ba75518670f75907a2780b5ef76bb3a1f42ab5aa /drivers/scsi/hpsa.c | |
parent | 26b937b2a349e9e6b294703232af264af7e277c4 (diff) |
[SCSI] hpsa: fix per device memory leak on driver unload
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/hpsa.c')
-rw-r--r-- | drivers/scsi/hpsa.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index b96962c39449..4dc9107456ae 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c | |||
@@ -4482,6 +4482,14 @@ static void hpsa_shutdown(struct pci_dev *pdev) | |||
4482 | #endif /* CONFIG_PCI_MSI */ | 4482 | #endif /* CONFIG_PCI_MSI */ |
4483 | } | 4483 | } |
4484 | 4484 | ||
4485 | static void __devexit hpsa_free_device_info(struct ctlr_info *h) | ||
4486 | { | ||
4487 | int i; | ||
4488 | |||
4489 | for (i = 0; i < h->ndevices; i++) | ||
4490 | kfree(h->dev[i]); | ||
4491 | } | ||
4492 | |||
4485 | static void __devexit hpsa_remove_one(struct pci_dev *pdev) | 4493 | static void __devexit hpsa_remove_one(struct pci_dev *pdev) |
4486 | { | 4494 | { |
4487 | struct ctlr_info *h; | 4495 | struct ctlr_info *h; |
@@ -4497,6 +4505,7 @@ static void __devexit hpsa_remove_one(struct pci_dev *pdev) | |||
4497 | iounmap(h->vaddr); | 4505 | iounmap(h->vaddr); |
4498 | iounmap(h->transtable); | 4506 | iounmap(h->transtable); |
4499 | iounmap(h->cfgtable); | 4507 | iounmap(h->cfgtable); |
4508 | hpsa_free_device_info(h); | ||
4500 | hpsa_free_sg_chain_blocks(h); | 4509 | hpsa_free_sg_chain_blocks(h); |
4501 | pci_free_consistent(h->pdev, | 4510 | pci_free_consistent(h->pdev, |
4502 | h->nr_cmds * sizeof(struct CommandList), | 4511 | h->nr_cmds * sizeof(struct CommandList), |