iris: Don't auto-flush/dirty on transfer unmap for coherent buffers
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 29 Aug 2019 00:50:13 +0000 (17:50 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 29 Aug 2019 05:11:05 +0000 (22:11 -0700)
commit90ca709f6de551892d84491ae2893301bc55fcbb
tree8ce9d85e5de503dae4739b064793d6cdd58d05a9
parent5f3eb6ef293f139f3504cb302b290a291629612d
iris: Don't auto-flush/dirty on transfer unmap for coherent buffers

When u_upload_mgr fills up a buffer, it unmaps and destroys it.  Our
unmap function was automatically performing the equivalent of a
FlushMappedBufferRange call in this case.  Because the buffer mapping
is persistent and coherent, we don't actually do any flushing when we
do the rest of the writes to the buffer - we were just doing one final
one at the end.  But we would be using the uploaded contents on the
GPU the whole time.

This certainly shouldn't be necessary for streaming buffers, and if
such flushing and dirtying is necessary for coherent buffers, this is
wildly insufficient.

Drops a small number of constant packets and PIPE_CONTROL flushes from
most benchmarks that I've looked at.  Doesn't seem to make much of an
impact on performance, however.

Thanks to Felix Degrood for noticing that we were emitting more
3DSTATE_CONSTANT_* packets than we needed to.
src/gallium/drivers/iris/iris_resource.c