fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-22 00:30:13 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

mesa: don't ever bind NullBufferObj to glBindBuffer(Base,Range) slots

Browse source 
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4466>
This commit is contained in:
Marek Olšák 2020-03-21 23:47:08 -04:00
parent f3cce7087a
commit 54525808aa
10 changed files with 96 additions and 90 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/mesa/drivers/dri/i965/brw_wm_surface_state.c
View file

@ -1329,7 +1329,7 @@ upload_buffer_surface(struct brw_context *brw,
{ {
struct gl_context *ctx = &brw->ctx; struct gl_context *ctx = &brw->ctx;
if (binding->BufferObject == ctx->Shared->NullBufferObj) { if (!binding->BufferObject) {
emit_null_surface_state(brw, NULL, out_offset); emit_null_surface_state(brw, NULL, out_offset);
} else { } else {
ptrdiff_t size = binding->BufferObject->Size - binding->Offset; ptrdiff_t size = binding->BufferObject->Size - binding->Offset;

2
src/mesa/drivers/dri/i965/genX_state_upload.c
View file

@ -3125,7 +3125,7 @@ genX(upload_push_constant_packets)(struct brw_context *brw)
const struct gl_buffer_binding *binding = const struct gl_buffer_binding *binding =
&ctx->UniformBufferBindings[block->Binding]; &ctx->UniformBufferBindings[block->Binding];
if (binding->BufferObject == ctx->Shared->NullBufferObj) { if (!binding->BufferObject) {
static unsigned msg_id = 0; static unsigned msg_id = 0;
_mesa_gl_debugf(ctx, &msg_id, MESA_DEBUG_SOURCE_API, _mesa_gl_debugf(ctx, &msg_id, MESA_DEBUG_SOURCE_API,
MESA_DEBUG_TYPE_UNDEFINED, MESA_DEBUG_TYPE_UNDEFINED,

124
src/mesa/main/bufferobj.c
View file

@ -872,7 +872,7 @@ _mesa_init_buffer_objects( struct gl_context *ctx )
for (i = 0; i < MAX_COMBINED_UNIFORM_BUFFERS; i++) { for (i = 0; i < MAX_COMBINED_UNIFORM_BUFFERS; i++) {
_mesa_reference_buffer_object(ctx, _mesa_reference_buffer_object(ctx,
&ctx->UniformBufferBindings[i].BufferObject, &ctx->UniformBufferBindings[i].BufferObject,
ctx->Shared->NullBufferObj); NULL);
ctx->UniformBufferBindings[i].Offset = -1; ctx->UniformBufferBindings[i].Offset = -1;
ctx->UniformBufferBindings[i].Size = -1; ctx->UniformBufferBindings[i].Size = -1;
} }
@ -880,7 +880,7 @@ _mesa_init_buffer_objects( struct gl_context *ctx )
for (i = 0; i < MAX_COMBINED_SHADER_STORAGE_BUFFERS; i++) { for (i = 0; i < MAX_COMBINED_SHADER_STORAGE_BUFFERS; i++) {
_mesa_reference_buffer_object(ctx, _mesa_reference_buffer_object(ctx,
&ctx->ShaderStorageBufferBindings[i].BufferObject, &ctx->ShaderStorageBufferBindings[i].BufferObject,
ctx->Shared->NullBufferObj); NULL);
ctx->ShaderStorageBufferBindings[i].Offset = -1; ctx->ShaderStorageBufferBindings[i].Offset = -1;
ctx->ShaderStorageBufferBindings[i].Size = -1; ctx->ShaderStorageBufferBindings[i].Size = -1;
} }
@ -888,7 +888,7 @@ _mesa_init_buffer_objects( struct gl_context *ctx )
for (i = 0; i < MAX_COMBINED_ATOMIC_BUFFERS; i++) { for (i = 0; i < MAX_COMBINED_ATOMIC_BUFFERS; i++) {
_mesa_reference_buffer_object(ctx, _mesa_reference_buffer_object(ctx,
&ctx->AtomicBufferBindings[i].BufferObject, &ctx->AtomicBufferBindings[i].BufferObject,
ctx->Shared->NullBufferObj); NULL);
ctx->AtomicBufferBindings[i].Offset = 0; ctx->AtomicBufferBindings[i].Offset = 0;
ctx->AtomicBufferBindings[i].Size = 0; ctx->AtomicBufferBindings[i].Size = 0;
} }
@ -1079,10 +1079,9 @@ _mesa_lookup_bufferobj_err(struct gl_context *ctx, GLuint buffer,
* of an existing buffer object. * of an existing buffer object.
* *
* If the buffer ID refers to an existing buffer object, a pointer * If the buffer ID refers to an existing buffer object, a pointer
* to the buffer object is returned. If the ID is zero, a pointer * to the buffer object is returned. If the ID is zero, NULL is returned.
* to the shared NullBufferObj is returned. If the ID is not zero * If the ID is not zero and does not refer to a valid buffer object, this
* and does not refer to a valid buffer object, this function * function returns NULL.
* returns NULL.
* *
* This function assumes that the caller has already locked the * This function assumes that the caller has already locked the
* hash table mutex by calling * hash table mutex by calling
@ -1091,9 +1090,12 @@ _mesa_lookup_bufferobj_err(struct gl_context *ctx, GLuint buffer,
struct gl_buffer_object * struct gl_buffer_object *
_mesa_multi_bind_lookup_bufferobj(struct gl_context *ctx, _mesa_multi_bind_lookup_bufferobj(struct gl_context *ctx,
const GLuint *buffers, const GLuint *buffers,
GLuint index, const char *caller) GLuint index, const char *caller,
bool *error)
{ {
struct gl_buffer_object *bufObj; struct gl_buffer_object *bufObj = NULL;
*error = false;
if (buffers[index] != 0) { if (buffers[index] != 0) {
bufObj = _mesa_lookup_bufferobj_locked(ctx, buffers[index]); bufObj = _mesa_lookup_bufferobj_locked(ctx, buffers[index]);
@ -1102,20 +1104,20 @@ _mesa_multi_bind_lookup_bufferobj(struct gl_context *ctx,
when they don't exist. */ when they don't exist. */
if (bufObj == &DummyBufferObject) if (bufObj == &DummyBufferObject)
bufObj = NULL; bufObj = NULL;
} else
bufObj = ctx->Shared->NullBufferObj;
if (!bufObj) { if (!bufObj) {
/* The ARB_multi_bind spec says: /* The ARB_multi_bind spec says:
* *
* "An INVALID_OPERATION error is generated if any value * "An INVALID_OPERATION error is generated if any value
* in <buffers> is not zero or the name of an existing * in <buffers> is not zero or the name of an existing
* buffer object (per binding)." * buffer object (per binding)."
*/ */
_mesa_error(ctx, GL_INVALID_OPERATION, _mesa_error(ctx, GL_INVALID_OPERATION,
"%s(buffers[%u]=%u is not zero or the name " "%s(buffers[%u]=%u is not zero or the name "
"of an existing buffer object)", "of an existing buffer object)",
caller, index, buffers[index]); caller, index, buffers[index]);
*error = true;
}
} }
return bufObj; return bufObj;
@ -1255,17 +1257,21 @@ set_buffer_multi_binding(struct gl_context *ctx,
gl_buffer_usage usage) gl_buffer_usage usage)
{ {
struct gl_buffer_object *bufObj; struct gl_buffer_object *bufObj;
if (binding->BufferObject && binding->BufferObject->Name == buffers[idx]) if (binding->BufferObject && binding->BufferObject->Name == buffers[idx])
bufObj = binding->BufferObject; bufObj = binding->BufferObject;
else else {
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, idx, caller); bool error;
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, idx, caller,
if (bufObj) { &error);
if (bufObj == ctx->Shared->NullBufferObj) if (error)
set_buffer_binding(ctx, binding, bufObj, -1, -1, !range, usage); return;
else
set_buffer_binding(ctx, binding, bufObj, offset, size, !range, usage);
} }
if (!bufObj)
set_buffer_binding(ctx, binding, bufObj, -1, -1, !range, usage);
else
set_buffer_binding(ctx, binding, bufObj, offset, size, !range, usage);
} }
static void static void
@ -1367,7 +1373,7 @@ bind_buffer_base_uniform_buffer(struct gl_context *ctx,
_mesa_reference_buffer_object(ctx, &ctx->UniformBuffer, bufObj); _mesa_reference_buffer_object(ctx, &ctx->UniformBuffer, bufObj);
if (bufObj == ctx->Shared->NullBufferObj) if (!bufObj)
bind_uniform_buffer(ctx, index, bufObj, -1, -1, GL_TRUE); bind_uniform_buffer(ctx, index, bufObj, -1, -1, GL_TRUE);
else else
bind_uniform_buffer(ctx, index, bufObj, 0, 0, GL_TRUE); bind_uniform_buffer(ctx, index, bufObj, 0, 0, GL_TRUE);
@ -1389,7 +1395,7 @@ bind_buffer_base_shader_storage_buffer(struct gl_context *ctx,
_mesa_reference_buffer_object(ctx, &ctx->ShaderStorageBuffer, bufObj); _mesa_reference_buffer_object(ctx, &ctx->ShaderStorageBuffer, bufObj);
if (bufObj == ctx->Shared->NullBufferObj) if (!bufObj)
bind_shader_storage_buffer(ctx, index, bufObj, -1, -1, GL_TRUE); bind_shader_storage_buffer(ctx, index, bufObj, -1, -1, GL_TRUE);
else else
bind_shader_storage_buffer(ctx, index, bufObj, 0, 0, GL_TRUE); bind_shader_storage_buffer(ctx, index, bufObj, 0, 0, GL_TRUE);
@ -1411,7 +1417,7 @@ bind_buffer_base_atomic_buffer(struct gl_context *ctx,
_mesa_reference_buffer_object(ctx, &ctx->AtomicBuffer, bufObj); _mesa_reference_buffer_object(ctx, &ctx->AtomicBuffer, bufObj);
if (bufObj == ctx->Shared->NullBufferObj) if (!bufObj)
bind_atomic_buffer(ctx, index, bufObj, -1, -1, GL_TRUE); bind_atomic_buffer(ctx, index, bufObj, -1, -1, GL_TRUE);
else else
bind_atomic_buffer(ctx, index, bufObj, 0, 0, GL_TRUE); bind_atomic_buffer(ctx, index, bufObj, 0, 0, GL_TRUE);
@ -1484,16 +1490,14 @@ delete_buffers(struct gl_context *ctx, GLsizei n, const GLuint *ids)
if (ctx->TransformFeedback.CurrentObject->Buffers[j] == bufObj) { if (ctx->TransformFeedback.CurrentObject->Buffers[j] == bufObj) {
_mesa_bind_buffer_base_transform_feedback(ctx, _mesa_bind_buffer_base_transform_feedback(ctx,
ctx->TransformFeedback.CurrentObject, ctx->TransformFeedback.CurrentObject,
j, ctx->Shared->NullBufferObj, j, NULL, false);
false);
} }
} }
/* unbind UBO binding points */ /* unbind UBO binding points */
for (j = 0; j < ctx->Const.MaxUniformBufferBindings; j++) { for (j = 0; j < ctx->Const.MaxUniformBufferBindings; j++) {
if (ctx->UniformBufferBindings[j].BufferObject == bufObj) { if (ctx->UniformBufferBindings[j].BufferObject == bufObj) {
bind_buffer_base_uniform_buffer(ctx, j, bind_buffer_base_uniform_buffer(ctx, j, NULL);
ctx->Shared->NullBufferObj);
} }
} }
@ -1504,8 +1508,7 @@ delete_buffers(struct gl_context *ctx, GLsizei n, const GLuint *ids)
/* unbind SSBO binding points */ /* unbind SSBO binding points */
for (j = 0; j < ctx->Const.MaxShaderStorageBufferBindings; j++) { for (j = 0; j < ctx->Const.MaxShaderStorageBufferBindings; j++) {
if (ctx->ShaderStorageBufferBindings[j].BufferObject == bufObj) { if (ctx->ShaderStorageBufferBindings[j].BufferObject == bufObj) {
bind_buffer_base_shader_storage_buffer(ctx, j, bind_buffer_base_shader_storage_buffer(ctx, j, NULL);
ctx->Shared->NullBufferObj);
} }
} }
@ -1516,8 +1519,7 @@ delete_buffers(struct gl_context *ctx, GLsizei n, const GLuint *ids)
/* unbind Atomci Buffer binding points */ /* unbind Atomci Buffer binding points */
for (j = 0; j < ctx->Const.MaxAtomicBufferBindings; j++) { for (j = 0; j < ctx->Const.MaxAtomicBufferBindings; j++) {
if (ctx->AtomicBufferBindings[j].BufferObject == bufObj) { if (ctx->AtomicBufferBindings[j].BufferObject == bufObj) {
bind_buffer_base_atomic_buffer(ctx, j, bind_buffer_base_atomic_buffer(ctx, j, NULL);
ctx->Shared->NullBufferObj);
} }
} }
@ -3713,7 +3715,7 @@ bind_buffer_range_uniform_buffer(struct gl_context *ctx, GLuint index,
struct gl_buffer_object *bufObj, struct gl_buffer_object *bufObj,
GLintptr offset, GLsizeiptr size) GLintptr offset, GLsizeiptr size)
{ {
if (bufObj == ctx->Shared->NullBufferObj) { if (!bufObj) {
offset = -1; offset = -1;
size = -1; size = -1;
} }
@ -3756,7 +3758,7 @@ bind_buffer_range_shader_storage_buffer(struct gl_context *ctx,
GLintptr offset, GLintptr offset,
GLsizeiptr size) GLsizeiptr size)
{ {
if (bufObj == ctx->Shared->NullBufferObj) { if (!bufObj) {
offset = -1; offset = -1;
size = -1; size = -1;
} }
@ -3798,7 +3800,7 @@ bind_buffer_range_atomic_buffer(struct gl_context *ctx, GLuint index,
struct gl_buffer_object *bufObj, struct gl_buffer_object *bufObj,
GLintptr offset, GLsizeiptr size) GLintptr offset, GLsizeiptr size)
{ {
if (bufObj == ctx->Shared->NullBufferObj) { if (!bufObj) {
offset = -1; offset = -1;
size = -1; size = -1;
} }
@ -3933,11 +3935,9 @@ error_check_bind_shader_storage_buffers(struct gl_context *ctx,
static void static void
unbind_uniform_buffers(struct gl_context *ctx, GLuint first, GLsizei count) unbind_uniform_buffers(struct gl_context *ctx, GLuint first, GLsizei count)
{ {
struct gl_buffer_object *bufObj = ctx->Shared->NullBufferObj;
for (int i = 0; i < count; i++) for (int i = 0; i < count; i++)
set_buffer_binding(ctx, &ctx->UniformBufferBindings[first + i], set_buffer_binding(ctx, &ctx->UniformBufferBindings[first + i],
bufObj, -1, -1, GL_TRUE, 0); NULL, -1, -1, GL_TRUE, 0);
} }
/** /**
@ -3948,11 +3948,9 @@ static void
unbind_shader_storage_buffers(struct gl_context *ctx, GLuint first, unbind_shader_storage_buffers(struct gl_context *ctx, GLuint first,
GLsizei count) GLsizei count)
{ {
struct gl_buffer_object *bufObj = ctx->Shared->NullBufferObj;
for (int i = 0; i < count; i++) for (int i = 0; i < count; i++)
set_buffer_binding(ctx, &ctx->ShaderStorageBufferBindings[first + i], set_buffer_binding(ctx, &ctx->ShaderStorageBufferBindings[first + i],
bufObj, -1, -1, GL_TRUE, 0); NULL, -1, -1, GL_TRUE, 0);
} }
static void static void
@ -4212,11 +4210,9 @@ unbind_xfb_buffers(struct gl_context *ctx,
struct gl_transform_feedback_object *tfObj, struct gl_transform_feedback_object *tfObj,
GLuint first, GLsizei count) GLuint first, GLsizei count)
{ {
struct gl_buffer_object * const bufObj = ctx->Shared->NullBufferObj;
for (int i = 0; i < count; i++) for (int i = 0; i < count; i++)
_mesa_set_transform_feedback_binding(ctx, tfObj, first + i, _mesa_set_transform_feedback_binding(ctx, tfObj, first + i,
bufObj, 0, 0); NULL, 0, 0);
} }
static void static void
@ -4325,12 +4321,16 @@ bind_xfb_buffers(struct gl_context *ctx,
if (boundBufObj && boundBufObj->Name == buffers[i]) if (boundBufObj && boundBufObj->Name == buffers[i])
bufObj = boundBufObj; bufObj = boundBufObj;
else else {
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i, caller); bool error;
bufObj = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i, caller,
&error);
if (error)
continue;
}
if (bufObj) _mesa_set_transform_feedback_binding(ctx, tfObj, index, bufObj,
_mesa_set_transform_feedback_binding(ctx, tfObj, index, bufObj, offset, size);
offset, size);
} }
_mesa_HashUnlockMutex(ctx->Shared->BufferObjects); _mesa_HashUnlockMutex(ctx->Shared->BufferObjects);
@ -4371,11 +4371,9 @@ error_check_bind_atomic_buffers(struct gl_context *ctx,
static void static void
unbind_atomic_buffers(struct gl_context *ctx, GLuint first, GLsizei count) unbind_atomic_buffers(struct gl_context *ctx, GLuint first, GLsizei count)
{ {
struct gl_buffer_object * const bufObj = ctx->Shared->NullBufferObj;
for (int i = 0; i < count; i++) for (int i = 0; i < count; i++)
set_buffer_binding(ctx, &ctx->AtomicBufferBindings[first + i], set_buffer_binding(ctx, &ctx->AtomicBufferBindings[first + i],
bufObj, -1, -1, GL_TRUE, 0); NULL, -1, -1, GL_TRUE, 0);
} }
static void static void
@ -4492,7 +4490,7 @@ bind_buffer_range(GLenum target, GLuint index, GLuint buffer, GLintptr offset,
} }
if (buffer == 0) { if (buffer == 0) {
bufObj = ctx->Shared->NullBufferObj; bufObj = NULL;
} else { } else {
bufObj = _mesa_lookup_bufferobj(ctx, buffer); bufObj = _mesa_lookup_bufferobj(ctx, buffer);
if (!_mesa_handle_bind_buffer_gen(ctx, buffer, if (!_mesa_handle_bind_buffer_gen(ctx, buffer,
@ -4590,7 +4588,7 @@ _mesa_BindBufferBase(GLenum target, GLuint index, GLuint buffer)
} }
if (buffer == 0) { if (buffer == 0) {
bufObj = ctx->Shared->NullBufferObj; bufObj = NULL;
} else { } else {
bufObj = _mesa_lookup_bufferobj(ctx, buffer); bufObj = _mesa_lookup_bufferobj(ctx, buffer);
if (!_mesa_handle_bind_buffer_gen(ctx, buffer, if (!_mesa_handle_bind_buffer_gen(ctx, buffer,

3
src/mesa/main/bufferobj.h
View file

@ -101,7 +101,8 @@ _mesa_lookup_bufferobj_err(struct gl_context *ctx, GLuint buffer,
extern struct gl_buffer_object * extern struct gl_buffer_object *
_mesa_multi_bind_lookup_bufferobj(struct gl_context *ctx, _mesa_multi_bind_lookup_bufferobj(struct gl_context *ctx,
const GLuint *buffers, const GLuint *buffers,
GLuint index, const char *caller); GLuint index, const char *caller,
bool *error);
extern void extern void
_mesa_initialize_buffer_object(struct gl_context *ctx, _mesa_initialize_buffer_object(struct gl_context *ctx,

9
src/mesa/main/get.c
View file

@ -2531,7 +2531,8 @@ find_value_indexed(const char *func, GLenum pname, GLuint index, union value *v)
goto invalid_value; goto invalid_value;
if (!ctx->Extensions.ARB_uniform_buffer_object) if (!ctx->Extensions.ARB_uniform_buffer_object)
goto invalid_enum; goto invalid_enum;
v->value_int = ctx->UniformBufferBindings[index].BufferObject->Name; buf = ctx->UniformBufferBindings[index].BufferObject;
v->value_int = buf ? buf->Name : 0;
return TYPE_INT; return TYPE_INT;
case GL_UNIFORM_BUFFER_START: case GL_UNIFORM_BUFFER_START:
@ -2558,7 +2559,8 @@ find_value_indexed(const char *func, GLenum pname, GLuint index, union value *v)
goto invalid_enum; goto invalid_enum;
if (index >= ctx->Const.MaxShaderStorageBufferBindings) if (index >= ctx->Const.MaxShaderStorageBufferBindings)
goto invalid_value; goto invalid_value;
v->value_int = ctx->ShaderStorageBufferBindings[index].BufferObject->Name; buf = ctx->ShaderStorageBufferBindings[index].BufferObject;
v->value_int = buf ? buf->Name : 0;
return TYPE_INT; return TYPE_INT;
case GL_SHADER_STORAGE_BUFFER_START: case GL_SHADER_STORAGE_BUFFER_START:
@ -2593,7 +2595,8 @@ find_value_indexed(const char *func, GLenum pname, GLuint index, union value *v)
goto invalid_enum; goto invalid_enum;
if (index >= ctx->Const.MaxAtomicBufferBindings) if (index >= ctx->Const.MaxAtomicBufferBindings)
goto invalid_value; goto invalid_value;
v->value_int = ctx->AtomicBufferBindings[index].BufferObject->Name; buf = ctx->AtomicBufferBindings[index].BufferObject;
v->value_int = buf ? buf->Name : 0;
return TYPE_INT; return TYPE_INT;
case GL_ATOMIC_COUNTER_BUFFER_START: case GL_ATOMIC_COUNTER_BUFFER_START:

28
src/mesa/main/transformfeedback.c
View file

@ -633,7 +633,7 @@ _mesa_validate_buffer_range_xfb(struct gl_context *ctx,
return false; return false;
} }
if (size <= 0 && (dsa || bufObj != ctx->Shared->NullBufferObj)) { if (size <= 0 && (dsa || bufObj)) {
/* OpenGL 4.5 core profile, 6.1, pdf page 82: "An INVALID_VALUE error is /* OpenGL 4.5 core profile, 6.1, pdf page 82: "An INVALID_VALUE error is
* generated by BindBufferRange if buffer is non-zero and size is less * generated by BindBufferRange if buffer is non-zero and size is less
* than or equal to zero." * than or equal to zero."
@ -709,20 +709,22 @@ lookup_transform_feedback_object_err(struct gl_context *ctx,
*/ */
static struct gl_buffer_object * static struct gl_buffer_object *
lookup_transform_feedback_bufferobj_err(struct gl_context *ctx, lookup_transform_feedback_bufferobj_err(struct gl_context *ctx,
GLuint buffer, const char* func) GLuint buffer, const char* func,
bool *error)
{ {
struct gl_buffer_object *bufObj; struct gl_buffer_object *bufObj = NULL;
*error = false;
/* OpenGL 4.5 core profile, 13.2, pdf page 444: buffer must be zero or the /* OpenGL 4.5 core profile, 13.2, pdf page 444: buffer must be zero or the
* name of an existing buffer object. * name of an existing buffer object.
*/ */
if (buffer == 0) { if (buffer) {
bufObj = ctx->Shared->NullBufferObj;
} else {
bufObj = _mesa_lookup_bufferobj(ctx, buffer); bufObj = _mesa_lookup_bufferobj(ctx, buffer);
if (!bufObj) { if (!bufObj) {
_mesa_error(ctx, GL_INVALID_VALUE, "%s(invalid buffer=%u)", func, _mesa_error(ctx, GL_INVALID_VALUE, "%s(invalid buffer=%u)", func,
buffer); buffer);
*error = true;
} }
} }
@ -742,9 +744,11 @@ _mesa_TransformFeedbackBufferBase(GLuint xfb, GLuint index, GLuint buffer)
return; return;
} }
bool error;
bufObj = lookup_transform_feedback_bufferobj_err(ctx, buffer, bufObj = lookup_transform_feedback_bufferobj_err(ctx, buffer,
"glTransformFeedbackBufferBase"); "glTransformFeedbackBufferBase",
if (!bufObj) { &error);
if (error) {
return; return;
} }
@ -765,9 +769,11 @@ _mesa_TransformFeedbackBufferRange(GLuint xfb, GLuint index, GLuint buffer,
return; return;
} }
bool error;
bufObj = lookup_transform_feedback_bufferobj_err(ctx, buffer, bufObj = lookup_transform_feedback_bufferobj_err(ctx, buffer,
"glTransformFeedbackBufferRange"); "glTransformFeedbackBufferRange",
if (!bufObj) { &error);
if (error) {
return; return;
} }
@ -793,7 +799,7 @@ bind_buffer_offset(struct gl_context *ctx,
struct gl_buffer_object *bufObj; struct gl_buffer_object *bufObj;
if (buffer == 0) { if (buffer == 0) {
bufObj = ctx->Shared->NullBufferObj; bufObj = NULL;
} else { } else {
bufObj = _mesa_lookup_bufferobj(ctx, buffer); bufObj = _mesa_lookup_bufferobj(ctx, buffer);
if (!no_error && !bufObj) { if (!no_error && !bufObj) {

4
src/mesa/main/transformfeedback.h
View file

@ -165,11 +165,11 @@ _mesa_set_transform_feedback_binding(struct gl_context *ctx,
{ {
_mesa_reference_buffer_object(ctx, &tfObj->Buffers[index], bufObj); _mesa_reference_buffer_object(ctx, &tfObj->Buffers[index], bufObj);
tfObj->BufferNames[index] = bufObj->Name; tfObj->BufferNames[index] = bufObj ? bufObj->Name : 0;
tfObj->Offset[index] = offset; tfObj->Offset[index] = offset;
tfObj->RequestedSize[index] = size; tfObj->RequestedSize[index] = size;
if (bufObj != ctx->Shared->NullBufferObj) if (bufObj)
bufObj->UsageHistory |= USAGE_TRANSFORM_FEEDBACK_BUFFER; bufObj->UsageHistory |= USAGE_TRANSFORM_FEEDBACK_BUFFER;
} }

10
src/mesa/main/varray.c
View file

@ -3167,13 +3167,11 @@ vertex_array_vertex_buffers(struct gl_context *ctx,
else if (binding->BufferObj && binding->BufferObj->Name == buffers[i]) else if (binding->BufferObj && binding->BufferObj->Name == buffers[i])
vbo = binding->BufferObj; vbo = binding->BufferObj;
else { else {
vbo = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i, func); bool error;
if (!vbo) vbo = _mesa_multi_bind_lookup_bufferobj(ctx, buffers, i, func,
&error);
if (error)
continue; continue;
/* TODO: remove this hack */
if (vbo == ctx->Shared->NullBufferObj)
vbo = NULL;
} }
} else { } else {
vbo = NULL; vbo = NULL;

2
src/mesa/state_tracker/st_atom_constbuf.c
View file

@ -212,7 +212,7 @@ st_bind_ubos(struct st_context *st, struct gl_program *prog,
&st->ctx->UniformBufferBindings[prog->sh.UniformBlocks[i]->Binding]; &st->ctx->UniformBufferBindings[prog->sh.UniformBlocks[i]->Binding];
st_obj = st_buffer_object(binding->BufferObject); st_obj = st_buffer_object(binding->BufferObject);
cb.buffer = st_obj->buffer; cb.buffer = st_obj ? st_obj->buffer : NULL;
if (cb.buffer) { if (cb.buffer) {
cb.buffer_offset = binding->Offset; cb.buffer_offset = binding->Offset;

2
src/mesa/state_tracker/st_atom_storagebuf.c
View file

@ -58,7 +58,7 @@ st_bind_ssbos(struct st_context *st, struct gl_program *prog,
prog->sh.ShaderStorageBlocks[i]->Binding]; prog->sh.ShaderStorageBlocks[i]->Binding];
st_obj = st_buffer_object(binding->BufferObject); st_obj = st_buffer_object(binding->BufferObject);
sb->buffer = st_obj->buffer; sb->buffer = st_obj ? st_obj->buffer : NULL;
if (sb->buffer) { if (sb->buffer) {
sb->buffer_offset = binding->Offset; sb->buffer_offset = binding->Offset;