drm/amdgpu: cleanup gem init/finit

Remove the double housekeeping and use something sane to
forcefuly delete BOs on unload.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 20 insertions, 11 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
index 0a80febbc1b2..2e26a517f2d6 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
@@ -84,22 +84,31 @@ retry:
         }
         *obj = &robj->gem_base;
 
-        mutex_lock(&adev->gem.mutex);
-        list_add_tail(&robj->list, &adev->gem.objects);
-        mutex_unlock(&adev->gem.mutex);
-
         return 0;
 }
 
-int amdgpu_gem_init(struct amdgpu_device *adev)
+void amdgpu_gem_force_release(struct amdgpu_device *adev)
 {
-        INIT_LIST_HEAD(&adev->gem.objects);
-        return 0;
-}
+        struct drm_device *ddev = adev->ddev;
+        struct drm_file *file;
 
-void amdgpu_gem_fini(struct amdgpu_device *adev)
-{
-        amdgpu_bo_force_delete(adev);
+        mutex_lock(&ddev->struct_mutex);
+
+        list_for_each_entry(file, &ddev->filelist, lhead) {
+                struct drm_gem_object *gobj;
+                int handle;
+
+                WARN_ONCE(1, "Still active user space clients!\n");
+                spin_lock(&file->table_lock);
+                idr_for_each_entry(&file->object_idr, gobj, handle) {
+                        WARN_ONCE(1, "And also active allocations!\n");
+                        drm_gem_object_unreference(gobj);
+                }
+                idr_destroy(&file->object_idr);
+                spin_unlock(&file->table_lock);
+        }
+
+        mutex_unlock(&ddev->struct_mutex);
 }
 
 /*