/* 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 <functional>
+
#include <signal.h> /* Signal handling */
#include <stdlib.h>
#include "src/internal_config.h"
XBT_LOG_NEW_CATEGORY(simix, "All SIMIX categories");
XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_kernel, simix, "Logging specific to SIMIX (kernel)");
-smx_global_t simix_global = NULL;
-static xbt_heap_t simix_timers = NULL;
+smx_global_t simix_global = nullptr;
+static xbt_heap_t simix_timers = nullptr;
/** @brief Timer datatype */
typedef struct s_smx_timer {
- double date;
- void(* func)(void*);
- void* args;
+ double date = 0.0;
+ std::packaged_task<void()> callback;
+
+ s_smx_timer() {}
+ s_smx_timer(double date, std::packaged_task<void()> callback)
+ : date(date), callback(std::move(callback)) {}
} s_smx_timer_t;
-void (*SMPI_switch_data_segment)(int) = NULL;
+void (*SMPI_switch_data_segment)(int) = nullptr;
int _sg_do_verbose_exit = 1;
static void inthandler(int ignored)
if (!(old_stack.ss_flags & SS_DISABLE)) {
XBT_DEBUG("An alternate stack was already installed (sp=%p, size=%zd, flags=%x). Restore it.",
old_stack.ss_sp, old_stack.ss_size, old_stack.ss_flags);
- sigaltstack(&old_stack, NULL);
+ sigaltstack(&old_stack, nullptr);
}
struct sigaction action, old_action;
- action.sa_sigaction = segvhandler;
+ action.sa_sigaction = &segvhandler;
action.sa_flags = SA_ONSTACK | SA_RESETHAND | SA_SIGINFO;
sigemptyset(&action.sa_mask);
XBT_DEBUG("A signal handler was already installed for SIGSEGV (%p). Restore it.",
(old_action.sa_flags & SA_SIGINFO) ?
(void*)old_action.sa_sigaction : (void*)old_action.sa_handler);
- sigaction(SIGSEGV, &old_action, NULL);
+ sigaction(SIGSEGV, &old_action, nullptr);
}
}
static void kill_process(smx_process_t process)
{
- SIMIX_process_kill(process, NULL);
+ SIMIX_process_kill(process, nullptr);
}
static std::function<void()> maestro_code;
simgrid::mc::Client::initialize();
#endif
- s_smx_process_t proc;
-
if (!simix_global) {
simix_global = xbt_new0(s_smx_global_t, 1);
- simix_global->process_to_run = xbt_dynar_new(sizeof(smx_process_t), NULL);
- simix_global->process_that_ran = xbt_dynar_new(sizeof(smx_process_t), NULL);
+ simgrid::simix::Process proc;
+ simix_global->process_to_run = xbt_dynar_new(sizeof(smx_process_t), nullptr);
+ simix_global->process_that_ran = xbt_dynar_new(sizeof(smx_process_t), nullptr);
simix_global->process_list = xbt_swag_new(xbt_swag_offset(proc, process_hookup));
simix_global->process_to_destroy = xbt_swag_new(xbt_swag_offset(proc, destroy_hookup));
- simix_global->maestro_process = NULL;
- simix_global->registered_functions = xbt_dict_new_homogeneous(NULL);
+ simix_global->maestro_process = nullptr;
+ simix_global->registered_functions = xbt_dict_new_homogeneous(nullptr);
- simix_global->create_process_function = SIMIX_process_create;
- simix_global->kill_process_function = kill_process;
- simix_global->cleanup_process_function = SIMIX_process_cleanup;
+ simix_global->create_process_function = &SIMIX_process_create;
+ simix_global->kill_process_function = &kill_process;
+ simix_global->cleanup_process_function = &SIMIX_process_cleanup;
simix_global->mutex = xbt_os_mutex_init();
surf_init(argc, argv); /* Initialize SURF structures */
// a context object with the current context mestro):
simgrid::simix::create_maestro(maestro_code);
- /* context exception handlers */
- __xbt_running_ctx_fetch = SIMIX_process_get_running_context;
- __xbt_ex_terminate = SIMIX_process_exception_terminate;
-
/* Prepare to display some more info when dying on Ctrl-C pressing */
signal(SIGINT, inthandler);
const char* name = storage->getName();
// TODO, create sg_storage_by_name
sg_storage_t s = xbt_lib_get_elm_or_null(storage_lib, name);
- xbt_assert(s != NULL, "Storage not found for name %s", name);
+ xbt_assert(s != nullptr, "Storage not found for name %s", name);
- SIMIX_storage_create(name, s, NULL);
+ SIMIX_storage_create(name, s, nullptr);
});
SIMIX_STORAGE_LEVEL = xbt_lib_add_level(storage_lib, SIMIX_storage_destroy);
}
- if (!simix_timers) {
- simix_timers = xbt_heap_new(8, &free);
- }
+ if (!simix_timers)
+ simix_timers = xbt_heap_new(8, [](void* p) {
+ delete static_cast<smx_timer_t>(p);
+ });
if (xbt_cfg_get_boolean("clean-atexit"))
atexit(SIMIX_clean);
void SIMIX_clean(void)
{
if (smx_cleaned) return; // to avoid double cleaning by java and C
+
+#if HAVE_SMPI
+ if (SIMIX_process_count()>0){
+ if(smpi_process_initialized()){
+ xbt_die("Process exited without calling MPI_Finalize - Killing simulation");
+ }else{
+ XBT_WARN("Process called exit when leaving - Skipping cleanups");
+ return;
+ }
+ }
+#endif
+
smx_cleaned = 1;
XBT_DEBUG("SIMIX_clean called. Simulation's over.");
if (!xbt_dynar_is_empty(simix_global->process_to_run) && SIMIX_get_clock() == 0.0) {
SIMIX_mailbox_exit();
xbt_heap_free(simix_timers);
- simix_timers = NULL;
+ simix_timers = nullptr;
/* Free the remaining data structures */
xbt_dynar_free(&simix_global->process_to_run);
xbt_dynar_free(&simix_global->process_that_ran);
xbt_swag_free(simix_global->process_to_destroy);
xbt_swag_free(simix_global->process_list);
- simix_global->process_list = NULL;
- simix_global->process_to_destroy = NULL;
+ simix_global->process_list = nullptr;
+ simix_global->process_to_destroy = nullptr;
xbt_dict_free(&(simix_global->registered_functions));
xbt_os_mutex_destroy(simix_global->mutex);
- simix_global->mutex = NULL;
+ simix_global->mutex = nullptr;
/* Let's free maestro now */
delete simix_global->maestro_process->context;
- xbt_free(simix_global->maestro_process->running_ctx);
- xbt_free(simix_global->maestro_process);
- simix_global->maestro_process = NULL;
-
- /* Restore the default exception setup */
- __xbt_running_ctx_fetch = &__xbt_ex_ctx_default;
- __xbt_ex_terminate = &__xbt_ex_terminate_default;
+ simix_global->maestro_process->context = nullptr;
+ delete simix_global->maestro_process;
+ simix_global->maestro_process = nullptr;
/* Finish context module and SURF */
SIMIX_context_mod_exit();
surf_exit();
xbt_free(simix_global);
- simix_global = NULL;
+ simix_global = nullptr;
return;
}
XBT_DEBUG("Handling process whose action terminated normally");
while ((action = surf_model_extract_done_action_set(model))) {
XBT_DEBUG(" Handling Action %p",action);
- if (action->getData() == NULL)
+ if (action->getData() == nullptr)
XBT_DEBUG("probably vcpu's action %p, skip", action);
else
SIMIX_simcall_exit((smx_synchro_t) action->getData());
//FIXME: make the timers being real callbacks
// (i.e. provide dispatchers that read and expand the args)
timer = (smx_timer_t) xbt_heap_pop(simix_timers);
- if (timer->func)
- timer->func(timer->args);
- xbt_free(timer);
+ try {
+ timer->callback();
+ }
+ catch(...) {
+ xbt_die("Exception throwed ouf of timer callback");
+ }
+ delete timer;
}
/* Wake up all processes waiting for a Surf action to finish */
XBT_DEBUG("Handling process whose action terminated normally");
while ((action = surf_model_extract_done_action_set(model))) {
XBT_DEBUG(" Handling Action %p",action);
- if (action->getData() == NULL)
+ if (action->getData() == nullptr)
XBT_DEBUG("probably vcpu's action %p, skip", action);
else
SIMIX_simcall_exit((smx_synchro_t) action->getData());
}
/* Autorestart all process */
- char *hostname = NULL;
+ char *hostname = nullptr;
xbt_dynar_foreach(host_that_restart,iter,hostname) {
XBT_INFO("Restart processes on host: %s",hostname);
SIMIX_host_autorestart(sg_host_by_name(hostname));
*/
smx_timer_t SIMIX_timer_set(double date, void (*function)(void*), void *arg)
{
- smx_timer_t timer = xbt_new0(s_smx_timer_t, 1);
+ smx_timer_t timer = new s_smx_timer_t(date,
+ std::packaged_task<void()>(std::bind(function, arg)));
+ xbt_heap_push(simix_timers, timer, date);
+ return timer;
+}
- timer->date = date;
- timer->func = function;
- timer->args = arg;
+smx_timer_t SIMIX_timer_set(double date, std::packaged_task<void()> callback)
+{
+ smx_timer_t timer = new s_smx_timer_t(date, std::move(callback));
xbt_heap_push(simix_timers, timer, date);
return timer;
}
+
/** @brief cancels a timer that was added earlier */
void SIMIX_timer_remove(smx_timer_t timer) {
xbt_heap_rm_elm(simix_timers, timer, timer->date);
}
-/** @brief Returns the date at which the timer will trigger (or 0 if NULL timer) */
+/** @brief Returns the date at which the timer will trigger (or 0 if nullptr timer) */
double SIMIX_timer_get_date(smx_timer_t timer) {
return timer?timer->date:0;
}
* to call SIMIX_process_create().
* \param function create process function
*/
-void SIMIX_function_register_process_create(smx_creation_func_t
- function)
+void SIMIX_function_register_process_create(smx_creation_func_t function)
{
simix_global->create_process_function = function;
}
void SIMIX_display_process_status(void)
{
- if (simix_global->process_list == NULL) {
+ if (simix_global->process_list == nullptr) {
return;
}
- smx_process_t process = NULL;
+ smx_process_t process = nullptr;
int nbprocess = xbt_swag_size(simix_global->process_list);
XBT_INFO("%d processes are still running, waiting for something.", nbprocess);
*/
XBT_INFO("Process %lu (%s@%s): waiting for %s synchro %p (%s) in state %d to finish",
- process->pid, process->name, sg_host_get_name(process->host),
+ process->pid, process->name.c_str(), sg_host_get_name(process->host),
synchro_description, process->waiting_synchro,
- process->waiting_synchro->name, (int)process->waiting_synchro->state);
+ process->waiting_synchro->name.c_str(), (int)process->waiting_synchro->state);
}
else {
- XBT_INFO("Process %lu (%s@%s)", process->pid, process->name, sg_host_get_name(process->host));
+ XBT_INFO("Process %lu (%s@%s)", process->pid, process->name.c_str(), sg_host_get_name(process->host));
}
}
}
int SIMIX_is_maestro()
{
- return simix_global==NULL /*SimDag*/|| SIMIX_process_self() == simix_global->maestro_process;
+ return simix_global==nullptr /*SimDag*/|| SIMIX_process_self() == simix_global->maestro_process;
}