Revert "gallium/trace: Fix helgrind complaint about one-time init"
authorErik Faye-Lund <erik.faye-lund@collabora.com>
Tue, 24 Nov 2020 22:33:14 +0000 (23:33 +0100)
committerMarge Bot <eric+marge@anholt.net>
Wed, 25 Nov 2020 09:44:11 +0000 (09:44 +0000)
This reverts commit 1aa055539f32fcb3c78d908d1635bb3a3d517bc2.

Acked-by: Rob Clark <robdclark@chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7760>

src/gallium/auxiliary/driver_trace/tr_screen.c

index 90ee53617a1bd5f4f29abeda96e9905d11e62bb5..56a70de894ae6da6f49877d9d10a36d661bdc665 100644 (file)
@@ -670,11 +670,15 @@ trace_screen_destroy(struct pipe_screen *_screen)
 bool
 trace_enabled(void)
 {
-   do_once {
-      if(trace_dump_trace_begin()) {
-         trace_dumping_start();
-         trace = true;
-      }
+   static bool firstrun = true;
+
+   if (!firstrun)
+      return trace;
+   firstrun = false;
+
+   if(trace_dump_trace_begin()) {
+      trace_dumping_start();
+      trace = true;
    }
 
    return trace;