commit 3af0711c402033a8ad7649dc0509e4f7ddc1b640
author Erik Faye-Lund <erik.faye-lund@collabora.com> Fri Jul 10 13:12:40 2020 +0200
committer Marge Bot <eric+marge@anholt.net> Thu Jul 16 10:49:22 2020 +0000
tree b70c4f655a80dd5b3dd9630c6a3b11a9d5aec36a
parent a0f84396913d478c926626593514de1ab6ebb439

mesa/main: use p_atomic_inc_return instead of locking

There's no good reason for using a mutex here, as we have a simpler
primitive; atomic integers. So let's use that instead, to simplify
things a bit.

Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5901>

src/mesa/main/debug_output.c

diff --git a/src/mesa/main/debug_output.c b/src/mesa/main/debug_output.c
index 6d4c8e7..6527aea 100644
--- a/src/mesa/main/debug_output.c
+++ b/src/mesa/main/debug_output.c
@@ -37,8 +37,7 @@
 #include "util/u_memory.h"
 
 
-static simple_mtx_t DynamicIDMutex = _SIMPLE_MTX_INITIALIZER_NP;
-static GLuint NextDynamicID = 1;
+static GLuint PrevDynamicID = 0;
 
 
 /**
@@ -194,10 +193,7 @@
 _mesa_debug_get_id(GLuint *id)
 {
    if (!(*id)) {
-      simple_mtx_lock(&DynamicIDMutex);
-      if (!(*id))
-         *id = NextDynamicID++;
-      simple_mtx_unlock(&DynamicIDMutex);
+      *id = p_atomic_inc_return(&PrevDynamicID);
    }
 }