[Intel-gfx] [PATCH] drm/i915: Use PIPE_CONTROL for flushing on gen6+.
Eric Anholt
eric at anholt.net
Tue Oct 4 00:59:29 CEST 2011
On Mon, 3 Oct 2011 13:00:16 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> On Mon, 26 Sep 2011 11:59:23 -0700
> Kenneth Graunke <kenneth at whitecape.org> wrote:
>
> > + /* Just flush everything for now */
> > + flags |= PIPE_CONTROL_WC_FLUSH;
> > + flags |= PIPE_CONTROL_IS_FLUSH;
> > + flags |= PIPE_CONTROL_TC_FLUSH;
> > + flags |= PIPE_CONTROL_DEPTH_FLUSH;
> > + flags |= PIPE_CONTROL_VFC;
>
> This is the only bit I'd like to see changed. While we still have the
> domain tracking code we may as well try to honor it and limit our
> flushing here like we do with MI_FLUSH.
>
> Unless someone has a "remove all domain tracking" patch already posted
> that is. :)
I don't think we "might as well try to honor it". Working out the
workarounds for various combinations is difficult to do even for a fixed
set of bits. Let's not make the workarounds more complicated by varying
them, when experiments showed no evidence for removing bits improving
performance.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20111003/e46468c2/attachment.sig>
More information about the Intel-gfx
mailing list