diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-10-04 10:33:04 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-10-04 11:11:54 -0400 |
commit | 7b4f3990a22fbe800945f12001bc30db374d0af5 (patch) | |
tree | 2e894f736ba0c4c05b481e7db8d8a122c70a9850 /drivers/gpu/drm/i915/i915_dma.c | |
parent | 2c6be944111a873ce96865f1a6033056bdf0d0e2 (diff) |
drm/i915: Avoid circular locking from intel_fbdev_fini()
lockdep spots that the fb_info->lock takes the dev->struct_mutex during
init (due to the device probing) and so we can not hold
dev->struct_mutex when unregistering the framebuffer. Simply reverse the
order of initialisation during cleanup and so do the intel_fbdev_fini()
before the intel_modeset_cleanup.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_dma.c')
-rw-r--r-- | drivers/gpu/drm/i915/i915_dma.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c index df86d04f7968..726c3736082f 100644 --- a/drivers/gpu/drm/i915/i915_dma.c +++ b/drivers/gpu/drm/i915/i915_dma.c | |||
@@ -2106,6 +2106,7 @@ int i915_driver_unload(struct drm_device *dev) | |||
2106 | acpi_video_unregister(); | 2106 | acpi_video_unregister(); |
2107 | 2107 | ||
2108 | if (drm_core_check_feature(dev, DRIVER_MODESET)) { | 2108 | if (drm_core_check_feature(dev, DRIVER_MODESET)) { |
2109 | intel_fbdev_fini(dev); | ||
2109 | intel_modeset_cleanup(dev); | 2110 | intel_modeset_cleanup(dev); |
2110 | 2111 | ||
2111 | /* | 2112 | /* |