I think we should always add such a "fake" action to handle failures (not just when...

[simgrid.git] / src / simix / smx_synchro.c

diff --git a/src/simix/smx_synchro.c b/src/simix/smx_synchro.c
index 699cf88..70f2249 100644 (file)
--- a/src/simix/smx_synchro.c
+++ b/src/simix/smx_synchro.c
@@ -181,16 +181,16 @@ void SIMIX_cond_wait(smx_cond_t cond, smx_mutex_t mutex)
   cond->mutex = mutex;
 
   SIMIX_mutex_unlock(mutex);
-  /* create an action null only if there are no actions already on the condition, usefull if the host crashs */
-  if (xbt_fifo_size(cond->actions) == 0) {
+  /* always create an action null in case there is a host failure */
+/*   if (xbt_fifo_size(cond->actions) == 0) { */
     act_sleep = SIMIX_action_sleep(SIMIX_host_self(), -1);
     SIMIX_register_action_to_condition(act_sleep, cond);
     __SIMIX_cond_wait(cond);
     SIMIX_unregister_action_to_condition(act_sleep, cond);
     SIMIX_action_destroy(act_sleep);
-  } else {
-    __SIMIX_cond_wait(cond);
-  }
+/*   } else { */
+/*     __SIMIX_cond_wait(cond); */
+/*   } */
   /* get the mutex again */
   SIMIX_mutex_lock(cond->mutex);