X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/d2b02bef7e30bc5239a13c9429a843fb5782a8dc..813e836067a1aa922c5fb1432300b7e6390ee352:/src/bindings/python/simgrid_python.cpp diff --git a/src/bindings/python/simgrid_python.cpp b/src/bindings/python/simgrid_python.cpp index 92992368be..ce60453926 100644 --- a/src/bindings/python/simgrid_python.cpp +++ b/src/bindings/python/simgrid_python.cpp @@ -1,4 +1,4 @@ -/* Copyright (c) 2018-2019. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2018-2020. 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. */ @@ -25,11 +25,13 @@ #include #include #include -#include #include +#include #include #include +#include +#include #include #include #include @@ -43,11 +45,9 @@ using simgrid::s4u::Mailbox; XBT_LOG_NEW_DEFAULT_CATEGORY(python, "python"); -PYBIND11_DECLARE_HOLDER_TYPE(T, boost::intrusive_ptr); - namespace { -static std::string get_simgrid_version() +std::string get_simgrid_version() { int major; int minor; @@ -56,221 +56,302 @@ static std::string get_simgrid_version() return simgrid::xbt::string_printf("%i.%i.%i", major, minor, patch); } -static std::string simgrid_version = get_simgrid_version(); +/* Classes GilScopedAcquire and GilScopedRelease have the same purpose as pybind11::gil_scoped_acquire and + * pybind11::gil_scoped_release. Refer to the manual of pybind11 for details: + * https://pybind11.readthedocs.io/en/stable/advanced/misc.html#global-interpreter-lock-gil + * + * The pybind11 versions are however too sophisticated (using TLS for example) and don't work well with all kinds of + * contexts. + * See also https://github.com/pybind/pybind11/issues/1276, which may be related. + * + * Briefly, GilScopedAcquire can be used on actor creation to acquire a new PyThreadState. The PyThreadState has to be + * released for context switches (i.e. before simcalls). That's the purpose of GilScopedRelease. + * + * Like their pybind11 counterparts, both classes use a RAII pattern. + */ +class XBT_PRIVATE GilScopedAcquire { + static PyThreadState* acquire() + { + PyThreadState* state = PyThreadState_New(PyInterpreterState_Head()); + PyEval_AcquireThread(state); + return state; + } + static void release(PyThreadState* state) + { + PyEval_ReleaseThread(state); + PyThreadState_Clear(state); + PyThreadState_Delete(state); + } + + std::unique_ptr thread_state{acquire(), &release}; + +public: + void reset() { thread_state.reset(); } +}; + +class XBT_PRIVATE GilScopedRelease { + std::unique_ptr thread_state{PyEval_SaveThread(), + &PyEval_RestoreThread}; +}; } // namespace +PYBIND11_DECLARE_HOLDER_TYPE(T, boost::intrusive_ptr) + PYBIND11_MODULE(simgrid, m) { - m.doc() = "SimGrid userspace API"; - m.attr("simgrid_version") = simgrid_version; + m.attr("simgrid_version") = get_simgrid_version(); // Internal exception used to kill actors and sweep the RAII chimney (free objects living on the stack) static py::object pyForcefulKillEx(py::register_exception(m, "ActorKilled")); /* this_actor namespace */ - void (*sleep_for_fun)(double) = &simgrid::s4u::this_actor::sleep_for; // pick the right overload - void (*sleep_until_fun)(double) = &simgrid::s4u::this_actor::sleep_until; - - py::module m2 = m.def_submodule("this_actor", "Bindings of the s4u::this_actor namespace."); - m2.def("info", [](char* s) { XBT_INFO("%s", s); }, "Display a logging message of default priority."); - m2.def("error", [](char* s) { XBT_ERROR("%s", s); }, "Display a logging message of 'error' priority."); - m2.def("execute", py::overload_cast(&simgrid::s4u::this_actor::execute), - "Block the current actor, computing the given amount of flops at the given priority, see :cpp:func:`void " - "simgrid::s4u::this_actor::execute(double, double)`", - py::arg("flops"), py::arg("priority") = 1); - m2.def("exec_init", [](double flops){return simgrid::s4u::this_actor::exec_init(flops);}); - m2.def("get_host", &simgrid::s4u::this_actor::get_host, "Retrieves host on which the current actor is located"); - m2.def("migrate", &simgrid::s4u::this_actor::migrate, "Moves the current actor to another host, see :cpp:func:`void simgrid::s4u::this_actor::migrate()`", - py::arg("dest")); - m2.def("sleep_for", sleep_for_fun, - "Block the actor sleeping for that amount of seconds, see :cpp:func:`void simgrid::s4u::this_actor::sleep_for`", py::arg("duration")); - m2.def("sleep_until", sleep_until_fun, - "Block the actor sleeping until the specified timestamp, see :cpp:func:`void simgrid::s4u::this_actor::sleep_until`", py::arg("duration")); - m2.def("suspend", &simgrid::s4u::this_actor::suspend, "Suspend the current actor, that is blocked until resume()ed by another actor. see :cpp:func:`void simgrid::s4u::this_actor::suspend`"); - m2.def("yield_", &simgrid::s4u::this_actor::yield, - "Yield the actor, see :cpp:func:`void simgrid::s4u::this_actor::yield()`"); - m2.def("exit", &simgrid::s4u::this_actor::exit, "kill the current actor"); - m2.def("on_exit", - [](py::object fun) { - ActorPtr act = Actor::self(); - simgrid::s4u::this_actor::on_exit([act, fun](bool /*failed*/) { - try { - fun(); - } catch (py::error_already_set& e) { - xbt_die("Error while executing the on_exit lambda: %s", e.what()); - } - }); - }, - ""); + m.def_submodule("this_actor", "Bindings of the s4u::this_actor namespace.") + .def( + "info", [](const char* s) { XBT_INFO("%s", s); }, "Display a logging message of 'info' priority.") + .def( + "error", [](const char* s) { XBT_ERROR("%s", s); }, "Display a logging message of 'error' priority.") + .def("execute", py::overload_cast(&simgrid::s4u::this_actor::execute), + py::call_guard(), + "Block the current actor, computing the given amount of flops at the given priority.", py::arg("flops"), + py::arg("priority") = 1) + .def("exec_init", py::overload_cast(&simgrid::s4u::this_actor::exec_init), + py::call_guard()) + .def("get_host", &simgrid::s4u::this_actor::get_host, "Retrieves host on which the current actor is located") + .def("set_host", &simgrid::s4u::this_actor::set_host, py::call_guard(), + "Moves the current actor to another host.", py::arg("dest")) + .def("sleep_for", static_cast(&simgrid::s4u::this_actor::sleep_for), + py::call_guard(), "Block the actor sleeping for that amount of seconds.", + py::arg("duration")) + .def("sleep_until", static_cast(&simgrid::s4u::this_actor::sleep_until), + py::call_guard(), "Block the actor sleeping until the specified timestamp.", + py::arg("duration")) + .def("suspend", &simgrid::s4u::this_actor::suspend, py::call_guard(), + "Suspend the current actor, that is blocked until resume()ed by another actor.") + .def("yield_", &simgrid::s4u::this_actor::yield, py::call_guard(), "Yield the actor") + .def("exit", &simgrid::s4u::this_actor::exit, py::call_guard(), "kill the current actor") + .def( + "on_exit", + [](py::object fun) { + fun.inc_ref(); // FIXME: why is this needed for tests like actor-kill and actor-lifetime? + simgrid::s4u::this_actor::on_exit([fun](bool /*failed*/) { + GilScopedAcquire py_context; // need a new context for callback + try { + fun(); + } catch (const py::error_already_set& e) { + std::string what = e.what(); + py_context.reset(); + xbt_die("Error while executing the on_exit lambda: %s", what.c_str()); + } + }); + }, + py::call_guard(), ""); /* Class Engine */ - py::class_(m, "Engine", "Simulation Engine, see :ref:`class s4u::Engine `") - .def(py::init([](std::vector args) -> simgrid::s4u::Engine* { - static char noarg[] = {'\0'}; - int argc = args.size(); - std::unique_ptr argv(new char*[argc + 1]); - for (int i = 0; i != argc; ++i) - argv[i] = args[i].empty() ? noarg : &args[i].front(); - argv[argc] = nullptr; + py::class_(m, "Engine", "Simulation Engine") + .def(py::init([](std::vector args) { + auto argc = static_cast(args.size()); + std::vector argv(args.size() + 1); // argv[argc] is nullptr + std::transform(begin(args), end(args), begin(argv), [](std::string& s) { return &s.front(); }); // Currently this can be dangling, we should wrap this somehow. - return new simgrid::s4u::Engine(&argc, argv.get()); + return new simgrid::s4u::Engine(&argc, argv.data()); })) - .def_static("get_clock", &Engine::get_clock, "The simulation time, ie the amount of simulated seconds since the simulation start.") + .def_static("get_clock", &Engine::get_clock, + "The simulation time, ie the amount of simulated seconds since the simulation start.") .def("get_all_hosts", &Engine::get_all_hosts, "Returns the list of all hosts found in the platform") - .def("load_platform", &Engine::load_platform, - "Load a platform file describing the environment, see :cpp:func:`simgrid::s4u::Engine::load_platform()`") - .def("load_deployment", &Engine::load_deployment, - "Load a deployment file and launch the actors that it contains, see " - ":cpp:func:`simgrid::s4u::Engine::load_deployment()`") - .def("run", &Engine::run, "Run the simulation") - .def("register_actor", - [](Engine* e, const std::string& name, py::object fun_or_class) { - e->register_actor(name, [fun_or_class](std::vector args) { - try { - /* Convert the std::vector into a py::tuple */ - py::tuple params(args.size() - 1); - for (size_t i = 1; i < args.size(); i++) - params[i - 1] = py::cast(args[i]); + .def("load_platform", &Engine::load_platform, "Load a platform file describing the environment") + .def("load_deployment", &Engine::load_deployment, "Load a deployment file and launch the actors that it contains") + .def("run", &Engine::run, py::call_guard(), "Run the simulation") + .def( + "register_actor", + [](Engine* e, const std::string& name, py::object fun_or_class) { + e->register_actor(name, [fun_or_class](std::vector args) { + GilScopedAcquire py_context; + try { + /* Convert the std::vector into a py::tuple */ + py::tuple params(args.size() - 1); + for (size_t i = 1; i < args.size(); i++) + params[i - 1] = py::cast(args[i]); - py::object res = fun_or_class(*params); + py::object res = fun_or_class(*params); - /* If I was passed a class, I just built an instance, so I need to call it now */ - if (py::isinstance(res)) - res(); - } catch (py::error_already_set& ex) { - if (ex.matches(pyForcefulKillEx)) { - XBT_VERB("Actor killed"); - /* Stop here that ForcefulKill exception which was meant to free the RAII stuff on the stack */ - } else { - throw; - } - } - }); - }, - "Registers the main function of an actor, see :cpp:func:`simgrid::s4u::Engine::register_actor()`"); + /* If I was passed a class, I just built an instance, so I need to call it now */ + if (py::isinstance(res)) + res(); + } catch (const py::error_already_set& ex) { + bool ffk = ex.matches(pyForcefulKillEx); + py_context.reset(); + if (ffk) { + XBT_VERB("Actor killed"); + /* Forward that ForcefulKill exception */ + simgrid::ForcefulKillException::do_throw(); + } + throw; + } + }); + }, + "Registers the main function of an actor"); /* Class Host */ - py::class_>(m, "Host", "Simulation Engine, see :ref:`class s4u::Host `") + py::class_>(m, "Host", "Simulated host") .def("by_name", &Host::by_name, "Retrieves a host from its name, or die") - .def("get_pstate_count", &Host::get_pstate_count, "Retrieve the cound of defined pstate levels, see :cpp:func:`simgrid::s4u::Host::get_pstate_count`") - .def("get_pstate_speed", &Host::get_pstate_speed, "Retrieve the maximal speed at the given pstate, see :cpp:func:`simgrid::s4u::Host::get_pstate_speed`") - .def_property("pstate", &Host::get_pstate, &Host::set_pstate, "The current pstate") - - .def("current", &Host::current, "Retrieves the host on which the running actor is located, see :cpp:func:`simgrid::s4u::Host::current()`") - .def_property_readonly("name", [](Host* self) -> const std::string { - return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC - }, "The name of this host") - .def_property_readonly("load", &Host::get_load, - "Returns the current computation load (in flops per second). This is the currently achieved speed. See :cpp:func:`simgrid::s4u::Host::get_load()`") - .def_property_readonly("speed", &Host::get_speed, - "The peak computing speed in flops/s at the current pstate, taking the external load into account. This is the max potential speed. See :cpp:func:`simgrid::s4u::Host::get_speed()`"); + .def("get_pstate_count", &Host::get_pstate_count, "Retrieve the count of defined pstate levels") + .def("get_pstate_speed", &Host::get_pstate_speed, "Retrieve the maximal speed at the given pstate") + .def_property( + "pstate", &Host::get_pstate, + [](Host* h, int i) { + GilScopedRelease gil_guard; + h->set_pstate(i); + }, + "The current pstate") + .def("current", &Host::current, py::call_guard(), + "Retrieves the host on which the running actor is located.") + .def_property_readonly( + "name", + [](const Host* self) { + return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC + }, + "The name of this host") + .def_property_readonly( + "load", &Host::get_load, + "Returns the current computation load (in flops per second). This is the currently achieved speed.") + .def_property_readonly( + "speed", &Host::get_speed, + "The peak computing speed in flops/s at the current pstate, taking the external load into account. " + "This is the max potential speed."); /* Class Mailbox */ - py::class_>(m, "Mailbox", "Mailbox, see :ref:`class s4u::Mailbox `") - .def("__str__", [](Mailbox* self) -> const std::string { - return std::string("Mailbox(") + self->get_cname() + ")"; - }, "Textual representation of the Mailbox`") - .def("by_name", &Mailbox::by_name, "Retrieve a Mailbox from its name, see :cpp:func:`simgrid::s4u::Mailbox::by_name()`") - .def_property_readonly("name", [](Mailbox* self) -> const std::string { - return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC - }, "The name of that mailbox, see :cpp:func:`simgrid::s4u::Mailbox::get_name()`") - .def("put", [](Mailbox* self, py::object data, int size) { - data.inc_ref(); - self->put(data.ptr(), size); - }, "Blocking data transmission, see :cpp:func:`void simgrid::s4u::Mailbox::put(void*, uint64_t)`") - .def("put_async", [](Mailbox* self, py::object data, int size) -> simgrid::s4u::CommPtr { - data.inc_ref(); - return self->put_async(data.ptr(), size); - }, "Non-blocking data transmission, see :cpp:func:`void simgrid::s4u::Mailbox::put_async(void*, uint64_t)`") - .def("get", [](Mailbox* self) -> py::object { - py::object data = pybind11::reinterpret_steal(static_cast(self->get())); - data.dec_ref(); - return data; - }, "Blocking data reception, see :cpp:func:`void* simgrid::s4u::Mailbox::get()`"); + py::class_>(m, "Mailbox", "Mailbox") + .def( + "__str__", [](const Mailbox* self) { return std::string("Mailbox(") + self->get_cname() + ")"; }, + "Textual representation of the Mailbox`") + .def("by_name", &Mailbox::by_name, py::call_guard(), "Retrieve a Mailbox from its name") + .def_property_readonly( + "name", + [](const Mailbox* self) { + return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC + }, + "The name of that mailbox") + .def( + "put", + [](Mailbox* self, py::object data, int size) { + data.inc_ref(); + self->put(data.ptr(), size); + }, + py::call_guard(), "Blocking data transmission") + .def( + "put_async", + [](Mailbox* self, py::object data, int size) { + data.inc_ref(); + return self->put_async(data.ptr(), size); + }, + py::call_guard(), "Non-blocking data transmission") + .def( + "get", + [](Mailbox* self) { + py::object data = pybind11::reinterpret_steal(self->get()); + // data.dec_ref(); // FIXME: why does it break python-actor-create? + return data; + }, + py::call_guard(), "Blocking data reception") + .def("set_receiver", + [](Mailbox* self, ActorPtr actor) { + self->set_receiver(actor); + }, + py::call_guard(), + "Sets the actor as permanent receiver"); /* Class Comm */ - py::class_(m, "Comm", - "Communication, see :ref:`class s4u::Comm `") - .def("test", [](simgrid::s4u::CommPtr self) { return self->test(); }, - "Test whether the communication is terminated, see :cpp:func:`simgrid::s4u::Comm::test()`") - .def("wait", [](simgrid::s4u::CommPtr self) { self->wait(); }, - "Block until the completion of that communication, see :cpp:func:`simgrid::s4u::Comm::wait()`") - .def("wait_all", [](std::vector* comms) { simgrid::s4u::Comm::wait_all(comms); }, - "Block until the completion of all communications in the list, see " - ":cpp:func:`simgrid::s4u::Comm::wait_all()`") - .def( - "wait_any", [](std::vector* comms) { return simgrid::s4u::Comm::wait_any(comms); }, - "Block until the completion of any communication in the list and return the index of the terminated one, see " - ":cpp:func:`simgrid::s4u::Comm::wait_any()`"); - py::class_(m, "Exec", "Execution, see :ref:`class s4u::Exec `") - .def_property_readonly("remaining", [](simgrid::s4u::ExecPtr self) { return self->get_remaining(); }, - "Amount of flops that remain to be computed until completion, see :cpp:func:`simgrid::s4u::Exec::get_remaining()`") - .def_property_readonly("remaining_ratio", [](simgrid::s4u::ExecPtr self) { return self->get_remaining_ratio(); }, - "Amount of work remaining until completion from 0 (completely done) to 1 (nothing done yet). See :cpp:func:`simgrid::s4u::Exec::get_remaining_ratio()`") - .def_property("host", - [](simgrid::s4u::ExecPtr self) { - simgrid::s4u::ExecSeqPtr seq = boost::dynamic_pointer_cast(self); - if (seq != nullptr) - return seq->get_host(); - xbt_throw_unimplemented(__FILE__, __LINE__, "host of parallel executions is not implemented in python yet."); - }, - [](simgrid::s4u::ExecPtr self, simgrid::s4u::Host* host) { self->set_host(host); }, - "Host on which this execution runs. See :cpp:func:`simgrid::s4u::ExecSeq::get_host()`") - .def("test", [](simgrid::s4u::ExecPtr self) { return self->test(); }, - "Test whether the execution is terminated, see :cpp:func:`simgrid::s4u::Exec::test()`") - .def("cancel", [](simgrid::s4u::ExecPtr self) { self->cancel(); }, - "Cancel that execution, see :cpp:func:`simgrid::s4u::Exec::cancel()`") - .def("start", [](simgrid::s4u::ExecPtr self) { return self->start(); }, - "Start that execution, see :cpp:func:`simgrid::s4u::Exec::start()`") - .def("wait", [](simgrid::s4u::ExecPtr self) { return self->wait(); }, - "Block until the completion of that execution, see :cpp:func:`simgrid::s4u::Exec::wait()`"); + py::class_(m, "Comm", "Communication") + .def("test", &simgrid::s4u::Comm::test, py::call_guard(), + "Test whether the communication is terminated.") + .def("wait", &simgrid::s4u::Comm::wait, py::call_guard(), + "Block until the completion of that communication.") + .def("wait_all", &simgrid::s4u::Comm::wait_all, py::call_guard(), + "Block until the completion of all communications in the list.") + .def("wait_any", &simgrid::s4u::Comm::wait_any, py::call_guard(), + "Block until the completion of any communication in the list and return the index of the terminated one."); + + /* Class Exec */ + py::class_(m, "Exec", "Execution") + .def_property_readonly( + "remaining", + [](simgrid::s4u::ExecPtr self) { + GilScopedRelease gil_guard; + return self->get_remaining(); + }, + "Amount of flops that remain to be computed until completion.") + .def_property_readonly( + "remaining_ratio", + [](simgrid::s4u::ExecPtr self) { + GilScopedRelease gil_guard; + return self->get_remaining_ratio(); + }, + "Amount of work remaining until completion from 0 (completely done) to 1 (nothing done " + "yet).") + .def_property("host", &simgrid::s4u::Exec::get_host, &simgrid::s4u::Exec::set_host, + "Host on which this execution runs. Only the first host is returned for parallel executions.") + .def("test", &simgrid::s4u::Exec::test, py::call_guard(), + "Test whether the execution is terminated.") + .def("cancel", &simgrid::s4u::Exec::cancel, py::call_guard(), "Cancel that execution.") + .def("start", &simgrid::s4u::Exec::start, py::call_guard(), "Start that execution.") + .def("wait", &simgrid::s4u::Exec::wait, py::call_guard(), + "Block until the completion of that execution."); /* Class Actor */ py::class_(m, "Actor", "An actor is an independent stream of execution in your distributed " - "application, see :ref:`class s4u::Actor `") - - .def("create", - [](py::str name, py::object host, py::object fun, py::args args) { - - return simgrid::s4u::Actor::create(name, host.cast(), [fun, args]() { - - try { - fun(*args); - } catch (py::error_already_set& ex) { - if (ex.matches(pyForcefulKillEx)) { - XBT_VERB("Actor killed"); - /* Stop here that ForcefulKill exception which was meant to free the RAII stuff on the stack */ - } else { - throw; - } - } - }); - }, - "Create an actor from a function or an object, see :cpp:func:`simgrid::s4u::Actor::create()`") - .def_property("host", &Actor::get_host, &Actor::migrate, "The host on which this actor is located") + "application") + .def( + "create", + [](py::str name, Host* host, py::object fun, py::args args) { + fun.inc_ref(); // FIXME: why is this needed for tests like exec-async, exec-dvfs and exec-remote? + args.inc_ref(); // FIXME: why is this needed for tests like actor-migrate? + return simgrid::s4u::Actor::create(name, host, [fun, args]() { + GilScopedAcquire py_context; + try { + fun(*args); + } catch (const py::error_already_set& ex) { + bool ffk = ex.matches(pyForcefulKillEx); + py_context.reset(); + if (ffk) { + XBT_VERB("Actor killed"); + /* Forward that ForcefulKill exception */ + simgrid::ForcefulKillException::do_throw(); + } + throw; + } + }); + }, + py::call_guard(), "Create an actor from a function or an object.") + .def_property( + "host", &Actor::get_host, + [](Actor* a, Host* h) { + GilScopedRelease gil_guard; + a->set_host(h); + }, + "The host on which this actor is located") .def_property_readonly("name", &Actor::get_cname, "The name of this actor.") .def_property_readonly("pid", &Actor::get_pid, "The PID (unique identifier) of this actor.") .def_property_readonly("ppid", &Actor::get_ppid, "The PID (unique identifier) of the actor that created this one.") .def("by_pid", &Actor::by_pid, "Retrieve an actor by its PID") - .def("daemonize", &Actor::daemonize, - "This actor will be automatically terminated when the last non-daemon actor finishes, see :cpp:func:`void " - "simgrid::s4u::Actor::daemonize()`") + .def("daemonize", &Actor::daemonize, py::call_guard(), + "This actor will be automatically terminated when the last non-daemon actor finishes (more info in the C++ " + "documentation).") .def("is_daemon", &Actor::is_daemon, "Returns True if that actor is a daemon and will be terminated automatically when the last non-daemon actor " "terminates.") - .def("join", py::overload_cast(&Actor::join), - "Wait for the actor to finish, see :cpp:func:`void simgrid::s4u::Actor::join(double)`", py::arg("timeout")) - .def("kill", [](ActorPtr act) { act->kill(); }, "Kill that actor") - .def("kill_all", &Actor::kill_all, "Kill all actors but the caller.") - .def("migrate", &Actor::migrate, - "Moves that actor to another host, see :cpp:func:`void simgrid::s4u::Actor::migrate()`", py::arg("dest")) - .def("self", &Actor::self, "Retrieves the current actor, see :cpp:func:`void simgrid::s4u::Actor::self()`") + .def("join", py::overload_cast(&Actor::join, py::const_), py::call_guard(), + "Wait for the actor to finish (more info in the C++ documentation).", py::arg("timeout")) + .def("kill", &Actor::kill, py::call_guard(), "Kill that actor") + .def("kill_all", &Actor::kill_all, py::call_guard(), "Kill all actors but the caller.") + .def("self", &Actor::self, "Retrieves the current actor.") .def("is_suspended", &Actor::is_suspended, "Returns True if that actor is currently suspended.") - .def("suspend", &Actor::suspend, "Suspend that actor, that is blocked until resume()ed by another actor.") - .def("resume", &Actor::resume, "Resume that actor, that was previously suspend()ed."); + .def("suspend", &Actor::suspend, py::call_guard(), + "Suspend that actor, that is blocked until resume()ed by another actor.") + .def("resume", &Actor::resume, py::call_guard(), + "Resume that actor, that was previously suspend()ed."); }