X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/4052b6d9960bd9792127d006c4f359b946cb7baa..ebc355d0c96552d0bc2aa301d90723490337bca3:/src/s4u/s4u_Actor.cpp diff --git a/src/s4u/s4u_Actor.cpp b/src/s4u/s4u_Actor.cpp index 1267fdb683..fa4677e9f7 100644 --- a/src/s4u/s4u_Actor.cpp +++ b/src/s4u/s4u_Actor.cpp @@ -1,13 +1,19 @@ -/* Copyright (c) 2006-2018. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2006-2019. The SimGrid Team. All rights reserved. */ /* 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 "simgrid/Exception.hpp" #include "simgrid/actor.h" #include "simgrid/s4u/Actor.hpp" #include "simgrid/s4u/Exec.hpp" #include "simgrid/s4u/Host.hpp" +#include "simgrid/s4u/VirtualMachine.hpp" +#include "src/kernel/activity/ExecImpl.hpp" #include "src/simix/smx_private.hpp" +#include "src/surf/HostImpl.hpp" + +#include #include XBT_LOG_NEW_DEFAULT_CATEGORY(s4u_actor, "S4U actors"); @@ -18,6 +24,8 @@ namespace s4u { simgrid::xbt::signal s4u::Actor::on_creation; simgrid::xbt::signal s4u::Actor::on_suspend; simgrid::xbt::signal s4u::Actor::on_resume; +simgrid::xbt::signal s4u::Actor::on_sleep; +simgrid::xbt::signal s4u::Actor::on_wake_up; simgrid::xbt::signal s4u::Actor::on_migration_start; simgrid::xbt::signal s4u::Actor::on_migration_end; simgrid::xbt::signal s4u::Actor::on_destruction; @@ -25,25 +33,39 @@ simgrid::xbt::signal s4u::Actor::on_destruction; // ***** Actor creation ***** ActorPtr Actor::self() { - smx_context_t self_context = SIMIX_context_self(); + smx_context_t self_context = simgrid::kernel::context::Context::self(); if (self_context == nullptr) return simgrid::s4u::ActorPtr(); - return self_context->process()->iface(); + return self_context->get_actor()->iface(); +} +ActorPtr Actor::init(const std::string& name, s4u::Host* host) +{ + smx_actor_t self = SIMIX_process_self(); + kernel::actor::ActorImpl* actor = simix::simcall([self, &name, host] { return self->init(name, host).get(); }); + return actor->ciface(); +} + +ActorPtr Actor::start(const std::function& code) +{ + simgrid::simix::simcall([this, &code] { pimpl_->start(code); }); + return this; } -ActorPtr Actor::create(const char* name, s4u::Host* host, std::function code) +ActorPtr Actor::create(const std::string& name, s4u::Host* host, const std::function& code) { - simgrid::kernel::actor::ActorImpl* actor = simcall_process_create(name, std::move(code), nullptr, host, nullptr); + smx_actor_t self = SIMIX_process_self(); + kernel::actor::ActorImpl* actor = + simix::simcall([self, &name, host, &code] { return self->init(name, host)->start(code); }); + return actor->iface(); } -ActorPtr Actor::create(const char* name, s4u::Host* host, const char* function, std::vector args) +ActorPtr Actor::create(const std::string& name, s4u::Host* host, const std::string& function, + std::vector args) { simgrid::simix::ActorCodeFactory& factory = SIMIX_get_actor_code_factory(function); - simgrid::simix::ActorCode code = factory(std::move(args)); - simgrid::kernel::actor::ActorImpl* actor = simcall_process_create(name, std::move(code), nullptr, host, nullptr); - return actor->iface(); + return create(name, host, factory(std::move(args))); } void intrusive_ptr_add_ref(Actor* actor) @@ -69,23 +91,34 @@ void Actor::join(double timeout) void Actor::set_auto_restart(bool autorestart) { - simgrid::simix::simcall([this, autorestart]() { pimpl_->auto_restart = autorestart; }); + simgrid::simix::simcall([this, autorestart]() { + xbt_assert(autorestart && not pimpl_->has_to_auto_restart()); // FIXME: handle all cases + pimpl_->set_auto_restart(autorestart); + + simgrid::kernel::actor::ProcessArg* arg = new simgrid::kernel::actor::ProcessArg(pimpl_->get_host(), pimpl_); + XBT_DEBUG("Adding Process %s to the actors_at_boot_ list of Host %s", arg->name.c_str(), arg->host->get_cname()); + pimpl_->get_host()->pimpl_->actors_at_boot_.emplace_back(arg); + }); } -void Actor::on_exit(int_f_pvoid_pvoid_t fun, void* data) +void Actor::on_exit(int_f_pvoid_pvoid_t fun, + void* data) /* deprecated: cleanup SIMIX_process_on_exit: change prototype of second parameter and + remove the last one */ { simgrid::simix::simcall([this, fun, data] { SIMIX_process_on_exit(pimpl_, fun, data); }); } -/** @brief Moves the actor to another host - * - * If the actor is currently blocked on an execution activity, the activity is also - * migrated to the new host. If it's blocked on another kind of activity, an error is - * raised as the mandated code is not written yet. Please report that bug if you need it. - * - * Asynchronous activities started by the actor are not migrated automatically, so you have - * to take care of this yourself (only you knows which ones should be migrated). - */ +void Actor::on_exit(const std::function& fun, void* data) /* deprecated */ +{ + on_exit([fun, data](bool exit) { fun(exit, data); }); +} + +void Actor::on_exit(const std::function& fun) +{ + simgrid::simix::simcall( + [this, fun] { SIMIX_process_on_exit(pimpl_, [fun](int a, void* /*data*/) { fun(a != 0); }, nullptr); }); +} + void Actor::migrate(Host* new_host) { s4u::Actor::on_migration_start(this); @@ -99,7 +132,7 @@ void Actor::migrate(Host* new_host) xbt_assert(exec.get() != nullptr, "We can only migrate blocked actors when they are blocked on executions."); exec->migrate(new_host); } - SIMIX_process_change_host(this->pimpl_, new_host); + this->pimpl_->set_host(new_host); }); s4u::Actor::on_migration_end(this); @@ -107,7 +140,7 @@ void Actor::migrate(Host* new_host) s4u::Host* Actor::get_host() { - return this->pimpl_->host; + return this->pimpl_->get_host(); } void Actor::daemonize() @@ -117,7 +150,7 @@ void Actor::daemonize() bool Actor::is_daemon() const { - return this->pimpl_->isDaemon(); + return this->pimpl_->is_daemon(); } const simgrid::xbt::string& Actor::get_name() const @@ -132,12 +165,12 @@ const char* Actor::get_cname() const aid_t Actor::get_pid() const { - return this->pimpl_->pid; + return this->pimpl_->get_pid(); } aid_t Actor::get_ppid() const { - return this->pimpl_->ppid; + return this->pimpl_->get_ppid(); } void Actor::suspend() @@ -152,28 +185,28 @@ void Actor::resume() s4u::Actor::on_resume(this); } -int Actor::is_suspended() +bool Actor::is_suspended() { - return simgrid::simix::simcall([this] { return pimpl_->suspended; }); + return simgrid::simix::simcall([this] { return pimpl_->is_suspended(); }); } -void Actor::set_kill_time(double time) +void Actor::set_kill_time(double kill_time) { - simcall_process_set_kill_time(pimpl_, time); + simgrid::simix::simcall([this, kill_time] { pimpl_->set_kill_time(kill_time); }); } -/** \brief Get the kill time of an actor(or 0 if unset). */ +/** @brief Get the kill time of an actor(or 0 if unset). */ double Actor::get_kill_time() { - return SIMIX_timer_get_date(pimpl_->kill_timer); + return pimpl_->get_kill_time(); } -void Actor::kill(aid_t pid) +void Actor::kill(aid_t pid) // deprecated { smx_actor_t killer = SIMIX_process_self(); - smx_actor_t process = SIMIX_process_from_PID(pid); - if (process != nullptr) { - simgrid::simix::simcall([killer, process] { SIMIX_process_kill(process, killer); }); + smx_actor_t victim = SIMIX_process_from_PID(pid); + if (victim != nullptr) { + simgrid::simix::simcall([killer, victim] { killer->kill(victim); }); } else { std::ostringstream oss; oss << "kill: (" << pid << ") - No such actor" << std::endl; @@ -184,8 +217,12 @@ void Actor::kill(aid_t pid) void Actor::kill() { smx_actor_t process = SIMIX_process_self(); - simgrid::simix::simcall( - [this, process] { SIMIX_process_kill(pimpl_, (pimpl_ == simix_global->maestro_process) ? pimpl_ : process); }); + simgrid::simix::simcall([this, process] { + if (pimpl_ == simix_global->maestro_process) + pimpl_->exit(); + else + process->kill(pimpl_); + }); } smx_actor_t Actor::get_impl() @@ -207,23 +244,23 @@ ActorPtr Actor::by_pid(aid_t pid) void Actor::kill_all() { smx_actor_t self = SIMIX_process_self(); - simgrid::simix::simcall([&self] { SIMIX_process_killall(self); }); + simgrid::simix::simcall([self] { self->kill_all(); }); } -std::map* Actor::get_properties() +std::unordered_map* Actor::get_properties() { - return simgrid::simix::simcall([this] { return this->pimpl_->getProperties(); }); + return simgrid::simix::simcall([this] { return this->pimpl_->get_properties(); }); } /** Retrieve the property value (or nullptr if not set) */ -const char* Actor::get_property(const char* key) +const char* Actor::get_property(const std::string& key) { - return simgrid::simix::simcall([this, key] { return pimpl_->getProperty(key); }); + return simgrid::simix::simcall([this, &key] { return pimpl_->get_property(key); }); } -void Actor::set_property(const char* key, const char* value) +void Actor::set_property(const std::string& key, const std::string& value) { - simgrid::simix::simcall([this, key, value] { pimpl_->setProperty(key, value); }); + simgrid::simix::simcall([this, &key, &value] { pimpl_->set_property(key, value); }); } Actor* Actor::restart() @@ -252,8 +289,14 @@ bool is_maestro() void sleep_for(double duration) { - if (duration > 0) + if (duration > 0) { + smx_actor_t actor = SIMIX_process_self(); + simgrid::s4u::Actor::on_sleep(actor->iface()); + simcall_process_sleep(duration); + + simgrid::s4u::Actor::on_wake_up(actor->iface()); + } } void yield() @@ -265,38 +308,81 @@ XBT_PUBLIC void sleep_until(double timeout) { double now = SIMIX_get_clock(); if (timeout > now) - simcall_process_sleep(timeout - now); + sleep_for(timeout - now); } void execute(double flops) { - get_host()->execute(flops); + execute(flops, 1.0 /* priority */); } void execute(double flops, double priority) { - get_host()->execute(flops, priority); + exec_init(flops)->set_priority(priority)->start()->wait(); } -void parallel_execute(int host_nb, s4u::Host** host_list, double* flops_amount, double* bytes_amount, double timeout) +void parallel_execute(const std::vector& hosts, const std::vector& flops_amounts, + const std::vector& bytes_amounts) +{ + parallel_execute(hosts, flops_amounts, bytes_amounts, -1); +} + +void parallel_execute(const std::vector& hosts, const std::vector& flops_amounts, + const std::vector& bytes_amounts, double timeout) +{ + xbt_assert(hosts.size() > 0, "Your parallel executions must span over at least one host."); + xbt_assert(hosts.size() == flops_amounts.size() || flops_amounts.empty(), + "Host count (%zu) does not match flops_amount count (%zu).", hosts.size(), flops_amounts.size()); + xbt_assert(hosts.size() * hosts.size() == bytes_amounts.size() || bytes_amounts.empty(), + "bytes_amounts must be a matrix of size host_count * host_count (%zu*%zu), but it's of size %zu.", + hosts.size(), hosts.size(), flops_amounts.size()); + /* Check that we are not mixing VMs and PMs in the parallel task */ + bool is_a_vm = (nullptr != dynamic_cast(hosts.front())); + xbt_assert(std::all_of(hosts.begin(), hosts.end(), + [is_a_vm](s4u::Host* elm) { + bool tmp_is_a_vm = (nullptr != dynamic_cast(elm)); + return is_a_vm == tmp_is_a_vm; + }), + "parallel_execute: mixing VMs and PMs is not supported (yet)."); + /* checking for infinite values */ + xbt_assert(std::all_of(flops_amounts.begin(), flops_amounts.end(), [](double elm) { return std::isfinite(elm); }), + "flops_amounts comprises infinite values!"); + xbt_assert(std::all_of(bytes_amounts.begin(), bytes_amounts.end(), [](double elm) { return std::isfinite(elm); }), + "flops_amounts comprises infinite values!"); + + exec_init(hosts, flops_amounts, bytes_amounts)->set_timeout(timeout)->wait(); +} + +// deprecated +void parallel_execute(int host_nb, s4u::Host* const* host_list, const double* flops_amount, const double* bytes_amount, + double timeout) { smx_activity_t s = - simcall_execution_parallel_start(nullptr, host_nb, host_list, flops_amount, bytes_amount, /* rate */ -1, timeout); + simcall_execution_parallel_start("", host_nb, host_list, flops_amount, bytes_amount, /* rate */ -1, timeout); simcall_execution_wait(s); + delete[] flops_amount; + delete[] bytes_amount; } -void parallel_execute(int host_nb, sg_host_t* host_list, double* flops_amount, double* bytes_amount) +// deprecated +void parallel_execute(int host_nb, s4u::Host* const* host_list, const double* flops_amount, const double* bytes_amount) { - parallel_execute(host_nb, host_list, flops_amount, bytes_amount, /* timeout */ -1); + smx_activity_t s = simcall_execution_parallel_start("", host_nb, host_list, flops_amount, bytes_amount, + /* rate */ -1, /*timeout*/ -1); + simcall_execution_wait(s); + delete[] flops_amount; + delete[] bytes_amount; } ExecPtr exec_init(double flops_amount) { - ExecPtr res = ExecPtr(new Exec()); - res->host_ = get_host(); - res->flops_amount_ = flops_amount; - res->set_remaining(flops_amount); - return res; + return ExecPtr(new ExecSeq(get_host(), flops_amount)); +} + +ExecPtr exec_init(const std::vector& hosts, const std::vector& flops_amounts, + const std::vector& bytes_amounts) +{ + return ExecPtr(new ExecPar(hosts, flops_amounts, bytes_amounts)); } ExecPtr exec_async(double flops) @@ -308,12 +394,12 @@ ExecPtr exec_async(double flops) aid_t get_pid() { - return SIMIX_process_self()->pid; + return SIMIX_process_self()->get_pid(); } aid_t get_ppid() { - return SIMIX_process_self()->ppid; + return SIMIX_process_self()->get_ppid(); } std::string get_name() @@ -328,7 +414,7 @@ const char* get_cname() Host* get_host() { - return SIMIX_process_self()->host; + return SIMIX_process_self()->get_host(); } void suspend() @@ -346,21 +432,20 @@ void resume() simgrid::s4u::Actor::on_resume(process->iface()); } -bool is_suspended() +void exit() { - smx_actor_t process = SIMIX_process_self(); - return simgrid::simix::simcall([process] { return process->suspended; }); + smx_actor_t actor = SIMIX_process_self(); + simgrid::simix::simcall([actor] { actor->exit(); }); } -void kill() +void on_exit(const std::function& fun) { - smx_actor_t process = SIMIX_process_self(); - simgrid::simix::simcall([process] { SIMIX_process_kill(process, process); }); + SIMIX_process_self()->iface()->on_exit(fun); } -void on_exit(int_f_pvoid_pvoid_t fun, void* data) +void on_exit(const std::function& fun, void* data) /* deprecated */ { - SIMIX_process_self()->iface()->on_exit(fun, data); + SIMIX_process_self()->iface()->on_exit([fun, data](bool exit) { fun(exit, data); }); } /** @brief Moves the current actor to another host @@ -396,13 +481,17 @@ Host* getHost() /* deprecated */ { return get_host(); } -bool isSuspended() /* deprecated */ +void on_exit(int_f_pvoid_pvoid_t fun, void* data) /* deprecated */ { - return is_suspended(); + SIMIX_process_self()->iface()->on_exit([fun, data](int a) { fun((void*)(intptr_t)a, data); }); } -void onExit /* deprecated */ (int_f_pvoid_pvoid_t fun, void* data) +void onExit(int_f_pvoid_pvoid_t fun, void* data) /* deprecated */ { - on_exit(fun, data); + on_exit([fun, data](int a) { fun((void*)(intptr_t)a, data); }); +} +void kill() /* deprecated */ +{ + exit(); } } // namespace this_actor @@ -411,12 +500,12 @@ void onExit /* deprecated */ (int_f_pvoid_pvoid_t fun, void* data) /* **************************** Public C interface *************************** */ -/** \ingroup m_actor_management - * \brief Returns the process ID of \a actor. +/** @ingroup m_actor_management + * @brief Returns the process ID of @a actor. * - * This function checks whether \a actor is a valid pointer and return its PID (or 0 in case of problem). + * This function checks whether @a actor is a valid pointer and return its PID (or 0 in case of problem). */ -int sg_actor_get_PID(sg_actor_t actor) +aid_t sg_actor_get_PID(sg_actor_t actor) { /* Do not raise an exception here: this function is called by the logs * and the exceptions, so it would be called back again and again */ @@ -425,23 +514,23 @@ int sg_actor_get_PID(sg_actor_t actor) return actor->get_pid(); } -/** \ingroup m_actor_management - * \brief Returns the process ID of the parent of \a actor. +/** @ingroup m_actor_management + * @brief Returns the process ID of the parent of @a actor. * - * This function checks whether \a actor is a valid pointer and return its parent's PID. + * This function checks whether @a actor is a valid pointer and return its parent's PID. * Returns -1 if the actor has not been created by any other actor. */ -int sg_actor_get_PPID(sg_actor_t actor) +aid_t sg_actor_get_PPID(sg_actor_t actor) { return actor->get_ppid(); } -/** \ingroup m_actor_management +/** @ingroup m_actor_management * - * \brief Return a #sg_actor_t given its PID. + * @brief Return a #sg_actor_t given its PID. * - * This function search in the list of all the created sg_actor_t for a sg_actor_t whose PID is equal to \a PID. - * If none is found, \c nullptr is returned. + * This function search in the list of all the created sg_actor_t for a sg_actor_t whose PID is equal to @a PID. + * If none is found, @c nullptr is returned. Note that the PID are unique in the whole simulation, not only on a given host. */ sg_actor_t sg_actor_by_PID(aid_t pid) @@ -449,8 +538,8 @@ sg_actor_t sg_actor_by_PID(aid_t pid) return simgrid::s4u::Actor::by_pid(pid).get(); } -/** \ingroup m_actor_management - * \brief Return the name of an actor. +/** @ingroup m_actor_management + * @brief Return the name of an actor. */ const char* sg_actor_get_name(sg_actor_t actor) { @@ -462,20 +551,20 @@ sg_host_t sg_actor_get_host(sg_actor_t actor) return actor->get_host(); } -/** \ingroup m_actor_management - * \brief Returns the value of a given actor property +/** @ingroup m_actor_management + * @brief Returns the value of a given actor property * - * \param actor an actor - * \param name a property name - * \return value of a property (or nullptr if the property is not set) + * @param actor an actor + * @param name a property name + * @return value of a property (or nullptr if the property is not set) */ const char* sg_actor_get_property_value(sg_actor_t actor, const char* name) { return actor->get_property(name); } -/** \ingroup m_actor_management - * \brief Return the list of properties +/** @ingroup m_actor_management + * @brief Return the list of properties * * This function returns all the parameters associated with an actor */ @@ -483,17 +572,17 @@ xbt_dict_t sg_actor_get_properties(sg_actor_t actor) { xbt_assert(actor != nullptr, "Invalid parameter: First argument must not be nullptr"); xbt_dict_t as_dict = xbt_dict_new_homogeneous(xbt_free_f); - std::map* props = actor->get_properties(); + std::unordered_map* props = actor->get_properties(); if (props == nullptr) return nullptr; - for (auto const& elm : *props) { - xbt_dict_set(as_dict, elm.first.c_str(), xbt_strdup(elm.second.c_str()), nullptr); + for (auto const& kv : *props) { + xbt_dict_set(as_dict, kv.first.c_str(), xbt_strdup(kv.second.c_str()), nullptr); } return as_dict; } -/** \ingroup m_actor_management - * \brief Suspend the actor. +/** @ingroup m_actor_management + * @brief Suspend the actor. * * This function suspends the actor by suspending the task on which it was waiting for the completion. */ @@ -503,8 +592,8 @@ void sg_actor_suspend(sg_actor_t actor) actor->suspend(); } -/** \ingroup m_actor_management - * \brief Resume a suspended actor. +/** @ingroup m_actor_management + * @brief Resume a suspended actor. * * This function resumes a suspended actor by resuming the task on which it was waiting for the completion. */ @@ -514,8 +603,8 @@ void sg_actor_resume(sg_actor_t actor) actor->resume(); } -/** \ingroup m_actor_management - * \brief Returns true if the actor is suspended . +/** @ingroup m_actor_management + * @brief Returns true if the actor is suspended . * * This checks whether an actor is suspended or not by inspecting the task on which it was waiting for the completion. */ @@ -525,14 +614,24 @@ int sg_actor_is_suspended(sg_actor_t actor) } /** - * \ingroup m_actor_management - * \brief Restarts an actor from the beginning. + * @ingroup m_actor_management + * @brief Restarts an actor from the beginning. */ sg_actor_t sg_actor_restart(sg_actor_t actor) { return actor->restart(); } +/** + * @ingroup m_actor_management + * @brief Sets the "auto-restart" flag of the actor. + * If the flag is set to 1, the actor will be automatically restarted when its host comes back up. + */ +void sg_actor_set_auto_restart(sg_actor_t actor, int auto_restart) +{ + actor->set_auto_restart(auto_restart); +} + /** @ingroup m_actor_management * @brief This actor will be terminated automatically when the last non-daemon actor finishes */ @@ -541,21 +640,21 @@ void sg_actor_daemonize(sg_actor_t actor) actor->daemonize(); } -/** \ingroup m_actor_management - * \brief Migrates an actor to another location. +/** @ingroup m_actor_management + * @brief Migrates an actor to another location. * - * This function changes the value of the #sg_host_t on which \a actor is running. + * This function changes the value of the #sg_host_t on which @a actor is running. */ void sg_actor_migrate(sg_actor_t process, sg_host_t host) { process->migrate(host); } -/** \ingroup m_actor_management - * \brief Wait for the completion of a #sg_actor_t. +/** @ingroup m_actor_management + * @brief Wait for the completion of a #sg_actor_t. * - * \param actor the actor to wait for - * \param timeout wait until the actor is over, or the timeout expires + * @param actor the actor to wait for + * @param timeout wait until the actor is over, or the timeout expires */ void sg_actor_join(sg_actor_t actor, double timeout) { @@ -572,11 +671,11 @@ void sg_actor_kill_all() simgrid::s4u::Actor::kill_all(); } -/** \ingroup m_actor_management - * \brief Set the kill time of an actor. +/** @ingroup m_actor_management + * @brief Set the kill time of an actor. * - * \param actor an actor - * \param kill_time the time when the actor is killed. + * @param actor an actor + * @param kill_time the time when the actor is killed. */ void sg_actor_set_kill_time(sg_actor_t actor, double kill_time) { @@ -588,3 +687,36 @@ void sg_actor_yield() { simgrid::s4u::this_actor::yield(); } + +void sg_actor_sleep_for(double duration) +{ + simgrid::s4u::this_actor::sleep_for(duration); +} + +sg_actor_t sg_actor_attach(const char* name, void* data, sg_host_t host, xbt_dict_t properties) +{ + xbt_assert(host != nullptr, "Invalid parameters: host and code params must not be nullptr"); + 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); + + /* Let's create the process: SIMIX may decide to start it right now, even before returning the flow control to us */ + smx_actor_t actor = nullptr; + try { + actor = simgrid::kernel::actor::ActorImpl::attach(name, data, host, &props).get(); + } catch (simgrid::HostFailureException const&) { + xbt_die("Could not attach"); + } + + simgrid::s4u::this_actor::yield(); + return actor->ciface(); +} + +void sg_actor_detach() +{ + simgrid::kernel::actor::ActorImpl::detach(); +}