<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 15, 2015 at 5:31 AM Daniel Vetter <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jul 14, 2015 at 04:29:13PM -0300, Paulo Zanoni wrote:<br>
> From: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com" target="_blank">paulo.r.zanoni@intel.com</a>><br>
><br>
> Use the appropriate call.<br></blockquote><div><br></div><div>Good!</div><div><br></div><div>Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
><br>
> I know there's a discussion about whether we need this call here at<br>
> all, but removing the call means we'll only update FBC after we get<br>
> the page flip IRQ. So the user may only see the new frame a little<br>
> after it should. Let's wait just a little bit more before removing<br>
> this call since we can rely in the HW tracking for accurate flips.<br></blockquote><div><br></div><div>I'm in favor of remove or at least move to frontbuffer prepare for flip..</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Afaik fbc2 on g4x+ does correctly invalidate the fbc cache when flipping<br>
the frontbuffer. This was needed for fbc1 which just didn't handle flips -<br>
there we had to essentially disable and then re-enable fbc to make sure<br>
the update was tracked correctly.<br>
<br>
The other problem is correctly synchronizing the fence update (for hw gtt<br>
tracking) against the flip. I'm not sure whether we still have races left<br>
there, but simply updating the fence window before we flip should be all<br>
that's required.</blockquote><div><br></div><div>But anyway I'm also in favor of removing the old fbc...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-Daniel<br>
<br>
><br>
> Signed-off-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com" target="_blank">paulo.r.zanoni@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 37b2528..6e3ba5f 100644<br>
> --- a/drivers/gpu/drm/i915/intel_display.c<br>
> +++ b/drivers/gpu/drm/i915/intel_display.c<br>
> @@ -11544,7 +11544,7 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc,<br>
>                         to_intel_plane(primary)->frontbuffer_bit);<br>
>       mutex_unlock(&dev->struct_mutex);<br>
><br>
> -     intel_fbc_disable(dev_priv);<br>
> +     intel_fbc_disable_crtc(intel_crtc);<br>
>       intel_frontbuffer_flip_prepare(dev,<br>
>                                      to_intel_plane(primary)->frontbuffer_bit);<br>
><br>
> --<br>
> 2.1.4<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>
<br>
--<br>
Daniel Vetter<br>
Software Engineer, Intel Corporation<br>
<a href="http://blog.ffwll.ch" rel="noreferrer" target="_blank">http://blog.ffwll.ch</a><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>