anv: Improve flushing around STATE_BASE_ADDRESS
authorJason Ekstrand <jason.ekstrand@intel.com>
Tue, 31 Jan 2017 03:53:17 +0000 (19:53 -0800)
committerJason Ekstrand <jason.ekstrand@intel.com>
Wed, 1 Feb 2017 02:49:44 +0000 (18:49 -0800)
commit92128590bc78bcbbfb19144c7004b31d6405bbcb
tree6d148ad3809761705a4a50f0d426f40a1089933e
parentf1f9794118008bcdc13d93ee709022d21cc4156d
anv: Improve flushing around STATE_BASE_ADDRESS

It is not clear from the docs exactly how pipelined STATE_BASE_ADDRESS
actually is.  We know from experimentation that we need to flush the
render cache prior to emitting STATE_BASE_ADDRESS and invalidate the
texture cache afterwards.  The only thing the PRM says is that, on gen8+
we're supposed to invalidate the state cache after STATE_BASE_ADDRESS
but experimentation has indicated that doing so does nothing whatsoever.

Since we don't really know, let's do just a bit more flushing in the
hopes that this won't be a problem again.  In particular:

 1) Do a CS stall before we emit STATE_BASE_ADDRESS since we don't
    really know whether or not it's pipelined.

 2) Do a data cache flush in case what runs before STATE_BASE_ADDRESS
    is a compute shader.

 3) Invalidate the state and constant caches after STATE_BASE_ADDRESS
    because the state may be getting cached there (we don't really know).

Reported-by: Mark Janes <mark.a.janes@intel.com>
Tested-by: Mark Janes <mark.a.janes@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Cc: "13.0 17.0" <mesa-stable@lists.freedesktop.org>
src/intel/vulkan/genX_cmd_buffer.c