drivers/perf: hisi: hns3: Actually use devm_add_action_or_reset()
authorHao Chen <chenhao418@huawei.com>
Thu, 25 Apr 2024 12:46:27 +0000 (20:46 +0800)
committerWill Deacon <will@kernel.org>
Sun, 28 Apr 2024 07:49:31 +0000 (08:49 +0100)
commit582c1aeee0a9e73010cf1c4cef338709860deeb0
treeb66f93aa8862e5c08fe7b44bfc1874723f922c44
parent81bdd60a3d1d3b05e6cc6674845afb1694dd3a0e
drivers/perf: hisi: hns3: Actually use devm_add_action_or_reset()

pci_alloc_irq_vectors() allocates an irq vector. When devm_add_action()
fails, the irq vector is not freed, which leads to a memory leak.

Replace the devm_add_action with devm_add_action_or_reset to ensure
the irq vector can be destroyed when it fails.

Fixes: 66637ab137b4 ("drivers/perf: hisi: add driver for HNS3 PMU")
Signed-off-by: Hao Chen <chenhao418@huawei.com>
Signed-off-by: Junhao He <hejunhao3@huawei.com>
Reviewed-by: Jijie Shao <shaojijie@huawei.com>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Link: https://lore.kernel.org/r/20240425124627.13764-4-hejunhao3@huawei.com
Signed-off-by: Will Deacon <will@kernel.org>
drivers/perf/hisilicon/hns3_pmu.c