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

mesa: fix ID usage for buffer warnings

Browse source 
We need a different ID pointer for each call site.

Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
This commit is contained in:
Brian Paul 2015-12-09 16:00:55 -07:00
parent de5bb7fe78
commit e1815bcc47
1 changed files with 12 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -60,16 +60,16 @@
/**
* Helper to warn of possible performance issues, such as frequently
* updating a buffer created with GL_STATIC_DRAW.
* updating a buffer created with GL_STATIC_DRAW. Called via the macro
* below.
*/
static void
buffer_usage_warning(struct gl_context *ctx, const char *fmt, ...)
buffer_usage_warning(struct gl_context *ctx, GLuint *id, const char *fmt, ...)
{
va_list args;
GLuint msg_id = 0;
va_start(args, fmt);
_mesa_gl_vdebug(ctx, &msg_id,
_mesa_gl_vdebug(ctx, id,
MESA_DEBUG_SOURCE_API,
MESA_DEBUG_TYPE_PERFORMANCE,
MESA_DEBUG_SEVERITY_MEDIUM,
@ -77,6 +77,12 @@ buffer_usage_warning(struct gl_context *ctx, const char *fmt, ...)
va_end(args);
}
#define BUFFER_USAGE_WARNING(CTX, FMT, ...) \
do { \
static GLuint id = 0; \
buffer_usage_warning(CTX, &id, FMT, ##__VA_ARGS__); \
} while (0)
/**
* Used as a placeholder for buffer objects between glGenBuffers() and
@ -1713,7 +1719,7 @@ _mesa_buffer_sub_data(struct gl_context *ctx, struct gl_buffer_object *bufObj,
/* If the application declared the buffer as static draw/copy or stream
* draw, it should not be frequently modified with glBufferSubData.
*/
buffer_usage_warning(ctx,
BUFFER_USAGE_WARNING(ctx,
"using %s(buffer %u, offset %u, size %u) to "
"update a %s buffer",
func, bufObj->Name, offset, size,
@ -2432,7 +2438,7 @@ _mesa_map_buffer_range(struct gl_context *ctx,
if ((bufObj->Usage == GL_STATIC_DRAW ||
bufObj->Usage == GL_STATIC_COPY) &&
bufObj->NumMapBufferWriteCalls >= BUFFER_WARNING_CALL_COUNT) {
buffer_usage_warning(ctx,
BUFFER_USAGE_WARNING(ctx,
"using %s(buffer %u, offset %u, length %u) to "
"update a %s buffer",
func, bufObj->Name, offset, length,