snRuntime: Fix synchronization bug

Since only the DMA core initializes the allocator structs, but all cores make use of it, it is important to place a barrier between the initialization and its successive use.
pulp-platform · Jul 4, 2024 · 8231ab3 · 8231ab3
1 parent 5ab7d18
commit 8231ab3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/sw/snRuntime/src/alloc.h b/sw/snRuntime/src/alloc.h
@@ -91,6 +91,8 @@ inline void snrt_alloc_init() {
         snrt_l3_allocator()->size = 0;
         snrt_l3_allocator()->next = snrt_l3_allocator()->base;
     }
+    // Synchronize with other cores
+    snrt_cluster_hw_barrier();
 }
 
 // TODO colluca: optimize by using DMA

diff --git a/sw/snRuntime/src/start.c b/sw/snRuntime/src/start.c
@@ -139,8 +139,9 @@ void snrt_main() {
 #endif
 
 #if defined(SNRT_INIT_BSS) || defined(SNRT_INIT_CLS)
-    // Single DMA wait call for both snrt_init_bss() and snrt_init_cls()
+    // Single DMA wait call and barrier for both snrt_init_bss() and snrt_init_cls()
     if (snrt_is_dm_core()) snrt_dma_wait_all();
+    snrt_cluster_hw_barrier();
 #endif
 
 #ifdef SNRT_CRT0_CALLBACK3