drm: cleanup debugfs in drm_unplug_minor()
authorDavid Herrmann <dh.herrmann@gmail.com>
Sun, 20 Oct 2013 16:55:43 +0000 (18:55 +0200)
committerDave Airlie <airlied@redhat.com>
Wed, 6 Nov 2013 04:53:24 +0000 (14:53 +1000)
There is no reason to delay debugfs-cleanup to drm_put_minor(). We should
forbid any access to debugfs files once the device is dead. Chances they
oops once a card was unplugged are very high, anyway.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_stub.c

index 76b4d718af601ce460579c3331de3618f5e6e091..b37b0d99f262a887ab11e7a049e7ff5eff052d5f 100644 (file)
@@ -341,6 +341,10 @@ static void drm_unplug_minor(struct drm_minor *minor)
        if (!minor || !device_is_registered(minor->kdev))
                return;
 
+#if defined(CONFIG_DEBUG_FS)
+       drm_debugfs_cleanup(minor);
+#endif
+
        drm_sysfs_device_remove(minor);
 }
 
@@ -360,10 +364,6 @@ static void drm_put_minor(struct drm_minor *minor)
 
        DRM_DEBUG("release secondary minor %d\n", minor->index);
 
-#if defined(CONFIG_DEBUG_FS)
-       drm_debugfs_cleanup(minor);
-#endif
-
        drm_unplug_minor(minor);
 
        idr_remove(&drm_minors_idr, minor->index);