[PATCH] drm/amd/amdgpu: fix bug fail to remove debugfs when rmmod
Wang Hongcheng
Annie.Wang at amd.com
Thu Aug 10 09:12:18 UTC 2017
Some debug files are forgotten to remove at fini. Remove them
all in pci_remove.
BUG: SWDEV-129297
Signed-off-by: Wang Hongcheng <Annie.Wang at amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 19 +++++++++++++++++++
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 ++
3 files changed, 23 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index c28069e4..b542191 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -1250,6 +1250,8 @@ struct amdgpu_debugfs {
int amdgpu_debugfs_add_files(struct amdgpu_device *adev,
const struct drm_info_list *files,
unsigned nfiles);
+int amdgpu_debugfs_cleanup(struct amdgpu_device *adev);
+
int amdgpu_debugfs_fence_init(struct amdgpu_device *adev);
#if defined(CONFIG_DEBUG_FS)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 7e40071..7594abb 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -3215,6 +3215,25 @@ int amdgpu_debugfs_add_files(struct amdgpu_device *adev,
return 0;
}
+int amdgpu_debugfs_cleanup(struct amdgpu_device *adev)
+{
+ struct drm_info_node *node, *tmp;
+
+ if (!&adev->ddev->primary->debugfs_root)
+ return 0;
+
+ mutex_lock(&adev->ddev->primary->debugfs_lock);
+ list_for_each_entry_safe(node, tmp,
+ &adev->ddev->primary->debugfs_list, list) {
+ debugfs_remove(node->dent);
+ list_del(&node->list);
+ kfree(node);
+ }
+ mutex_unlock(&adev->ddev->primary->debugfs_lock);
+
+ return 0;
+}
+
#if defined(CONFIG_DEBUG_FS)
static ssize_t amdgpu_debugfs_regs_read(struct file *f, char __user *buf,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index 13c91a6..4f5af93 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -594,7 +594,9 @@ static void
amdgpu_pci_remove(struct pci_dev *pdev)
{
struct drm_device *dev = pci_get_drvdata(pdev);
+ struct amdgpu_device *adev = dev->dev_private;
+ amdgpu_debugfs_cleanup(adev);
drm_dev_unregister(dev);
drm_dev_unref(dev);
}
--
2.7.4
More information about the amd-gfx
mailing list