Skip to content

Commit e592560

Browse files
committed
dmaengine: idxd: Fix refcount underflow on module unload
JIRA: https://issues.redhat.com/browse/RHEL-114133 Upstream-Status: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git commit b7cb9a0 Author: Yi Sun <yi.sun@intel.com> Date: Tue Jul 29 23:03:13 2025 +0800 dmaengine: idxd: Fix refcount underflow on module unload A recent refactor introduced a misplaced put_device() call, resulting in a reference count underflow during module unload. There is no need to add additional put_device() calls for idxd groups, engines, or workqueues. Although the commit claims: "Note, this also fixes the missing put_device() for idxd groups, engines, and wqs." It appears no such omission actually existed. The required cleanup is already handled by the call chain: idxd_unregister_devices() -> device_unregister() -> put_device() Extend idxd_cleanup() to handle the remaining necessary cleanup and remove idxd_cleanup_internals(), which duplicates deallocation logic for idxd, engines, groups, and workqueues. Memory management is also properly handled through the Linux device model. Fixes: a409e91 ("dmaengine: idxd: Refactor remove call with idxd_cleanup() helper") Signed-off-by: Yi Sun <yi.sun@intel.com> Tested-by: Shuai Xue <xueshuai@linux.alibaba.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> Acked-by: Vinicius Costa Gomes <vinicius.gomes@intel.com> Link: https://lore.kernel.org/r/20250729150313.1934101-3-yi.sun@intel.com Signed-off-by: Vinod Koul <vkoul@kernel.org> (cherry picked from commit b7cb9a0) Signed-off-by: Jerry Snitselaar <jsnitsel@redhat.com>
1 parent af922b4 commit e592560

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

drivers/dma/idxd/init.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1320,7 +1320,10 @@ static void idxd_remove(struct pci_dev *pdev)
13201320
device_unregister(idxd_confdev(idxd));
13211321
idxd_shutdown(pdev);
13221322
idxd_device_remove_debugfs(idxd);
1323-
idxd_cleanup(idxd);
1323+
perfmon_pmu_remove(idxd);
1324+
idxd_cleanup_interrupts(idxd);
1325+
if (device_pasid_enabled(idxd))
1326+
idxd_disable_system_pasid(idxd);
13241327
pci_iounmap(pdev, idxd->reg_base);
13251328
put_device(idxd_confdev(idxd));
13261329
pci_disable_device(pdev);

0 commit comments

Comments
 (0)