[PATCH] drm/i915: Really fix the debug unregister code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This smells like a missing git add, but unregister was not handled
properly in:
commit c032d5491a4d8d56d8364f4919364815b55d3437
Author: Daniel Vetter <daniel.vetter at ffwll.ch>
Date:   Thu Jul 4 20:39:49 2013 +0200

    drm/i915: convert debugfs creation/destruction to table

fops is already the pointer that we want when incrementally removing.

One really bad side effect of this failure is all tests in our test
suite will fail after module_reload fails.

This fix was actually lazily authored by Chris.

CC: Chris Wilson <chris at chris-wilson.co.uk>
Reported-and-tested-by: Ben Widawsky <ben at bwidawsk.net>
Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
---
 drivers/gpu/drm/i915/i915_debugfs.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index b49d731..d870f27 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -2420,8 +2420,7 @@ void i915_debugfs_cleanup(struct drm_minor *minor)
 				 1, minor);
 	for (i = 0; i < ARRAY_SIZE(i915_debugfs_files); i++) {
 		struct drm_info_list *info_list =
-			(struct drm_info_list *) &i915_debugfs_files[i].fops;
-
+			(struct drm_info_list *)i915_debugfs_files[i].fops;
 		drm_debugfs_remove_files(info_list, 1, minor);
 	}
 }
-- 
1.8.3.2



[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux