amdgcn, openmp: Fix concurrency in low-latency allocator

The previous code works fine on Fiji and Vega 10 devices, but bogs down in The
spin locks on Vega 20 or newer.  Adding the sleep instructions fixes the
problem.

libgomp/ChangeLog:

	* basic-allocator.c (basic_alloc_free): Use BASIC_ALLOC_YIELD.
	(basic_alloc_realloc): Use BASIC_ALLOC_YIELD.

diff --git a/libgomp/ChangeLog.omp b/libgomp/ChangeLog.omp
index 09cf9c6f3c14c8b2bc1ff0848cbc867c865e14b4..32bc3b0c82cea1c8c5e2c390993596b84bff2dac 100644 (file)
--- a/libgomp/ChangeLog.omp
+++ b/libgomp/ChangeLog.omp
@@ -1,3 +1,8 @@
+2023-04-20  Andrew Stubbs  <ams@codesourcery.com>
+
+       * basic-allocator.c (basic_alloc_free): Use BASIC_ALLOC_YIELD.
+       (basic_alloc_realloc): Use BASIC_ALLOC_YIELD.
+
 2023-04-03  Thomas Schwinge  <thomas@codesourcery.com>
 
        * target.c (gomp_map_vars_internal)

diff --git a/libgomp/basic-allocator.c b/libgomp/basic-allocator.c
index b4b9e4ba13a74be0b366c6669af5d02e186dce5f..a61828e48a0dbaf5f4490bba2d5362f2243e6593 100644 (file)
--- a/libgomp/basic-allocator.c
+++ b/libgomp/basic-allocator.c
@@ -188,6 +188,7 @@ basic_alloc_free (char *heap, void *addr, size_t size)
          break;
        }
       /* Spin.  */
+      BASIC_ALLOC_YIELD;
     }
   while (1);
 
@@ -267,6 +268,7 @@ basic_alloc_realloc (char *heap, void *addr, size_t oldsize,
          break;
        }
       /* Spin.  */
+      BASIC_ALLOC_YIELD;
     }
   while (1);