mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2025-12-30 10:00:14 +01:00
mesa: merge bind_atomic_buffers_{base|range}
Reduced code duplication should make the code more maintainable. Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
This commit is contained in:
Nicolai Hähnle
parent
5eb104d6ab
commit
da5d4583e5
1 changed files with 49 additions and 106 deletions
|
|
@ -3580,82 +3580,18 @@ unbind_atomic_buffers(struct gl_context *ctx, GLuint first, GLsizei count)
|
|||
}
|
||||
|
||||
static void
|
||||
bind_atomic_buffers_base(struct gl_context *ctx,
|
||||
GLuint first,
|
||||
GLsizei count,
|
||||
const GLuint *buffers)
|
||||
bind_atomic_buffers(struct gl_context *ctx,
|
||||
GLuint first,
|
||||
GLsizei count,
|
||||
const GLuint *buffers,
|
||||
bool range,
|
||||
const GLintptr *offsets,
|
||||
const GLsizeiptr *sizes,
|
||||
const char *caller)
|
||||
{
|
||||
GLint i;
|
||||
|
||||
if (!error_check_bind_atomic_buffers(ctx, first, count,
|
||||
"glBindBuffersBase"))
|
||||
return;
|
||||
|
||||
/* Assume that at least one binding will be changed */
|
||||
FLUSH_VERTICES(ctx, 0);
|
||||
ctx->NewDriverState |= ctx->DriverFlags.NewAtomicBuffer;
|
||||
|
||||
if (!buffers) {
|
||||
/* The ARB_multi_bind spec says:
|
||||
*
|
||||
* "If <buffers> is NULL, all bindings from <first> through
|
||||
* <first>+<count>-1 are reset to their unbound (zero) state."
|
||||
*/
|
||||
unbind_atomic_buffers(ctx, first, count);
|
||||
return;
|
||||
}
|
||||
|
||||
/* Note that the error semantics for multi-bind commands differ from
|
||||
* those of other GL commands.
|
||||
*
|
||||
* The Issues section in the ARB_multi_bind spec says:
|
||||
*
|
||||
* "(11) Typically, OpenGL specifies that if an error is generated by a
|
||||
* command, that command has no effect. This is somewhat
|
||||
* unfortunate for multi-bind commands, because it would require a
|
||||
* first pass to scan the entire list of bound objects for errors
|
||||
* and then a second pass to actually perform the bindings.
|
||||
* Should we have different error semantics?
|
||||
*
|
||||
* RESOLVED: Yes. In this specification, when the parameters for
|
||||
* one of the <count> binding points are invalid, that binding point
|
||||
* is not updated and an error will be generated. However, other
|
||||
* binding points in the same command will be updated if their
|
||||
* parameters are valid and no other error occurs."
|
||||
*/
|
||||
|
||||
_mesa_begin_bufferobj_lookups(ctx);
|
||||
|
||||
for (i = 0; i < count; i++) {
|
||||
struct gl_atomic_buffer_binding *binding =
|
||||
&ctx->AtomicBufferBindings[first + i];
|
||||
struct gl_buffer_object *bufObj;
|
||||
|
||||
if (binding->BufferObject && binding->BufferObject->Name == buffers[i])
|
||||
bufObj = binding->BufferObject;
|
||||
else
|
||||
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i,
|
||||
"glBindBuffersBase");
|
||||
|
||||
if (bufObj)
|
||||
set_atomic_buffer_binding(ctx, binding, bufObj, 0, 0);
|
||||
}
|
||||
|
||||
_mesa_end_bufferobj_lookups(ctx);
|
||||
}
|
||||
|
||||
static void
|
||||
bind_atomic_buffers_range(struct gl_context *ctx,
|
||||
GLuint first,
|
||||
GLsizei count,
|
||||
const GLuint *buffers,
|
||||
const GLintptr *offsets,
|
||||
const GLsizeiptr *sizes)
|
||||
{
|
||||
GLint i;
|
||||
|
||||
if (!error_check_bind_atomic_buffers(ctx, first, count,
|
||||
"glBindBuffersRange"))
|
||||
if (!error_check_bind_atomic_buffers(ctx, first, count, caller))
|
||||
return;
|
||||
|
||||
/* Assume that at least one binding will be changed */
|
||||
|
|
@ -3700,45 +3636,51 @@ bind_atomic_buffers_range(struct gl_context *ctx,
|
|||
struct gl_atomic_buffer_binding *binding =
|
||||
&ctx->AtomicBufferBindings[first + i];
|
||||
struct gl_buffer_object *bufObj;
|
||||
GLintptr offset = 0;
|
||||
GLsizeiptr size = 0;
|
||||
|
||||
if (!bind_buffers_check_offset_and_size(ctx, i, offsets, sizes))
|
||||
continue;
|
||||
if (range) {
|
||||
if (!bind_buffers_check_offset_and_size(ctx, i, offsets, sizes))
|
||||
continue;
|
||||
|
||||
/* The ARB_multi_bind spec says:
|
||||
*
|
||||
* "An INVALID_VALUE error is generated by BindBuffersRange if any
|
||||
* pair of values in <offsets> and <sizes> does not respectively
|
||||
* satisfy the constraints described for those parameters for the
|
||||
* specified target, as described in section 6.7.1 (per binding)."
|
||||
*
|
||||
* Section 6.7.1 refers to table 6.5, which says:
|
||||
*
|
||||
* "┌───────────────────────────────────────────────────────────────┐
|
||||
* │ Atomic counter array bindings (see sec. 7.7.2) │
|
||||
* ├───────────────────────┬───────────────────────────────────────┤
|
||||
* │ ... │ ... │
|
||||
* │ offset restriction │ multiple of 4 │
|
||||
* │ ... │ ... │
|
||||
* │ size restriction │ none │
|
||||
* └───────────────────────┴───────────────────────────────────────┘"
|
||||
*/
|
||||
if (offsets[i] & (ATOMIC_COUNTER_SIZE - 1)) {
|
||||
_mesa_error(ctx, GL_INVALID_VALUE,
|
||||
"glBindBuffersRange(offsets[%u]=%" PRId64
|
||||
" is misaligned; it must be a multiple of %d when "
|
||||
"target=GL_ATOMIC_COUNTER_BUFFER)",
|
||||
i, (int64_t) offsets[i], ATOMIC_COUNTER_SIZE);
|
||||
continue;
|
||||
/* The ARB_multi_bind spec says:
|
||||
*
|
||||
* "An INVALID_VALUE error is generated by BindBuffersRange if any
|
||||
* pair of values in <offsets> and <sizes> does not respectively
|
||||
* satisfy the constraints described for those parameters for the
|
||||
* specified target, as described in section 6.7.1 (per binding)."
|
||||
*
|
||||
* Section 6.7.1 refers to table 6.5, which says:
|
||||
*
|
||||
* "┌───────────────────────────────────────────────────────────────┐
|
||||
* │ Atomic counter array bindings (see sec. 7.7.2) │
|
||||
* ├───────────────────────┬───────────────────────────────────────┤
|
||||
* │ ... │ ... │
|
||||
* │ offset restriction │ multiple of 4 │
|
||||
* │ ... │ ... │
|
||||
* │ size restriction │ none │
|
||||
* └───────────────────────┴───────────────────────────────────────┘"
|
||||
*/
|
||||
if (offsets[i] & (ATOMIC_COUNTER_SIZE - 1)) {
|
||||
_mesa_error(ctx, GL_INVALID_VALUE,
|
||||
"glBindBuffersRange(offsets[%u]=%" PRId64
|
||||
" is misaligned; it must be a multiple of %d when "
|
||||
"target=GL_ATOMIC_COUNTER_BUFFER)",
|
||||
i, (int64_t) offsets[i], ATOMIC_COUNTER_SIZE);
|
||||
continue;
|
||||
}
|
||||
|
||||
offset = offsets[i];
|
||||
size = sizes[i];
|
||||
}
|
||||
|
||||
if (binding->BufferObject && binding->BufferObject->Name == buffers[i])
|
||||
bufObj = binding->BufferObject;
|
||||
else
|
||||
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i,
|
||||
"glBindBuffersRange");
|
||||
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i, caller);
|
||||
|
||||
if (bufObj)
|
||||
set_atomic_buffer_binding(ctx, binding, bufObj, offsets[i], sizes[i]);
|
||||
set_atomic_buffer_binding(ctx, binding, bufObj, offset, size);
|
||||
}
|
||||
|
||||
_mesa_end_bufferobj_lookups(ctx);
|
||||
|
|
@ -3903,8 +3845,8 @@ _mesa_BindBuffersRange(GLenum target, GLuint first, GLsizei count,
|
|||
"glBindBuffersRange");
|
||||
return;
|
||||
case GL_ATOMIC_COUNTER_BUFFER:
|
||||
bind_atomic_buffers_range(ctx, first, count, buffers,
|
||||
offsets, sizes);
|
||||
bind_atomic_buffers(ctx, first, count, buffers, true, offsets, sizes,
|
||||
"glBindBuffersRange");
|
||||
return;
|
||||
default:
|
||||
_mesa_error(ctx, GL_INVALID_ENUM, "glBindBuffersRange(target=%s)",
|
||||
|
|
@ -3938,7 +3880,8 @@ _mesa_BindBuffersBase(GLenum target, GLuint first, GLsizei count,
|
|||
"glBindBuffersBase");
|
||||
return;
|
||||
case GL_ATOMIC_COUNTER_BUFFER:
|
||||
bind_atomic_buffers_base(ctx, first, count, buffers);
|
||||
bind_atomic_buffers(ctx, first, count, buffers, false, NULL, NULL,
|
||||
"glBindBuffersBase");
|
||||
return;
|
||||
default:
|
||||
_mesa_error(ctx, GL_INVALID_ENUM, "glBindBuffersBase(target=%s)",
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue