mirrors
/
mesa
mirror of https://gitlab.freedesktop.org/mesa/mesa
1
0
Fork 0
Code Issues 9 Packages Projects Releases Wiki Activity

util: Fixes debug_get_option_* thread safety by set initialized=true after the value get 

Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Acked-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18764>
This commit is contained in:
Yonggang Luo 2022-09-24 15:28:52 +08:00 committed by Marge Bot
parent e2c3739a3f
commit 171c3c551a
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/util/u_debug.h
View File

@ -46,6 +46,7 @@
#endif
#include "util/os_misc.h"
#include "util/u_atomic.h"
#include "util/detect_os.h"
#include "util/macros.h"
@ -374,9 +375,9 @@ debug_get_option_ ## suffix (void) \
{ \
static bool initialized = false; \
static const char * value; \
if (!initialized) { \
initialized = true; \
if (unlikely(!p_atomic_read_relaxed(&initialized))) { \
value = debug_get_option(name, dfault); \
p_atomic_set(&initialized, true); \
} \
return value; \
}
@ -397,9 +398,9 @@ debug_get_option_ ## sufix (void) \
{ \
static bool initialized = false; \
static bool value; \
if (!initialized) { \
initialized = true; \
if (unlikely(!p_atomic_read_relaxed(&initialized))) { \
value = debug_get_bool_option(name, dfault); \
p_atomic_set(&initialized, true); \
} \
return value; \
}
@ -410,9 +411,9 @@ debug_get_option_ ## sufix (void) \
{ \
static bool initialized = false; \
static long value; \
if (!initialized) { \
initialized = true; \
if (unlikely(!p_atomic_read_relaxed(&initialized))) { \
value = debug_get_num_option(name, dfault); \
p_atomic_set(&initialized, true); \
} \
return value; \
}
@ -423,9 +424,9 @@ debug_get_option_ ## sufix (void) \
{ \
static bool initialized = false; \
static unsigned long value; \
if (!initialized) { \
initialized = true; \
if (unlikely(!p_atomic_read_relaxed(&initialized))) { \
value = debug_get_flags_option(name, flags, dfault); \
p_atomic_set(&initialized, true); \
} \
return value; \
}