From: Paulo Zanoni <[email protected]>

Neither intel_fbc_disable() nor intel_frontbuffer_flip_prepare() need
that lock.

That intel_fbc_disable() call over there can also certainly be
improved, but let's do it on its own commit.

Signed-off-by: Paulo Zanoni <[email protected]>
---
 drivers/gpu/drm/i915/intel_display.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/intel_display.c 
b/drivers/gpu/drm/i915/intel_display.c
index eab5aa1..0d98332 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -11445,11 +11445,11 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc,
 
        i915_gem_track_fb(intel_fb_obj(work->old_fb), obj,
                          to_intel_plane(primary)->frontbuffer_bit);
+       mutex_unlock(&dev->struct_mutex);
 
        intel_fbc_disable(dev);
        intel_frontbuffer_flip_prepare(dev,
                                       
to_intel_plane(primary)->frontbuffer_bit);
-       mutex_unlock(&dev->struct_mutex);
 
        trace_i915_flip_request(intel_crtc->plane, obj);
 
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to