info: fix ULTs stacks dump works only once

1/one returned value must be tested instead of 0 to detect that last "parked" XStream is done in ABTI_info_check_print_all_thread_stacks() and thus that print_stack_flag can be reset to PRINT_STACK_FLAG_UNSET to allow for a next dump to be started. DAOS-14248 ticket, Argobots issue pmodels#393. Signed-off-by: Bruno Faccini <[email protected]>
bfaccini · Aug 31, 2023 · 0c4d79c · 0c4d79c
1 parent 36a41b5
commit 0c4d79c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/info.c b/src/info.c
@@ -1097,7 +1097,8 @@ void ABTI_info_check_print_all_thread_stacks(void)
 
     /* Decrement the barrier value. */
     int dec_value = ABTD_atomic_fetch_sub_int(&print_stack_barrier, 1);
-    if (dec_value == 0) {
+    /* previous value should be 1 ! */
+    if (dec_value == 1) {
         /* The last execution stream resets the flag. */
         ABTD_atomic_release_store_int(&print_stack_flag,
                                       PRINT_STACK_FLAG_UNSET);