iris: Fix resource tracking for CS thread ID buffer
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 27 Jun 2019 06:38:59 +0000 (23:38 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 27 Jun 2019 15:12:22 +0000 (08:12 -0700)
commit340df53d6a9dea21a968bb040232ffa66db53add
tree15eac714d4e970d45919936944c3386f889ed193
parent16d334951e7302a40c2d7814a5f8aae32ec5732c
iris: Fix resource tracking for CS thread ID buffer

Today, we stream the compute shader thread IDs simply because they're
(annoyingly) relative to dynamic state base address.  We could upload
them once at compile time, but we'd need a separate non-streaming
uploader for IRIS_MEMZONE_DYNAMIC, and I'm not sure it's worth it.

stream_state pins the buffer for use in the current batch, but also
returns a reference to the pipe_resource.  We dropped this reference
on the floor, leaking a reference basically every time we dispatched
a compute shader after switching to a new one.

The reason it returns a reference is so that we can hold on to it and
re-pin it in iris_restore_compute_saved_bos, which we were also failing
to do.  So if we actually filled up a batch with repeated dispatches to
the same compute shader, and flushed, then continued dispatching, we
would fail to pin it and likely GPU hang.
src/gallium/drivers/iris/iris_context.h
src/gallium/drivers/iris/iris_state.c