Except two minor comments, series is:
Reviewed-by: Samuel Pitoiset <[email protected]>
Nice refactoring. :)
On 09/15/2017 05:57 AM, Dave Airlie wrote:
From: Dave Airlie <[email protected]>
This respects the size from the range setting like ssbo.
Signed-off-by: Dave Airlie <[email protected]>
---
src/mesa/state_tracker/st_atom_atomicbuf.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/state_tracker/st_atom_atomicbuf.c
b/src/mesa/state_tracker/st_atom_atomicbuf.c
index 7ebcd08..ee5944f 100644
--- a/src/mesa/state_tracker/st_atom_atomicbuf.c
+++ b/src/mesa/state_tracker/st_atom_atomicbuf.c
@@ -62,6 +62,12 @@ st_bind_atomics(struct st_context *st, struct gl_program
*prog,
sb.buffer = st_obj->buffer;
sb.buffer_offset = binding->Offset;
sb.buffer_size = st_obj->buffer->width0 - binding->Offset;
+
+ /* AutomaticSize is FALSE if the buffer was set with BindBufferRange.
+ * Take the minimum just to be sure.
+ */
+ if (!binding->AutomaticSize)
+ sb.buffer_size = MIN2(sb.buffer_size, (unsigned) binding->Size);
}
st->pipe->set_shader_buffers(st->pipe, shader_type,
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
