[Mesa-dev] [PATCH 3/7] st/mesa: simplify MaxAtomicBufferSize-logic
Erik Faye-Lund
erik.faye-lund at collabora.com
Thu Aug 30 13:40:14 UTC 2018
MaxAtomicCounters has already been assigned in the loop above in the
ssbo_atomic = true case, so this will calculate the same value as the
default.
While we're at it, fixup indentation on the MaxAtomicBufferBindings
assign.
Signed-off-by: Erik Faye-Lund <erik.faye-lund at collabora.com>
---
src/mesa/state_tracker/st_extensions.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c b/src/mesa/state_tracker/st_extensions.c
index 83fc09f52b..9ecdd26edd 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src/mesa/state_tracker/st_extensions.c
@@ -441,12 +441,11 @@ void st_init_limits(struct pipe_screen *screen,
c->NumProgramBinaryFormats = 1;
c->MaxAtomicBufferBindings =
- c->Program[MESA_SHADER_FRAGMENT].MaxAtomicBuffers;
+ c->Program[MESA_SHADER_FRAGMENT].MaxAtomicBuffers;
+ c->MaxAtomicBufferSize =
+ c->Program[MESA_SHADER_FRAGMENT].MaxAtomicCounters * ATOMIC_COUNTER_SIZE;
if (!ssbo_atomic) {
- /* for separate atomic buffers - there atomic buffer size will be
- limited */
- c->MaxAtomicBufferSize = c->Program[MESA_SHADER_FRAGMENT].MaxAtomicCounters * ATOMIC_COUNTER_SIZE;
/* on all HW with separate atomic (evergreen) the following
lines are true. not sure it's worth adding CAPs for this at this
stage. */
--
2.17.1
More information about the mesa-dev
mailing list