smx_mutex_t m = xbt_new0(s_smx_mutex_t, 1);
s_smx_process_t p; /* useful to initialize sleeping swag */
/* structures initialization */
- m->using = 0;
+ m->refcount = 0;
m->sleeping = xbt_swag_new(xbt_swag_offset(p, synchro_hookup));
return m;
}
xbt_assert0((mutex != NULL), "Invalid parameters");
- if (mutex->using) {
+ if (mutex->refcount ) {
/* somebody using the mutex, block */
xbt_swag_insert(self, mutex->sleeping);
self->simdata->mutex = mutex;
xbt_context_yield();
}
- mutex->using = 1;
+ mutex->refcount = 1;
} else {
/* mutex free */
- mutex->using = 1;
+ mutex->refcount = 1;
}
return;
}
{
xbt_assert0((mutex != NULL), "Invalid parameters");
- if (mutex->using)
+ if (mutex->refcount )
return 0;
else {
- mutex->using = 1;
+ mutex->refcount = 1;
return 1;
}
}
if (xbt_swag_size(mutex->sleeping) > 0) {
p = xbt_swag_extract(mutex->sleeping);
- mutex->using = 0;
+ mutex->refcount = 0;
xbt_swag_insert(p, simix_global->process_to_run);
} else {
/* nobody to wake up */
- mutex->using = 0;
+ mutex->refcount = 0;
}
return;
}
*/
void SIMIX_cond_signal(smx_cond_t cond)
{
+ smx_process_t proc = NULL;
DEBUG1("Signal condition %p", cond);
xbt_assert0((cond != NULL), "Invalid parameters");
- smx_process_t proc = NULL;
+
if (xbt_swag_size(cond->sleeping) >= 1) {
proc = xbt_swag_extract(cond->sleeping);
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_register_condition_to_action(act_sleep, cond);
__SIMIX_cond_wait(cond);
- xbt_fifo_pop(act_sleep->cond_list);
+ 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);
void SIMIX_cond_wait_timeout(smx_cond_t cond, smx_mutex_t mutex,
double max_duration)
{
- xbt_assert0((mutex != NULL), "Invalid parameters");
+
smx_action_t act_sleep;
+ xbt_assert0((mutex != NULL), "Invalid parameters");
DEBUG1("Timed wait condition %p", cond);
cond->mutex = mutex;
if (max_duration >= 0) {
act_sleep = SIMIX_action_sleep(SIMIX_host_self(), max_duration);
SIMIX_register_action_to_condition(act_sleep, cond);
- SIMIX_register_condition_to_action(act_sleep, cond);
__SIMIX_cond_wait(cond);
- xbt_fifo_remove(act_sleep->cond_list, cond);
+ SIMIX_unregister_action_to_condition(act_sleep, cond);
if (SIMIX_action_get_state(act_sleep) == SURF_ACTION_DONE) {
SIMIX_action_destroy(act_sleep);
THROW0(timeout_error, 0, "Condition timeout");
*/
void SIMIX_cond_broadcast(smx_cond_t cond)
{
- xbt_assert0((cond != NULL), "Invalid parameters");
smx_process_t proc = NULL;
smx_process_t proc_next = NULL;
+ xbt_assert0((cond != NULL), "Invalid parameters");
+
DEBUG1("Broadcast condition %p", cond);
xbt_swag_foreach_safe(proc, proc_next, cond->sleeping) {
xbt_swag_remove(proc, cond->sleeping);
"Cannot destroy conditional since someone is still using it");
xbt_swag_free(cond->sleeping);
+ DEBUG1("%d actions registered", xbt_fifo_size(cond->actions));
+ __SIMIX_cond_display_actions(cond);
xbt_fifo_foreach(cond->actions, item, action, smx_action_t) {
- SIMIX_unregister_condition_to_action(action, cond);
+ SIMIX_unregister_action_to_condition(action, cond);
}
+ __SIMIX_cond_display_actions(cond);
+
xbt_fifo_free(cond->actions);
xbt_free(cond);
return;
}
}
-/**
- * \brief Set a condition to an action
- *
- * Creates the "link" between an action and a condition. You have to call this function when you create an action and want to wait its ending.
- * \param action SIMIX action
- * \param cond SIMIX cond
- */
-void SIMIX_register_condition_to_action(smx_action_t action,
- smx_cond_t cond)
+void SIMIX_cond_display_info(smx_cond_t cond)
{
- xbt_assert0((action != NULL) && (cond != NULL), "Invalid parameters");
-
- DEBUG2("Register condition %p to action %p", cond, action);
- xbt_fifo_push(action->cond_list, cond);
-}
-
-/**
- * \brief Unset a condition to an action
- *
- * Destroys the "link" between an action and a condition.
- * \param action SIMIX action
- * \param cond SIMIX cond
- */
-void SIMIX_unregister_condition_to_action(smx_action_t action,
- smx_cond_t cond)
-{
- xbt_assert0((action != NULL) && (cond != NULL), "Invalid parameters");
+ if (cond == NULL)
+ return;
+ else {
+ smx_process_t process = NULL;
- while (xbt_fifo_remove(action->cond_list, cond)) {
+ INFO0("Blocked process on this condition:");
+ xbt_swag_foreach(process,cond->sleeping) {
+ INFO2("\t %s running on host %s",process->name,process->simdata->smx_host->name);
+ }
}
}