gallium: Pack the atomic counters just above the SSBOs.
authorEric Anholt <eric@anholt.net>
Fri, 20 Dec 2019 21:30:04 +0000 (13:30 -0800)
committerEric Anholt <eric@anholt.net>
Tue, 21 Jan 2020 18:06:23 +0000 (10:06 -0800)
commitd5a3971457b8a9d5269050c5aa1d09a6197b5eef
tree70838a4354f1081225b8549e73ce6e0db7f0ee33
parent10dc4ac4c5d6dbe3df1f2b75229804e7aa5f86f1
gallium: Pack the atomic counters just above the SSBOs.

We carve out half the SSBO space for atomics, and we were just binding
them way up there.  freedreno was then using a remapping table to map the
sparse buffer index back down, since space in the descriptor array is a
shared resource that may limit parallelism.  That remapping table
generated inside of the ir3 compiler is getting thoroughly in the way of
implementing vulkan descriptor sets.

We will be able to get rid of the freedreno's remapping table, and
hopefully save shared resources on other hardware, by packing the atomics
tightly above the SSBOs (like i965 does).  We already rebind the shader
buffers on program change if either the old or new program has SSBOs or
ABOs, so this doesn't necessarily increase the program state change cost
(the only cost increase I can come up with is if you're using the same
atomic counter without rebinding it across changes of programs with
varying SSBO counts, meaning it would now bounce around index space).

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3240>
src/mesa/state_tracker/st_atom_atomicbuf.c
src/mesa/state_tracker/st_atom_list.h
src/mesa/state_tracker/st_atom_storagebuf.c
src/mesa/state_tracker/st_context.h
src/mesa/state_tracker/st_glsl_to_nir.cpp
src/mesa/state_tracker/st_glsl_to_tgsi.cpp