X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/e4921252225dbd0ea1ec9fcd90be2134df879a0e..905e26b959375a249abcfdf1599f6178fe031f44:/src/simix/smx_process.cpp diff --git a/src/simix/smx_process.cpp b/src/simix/smx_process.cpp index bc84eb7727..9d23a075aa 100644 --- a/src/simix/smx_process.cpp +++ b/src/simix/smx_process.cpp @@ -4,24 +4,55 @@ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ +#include +#include +#include +#include + +#include + +#include +#include +#include +#include +#include + +#include + +#include + +#include "src/surf/surf_interface.hpp" #include "smx_private.h" -#include "xbt/sysdep.h" -#include "xbt/log.h" -#include "xbt/dict.h" -#include "mc/mc.h" #include "src/mc/mc_replay.h" -#include "src/mc/mc_client.h" -#include "src/simix/smx_private.hpp" +#include "src/mc/Client.hpp" +#include "src/msg/msg_private.h" +#include "src/kernel/activity/SynchroSleep.hpp" +#include "src/kernel/activity/SynchroRaw.hpp" +#include "src/kernel/activity/SynchroIo.hpp" #ifdef HAVE_SMPI #include "src/smpi/private.h" #endif -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_process, simix, - "Logging specific to SIMIX (process)"); +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_process, simix, "Logging specific to SIMIX (process)"); unsigned long simix_process_maxpid = 0; +/** Increase the refcount for this process */ +smx_process_t SIMIX_process_ref(smx_process_t process) +{ + if (process != nullptr) + intrusive_ptr_add_ref(process); + return process; +} + +/** Decrease the refcount for this process */ +void SIMIX_process_unref(smx_process_t process) +{ + if (process != nullptr) + intrusive_ptr_release(process); +} + /** * \brief Returns the current agent. * @@ -29,11 +60,11 @@ unsigned long simix_process_maxpid = 0; * * \return The SIMIX process */ -smx_process_t SIMIX_process_self(void) +smx_process_t SIMIX_process_self() { smx_context_t self_context = SIMIX_context_self(); - return self_context ? SIMIX_context_get_process(self_context) : NULL; + return (self_context != nullptr) ? self_context->process() : nullptr; } /** @@ -51,57 +82,59 @@ int SIMIX_process_has_pending_comms(smx_process_t process) { void SIMIX_process_cleanup(smx_process_t process) { XBT_DEBUG("Cleanup process %s (%p), waiting synchro %p", - process->name, process, process->waiting_synchro); + process->name.c_str(), process, process->waiting_synchro); + process->finished = true; SIMIX_process_on_exit_runall(process); /* Unregister from the kill timer if any */ - if (process->kill_timer != NULL) - SIMIX_timer_remove(process->kill_timer); + if (process->kill_timer != nullptr) + SIMIX_timer_remove(process->kill_timer); xbt_os_mutex_acquire(simix_global->mutex); /* cancel non-blocking communications */ - smx_synchro_t synchro; - while ((synchro = (smx_synchro_t) xbt_fifo_pop(process->comms))) { + smx_synchro_t synchro = static_cast(xbt_fifo_pop(process->comms)); + while (synchro != nullptr) { + simgrid::kernel::activity::Comm *comm = static_cast(synchro); /* make sure no one will finish the comm after this process is destroyed, * because src_proc or dst_proc would be an invalid pointer */ - SIMIX_comm_cancel(synchro); + comm->cancel(); - if (synchro->comm.src_proc == process) { + if (comm->src_proc == process) { XBT_DEBUG("Found an unfinished send comm %p (detached = %d), state %d, src = %p, dst = %p", - synchro, synchro->comm.detached, (int)synchro->state, synchro->comm.src_proc, synchro->comm.dst_proc); - synchro->comm.src_proc = NULL; + comm, comm->detached, (int)comm->state, comm->src_proc, comm->dst_proc); + comm->src_proc = nullptr; /* I'm not supposed to destroy a detached comm from the sender side, */ - if (!synchro->comm.detached) - SIMIX_comm_destroy(synchro); + if (comm->detached) + XBT_DEBUG("Don't destroy it since it's a detached comm and I'm the sender"); else - XBT_DEBUG("Don't destroy it since it's a detached comm"); + comm->unref(); } - else if (synchro->comm.dst_proc == process){ + else if (comm->dst_proc == process){ XBT_DEBUG("Found an unfinished recv comm %p, state %d, src = %p, dst = %p", - synchro, (int)synchro->state, synchro->comm.src_proc, synchro->comm.dst_proc); - synchro->comm.dst_proc = NULL; + comm, (int)comm->state, comm->src_proc, comm->dst_proc); + comm->dst_proc = nullptr; - if (synchro->comm.detached && synchro->comm.refcount == 1 - && synchro->comm.src_proc != NULL) { + if (comm->detached && comm->src_proc != nullptr) { /* the comm will be freed right now, remove it from the sender */ - xbt_fifo_remove(synchro->comm.src_proc->comms, synchro); + xbt_fifo_remove(comm->src_proc->comms, comm); } - SIMIX_comm_destroy(synchro); - } - else { - xbt_die("Communication synchro %p is in my list but I'm not the sender " - "or the receiver", synchro); + + comm->unref(); + } else { + xbt_die("Communication synchro %p is in my list but I'm not the sender nor the receiver", synchro); } + synchro = static_cast(xbt_fifo_pop(process->comms)); } XBT_DEBUG("%p should not be run anymore",process); xbt_swag_remove(process, simix_global->process_list); - xbt_swag_remove(process, sg_host_simix(process->host)->process_list); + if (process->host) + xbt_swag_remove(process, sg_host_simix(process->host)->process_list); xbt_swag_insert(process, simix_global->process_to_destroy); process->context->iwannadie = 0; @@ -114,47 +147,64 @@ void SIMIX_process_cleanup(smx_process_t process) * Should be called some time to time to free the memory allocated for processes * that have finished (or killed). */ -void SIMIX_process_empty_trash(void) +void SIMIX_process_empty_trash() { - smx_process_t process = NULL; + smx_process_t process = nullptr; while ((process = (smx_process_t) xbt_swag_extract(simix_global->process_to_destroy))) { XBT_DEBUG("Getting rid of %p",process); + intrusive_ptr_release(process); + } +} - SIMIX_context_free(process->context); - - /* Free the exception allocated at creation time */ - free(process->running_ctx); - xbt_dict_free(&process->properties); +namespace simgrid { +namespace simix { - xbt_fifo_free(process->comms); +Process::~Process() +{ + delete this->context; + if (this->properties) + xbt_dict_free(&this->properties); + if (this->comms != nullptr) + xbt_fifo_free(this->comms); + if (this->on_exit) + xbt_dynar_free(&this->on_exit); +} - xbt_dynar_free(&process->on_exit); +void create_maestro(std::function code) +{ + smx_process_t maestro = nullptr; + /* Create maestro process and intilialize it */ + maestro = new simgrid::simix::Process(); + maestro->pid = simix_process_maxpid++; + maestro->ppid = -1; + maestro->name = ""; + maestro->data = nullptr; - xbt_free(process->name); - xbt_free(process); + if (!code) { + maestro->context = SIMIX_context_new(std::function(), nullptr, maestro); + } else { + if (!simix_global) + xbt_die("simix is not initialized, please call MSG_init first"); + maestro->context = + simix_global->context_factory->create_maestro(code, maestro); } + + maestro->simcall.issuer = maestro; + simix_global->maestro_process = maestro; +} + +} } /** * \brief Creates and runs the maestro process */ -void SIMIX_create_maestro_process() +void SIMIX_maestro_create(void (*code)(void*), void* data) { - smx_process_t maestro = NULL; - - /* Create maestro process and intilialize it */ - maestro = xbt_new0(s_smx_process_t, 1); - maestro->pid = simix_process_maxpid++; - maestro->ppid = -1; - maestro->name = (char *) ""; - maestro->running_ctx = (xbt_running_ctx_t*) xbt_malloc0(sizeof(xbt_running_ctx_t)); - XBT_RUNNING_CTX_INITIALIZE(maestro->running_ctx); - maestro->context = SIMIX_context_new(NULL, 0, NULL, NULL, maestro); - maestro->simcall.issuer = maestro; - simix_global->maestro_process = maestro; - return; + simgrid::simix::create_maestro(std::bind(code, data)); } + /** * \brief Stops a process. * @@ -163,63 +213,21 @@ void SIMIX_create_maestro_process() * and stops its context. */ void SIMIX_process_stop(smx_process_t arg) { + arg->finished = true; /* execute the on_exit functions */ SIMIX_process_on_exit_runall(arg); - /* Add the process to the list of process to restart, only if - * the host is down - */ - if (arg->auto_restart && !sg_host_get_state(arg->host)) { - SIMIX_host_add_auto_restart_process(arg->host,arg->name,arg->code, arg->data, + /* Add the process to the list of process to restart, only if the host is down */ + if (arg->auto_restart && arg->host->isOff()) { + SIMIX_host_add_auto_restart_process(arg->host, arg->name.c_str(), + arg->code, arg->data, sg_host_get_name(arg->host), SIMIX_timer_get_date(arg->kill_timer), - arg->argc,arg->argv,arg->properties, + arg->properties, arg->auto_restart); } - XBT_DEBUG("Process %s (%s) is dead",arg->name,sg_host_get_name(arg->host)); - /* stop the context */ - SIMIX_context_stop(arg->context); -} - -/** - * \brief Same as SIMIX_process_create() but with only one argument (used by timers). - * This function frees the argument. - * \return the process created - */ -smx_process_t SIMIX_process_create_from_wrapper(smx_process_arg_t args) { - - smx_process_t process = simix_global->create_process_function( - args->name, - args->code, - args->data, - args->hostname, - args->kill_time, - args->argc, - args->argv, - args->properties, - args->auto_restart, - NULL); - xbt_free(args); - return process; -} - - -void* simcall_HANDLER_process_create(smx_simcall_t simcall, - const char *name, - xbt_main_func_t code, - void *data, - const char *hostname, - double kill_time, - int argc, char **argv, - xbt_dict_t properties, - int auto_restart){ - return (void*)SIMIX_process_create(name, code, data, hostname, - kill_time, argc, argv, properties, auto_restart, - simcall->issuer); -} - -static void kill_process(void* process) -{ - simix_global->kill_process_function((smx_process_t) process); + XBT_DEBUG("Process %s (%s) is dead", + arg->name.c_str(), sg_host_get_name(arg->host)); + arg->context->stop(); } /** @@ -233,35 +241,31 @@ static void kill_process(void* process) */ smx_process_t SIMIX_process_create( const char *name, - xbt_main_func_t code, + std::function code, void *data, const char *hostname, double kill_time, - int argc, char **argv, xbt_dict_t properties, int auto_restart, smx_process_t parent_process) { - smx_process_t process = NULL; + smx_process_t process = nullptr; sg_host_t host = sg_host_by_name(hostname); XBT_DEBUG("Start process %s on host '%s'", name, hostname); - if (!sg_host_get_state(host)) { - int i; + if (host->isOff()) { XBT_WARN("Cannot launch process '%s' on failed host '%s'", name, hostname); - for (i = 0; i < argc; i++) - xbt_free(argv[i]); - xbt_free(argv); + return nullptr; } else { - process = xbt_new0(s_smx_process_t, 1); + process = new simgrid::simix::Process(); - xbt_assert(((code != NULL) && (host != NULL)), "Invalid parameters"); + xbt_assert(code && host != nullptr, "Invalid parameters"); /* Process data */ process->pid = simix_process_maxpid++; - process->name = xbt_strdup(name); + process->name = simgrid::xbt::string(name); process->host = host; process->data = data; process->comms = xbt_fifo_new(); @@ -269,17 +273,18 @@ smx_process_t SIMIX_process_create( /* Initiliaze data segment to default value */ SIMIX_segment_index_set(process, -1); - if (parent_process != NULL) { - process->ppid = SIMIX_process_get_PID(parent_process); - /* SMPI process have their own data segment and - each other inherit from their father */ - if(smpi_privatize_global_variables){ - if( parent_process->pid != 0){ + if (parent_process != nullptr) { + process->ppid = parent_process->pid; + /* SMPI process have their own data segment and each other inherit from their father */ +#if HAVE_SMPI + if( smpi_privatize_global_variables) { + if (parent_process->pid != 0) { SIMIX_segment_index_set(process, parent_process->segment_index); } else { SIMIX_segment_index_set(process, process->pid - 1); } } +#endif } else { process->ppid = -1; } @@ -287,19 +292,11 @@ smx_process_t SIMIX_process_create( /* Process data for auto-restart */ process->auto_restart = auto_restart; process->code = code; - process->argc = argc; - process->argv = argv; - - XBT_VERB("Create context %s", process->name); - process->context = SIMIX_context_new(code, argc, argv, simix_global->cleanup_process_function, process); - - process->running_ctx = (xbt_running_ctx_t*) xbt_malloc0(sizeof(xbt_running_ctx_t)); - XBT_RUNNING_CTX_INITIALIZE(process->running_ctx); - - if(MC_is_active()){ - MC_ignore_heap(process->running_ctx, sizeof(*process->running_ctx)); - } + XBT_VERB("Create context %s", process->name.c_str()); + process->context = SIMIX_context_new( + std::move(code), + simix_global->cleanup_process_function, process); /* Add properties */ process->properties = properties; @@ -307,22 +304,123 @@ smx_process_t SIMIX_process_create( /* Add the process to it's host process list */ xbt_swag_insert(process, sg_host_simix(host)->process_list); - XBT_DEBUG("Start context '%s'", process->name); + XBT_DEBUG("Start context '%s'", process->name.c_str()); /* Now insert it in the global process list and in the process to run list */ xbt_swag_insert(process, simix_global->process_list); - XBT_DEBUG("Inserting %s(%s) in the to_run list", process->name, sg_host_get_name(host)); + XBT_DEBUG("Inserting %s(%s) in the to_run list", + process->name.c_str(), sg_host_get_name(host)); xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, process); if (kill_time > SIMIX_get_clock() && simix_global->kill_process_function) { - XBT_DEBUG("Process %s(%s) will be kill at time %f", process->name, - sg_host_get_name(process->host), kill_time); - process->kill_timer = SIMIX_timer_set(kill_time, kill_process, process); + XBT_DEBUG("Process %s(%s) will be kill at time %f", + process->name.c_str(), sg_host_get_name(process->host), kill_time); + process->kill_timer = SIMIX_timer_set(kill_time, [=]() { + simix_global->kill_process_function(process); + }); + } + + /* Tracing the process creation */ + TRACE_msg_process_create(process->name.c_str(), process->pid, process->host); + } + return process; +} + +smx_process_t SIMIX_process_attach( + const char* name, + void *data, + const char* hostname, + xbt_dict_t properties, + smx_process_t parent_process) +{ + // This is mostly a copy/paste from SIMIX_process_new(), + // it'd be nice to share some code between those two functions. + + sg_host_t host = sg_host_by_name(hostname); + XBT_DEBUG("Attach process %s on host '%s'", name, hostname); + + if (host->isOff()) { + XBT_WARN("Cannot launch process '%s' on failed host '%s'", + name, hostname); + return nullptr; + } + + smx_process_t process = new simgrid::simix::Process(); + /* Process data */ + process->pid = simix_process_maxpid++; + process->name = std::string(name); + process->host = host; + process->data = data; + process->comms = xbt_fifo_new(); + process->simcall.issuer = process; + process->ppid = -1; + /* Initiliaze data segment to default value */ + SIMIX_segment_index_set(process, -1); + if (parent_process != nullptr) { + process->ppid = parent_process->pid; + /* SMPI process have their own data segment and each other inherit from their father */ +#if HAVE_SMPI + if (smpi_privatize_global_variables) { + if (parent_process->pid != 0) { + SIMIX_segment_index_set(process, parent_process->segment_index); + } else { + SIMIX_segment_index_set(process, process->pid - 1); + } } +#endif } + + /* Process data for auto-restart */ + process->auto_restart = false; + process->code = nullptr; + + XBT_VERB("Create context %s", process->name.c_str()); + if (!simix_global) + xbt_die("simix is not initialized, please call MSG_init first"); + process->context = simix_global->context_factory->attach( + simix_global->cleanup_process_function, process); + + /* Add properties */ + process->properties = properties; + + /* Add the process to it's host process list */ + xbt_swag_insert(process, sg_host_simix(host)->process_list); + + /* Now insert it in the global process list and in the process to run list */ + xbt_swag_insert(process, simix_global->process_list); + XBT_DEBUG("Inserting %s(%s) in the to_run list", + process->name.c_str(), sg_host_get_name(host)); + xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, process); + + /* Tracing the process creation */ + TRACE_msg_process_create(process->name.c_str(), process->pid, process->host); + + auto context = dynamic_cast(process->context); + if (!context) + xbt_die("Not a suitable context"); + + context->attach_start(); return process; } +void SIMIX_process_detach() +{ + auto context = dynamic_cast(SIMIX_context_self()); + if (!context) + xbt_die("Not a suitable context"); + + simix_global->cleanup_process_function(context->process()); + + // Let maestro ignore we are still alive: + // xbt_swag_remove(context->process(), simix_global->process_list); + + // TODDO, Remove from proces list: + // xbt_swag_remove(process, sg_host_simix(host)->process_list); + + context->attach_stop(); + // delete context; +} + /** * \brief Executes the processes from simix_global->process_to_run. * @@ -332,7 +430,7 @@ smx_process_t SIMIX_process_create( * The two lists are swapped so, be careful when using them before and after a * call to this function. */ -void SIMIX_process_runall(void) +void SIMIX_process_runall() { SIMIX_context_runall(); @@ -356,56 +454,63 @@ void simcall_HANDLER_process_kill(smx_simcall_t simcall, smx_process_t process) */ void SIMIX_process_kill(smx_process_t process, smx_process_t issuer) { - XBT_DEBUG("Killing process %s on %s", process->name, sg_host_get_name(process->host)); + XBT_DEBUG("Killing process %s on %s", + process->name.c_str(), sg_host_get_name(process->host)); process->context->iwannadie = 1; process->blocked = 0; process->suspended = 0; - process->doexception = 0; + process->exception = nullptr; /* destroy the blocking synchro if any */ if (process->waiting_synchro) { - switch (process->waiting_synchro->type) { + simgrid::kernel::activity::Exec *exec = dynamic_cast(process->waiting_synchro); + simgrid::kernel::activity::Comm *comm = dynamic_cast(process->waiting_synchro); + simgrid::kernel::activity::Sleep *sleep = dynamic_cast(process->waiting_synchro); + simgrid::kernel::activity::Raw *raw = dynamic_cast(process->waiting_synchro); + simgrid::kernel::activity::Io *io = dynamic_cast(process->waiting_synchro); - case SIMIX_SYNC_EXECUTE: - case SIMIX_SYNC_PARALLEL_EXECUTE: - SIMIX_process_execution_destroy(process->waiting_synchro); - break; + if (exec != nullptr) { + exec->unref(); - case SIMIX_SYNC_COMMUNICATE: + } else if (comm != nullptr) { xbt_fifo_remove(process->comms, process->waiting_synchro); - SIMIX_comm_cancel(process->waiting_synchro); - xbt_fifo_remove(process->waiting_synchro->simcalls, &process->simcall); - SIMIX_comm_destroy(process->waiting_synchro); - break; + comm->cancel(); - case SIMIX_SYNC_SLEEP: - SIMIX_process_sleep_destroy(process->waiting_synchro); - break; + // Remove first occurrence of &process->simcall: + auto i = boost::range::find( + process->waiting_synchro->simcalls, + &process->simcall); + if (i != process->waiting_synchro->simcalls.end()) + process->waiting_synchro->simcalls.remove(&process->simcall); - case SIMIX_SYNC_JOIN: + comm->unref(); + + } else if (sleep != nullptr) { SIMIX_process_sleep_destroy(process->waiting_synchro); - break; - case SIMIX_SYNC_SYNCHRO: + } else if (raw != nullptr) { SIMIX_synchro_stop_waiting(process, &process->simcall); - SIMIX_synchro_destroy(process->waiting_synchro); - break; + delete process->waiting_synchro; - case SIMIX_SYNC_IO: + } else if (io != nullptr) { SIMIX_io_destroy(process->waiting_synchro); - break; - } - process->waiting_synchro = NULL; + /* + switch (process->waiting_synchro->type) { + case SIMIX_SYNC_JOIN: + SIMIX_process_sleep_destroy(process->waiting_synchro); + break; + } */ + + process->waiting_synchro = nullptr; } if(!xbt_dynar_member(simix_global->process_to_run, &(process)) && process != issuer) { - XBT_DEBUG("Inserting %s in the to_run list", process->name); + XBT_DEBUG("Inserting %s in the to_run list", process->name.c_str()); xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, process); } - } /** @brief Ask another process to raise the given exception @@ -423,38 +528,37 @@ void SIMIX_process_throw(smx_process_t process, xbt_errcat_t cat, int value, con /* cancel the blocking synchro if any */ if (process->waiting_synchro) { - switch (process->waiting_synchro->type) { - - case SIMIX_SYNC_EXECUTE: - case SIMIX_SYNC_PARALLEL_EXECUTE: - SIMIX_process_execution_cancel(process->waiting_synchro); - break; + simgrid::kernel::activity::Exec *exec = dynamic_cast(process->waiting_synchro); + if (exec != nullptr) { + SIMIX_execution_cancel(process->waiting_synchro); + } - case SIMIX_SYNC_COMMUNICATE: - xbt_fifo_remove(process->comms, process->waiting_synchro); - SIMIX_comm_cancel(process->waiting_synchro); - break; + simgrid::kernel::activity::Comm *comm = dynamic_cast(process->waiting_synchro); + if (comm != nullptr) { + xbt_fifo_remove(process->comms, comm); + comm->cancel(); + } - case SIMIX_SYNC_SLEEP: - case SIMIX_SYNC_JOIN: + simgrid::kernel::activity::Sleep *sleep = dynamic_cast(process->waiting_synchro); + if (sleep != nullptr) { SIMIX_process_sleep_destroy(process->waiting_synchro); if (!xbt_dynar_member(simix_global->process_to_run, &(process)) && process != SIMIX_process_self()) { - XBT_DEBUG("Inserting %s in the to_run list", process->name); + XBT_DEBUG("Inserting %s in the to_run list", process->name.c_str()); xbt_dynar_push_as(simix_global->process_to_run, smx_process_t, process); } - break; + } - case SIMIX_SYNC_SYNCHRO: + simgrid::kernel::activity::Raw *raw = dynamic_cast(process->waiting_synchro); + if (raw != nullptr) { SIMIX_synchro_stop_waiting(process, &process->simcall); - break; + } - case SIMIX_SYNC_IO: + simgrid::kernel::activity::Io *io = dynamic_cast(process->waiting_synchro); + if (io != nullptr) { SIMIX_io_destroy(process->waiting_synchro); - break; - } } - process->waiting_synchro = NULL; + process->waiting_synchro = nullptr; } @@ -467,7 +571,7 @@ void simcall_HANDLER_process_killall(smx_simcall_t simcall, int reset_pid) { */ void SIMIX_process_killall(smx_process_t issuer, int reset_pid) { - smx_process_t p = NULL; + smx_process_t p = nullptr; while ((p = (smx_process_t) xbt_swag_extract(simix_global->process_list))) { if (p != issuer) { @@ -487,10 +591,9 @@ void simcall_HANDLER_process_set_host(smx_simcall_t simcall, smx_process_t proce { process->new_host = dest; } -void SIMIX_process_change_host(smx_process_t process, - sg_host_t dest) +void SIMIX_process_change_host(smx_process_t process, sg_host_t dest) { - xbt_assert((process != NULL), "Invalid parameters"); + xbt_assert((process != nullptr), "Invalid parameters"); xbt_swag_remove(process, sg_host_simix(process->host)->process_list); process->host = dest; xbt_swag_insert(process, sg_host_simix(dest)->process_list); @@ -499,67 +602,38 @@ void SIMIX_process_change_host(smx_process_t process, void simcall_HANDLER_process_suspend(smx_simcall_t simcall, smx_process_t process) { - smx_synchro_t sync_suspend = - SIMIX_process_suspend(process, simcall->issuer); + smx_synchro_t sync_suspend = SIMIX_process_suspend(process, simcall->issuer); if (process != simcall->issuer) { SIMIX_simcall_answer(simcall); } else { - xbt_fifo_push(sync_suspend->simcalls, simcall); + sync_suspend->simcalls.push_back(simcall); process->waiting_synchro = sync_suspend; - SIMIX_host_execution_suspend(process->waiting_synchro); + process->waiting_synchro->suspend(); } /* If we are suspending ourselves, then just do not finish the simcall now */ } smx_synchro_t SIMIX_process_suspend(smx_process_t process, smx_process_t issuer) { - xbt_assert((process != NULL), "Invalid parameters"); - if (process->suspended) { - XBT_DEBUG("Process '%s' is already suspended", process->name); - return NULL; + XBT_DEBUG("Process '%s' is already suspended", process->name.c_str()); + return nullptr; } process->suspended = 1; - /* If we are suspending another process, and it is waiting on a sync, - suspend its synchronization. */ + /* If we are suspending another process that is waiting on a sync, suspend its synchronization. */ if (process != issuer) { - if (process->waiting_synchro) { - - switch (process->waiting_synchro->type) { + if (process->waiting_synchro) + process->waiting_synchro->suspend(); + /* If the other process is not waiting, its suspension is delayed to when the process is rescheduled. */ - case SIMIX_SYNC_EXECUTE: - case SIMIX_SYNC_PARALLEL_EXECUTE: - SIMIX_host_execution_suspend(process->waiting_synchro); - break; - - case SIMIX_SYNC_COMMUNICATE: - SIMIX_comm_suspend(process->waiting_synchro); - break; - - case SIMIX_SYNC_SLEEP: - SIMIX_process_sleep_suspend(process->waiting_synchro); - break; - - case SIMIX_SYNC_SYNCHRO: - /* Suspension is delayed to when the process is rescheduled. */ - break; - - default: - xbt_die("Internal error in SIMIX_process_suspend: unexpected synchronization type %d", - (int)process->waiting_synchro->type); - } - return NULL; - } else { - /* Suspension is delayed to when the process is rescheduled. */ - return NULL; - } + return nullptr; } else { /* FIXME: computation size is zero. Is it okay that bound is zero ? */ - return SIMIX_process_execute(process, "suspend", 0.0, 1.0, 0.0, 0); + return SIMIX_execution_start(process, "suspend", 0.0, 1.0, 0.0, 0); } } @@ -584,73 +658,43 @@ void SIMIX_process_resume(smx_process_t process, smx_process_t issuer) if (process != issuer) { if (process->waiting_synchro) { - - switch (process->waiting_synchro->type) { - - case SIMIX_SYNC_EXECUTE: - case SIMIX_SYNC_PARALLEL_EXECUTE: - SIMIX_host_execution_resume(process->waiting_synchro); - break; - - case SIMIX_SYNC_COMMUNICATE: - SIMIX_comm_resume(process->waiting_synchro); - break; - - case SIMIX_SYNC_SLEEP: - SIMIX_process_sleep_resume(process->waiting_synchro); - break; - - case SIMIX_SYNC_SYNCHRO: - /* I cannot resume it now. This is delayed to when the process is rescheduled at - * the end of the synchro. */ - break; - - default: - xbt_die("Internal error in SIMIX_process_resume: unexpected synchronization type %d", - (int)process->waiting_synchro->type); - } + process->waiting_synchro->resume(); } } else XBT_WARN("Strange. Process %p is trying to resume himself.", issuer); XBT_OUT(); } -int SIMIX_process_get_maxpid(void) { +int SIMIX_process_get_maxpid() { return simix_process_maxpid; } -int SIMIX_process_count(void) +int SIMIX_process_count() { return xbt_swag_size(simix_global->process_list); } -int SIMIX_process_get_PID(smx_process_t self){ - if (self == NULL) +int SIMIX_process_get_PID(smx_process_t self) +{ + if (self == nullptr) return 0; else return self->pid; } -int SIMIX_process_get_PPID(smx_process_t self){ - if (self == NULL) - return 0; - else - return self->ppid; -} - -void* SIMIX_process_self_get_data(smx_process_t self) +void* SIMIX_process_self_get_data() { - xbt_assert(self == SIMIX_process_self(), "This is not the current process"); + smx_process_t self = SIMIX_process_self(); if (!self) { - return NULL; + return nullptr; } return SIMIX_process_get_data(self); } -void SIMIX_process_self_set_data(smx_process_t self, void *data) +void SIMIX_process_self_set_data(void *data) { - xbt_assert(self == SIMIX_process_self(), "This is not the current process"); + smx_process_t self = SIMIX_process_self(); SIMIX_process_set_data(self, data); } @@ -665,41 +709,25 @@ void SIMIX_process_set_data(smx_process_t process, void *data) process->data = data; } -sg_host_t SIMIX_process_get_host(smx_process_t process) -{ - return process->host; -} - -xbt_main_func_t SIMIX_process_get_code(void){ - return SIMIX_process_self()->code; -} - /* needs to be public and without simcall because it is called by exceptions and logging events */ -const char* SIMIX_process_self_get_name(void) { +const char* SIMIX_process_self_get_name() { smx_process_t process = SIMIX_process_self(); - if (process == NULL || process == simix_global->maestro_process) - return ""; - - return SIMIX_process_get_name(process); -} + if (process == nullptr || process == simix_global->maestro_process) + return "maestro"; -const char* SIMIX_process_get_name(smx_process_t process) -{ - return process->name; + return process->name.c_str(); } smx_process_t SIMIX_process_get_by_name(const char* name) { smx_process_t proc; - - xbt_swag_foreach(proc, simix_global->process_list) - { - if(!strcmp(name, proc->name)) + xbt_swag_foreach(proc, simix_global->process_list) { + if (proc->name == name) return proc; } - return NULL; + return nullptr; } int SIMIX_process_is_suspended(smx_process_t process) @@ -714,19 +742,28 @@ xbt_dict_t SIMIX_process_get_properties(smx_process_t process) void simcall_HANDLER_process_join(smx_simcall_t simcall, smx_process_t process, double timeout) { + if (process->finished) { + // The joined process is already finished, just wake up the issuer process right away + simcall_process_sleep__set__result(simcall, SIMIX_DONE); + SIMIX_simcall_answer(simcall); + return; + } smx_synchro_t sync = SIMIX_process_join(simcall->issuer, process, timeout); - xbt_fifo_push(sync->simcalls, simcall); + sync->simcalls.push_back(simcall); simcall->issuer->waiting_synchro = sync; } -static int SIMIX_process_join_finish(smx_process_exit_status_t status, smx_synchro_t sync){ - if (sync->sleep.surf_sleep) { - surf_action_cancel(sync->sleep.surf_sleep); +static int SIMIX_process_join_finish(smx_process_exit_status_t status, smx_synchro_t synchro){ + simgrid::kernel::activity::Sleep *sleep = static_cast(synchro); + + if (sleep->surf_sleep) { + sleep->surf_sleep->cancel(); - smx_simcall_t simcall; - while ((simcall = (smx_simcall_t) xbt_fifo_shift(sync->simcalls))) { + while (!sleep->simcalls.empty()) { + smx_simcall_t simcall = sleep->simcalls.front(); + sleep->simcalls.pop_front(); simcall_process_sleep__set__result(simcall, SIMIX_DONE); - simcall->issuer->waiting_synchro = NULL; + simcall->issuer->waiting_synchro = nullptr; if (simcall->issuer->suspended) { XBT_DEBUG("Wait! This process is suspended and can't wake up now."); simcall->issuer->suspended = 0; @@ -735,17 +772,17 @@ static int SIMIX_process_join_finish(smx_process_exit_status_t status, smx_synch SIMIX_simcall_answer(simcall); } } - surf_action_unref(sync->sleep.surf_sleep); - sync->sleep.surf_sleep = NULL; + sleep->surf_sleep->unref(); + sleep->surf_sleep = nullptr; } - xbt_mallocator_release(simix_global->synchro_mallocator, sync); + sleep->unref(); return 0; } smx_synchro_t SIMIX_process_join(smx_process_t issuer, smx_process_t process, double timeout) { smx_synchro_t res = SIMIX_process_sleep(issuer, timeout); - res->type = SIMIX_SYNC_JOIN; + static_cast(res)->ref(); SIMIX_process_on_exit(process, (int_f_pvoid_pvoid_t)SIMIX_process_join_finish, res); return res; } @@ -759,7 +796,7 @@ void simcall_HANDLER_process_sleep(smx_simcall_t simcall, double duration) return; } smx_synchro_t sync = SIMIX_process_sleep(simcall->issuer, duration); - xbt_fifo_push(sync->simcalls, simcall); + sync->simcalls.push_back(simcall); simcall->issuer->waiting_synchro = sync; } @@ -768,89 +805,28 @@ smx_synchro_t SIMIX_process_sleep(smx_process_t process, double duration) sg_host_t host = process->host; /* check if the host is active */ - if (surf_host_get_state(surf_host_resource_priv(host)) != SURF_RESOURCE_ON) { - THROWF(host_error, 0, "Host %s failed, you cannot call this function", - sg_host_get_name(host)); - } + if (host->isOff()) + THROWF(host_error, 0, "Host %s failed, you cannot call this function", sg_host_get_name(host)); - smx_synchro_t synchro = (smx_synchro_t) xbt_mallocator_get(simix_global->synchro_mallocator); - synchro->type = SIMIX_SYNC_SLEEP; - synchro->name = NULL; - synchro->category = NULL; - - synchro->sleep.host = host; - synchro->sleep.surf_sleep = surf_host_sleep(host, duration); - - surf_action_set_data(synchro->sleep.surf_sleep, synchro); + simgrid::kernel::activity::Sleep *synchro = new simgrid::kernel::activity::Sleep(); + synchro->host = host; + synchro->surf_sleep = surf_host_sleep(host, duration); + synchro->surf_sleep->setData(synchro); XBT_DEBUG("Create sleep synchronization %p", synchro); return synchro; } -void SIMIX_post_process_sleep(smx_synchro_t synchro) -{ - smx_simcall_t simcall; - e_smx_state_t state; - xbt_assert(synchro->type == SIMIX_SYNC_SLEEP || synchro->type == SIMIX_SYNC_JOIN); - - while ((simcall = (smx_simcall_t) xbt_fifo_shift(synchro->simcalls))) { - - switch(surf_action_get_state(synchro->sleep.surf_sleep)){ - case SURF_ACTION_FAILED: - simcall->issuer->context->iwannadie = 1; - //SMX_EXCEPTION(simcall->issuer, host_error, 0, "Host failed"); - state = SIMIX_SRC_HOST_FAILURE; - break; - - case SURF_ACTION_DONE: - state = SIMIX_DONE; - break; - - default: - THROW_IMPOSSIBLE; - break; - } - if (surf_host_get_state(surf_host_resource_priv(simcall->issuer->host)) != SURF_RESOURCE_ON) { - simcall->issuer->context->iwannadie = 1; - } - simcall_process_sleep__set__result(simcall, state); - simcall->issuer->waiting_synchro = NULL; - if (simcall->issuer->suspended) { - XBT_DEBUG("Wait! This process is suspended and can't wake up now."); - simcall->issuer->suspended = 0; - simcall_HANDLER_process_suspend(simcall, simcall->issuer); - } else { - SIMIX_simcall_answer(simcall); - } - } - - SIMIX_process_sleep_destroy(synchro); -} - void SIMIX_process_sleep_destroy(smx_synchro_t synchro) { XBT_DEBUG("Destroy synchro %p", synchro); - xbt_assert(synchro->type == SIMIX_SYNC_SLEEP || synchro->type == SIMIX_SYNC_JOIN); + simgrid::kernel::activity::Sleep *sleep = static_cast(synchro); - if (synchro->sleep.surf_sleep) { - surf_action_unref(synchro->sleep.surf_sleep); - synchro->sleep.surf_sleep = NULL; + if (sleep->surf_sleep) { + sleep->surf_sleep->unref(); + sleep->surf_sleep = nullptr; + sleep->unref(); } - if (synchro->type == SIMIX_SYNC_SLEEP) - xbt_mallocator_release(simix_global->synchro_mallocator, synchro); -} - -void SIMIX_process_sleep_suspend(smx_synchro_t synchro) -{ - xbt_assert(synchro->type == SIMIX_SYNC_SLEEP); - surf_action_suspend(synchro->sleep.surf_sleep); -} - -void SIMIX_process_sleep_resume(smx_synchro_t synchro) -{ - XBT_DEBUG("Synchro state is %d on process_sleep_resume.", synchro->state); - xbt_assert(synchro->type == SIMIX_SYNC_SLEEP); - surf_action_resume(synchro->sleep.surf_sleep); } /** @@ -863,17 +839,17 @@ void SIMIX_process_sleep_resume(smx_synchro_t synchro) */ void SIMIX_process_yield(smx_process_t self) { - XBT_DEBUG("Yield process '%s'", self->name); + XBT_DEBUG("Yield process '%s'", self->name.c_str()); /* Go into sleep and return control to maestro */ - SIMIX_context_suspend(self->context); + self->context->suspend(); /* Ok, maestro returned control to us */ - XBT_DEBUG("Control returned to me: '%s'", self->name); + XBT_DEBUG("Control returned to me: '%s'", self->name.c_str()); if (self->new_host) { SIMIX_process_change_host(self, self->new_host); - self->new_host = NULL; + self->new_host = nullptr; } if (self->context->iwannadie){ @@ -883,15 +859,16 @@ void SIMIX_process_yield(smx_process_t self) if (self->suspended) { XBT_DEBUG("Hey! I'm suspended."); - xbt_assert(!self->doexception, "Gasp! This exception may be lost by subsequent calls."); + xbt_assert(self->exception != nullptr, "Gasp! This exception may be lost by subsequent calls."); self->suspended = 0; SIMIX_process_suspend(self, self); } - if (self->doexception) { + if (self->exception != nullptr) { XBT_DEBUG("Wait, maestro left me an exception"); - self->doexception = 0; - SMX_THROW(); + std::exception_ptr exception = std::move(self->exception); + self->exception = nullptr; + std::rethrow_exception(std::move(exception)); } if(SMPI_switch_data_segment && self->segment_index != -1){ @@ -899,12 +876,6 @@ void SIMIX_process_yield(smx_process_t self) } } -/* callback: context fetching */ -xbt_running_ctx_t *SIMIX_process_get_running_context(void) -{ - return SIMIX_process_self()->running_ctx; -} - /* callback: termination */ void SIMIX_process_exception_terminate(xbt_ex_t * e) { @@ -923,7 +894,7 @@ void SIMIX_process_set_context(smx_process_t p,smx_context_t c) { /** * \brief Returns the list of processes to run. */ -xbt_dynar_t SIMIX_process_get_runnable(void) +xbt_dynar_t SIMIX_process_get_runnable() { return simix_global->process_to_run; } @@ -935,27 +906,25 @@ smx_process_t SIMIX_process_from_PID(int PID) { smx_process_t proc; xbt_swag_foreach(proc, simix_global->process_list) { - if (proc->pid == (unsigned long) PID) + if (proc->pid == static_cast (PID)) return proc; } - return NULL; + return nullptr; } -/** @brief returns a dynar containg all currently existing processes */ -xbt_dynar_t SIMIX_processes_as_dynar(void) { +/** @brief returns a dynar containing all currently existing processes */ +xbt_dynar_t SIMIX_processes_as_dynar() { smx_process_t proc; - xbt_dynar_t res = xbt_dynar_new(sizeof(smx_process_t),NULL); + xbt_dynar_t res = xbt_dynar_new(sizeof(smx_process_t),nullptr); xbt_swag_foreach(proc, simix_global->process_list) { xbt_dynar_push(res,&proc); } return res; } - void SIMIX_process_on_exit_runall(smx_process_t process) { s_smx_process_exit_fun_t exit_fun; - smx_process_exit_status_t exit_status = (process->context->iwannadie) ? - SMX_EXIT_FAILURE : SMX_EXIT_SUCCESS; + smx_process_exit_status_t exit_status = (process->context->iwannadie) ? SMX_EXIT_FAILURE : SMX_EXIT_SUCCESS; while (!xbt_dynar_is_empty(process->on_exit)) { exit_fun = xbt_dynar_pop_as(process->on_exit,s_smx_process_exit_fun_t); (exit_fun.fun)((void*)exit_status, exit_fun.arg); @@ -966,7 +935,7 @@ void SIMIX_process_on_exit(smx_process_t process, int_f_pvoid_pvoid_t fun, void xbt_assert(process, "current process not found: are you in maestro context ?"); if (!process->on_exit) { - process->on_exit = xbt_dynar_new(sizeof(s_smx_process_exit_fun_t), NULL); + process->on_exit = xbt_dynar_new(sizeof(s_smx_process_exit_fun_t), nullptr); } s_smx_process_exit_fun_t exit_fun = {fun, data}; @@ -988,55 +957,82 @@ smx_process_t simcall_HANDLER_process_restart(smx_simcall_t simcall, smx_process } /** @brief Restart a process, starting it again from the beginning. */ smx_process_t SIMIX_process_restart(smx_process_t process, smx_process_t issuer) { - XBT_DEBUG("Restarting process %s on %s", process->name, sg_host_get_name(process->host)); + XBT_DEBUG("Restarting process %s on %s", process->name.c_str(), sg_host_get_name(process->host)); + //retrieve the arguments of the old process //FIXME: Factorize this with SIMIX_host_add_auto_restart_process ? - s_smx_process_arg_t arg; + simgrid::simix::ProcessArg arg; + arg.name = process->name; arg.code = process->code; arg.hostname = sg_host_get_name(process->host); arg.kill_time = SIMIX_timer_get_date(process->kill_timer); - arg.argc = process->argc; arg.data = process->data; - int i; - arg.argv = xbt_new(char*,process->argc + 1); - for (i = 0; i < arg.argc; i++) { - arg.argv[i] = xbt_strdup(process->argv[i]); - } - arg.argv[process->argc] = NULL; - arg.properties = NULL; + arg.properties = nullptr; arg.auto_restart = process->auto_restart; + //kill the old process - SIMIX_process_kill(process,issuer); - //start the new process - smx_process_t new_process; - if (simix_global->create_process_function) { - new_process = simix_global->create_process_function( - arg.argv[0], - arg.code, - arg.data, - arg.hostname, - arg.kill_time, - arg.argc, - arg.argv, - arg.properties, - arg.auto_restart, - NULL); - } else { - new_process = simcall_process_create( - arg.argv[0], - arg.code, - arg.data, - arg.hostname, - arg.kill_time, - arg.argc, - arg.argv, - arg.properties, - arg.auto_restart); + SIMIX_process_kill(process, issuer); - } - return new_process; + //start the new process + if (simix_global->create_process_function) + return simix_global->create_process_function(arg.name.c_str(), std::move(arg.code), arg.data, arg.hostname, + arg.kill_time, arg.properties, arg.auto_restart, nullptr); + else + return simcall_process_create(arg.name.c_str(), std::move(arg.code), arg.data, arg.hostname, arg.kill_time, + arg.properties, arg.auto_restart); } void SIMIX_segment_index_set(smx_process_t proc, int index){ proc->segment_index = index; } + +/** + * \ingroup simix_process_management + * \brief Creates and runs a new SIMIX process. + * + * The structure and the corresponding thread are created and put in the list of ready processes. + * + * \param name a name for the process. It is for user-level information and can be nullptr. + * \param code the main function of the process + * \param data a pointer to any data one may want to attach to the new object. It is for user-level information and can be nullptr. + * It can be retrieved with the function \ref simcall_process_get_data. + * \param hostname name of the host where the new agent is executed. + * \param kill_time time when the process is killed + * \param argc first argument passed to \a code + * \param argv second argument passed to \a code + * \param properties the properties of the process + * \param auto_restart either it is autorestarting or not. + */ +smx_process_t simcall_process_create(const char *name, + xbt_main_func_t code, + void *data, + const char *hostname, + double kill_time, + int argc, char **argv, + xbt_dict_t properties, + int auto_restart) +{ + if (name == nullptr) + name = ""; + auto wrapped_code = simgrid::xbt::wrapMain(code, argc, argv); + for (int i = 0; i != argc; ++i) + xbt_free(argv[i]); + xbt_free(argv); + smx_process_t res = simcall_process_create(name, + std::move(wrapped_code), + data, hostname, kill_time, properties, auto_restart); + return res; +} + +smx_process_t simcall_process_create( + const char *name, std::function code, void *data, + const char *hostname, double kill_time, + xbt_dict_t properties, int auto_restart) +{ + if (name == nullptr) + name = ""; + smx_process_t self = SIMIX_process_self(); + return simgrid::simix::kernelImmediate([&] { + return SIMIX_process_create(name, std::move(code), data, hostname, kill_time, properties, auto_restart, self); + }); +}