XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_smurf, simix,
"Logging specific to SIMIX (SMURF)");
-/* Array storing all requests to be handled in this scheduling round */
-static smx_req_t* req_vector=NULL;
-/* Positions of the previous array with non-null data */
-static xbt_heap_t req_positions;
+/* Requests to handle at the end of this round of scheduling user processes */
+static xbt_heap_t req_todo;
/* to protect the write actions in the heap */
static xbt_os_mutex_t sync_req_positions;
void SIMIX_request_init(void)
{
- req_vector = xbt_new0(smx_req_t,1);/* enough room for maestro's requests */
- req_positions = xbt_heap_new(5,NULL);
+ req_todo = xbt_heap_new(5,NULL);
sync_req_positions = xbt_os_mutex_init();
}
void SIMIX_request_destroy(void)
{
- free(req_vector);
- req_vector = NULL;
- xbt_heap_free(req_positions);
+ xbt_heap_free(req_todo);
xbt_os_mutex_destroy(sync_req_positions);
}
-void SIMIX_request_push(smx_req_t req)
+/* FIXME: we may want to save the initialization of issuer... */
+XBT_INLINE smx_req_t SIMIX_req_mine() {
+ smx_process_t issuer = SIMIX_process_self();
+ return &issuer->request;
+}
+
+void SIMIX_request_push()
{
- req->issuer = SIMIX_process_self();
- if (req->issuer != simix_global->maestro_process){
- req_vector[req->issuer->pid] = req;
- req->issuer->request = req;
+ smx_process_t issuer = SIMIX_process_self();
+ if (issuer != simix_global->maestro_process){
+ issuer->request.issuer = issuer;
- if (_surf_parallel_contexts)
+ if (smx_parallel_contexts)
xbt_os_mutex_acquire(sync_req_positions);
- xbt_heap_push(req_positions,req,req->issuer->pid);
- if (_surf_parallel_contexts)
+ xbt_heap_push(req_todo,&issuer->request,issuer->pid);
+ DEBUG4("Pushed request %s (%d) of %s; now %d requests waiting",
+ SIMIX_request_name(issuer->request.call), issuer->request.call,
+ issuer->name,xbt_heap_size(req_todo));
+ if (smx_parallel_contexts)
xbt_os_mutex_release(sync_req_positions);
- DEBUG2("Yield process '%s' on request of type %d", req->issuer->name, req->call);
+ DEBUG3("Yield process '%s' on request of type %s (%d)", issuer->name,
+ SIMIX_request_name(issuer->request.call), issuer->request.call);
SIMIX_process_yield();
} else {
- SIMIX_request_pre(req);
+ SIMIX_request_pre(&issuer->request, 0);
}
}
smx_req_t SIMIX_request_pop(void)
{
- return xbt_heap_pop(req_positions);
+ smx_req_t req = xbt_heap_pop(req_todo);
+ if(req)
+ DEBUG4("Popped request %s (%d) of %s; now %d requests waiting",
+ SIMIX_request_name(req->issuer->request.call),
+ req->issuer->request.call,
+ req->issuer->name,xbt_heap_size(req_todo));
+ return req;
}
void SIMIX_request_answer(smx_req_t req)
{
if (req->issuer != simix_global->maestro_process){
- req->issuer->request = NULL;
- xbt_swag_insert(req->issuer, simix_global->process_to_run);
+ DEBUG2("Answer request %s (%d)", SIMIX_request_name(req->call), req->call);
+ req->issuer->request.call = REQ_NO_REQ;
+ xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, req->issuer);
}
}
|| req->call == REQ_COMM_IRECV
|| req->call == REQ_COMM_WAIT
|| req->call == REQ_COMM_WAITANY
- || req->call == REQ_COMM_TEST;
+ || req->call == REQ_COMM_TEST
+ || req->call == REQ_COMM_TESTANY;
}
int SIMIX_request_is_enabled(smx_req_t req)
case REQ_COMM_WAIT:
/* FIXME: check also that src and dst processes are not suspended */
- if (req->comm_wait.comm->comm.src_proc
- && req->comm_wait.comm->comm.dst_proc)
+ /* If there is a timeout it will be always enabled because, if the
+ * communication is not ready, it can timeout.
+ * This avoids false positives on dead-locks */
+ if(req->comm_wait.timeout >= 0)
return TRUE;
- return FALSE;
+
+ act = req->comm_wait.comm;
+ return (act->comm.src_proc && act->comm.dst_proc);
break;
case REQ_COMM_WAITANY:
}
}
+int SIMIX_request_is_enabled_by_idx(smx_req_t req, unsigned int idx)
+{
+ smx_action_t act;
+
+ switch (req->call) {
+
+ case REQ_COMM_WAIT:
+ /* FIXME: check also that src and dst processes are not suspended */
+ act = req->comm_wait.comm;
+ return (act->comm.src_proc && act->comm.dst_proc);
+ break;
+
+ case REQ_COMM_WAITANY:
+ act = xbt_dynar_get_as(req->comm_waitany.comms, idx, smx_action_t);
+ return (act->comm.src_proc && act->comm.dst_proc);
+ break;
+
+ default:
+ return TRUE;
+ }
+}
-void SIMIX_request_pre(smx_req_t req)
+void SIMIX_request_pre(smx_req_t req, int value)
{
switch (req->call) {
+ case REQ_NO_REQ:
+ THROW2(arg_error,0,"Asked to do the noop syscall on %s@%s",
+ SIMIX_process_get_name(req->issuer),
+ SIMIX_host_get_name(SIMIX_process_get_host(req->issuer))
+ );
+ break;
case REQ_HOST_GET_BY_NAME:
req->host_get_by_name.result =
break;
case REQ_PROCESS_CREATE:
- req_vector = xbt_realloc(req_vector,sizeof(smx_req_t)*(SIMIX_process_get_maxpid()+2));
req->process_create.result = SIMIX_process_create(
req->process_create.name,
req->process_create.code,
break;
case REQ_COMM_WAITANY:
- SIMIX_pre_comm_waitany(req);
+ SIMIX_pre_comm_waitany(req, value);
break;
case REQ_COMM_WAIT:
break;
case REQ_COMM_TESTANY:
- SIMIX_pre_comm_testany(req);
+ SIMIX_pre_comm_testany(req, value);
break;
case REQ_COMM_GET_REMAINS: