Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Fix tne runall functions
authorthiery <thiery@48e7efb5-ca39-0410-a469-dd3cf9ba447f>
Mon, 13 Dec 2010 10:57:18 +0000 (10:57 +0000)
committerthiery <thiery@48e7efb5-ca39-0410-a469-dd3cf9ba447f>
Mon, 13 Dec 2010 10:57:18 +0000 (10:57 +0000)
git-svn-id: svn+ssh://scm.gforge.inria.fr/svn/simgrid/simgrid/trunk@9145 48e7efb5-ca39-0410-a469-dd3cf9ba447f

src/replay/state_machine_context.c
src/simix/smx_context_raw.c
src/simix/smx_context_ruby.c
src/simix/smx_context_sysv.c
src/simix/smx_context_thread.c

index ab48214..66145f2 100644 (file)
@@ -76,11 +76,11 @@ static void statem_ctx_resume(smx_context_t new_context) {
 static void statem_ctx_runall(xbt_dynar_t processes) {
   smx_context_t old_context;
   smx_process_t process;
 static void statem_ctx_runall(xbt_dynar_t processes) {
   smx_context_t old_context;
   smx_process_t process;
+  unsigned int cursor;
 
   INFO0("Run all");
 
 
   INFO0("Run all");
 
-  while (xbt_dynar_length(processes)){
-    process = xbt_dynar_pop_as(processes,smx_process_t);
+  xbt_dynar_foreach(processes, cursor, process) {
     statem_context_t ctx = (statem_context_t)SIMIX_process_get_context(process);
     old_context = smx_current_context;
     smx_current_context = SIMIX_process_get_context(process);
     statem_context_t ctx = (statem_context_t)SIMIX_process_get_context(process);
     old_context = smx_current_context;
     smx_current_context = SIMIX_process_get_context(process);
@@ -89,5 +89,6 @@ static void statem_ctx_runall(xbt_dynar_t processes) {
           ctx->syscall_id==0?NULL:SIMIX_request_get_result(ctx->syscall_id));
     smx_current_context = old_context;
   }
           ctx->syscall_id==0?NULL:SIMIX_request_get_result(ctx->syscall_id));
     smx_current_context = old_context;
   }
+  xbt_dynar_reset(processes);
 }
 
 }
 
index 57f8c71..2ca8451 100644 (file)
@@ -260,11 +260,12 @@ static void smx_ctx_raw_resume(smx_context_t context)
 static void smx_ctx_raw_runall(xbt_dynar_t processes)
 {
   smx_process_t process;
 static void smx_ctx_raw_runall(xbt_dynar_t processes)
 {
   smx_process_t process;
+  unsigned int cursor;
 
 
-  while (xbt_dynar_length(processes)){
-    process = xbt_dynar_pop_as(processes,smx_process_t);
+  xbt_dynar_foreach(processes, cursor, process) {
     smx_ctx_raw_resume(process->context);
   }
     smx_ctx_raw_resume(process->context);
   }
+  xbt_dynar_reset(processes);
 }
 
 static void smx_ctx_raw_resume_parallel(smx_context_t context)
 }
 
 static void smx_ctx_raw_resume_parallel(smx_context_t context)
index ce1a50e..4051bad 100644 (file)
@@ -114,11 +114,13 @@ static void smx_ctx_ruby_runall(xbt_dynar_t processes)
 {
   smx_process_t process;
   smx_context_t old_context;
 {
   smx_process_t process;
   smx_context_t old_context;
-  while (xbt_dynar_length(processes)){
-    process = xbt_dynar_pop_as(processes,smx_process_t);
+  unsigned int cursor;
+
+  xbt_dynar_foreach(processes, cursor, process) {
     old_context = smx_current_context;
     smx_current_context = process->context;
     smx_ctx_ruby_resume(smx_current_context);
     smx_current_context = old_context;
     old_context = smx_current_context;
     smx_current_context = process->context;
     smx_ctx_ruby_resume(smx_current_context);
     smx_current_context = old_context;
-  } 
+  }
+  xbt_dynar_reset(processes);
 }
 }
index b13220e..12408c3 100644 (file)
@@ -169,11 +169,12 @@ void smx_ctx_sysv_resume(smx_context_t context)
 void smx_ctx_sysv_runall(xbt_dynar_t processes)
 {
   smx_process_t process;
 void smx_ctx_sysv_runall(xbt_dynar_t processes)
 {
   smx_process_t process;
-  
-  while (xbt_dynar_length(processes)){
-    process = xbt_dynar_pop_as(processes,smx_process_t);
+  unsigned int cursor;
+
+  xbt_dynar_foreach(processes, cursor, process) {
     smx_ctx_sysv_resume(process->context);
   }
     smx_ctx_sysv_resume(process->context);
   }
+  xbt_dynar_reset(processes);
 }
 
 void smx_ctx_sysv_resume_parallel(smx_process_t process)
 }
 
 void smx_ctx_sysv_resume_parallel(smx_process_t process)
index e1fdecc..b2ec075 100644 (file)
@@ -145,11 +145,13 @@ static void smx_ctx_thread_suspend(smx_context_t context)
 static void smx_ctx_thread_runall_serial(xbt_dynar_t processes)
 {
   smx_process_t process;
 static void smx_ctx_thread_runall_serial(xbt_dynar_t processes)
 {
   smx_process_t process;
-  while (xbt_dynar_length(processes)){
-    process = xbt_dynar_pop_as(processes,smx_process_t);
+  unsigned int cursor;
+
+  xbt_dynar_foreach(processes, cursor, process) {
     xbt_os_sem_release(((smx_ctx_thread_t) process->context)->begin);
     xbt_os_sem_acquire(((smx_ctx_thread_t) process->context)->end);
   }
     xbt_os_sem_release(((smx_ctx_thread_t) process->context)->begin);
     xbt_os_sem_acquire(((smx_ctx_thread_t) process->context)->end);
   }
+  xbt_dynar_reset(processes);
 }
 
 static void smx_ctx_thread_runall_parallel(xbt_dynar_t processes)
 }
 
 static void smx_ctx_thread_runall_parallel(xbt_dynar_t processes)
@@ -160,10 +162,10 @@ static void smx_ctx_thread_runall_parallel(xbt_dynar_t processes)
   xbt_dynar_foreach(processes, index, process)
     xbt_os_sem_release(((smx_ctx_thread_t) process->context)->begin);
 
   xbt_dynar_foreach(processes, index, process)
     xbt_os_sem_release(((smx_ctx_thread_t) process->context)->begin);
 
-  while(xbt_dynar_length(processes)){
-     process = xbt_dynar_pop_as(processes,smx_process_t);
+  xbt_dynar_foreach(processes, index, process) {
      xbt_os_sem_acquire(((smx_ctx_thread_t) process->context)->end);
   }
      xbt_os_sem_acquire(((smx_ctx_thread_t) process->context)->end);
   }
+  xbt_dynar_reset(processes);
 }
 
 static smx_context_t smx_ctx_thread_self(void)
 }
 
 static smx_context_t smx_ctx_thread_self(void)