X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/efcde007ad6d08be411ca5b3d2bcfe4fa0615a7f..cabfcca7cb25b7aa54d7ee1b0a8b117940b85a8d:/include/simgrid/s4u/actor.hpp diff --git a/include/simgrid/s4u/actor.hpp b/include/simgrid/s4u/actor.hpp index fc067f1551..28dbdc4031 100644 --- a/include/simgrid/s4u/actor.hpp +++ b/include/simgrid/s4u/actor.hpp @@ -6,17 +6,29 @@ #ifndef SIMGRID_S4U_ACTOR_HPP #define SIMGRID_S4U_ACTOR_HPP +#include +#include +#include +#include #include +#include +#include +#include +#include + +#include + #include +#include + +#include #include #include namespace simgrid { namespace s4u { -/** @addtogroup s4u_actor - * - * @tableofcontents +/** @ingroup s4u_api * * An actor is an independent stream of execution in your distributed application. * @@ -115,41 +127,89 @@ namespace s4u { * * @{ */ - + /** @brief Simulation Agent (see \ref s4u_actor)*/ XBT_PUBLIC_CLASS Actor { + friend Mailbox; + friend simgrid::simix::Process; + smx_process_t pimpl_ = nullptr; + + /** Wrap a (possibly non-copyable) single-use task into a `std::function` */ + template + static std::function wrap_task(F f, Args... args) + { + typedef decltype(f(std::move(args)...)) R; + auto task = std::make_shared>( + simgrid::xbt::makeTask(std::move(f), std::move(args)...)); + return [=] { + (*task)(); + }; + } + + Actor(smx_process_t pimpl) : pimpl_(pimpl) {} + public: - Actor() : pimpl_(nullptr) {} - Actor(smx_process_t smx_proc) : - pimpl_(SIMIX_process_ref(smx_proc)) {} - ~Actor() + + // ***** No copy ***** + + Actor(Actor const&) = delete; + Actor& operator=(Actor const&) = delete; + + // ***** Reference count (delegated to pimpl_) ***** + + friend void intrusive_ptr_add_ref(Actor* actor) { - SIMIX_process_unref(pimpl_); + xbt_assert(actor != nullptr); + SIMIX_process_ref(actor->pimpl_); } + friend void intrusive_ptr_release(Actor* actor) + { + xbt_assert(actor != nullptr); + SIMIX_process_unref(actor->pimpl_); + } + using Ptr = boost::intrusive_ptr; + + // ***** Actor creation ***** + + /** Create an actor using a function + * + * If the actor is restarted, the actor has a fresh copy of the function. + */ + static Ptr createActor(const char* name, s4u::Host *host, double killTime, std::function code); - // Copy+move (with the copy-and-swap idiom): - Actor(Actor const& actor) : pimpl_(SIMIX_process_ref(actor.pimpl_)) {} - friend void swap(Actor& first, Actor& second) + static Ptr createActor(const char* name, s4u::Host *host, std::function code) { - using std::swap; - swap(first.pimpl_, second.pimpl_); + return createActor(name, host, -1.0, std::move(code)); } - Actor& operator=(Actor actor) + + /** Create an actor using code + * + * Using this constructor, move-only type can be used. The consequence is + * that we cannot copy the value and restart the process in its initial + * state. In order to use auto-restart, an explicit `function` must be passed + * instead. + */ + template::type + > + static Ptr createActor(const char* name, s4u::Host *host, F code, Args... args) { - swap(*this, actor); - return *this; + return createActor(name, host, wrap_task(std::move(code), std::move(args)...)); } - Actor(Actor&& actor) : pimpl_(nullptr) + + // Create actor from function name: + + static Ptr createActor(const char* name, s4u::Host *host, double killTime, + const char* function, std::vector args); + + static Ptr createActor(const char* name, s4u::Host *host, const char* function, + std::vector args) { - swap(*this, actor); + return createActor(name, host, -1.0, function, std::move(args)); } - Actor(const char* name, s4u::Host *host, double killTime, std::function code); - Actor(const char* name, s4u::Host *host, std::function code) - : Actor(name, host, -1, std::move(code)) {}; - template - Actor(const char* name, s4u::Host *host, C code) - : Actor(name, host, -1, std::function(std::move(code))) {} + // ***** Methods ***** /** Retrieves the actor that have the given PID (or NULL if not existing) */ //static Actor *byPid(int pid); not implemented @@ -176,6 +236,7 @@ public: void kill(); static void kill(int pid); + static Ptr forPid(int pid); /** * Wait for the actor to finish. @@ -187,15 +248,38 @@ public: /** Ask kindly to all actors to die. Only the issuer will survive. */ static void killAll(); -private: - smx_process_t pimpl_ = nullptr; + /** Returns the internal implementation of this actor */ + smx_process_t getImpl(); }; -/** @brief Static methods working on the current actor (see @ref s4u_actor) */ +using ActorPtr = Actor::Ptr; + +/** @ingroup s4u_api + * @brief Static methods working on the current actor (see @ref s4u::Actor) */ namespace this_actor { /** Block the actor sleeping for that amount of seconds (may throws hostFailure) */ - XBT_PUBLIC(void) sleep(double duration); + XBT_PUBLIC(void) sleep_for(double duration); + XBT_PUBLIC(void) sleep_until(double timeout); + + template + inline void sleep_for(std::chrono::duration duration) + { + auto seconds = std::chrono::duration_cast(duration); + this_actor::sleep_for(seconds.count()); + } + template + inline void sleep_until(const SimulationTimePoint& timeout_time) + { + auto timeout_native = std::chrono::time_point_cast(timeout_time); + this_actor::sleep_until(timeout_native.time_since_epoch().count()); + } + + XBT_ATTRIB_DEPRECATED("Use sleep_for()") + inline void sleep(double duration) + { + return sleep_for(duration); + } /** Block the actor, computing the given amount of flops */ XBT_PUBLIC(e_smx_state_t) execute(double flop); @@ -211,6 +295,11 @@ namespace this_actor { * See \ref Comm for the full communication API (including non blocking communications). */ XBT_PUBLIC(void) send(Mailbox &chan, void*payload, size_t simulatedSize); + + /** + * Return the PID of the current actor. + */ + XBT_PUBLIC(int) getPid(); };