[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