<div dir="ltr">Tested-by and Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 8, 2014 at 11:27 AM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel.vetter@ffwll.ch" target="_blank">daniel.vetter@ffwll.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We treat other plane updates in the same fashion. Spotted because<br>
Rodrigo kept reporting a bug in the PSR code where the frontbuffer was<br>
eternally stuck with a dirty cursor bit set.<br>
<br>
The psr testcase should have caught this, but that i-g-t is kaputt.<br>
Rodrigo is signed up to fix that.<br>
<br>
Cc: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>><br>
Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>><br>
---<br>
drivers/gpu/drm/i915/intel_display.c | 4 ++++<br>
1 file changed, 4 insertions(+)<br>
<br>
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c<br>
index fac7bdfe9ec8..91984710b841 100644<br>
--- a/drivers/gpu/drm/i915/intel_display.c<br>
+++ b/drivers/gpu/drm/i915/intel_display.c<br>
@@ -11677,6 +11677,10 @@ intel_cursor_plane_update(struct drm_plane *plane, struct drm_crtc *crtc,<br>
return intel_crtc_cursor_set_obj(crtc, obj, crtc_w, crtc_h);<br>
} else {<br>
intel_crtc_update_cursor(crtc, visible);<br>
+<br>
+ intel_frontbuffer_flip(crtc->dev,<br>
+ INTEL_FRONTBUFFER_CURSOR(intel_crtc->pipe));<br>
+<br>
return 0;<br>
}<br>
}<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.0.1<br>
<br>
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div>