X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/4d02714ee138a3bd9b02e0064b7a2aa26407e9e8..82567f27fa74ddc2b6afa8d9b682bdde15fe3f9c:/src/msg/msg_process.cpp diff --git a/src/msg/msg_process.cpp b/src/msg/msg_process.cpp index d95e2db3c4..9cdb6057f5 100644 --- a/src/msg/msg_process.cpp +++ b/src/msg/msg_process.cpp @@ -7,7 +7,6 @@ #include "simgrid/Exception.hpp" #include "simgrid/s4u/Host.hpp" #include "src/instr/instr_private.hpp" -#include "src/simix/ActorImpl.hpp" #include "src/simix/smx_private.hpp" XBT_LOG_NEW_DEFAULT_SUBCATEGORY(msg_process, msg, "Logging specific to MSG (process)"); @@ -18,7 +17,6 @@ std::string instr_pid(msg_process_t proc) } /******************************** Process ************************************/ - /** @brief Creates and runs a new #msg_process_t. * * Does exactly the same as #MSG_process_create_with_arguments but without providing standard arguments @@ -69,39 +67,35 @@ msg_process_t MSG_process_create_with_environment(const char *name, xbt_main_fun int argc, char **argv, xbt_dict_t properties) { xbt_assert(host != nullptr, "Invalid parameters: host param must not be nullptr"); - simgrid::simix::ActorCode function; if (code) function = simgrid::xbt::wrap_main(code, argc, static_cast(argv)); - std::unordered_map props; - xbt_dict_cursor_t cursor = nullptr; - char* key; - char* value; - xbt_dict_foreach (properties, cursor, key, value) - props[key] = value; - xbt_dict_free(&properties); + simgrid::s4u::ActorPtr actor; - smx_actor_t self = SIMIX_process_self(); - smx_actor_t actor = nullptr; try { - actor = simgrid::simix::simcall([name, function, data, host, &props, self] { - return simgrid::kernel::actor::ActorImpl::create(std::move(name), std::move(function), data, host, &props, self) - .get(); - }); + if (data != nullptr) { + actor = simgrid::s4u::Actor::init(std::move(name), host); + actor->extension()->set_user_data(data); + xbt_dict_cursor_t cursor = nullptr; + char* key; + char* value; + xbt_dict_foreach (properties, cursor, key, value) + actor->set_property(key, value); + actor->start(std::move(function)); + } else + actor = simgrid::s4u::Actor::create(std::move(name), host, std::move(function)); } catch (simgrid::HostFailureException const&) { - xbt_die("Could not create a new process on failed host %s.", host->get_cname()); + xbt_die("Could not launch a new process on failed host %s.", host->get_cname()); } + xbt_dict_free(&properties); for (int i = 0; i != argc; ++i) xbt_free(argv[i]); xbt_free(argv); - if (actor == nullptr) - return nullptr; - - MSG_process_yield(); - return actor->ciface(); + simgrid::s4u::this_actor::yield(); + return actor.get(); } /** @brief Returns the user data of a process. @@ -113,7 +107,7 @@ void* MSG_process_get_data(msg_process_t process) xbt_assert(process != nullptr, "Invalid parameter: first parameter must not be nullptr!"); /* get from SIMIX the MSG process data, and then the user data */ - return process->get_impl()->get_user_data(); + return process->extension()->get_user_data(); } /** @brief Sets the user data of a process. @@ -123,8 +117,7 @@ void* MSG_process_get_data(msg_process_t process) msg_error_t MSG_process_set_data(msg_process_t process, void *data) { xbt_assert(process != nullptr, "Invalid parameter: first parameter must not be nullptr!"); - - process->get_impl()->set_user_data(data); + process->extension()->set_user_data(data); return MSG_OK; } @@ -188,7 +181,7 @@ msg_process_t MSG_process_self() } smx_context_t MSG_process_get_smx_ctx(msg_process_t process) { // deprecated -- smx_context_t should die afterward - return process->get_impl()->context_; + return process->get_impl()->context_.get(); } /** @brief Add a function to the list of "on_exit" functions for the current process. * The on_exit functions are the functions executed when your process is killed.