<div dir="ltr">I had forgotten I had this patch and lost sometime yesterday debugging and end up on same fix again :/<div><br></div><div>Daniel, do you need a reviewer on this? could you please take a quickly look?</div><div><br></div><div>thanks in advance,</div><div>Rodrigo.</div><div><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 24, 2015 at 4:41 PM Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We also need to call the frontbuffer flip to trigger proper<br>
invalidations when disabling planes. Otherwise we will miss<br>
screen updates when disabling sprites or cursor.<br>
<br>
It was caught with kms_psr_sink_crc sprite_plane_onoff<br>
and cursor_plane_onoff subtests.<br>
<br>
This is probably a regression since I can also get this<br>
with the manual test case, but with so many changes on atomic<br>
modeset I couldn't track exactly when this was introduced.<br>
<br>
Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
---<br>
 drivers/gpu/drm/i915/intel_display.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c<br>
index af0bcfe..bb124cc 100644<br>
--- a/drivers/gpu/drm/i915/intel_display.c<br>
+++ b/drivers/gpu/drm/i915/intel_display.c<br>
@@ -11716,7 +11716,7 @@ int intel_plane_atomic_calc_changes(struct drm_crtc_state *crtc_state,<br>
                intel_crtc->atomic.update_wm_pre = true;<br>
        }<br>
<br>
-       if (visible)<br>
+       if (visible || was_visible)<br>
                intel_crtc->atomic.fb_bits |=<br>
                        to_intel_plane(plane)->frontbuffer_bit;<br>
<br>
--<br>
1.9.3<br>
<br>
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</blockquote></div></div></div></div>