Revert "drm/i915: Lock mode_config.mutex in intel_display_resume."
authorMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Tue, 4 Apr 2017 13:22:48 +0000 (15:22 +0200)
committerMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Wed, 12 Apr 2017 08:54:09 +0000 (10:54 +0200)
This reverts commit ea49c9acf2db7082f0406bb3a570cc6bad37082b.

mode_config.mutex was originally added to fix WARNs in connector
functions, but now that atomic nonblocking modeset support is
included, we will likely never hold any any lock at all.

The WARN mentioned in commit bbf35e9defb9a6d1 ("drm/i915:
Pass atomic state to intel_audio_codec_enable, v2."), so it's
safe to revert this now.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1491312168-18147-1-git-send-email-maarten.lankhorst@linux.intel.com
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c

index b6b40cd156443400894a5b2b8f0214bfbb165779..48a546210d8be4f99be2792030c515be805b2af4 100644 (file)
@@ -15559,13 +15559,6 @@ void intel_display_resume(struct drm_device *dev)
        if (state)
                state->acquire_ctx = &ctx;
 
-       /*
-        * This is a cludge because with real atomic modeset mode_config.mutex
-        * won't be taken. Unfortunately some probed state like
-        * audio_codec_enable is still protected by mode_config.mutex, so lock
-        * it here for now.
-        */
-       mutex_lock(&dev->mode_config.mutex);
        drm_modeset_acquire_init(&ctx, 0);
 
        while (1) {
@@ -15581,7 +15574,6 @@ void intel_display_resume(struct drm_device *dev)
 
        drm_modeset_drop_locks(&ctx);
        drm_modeset_acquire_fini(&ctx);
-       mutex_unlock(&dev->mode_config.mutex);
 
        if (ret)
                DRM_ERROR("Restoring old state failed with %i\n", ret);