SDL: Use atomic variables for thread communication (a7522)

SDL Commits
1 
From a75227aaebc5bc04ad5a9b3225c66db0a707951d Mon Sep 17 00:00:00 2001
From: Sam Lantinga <[EMAIL REDACTED]>
Date: Thu, 5 Sep 2024 05:35:03 -0700
Subject: [PATCH] Use atomic variables for thread communication

Fixes https://github.com/libsdl-org/SDL/issues/10711

(cherry picked from commit a0f36fb85b40338fb24e1f1479bdcc432c06aba6)
---
 test/testthread.c    | 12 ++++++------
 test/torturethread.c | 11 ++++++-----
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/test/testthread.c b/test/testthread.c
index 892535055ec30..031b87a6a4cf8 100644
--- a/test/testthread.c
+++ b/test/testthread.c
@@ -20,7 +20,7 @@
 #include "SDL_test.h"
 
 static SDL_TLSID tls;
-static int alive = 0;
+static SDL_atomic_t alive;
 static int testprio = 0;
 static SDLTest_CommonState *state;
 
@@ -58,7 +58,7 @@ ThreadFunc(void *data)
     SDL_TLSSet(tls, "baby thread", NULL);
     SDL_Log("Started thread %s: My thread id is %lu, thread data = %s\n",
             (char *)data, SDL_ThreadID(), (const char *)SDL_TLSGet(tls));
-    while (alive) {
+    while (SDL_AtomicGet(&alive)) {
         SDL_Log("Thread '%s' is alive!\n", (char *)data);
 
         if (testprio) {
@@ -79,7 +79,7 @@ killed(int sig)
 {
     SDL_Log("Killed with SIGTERM, waiting 5 seconds to exit\n");
     SDL_Delay(5 * 1000);
-    alive = 0;
+    SDL_AtomicSet(&alive, 0);
     quit(0);
 }
 
@@ -133,7 +133,7 @@ int main(int argc, char *argv[])
     SDL_TLSSet(tls, "main thread", NULL);
     SDL_Log("Main thread data initially: %s\n", (const char *)SDL_TLSGet(tls));
 
-    alive = 1;
+    SDL_AtomicSet(&alive, 1);
     thread = SDL_CreateThread(ThreadFunc, "One", "#1");
     if (!thread) {
         SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "Couldn't create thread: %s\n", SDL_GetError());
@@ -141,12 +141,12 @@ int main(int argc, char *argv[])
     }
     SDL_Delay(5 * 1000);
     SDL_Log("Waiting for thread #1\n");
-    alive = 0;
+    SDL_AtomicSet(&alive, 0);
     SDL_WaitThread(thread, NULL);
 
     SDL_Log("Main thread data finally: %s\n", (const char *)SDL_TLSGet(tls));
 
-    alive = 1;
+    SDL_AtomicSet(&alive, 1);
     (void)signal(SIGTERM, killed);
     thread = SDL_CreateThread(ThreadFunc, "Two", "#2");
     if (!thread) {
diff --git a/test/torturethread.c b/test/torturethread.c
index b612cda497766..b91baa1347eb0 100644
--- a/test/torturethread.c
+++ b/test/torturethread.c
@@ -34,8 +34,9 @@ quit(int rc)
 int SDLCALL
 SubThreadFunc(void *data)
 {
-    while (!*(int volatile *)data) {
-        ; /* SDL_Delay(10); */ /* do nothing */
+    SDL_atomic_t *flag = (SDL_atomic_t *)data;
+    while (!SDL_AtomicGet(flag)) {
+        SDL_Delay(10);
     }
     return 0;
 }
@@ -44,7 +45,7 @@ int SDLCALL
 ThreadFunc(void *data)
 {
     SDL_Thread *sub_threads[NUMTHREADS];
-    int flags[NUMTHREADS];
+    SDL_atomic_t flags[NUMTHREADS];
     int i;
     int tid = (int)(uintptr_t)data;
 
@@ -53,7 +54,7 @@ ThreadFunc(void *data)
     for (i = 0; i < NUMTHREADS; i++) {
         char name[64];
         (void)SDL_snprintf(name, sizeof(name), "Child%d_%d", tid, i);
-        flags[i] = 0;
+        SDL_AtomicSet(&flags[i], 0);
         sub_threads[i] = SDL_CreateThread(SubThreadFunc, name, &flags[i]);
     }
 
@@ -64,7 +65,7 @@ ThreadFunc(void *data)
 
     SDL_Log("Thread '%d' sending signals to subthreads\n", tid);
     for (i = 0; i < NUMTHREADS; i++) {
-        flags[i] = 1;
+        SDL_AtomicSet(&flags[i], 1);
         SDL_WaitThread(sub_threads[i], NULL);
     }