dmaengine: idxd: Add missing idxd cleanup to fix memory leak in remove call

axiqia · gregkh · commit 2b7a961cea0e · 2025-05-22T14:31:56.000+02:00
commit d5449ff upstream. The remove call stack is missing idxd cleanup to free bitmap, ida and the idxd_device. Call idxd_free() helper routines to make sure we exit gracefully. Fixes: bfe1d56 ("dmaengine: idxd: Init and probe for Intel data accelerators") Cc: stable@vger.kernel.org Suggested-by: Vinicius Costa Gomes <vinicius.gomes@intel.com> Signed-off-by: Shuai Xue <xueshuai@linux.alibaba.com> Reviewed-by: Fenghua Yu <fenghuay@nvidia.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> Link: https://lore.kernel.org/r/20250404120217.48772-9-xueshuai@linux.alibaba.com Signed-off-by: Vinod Koul <vkoul@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/dma/idxd/init.c b/drivers/dma/idxd/init.c
@@ -1319,6 +1319,7 @@ static void idxd_remove(struct pci_dev *pdev)
 	destroy_workqueue(idxd->wq);
 	perfmon_pmu_remove(idxd);
 	put_device(idxd_confdev(idxd));
+	idxd_free(idxd);
 }
 
 static struct pci_driver idxd_pci_driver = {

Original file line number	Diff line number	Diff line change
`@@ -1319,6 +1319,7 @@ static void idxd_remove(struct pci_dev *pdev)`
`1319`	`1319`	`destroy_workqueue(idxd->wq);`
`1320`	`1320`	`perfmon_pmu_remove(idxd);`
`1321`	`1321`	`put_device(idxd_confdev(idxd));`
	`1322`	`+ idxd_free(idxd);`
`1322`	`1323`	`}`
`1323`	`1324`
`1324`	`1325`	`static struct pci_driver idxd_pci_driver = {`