diff options
| author | Erik Faye-Lund <[email protected]> | 2020-07-10 13:12:40 +0200 |
|---|---|---|
| committer | Marge Bot <[email protected]> | 2020-07-16 10:49:22 +0000 |
| commit | 3af0711c402033a8ad7649dc0509e4f7ddc1b640 (patch) | |
| tree | b70c4f655a80dd5b3dd9630c6a3b11a9d5aec36a /src/mesa | |
| parent | a0f84396913d478c926626593514de1ab6ebb439 (diff) | |
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 <[email protected]>
Reviewed-by: Eric Engestrom <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5901>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/main/debug_output.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/mesa/main/debug_output.c b/src/mesa/main/debug_output.c index 6d4c8e72ee3..6527aea5771 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 @@ void _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); } } |