Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Use typed Mailbox::get<>() instead of using static_cast everywhere.
[simgrid.git] / src / bindings / python / simgrid_python.cpp
index b8301ad..ce60453 100644 (file)
@@ -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. */
@@ -31,6 +31,7 @@
 #include <simgrid/s4u/Mailbox.hpp>
 #include <simgrid/version.h>
 
+#include <algorithm>
 #include <memory>
 #include <string>
 #include <vector>
@@ -44,11 +45,9 @@ using simgrid::s4u::Mailbox;
 
 XBT_LOG_NEW_DEFAULT_CATEGORY(python, "python");
 
-PYBIND11_DECLARE_HOLDER_TYPE(T, boost::intrusive_ptr<T>)
-
 namespace {
 
-static std::string get_simgrid_version()
+std::string get_simgrid_version()
 {
   int major;
   int minor;
@@ -57,220 +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<PyThreadState, decltype(&release)> thread_state{acquire(), &release};
+
+public:
+  void reset() { thread_state.reset(); }
+};
+
+class XBT_PRIVATE GilScopedRelease {
+  std::unique_ptr<PyThreadState, decltype(&PyEval_RestoreThread)> thread_state{PyEval_SaveThread(),
+                                                                               &PyEval_RestoreThread};
+};
 
 } // namespace
 
+PYBIND11_DECLARE_HOLDER_TYPE(T, boost::intrusive_ptr<T>)
+
 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<simgrid::ForcefulKillException>(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<double, double>(&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("set_host", &simgrid::s4u::this_actor::set_host,
-         "Moves the current actor to another host, see :cpp:func:`void simgrid::s4u::this_actor::set_host()`",
-         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<double, double>(&simgrid::s4u::this_actor::execute),
+           py::call_guard<GilScopedRelease>(),
+           "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<double>(&simgrid::s4u::this_actor::exec_init),
+           py::call_guard<GilScopedRelease>())
+      .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<GilScopedRelease>(),
+           "Moves the current actor to another host.", py::arg("dest"))
+      .def("sleep_for", static_cast<void (*)(double)>(&simgrid::s4u::this_actor::sleep_for),
+           py::call_guard<GilScopedRelease>(), "Block the actor sleeping for that amount of seconds.",
+           py::arg("duration"))
+      .def("sleep_until", static_cast<void (*)(double)>(&simgrid::s4u::this_actor::sleep_until),
+           py::call_guard<GilScopedRelease>(), "Block the actor sleeping until the specified timestamp.",
+           py::arg("duration"))
+      .def("suspend", &simgrid::s4u::this_actor::suspend, py::call_guard<GilScopedRelease>(),
+           "Suspend the current actor, that is blocked until resume()ed by another actor.")
+      .def("yield_", &simgrid::s4u::this_actor::yield, py::call_guard<GilScopedRelease>(), "Yield the actor")
+      .def("exit", &simgrid::s4u::this_actor::exit, py::call_guard<GilScopedRelease>(), "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<GilScopedRelease>(), "");
 
   /* Class Engine */
-  py::class_<Engine>(m, "Engine", "Simulation Engine, see :ref:`class s4u::Engine <API_s4u_Engine>`")
-      .def(py::init([](std::vector<std::string> args) -> simgrid::s4u::Engine* {
-        static char noarg[] = {'\0'};
-        int argc            = args.size();
-        std::unique_ptr<char* []> 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_<Engine>(m, "Engine", "Simulation Engine")
+      .def(py::init([](std::vector<std::string> args) {
+        auto argc           = static_cast<int>(args.size());
+        std::vector<char*> 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<std::string> 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<GilScopedRelease>(), "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<std::string> 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<py::function>(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<py::function>(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_<simgrid::s4u::Host, std::unique_ptr<Host, py::nodelete>>(m, "Host", "Simulation Engine, see :ref:`class s4u::Host <API_s4u_Host>`")
+  py::class_<simgrid::s4u::Host, std::unique_ptr<Host, py::nodelete>>(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<GilScopedRelease>(),
+           "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_<simgrid::s4u::Mailbox, std::unique_ptr<Mailbox, py::nodelete>>(m, "Mailbox", "Mailbox, see :ref:`class s4u::Mailbox <API_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<py::object>(static_cast<PyObject*>(self->get()));
-         data.dec_ref();
-         return data;
-      }, "Blocking data reception, see :cpp:func:`void* simgrid::s4u::Mailbox::get()`");
+  py::class_<simgrid::s4u::Mailbox, std::unique_ptr<Mailbox, py::nodelete>>(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<GilScopedRelease>(), "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<GilScopedRelease>(), "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<GilScopedRelease>(), "Non-blocking data transmission")
+      .def(
+          "get",
+          [](Mailbox* self) {
+            py::object data = pybind11::reinterpret_steal<py::object>(self->get<PyObject>());
+            // data.dec_ref(); // FIXME: why does it break python-actor-create?
+            return data;
+          },
+          py::call_guard<GilScopedRelease>(), "Blocking data reception")
+      .def("set_receiver",
+        [](Mailbox* self, ActorPtr actor) {
+          self->set_receiver(actor);
+        },
+        py::call_guard<GilScopedRelease>(),
+        "Sets the actor as permanent receiver");
 
   /* Class Comm */
-  py::class_<simgrid::s4u::Comm, simgrid::s4u::CommPtr>(m, "Comm",
-                                                        "Communication, see :ref:`class s4u::Comm <API_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<simgrid::s4u::CommPtr>* 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<simgrid::s4u::CommPtr>* 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_<simgrid::s4u::Exec, simgrid::s4u::ExecPtr>(m, "Exec", "Execution, see :ref:`class s4u::Exec <API_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<simgrid::s4u::ExecSeq>(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_<simgrid::s4u::Comm, simgrid::s4u::CommPtr>(m, "Comm", "Communication")
+      .def("test", &simgrid::s4u::Comm::test, py::call_guard<GilScopedRelease>(),
+           "Test whether the communication is terminated.")
+      .def("wait", &simgrid::s4u::Comm::wait, py::call_guard<GilScopedRelease>(),
+           "Block until the completion of that communication.")
+      .def("wait_all", &simgrid::s4u::Comm::wait_all, py::call_guard<GilScopedRelease>(),
+           "Block until the completion of all communications in the list.")
+      .def("wait_any", &simgrid::s4u::Comm::wait_any, py::call_guard<GilScopedRelease>(),
+           "Block until the completion of any communication in the list and return the index of the terminated one.");
+
+  /* Class Exec */
+  py::class_<simgrid::s4u::Exec, simgrid::s4u::ExecPtr>(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<GilScopedRelease>(),
+           "Test whether the execution is terminated.")
+      .def("cancel", &simgrid::s4u::Exec::cancel, py::call_guard<GilScopedRelease>(), "Cancel that execution.")
+      .def("start", &simgrid::s4u::Exec::start, py::call_guard<GilScopedRelease>(), "Start that execution.")
+      .def("wait", &simgrid::s4u::Exec::wait, py::call_guard<GilScopedRelease>(),
+           "Block until the completion of that execution.");
 
   /* Class Actor */
   py::class_<simgrid::s4u::Actor, ActorPtr>(m, "Actor",
                                             "An actor is an independent stream of execution in your distributed "
-                                            "application, see :ref:`class s4u::Actor <API_s4u_Actor>`")
-
-      .def("create",
-           [](py::str name, py::object host, py::object fun, py::args args) {
-
-             return simgrid::s4u::Actor::create(name, host.cast<Host*>(), [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.")
-      .def_property("host", &Actor::get_host, &Actor::set_host, "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<GilScopedRelease>(), "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,
+      .def("daemonize", &Actor::daemonize, py::call_guard<GilScopedRelease>(),
            "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<double>(&Actor::join),
+      .def("join", py::overload_cast<double>(&Actor::join, py::const_), py::call_guard<GilScopedRelease>(),
            "Wait for the actor to finish (more info in the C++ documentation).", 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("kill", &Actor::kill, py::call_guard<GilScopedRelease>(), "Kill that actor")
+      .def("kill_all", &Actor::kill_all, py::call_guard<GilScopedRelease>(), "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<GilScopedRelease>(),
+           "Suspend that actor, that is blocked until resume()ed by another actor.")
+      .def("resume", &Actor::resume, py::call_guard<GilScopedRelease>(),
+           "Resume that actor, that was previously suspend()ed.");
 }