XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_smurf, simix,
"Logging specific to SIMIX (SMURF)");
-static xbt_dynar_t req_vector;
-static xbt_os_mutex_t sync_req_vector;
+/* 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)
{
- sync_req_vector = xbt_os_mutex_init();
- req_vector = xbt_dynar_new(sizeof(void *), NULL);
+ req_todo = xbt_heap_new(5,NULL);
+ sync_req_positions = xbt_os_mutex_init();
}
void SIMIX_request_destroy(void)
{
- xbt_os_mutex_destroy(sync_req_vector);
- xbt_dynar_free(&req_vector);
+ 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){
- if (_surf_parallel_contexts)
- xbt_os_mutex_acquire(sync_req_vector);
- xbt_dynar_set_as(req_vector, req->issuer->pid, smx_req_t, req);
- if (_surf_parallel_contexts)
- xbt_os_mutex_release(sync_req_vector);
- req->issuer->request = req;
- DEBUG2("Yield process '%s' on request of type %d", req->issuer->name, req->call);
+ smx_process_t issuer = SIMIX_process_self();
+ if (issuer != simix_global->maestro_process){
+ issuer->request.issuer = issuer;
+
+ if (smx_parallel_contexts)
+ xbt_os_mutex_acquire(sync_req_positions);
+ 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);
+
+ 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);
}
}
smx_req_t SIMIX_request_pop(void)
{
- smx_req_t request = NULL;
- while (xbt_dynar_length(req_vector)){
- request = xbt_dynar_pop_as(req_vector, smx_req_t);
- if (request)
- break;
- }
-
- return request;
+ return xbt_heap_pop(req_todo);
}
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);
+ req->issuer->request.call = REQ_NO_REQ;
+ xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, req->issuer);
}
}
void SIMIX_request_pre(smx_req_t req)
{
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 =