void SIMIX_post_synchro(smx_action_t action)
{
- if(surf_workstation_model->action_state_get(action->synchro.sleep) == SURF_ACTION_FAILED)
- action->state = SIMIX_FAILED;
+ if (surf_workstation_model->action_state_get(action->synchro.sleep) == SURF_ACTION_FAILED)
+ action->state = SIMIX_FAILED;
else if(surf_workstation_model->action_state_get(action->synchro.sleep) == SURF_ACTION_DONE)
- action->state = SIMIX_SRC_TIMEOUT;
-
- action->synchro.sleep->model_type->action_unref(action->synchro.sleep);
+ action->state = SIMIX_SRC_TIMEOUT;
SIMIX_synchro_finish(action);
}
static void SIMIX_synchro_finish(smx_action_t action)
{
smx_req_t req = xbt_fifo_shift(action->request_list);
-
- switch(action->state){
+
+ switch (action->state) {
+
case SIMIX_SRC_TIMEOUT:
TRY {
THROW0(timeout_error, 0, "Synchro's wait timeout");
req->issuer->doexception = 1;
}
break;
-
+
default:
THROW_IMPOSSIBLE;
break;
*/
void SIMIX_pre_mutex_lock(smx_req_t req)
{
- /*FIXME: check where to validate the arguments */
+ /* FIXME: check where to validate the arguments */
smx_action_t sync_act = NULL;
smx_mutex_t mutex = req->mutex_lock.mutex;
smx_process_t process = req->issuer;
-
+
if (mutex->locked) {
/* FIXME: check if the host is active ? */
/* Somebody using the mutex, use a synchro action to get host failures */
{
if (mutex->locked)
return 0;
- else {
- mutex->locked = 1;
- mutex->owner = issuer;
- return 1;
- }
+
+ mutex->locked = 1;
+ mutex->owner = issuer;
+ return 1;
}
/**
smx_process_t p; /*process to wake up */
/* If the mutex is not owned by the issuer do nothing */
- if(issuer != mutex->owner)
+ if (issuer != mutex->owner)
return;
-
+
if (xbt_swag_size(mutex->sleeping) > 0) {
p = xbt_swag_extract(mutex->sleeping);
SIMIX_synchro_destroy(p->waiting_action);
p->waiting_action = NULL;
mutex->owner = p;
- SIMIX_request_answer(p->request);
+ SIMIX_request_answer(&p->request);
} else {
/* nobody to wake up */
mutex->locked = 0;
*/
void SIMIX_mutex_destroy(smx_mutex_t mutex)
{
- if(mutex){
+ if (mutex){
xbt_swag_free(mutex->sleeping);
xbt_free(mutex);
}
smx_cond_t cond = req->cond_wait_timeout.cond;
smx_mutex_t mutex = req->cond_wait_timeout.mutex;
double timeout = req->cond_wait_timeout.timeout;
-
+
_SIMIX_cond_wait(cond, mutex, timeout, issuer, req);
}
smx_process_t issuer, smx_req_t req)
{
smx_action_t sync_act = NULL;
-
+
DEBUG1("Wait condition %p", cond);
-
+
/* If there is a mutex unlock it */
- /* FIXME: what happen if the issuer is not the owner of the mutex ? */
+ /* FIXME: what happens if the issuer is not the owner of the mutex? */
if (mutex != NULL) {
cond->mutex = mutex;
SIMIX_mutex_unlock(mutex, issuer);
smx_process_t proc = NULL;
smx_mutex_t mutex = NULL;
smx_req_t req = NULL;
-
+
DEBUG1("Signal condition %p", cond);
/* If there are processes waiting for the condition choose one and try
to make it acquire the mutex */
if ((proc = xbt_swag_extract(cond->sleeping))) {
-
+
/* Destroy waiter's synchro action */
SIMIX_synchro_destroy(proc->waiting_action);
proc->waiting_action = NULL;
/* Now transform the cond wait request into a mutex lock one */
- req = proc->request;
+ req = &proc->request;
if(req->call == REQ_COND_WAIT)
mutex = req->cond_wait.mutex;
else
req->call = REQ_MUTEX_LOCK;
req->mutex_lock.mutex = mutex;
-
+
SIMIX_pre_mutex_lock(req);
}
}
DEBUG1("Broadcast condition %p", cond);
/* Signal the condition until nobody is waiting on it */
- while(xbt_swag_size(cond->sleeping)){
+ while (xbt_swag_size(cond->sleeping)) {
SIMIX_cond_signal(cond);
}
}
{
DEBUG1("Destroy condition %p", cond);
- if (cond != NULL){
+ if (cond != NULL) {
xbt_assert0(xbt_swag_size(cond->sleeping) == 0,
"Cannot destroy conditional since someone is still using it");
xbt_swag_free(cond->sleeping);
xbt_free(cond);
- return;
}
}
void SIMIX_sem_destroy(smx_sem_t sem)
{
DEBUG1("Destroy semaphore %p", sem);
- if (sem != NULL){
+ if (sem != NULL) {
xbt_assert0(xbt_swag_size(sem->sleeping) == 0,
"Cannot destroy semaphore since someone is still using it");
xbt_swag_free(sem->sleeping);
*
* Unlock a process waiting on the semaphore.
* If no one was blocked, the semaphore capacity is increased by 1.
- * */
+ */
void SIMIX_sem_release(smx_sem_t sem)
{
smx_process_t proc;
-
+
DEBUG1("Sem release semaphore %p", sem);
if ((proc = xbt_swag_extract(sem->sleeping))) {
proc = xbt_swag_extract(sem->sleeping);
SIMIX_synchro_destroy(proc->waiting_action);
proc->waiting_action = NULL;
- SIMIX_request_answer(proc->request);
- }else if(sem->value < SMX_SEM_NOLIMIT){
+ SIMIX_request_answer(&proc->request);
+ } else if (sem->value < SMX_SEM_NOLIMIT) {
sem->value++;
}
}
smx_req_t req)
{
smx_action_t sync_act = NULL;
-
+
DEBUG2("Wait semaphore %p (timeout:%f)", sem, timeout);
- if(sem->value <= 0){
+ if (sem->value <= 0) {
sync_act = SIMIX_synchro_wait(issuer->smx_host, timeout);
xbt_fifo_unshift(sync_act->request_list, req);
issuer->waiting_action = sync_act;
xbt_swag_insert(issuer, sem->sleeping);
- }else{
+ } else {
sem->value--;
SIMIX_request_answer(req);
}