dmaengine: idxd: Avoid unnecessary destruction of file_ida
authorFenghua Yu <fenghua.yu@intel.com>
Tue, 30 Jan 2024 01:39:54 +0000 (17:39 -0800)
committerVinod Koul <vkoul@kernel.org>
Sat, 4 May 2024 12:22:04 +0000 (17:52 +0530)
commit76e43fa6a456787bad31b8d0daeabda27351a480
tree2d1dfd8bb33179364617cd1c2bee292f29e0573a
parent700b2e1eccb4490752227d4339c3d2c3d52d06a7
dmaengine: idxd: Avoid unnecessary destruction of file_ida

file_ida is allocated during cdev open and is freed accordingly
during cdev release. This sequence is guaranteed by driver file
operations. Therefore, there is no need to destroy an already empty
file_ida when the WQ cdev is removed.

Worse, ida_free() in cdev release may happen after destruction of
file_ida per WQ cdev. This can lead to accessing an id in file_ida
after it has been destroyed, resulting in a kernel panic.

Remove ida_destroy(&file_ida) to address these issues.

Fixes: e6fd6d7e5f0f ("dmaengine: idxd: add a device to represent the file opened")
Signed-off-by: Lijun Pan <lijun.pan@intel.com>
Signed-off-by: Fenghua Yu <fenghua.yu@intel.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/20240130013954.2024231-1-fenghua.yu@intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/idxd/cdev.c