freedreno: Stop being too clever by half

This wasn't taking into account a change in corresponding bit in
writeable_bitmask, causing problem if an SSBO was first bound for
read, and then rebound for write, we wouldn't update the buffers
valid range.  Instead just drop the premature optimization.

Signed-off-by: Rob Clark <robdclark@chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22224>
This commit is contained in:
Rob Clark 2023-03-27 15:06:37 -07:00 committed by Marge Bot
parent b123ee707d
commit 19a138adcf

View file

@ -172,11 +172,6 @@ fd_set_shader_buffers(struct pipe_context *pctx, enum pipe_shader_type shader,
struct pipe_shader_buffer *buf = &so->sb[n];
if (buffers && buffers[i].buffer) {
if ((buf->buffer == buffers[i].buffer) &&
(buf->buffer_offset == buffers[i].buffer_offset) &&
(buf->buffer_size == buffers[i].buffer_size))
continue;
buf->buffer_offset = buffers[i].buffer_offset;
buf->buffer_size = buffers[i].buffer_size;
pipe_resource_reference(&buf->buffer, buffers[i].buffer);