Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
rename simdata_process_t into MsgActorExt
[simgrid.git] / src / msg / msg_process.cpp
index 5d588bd..82d6774 100644 (file)
@@ -5,66 +5,57 @@
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
 #include "msg_private.h"
-#include "xbt/sysdep.h"
-#include "xbt/log.h"
-#include "src/simix/smx_process_private.h"
-#include "src/simix/smx_private.h"
+#include "simgrid/s4u/host.hpp"
+#include "src/simix/ActorImpl.hpp"
 
-XBT_LOG_NEW_DEFAULT_SUBCATEGORY(msg_process, msg,
-                                "Logging specific to MSG (process)");
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(msg_process, msg, "Logging specific to MSG (process)");
 
 /** @addtogroup m_process_management
- * \htmlonly <!-- DOXYGEN_NAVBAR_LABEL="Processes" --> \endhtmlonly
  *
- *  Processes (#msg_process_t) are independent agents that can do stuff on their own. They are in charge of executing your code interacting with the simulated world.
+ *  Processes (#msg_process_t) are independent agents that can do stuff on their own. They are in charge of executing
+ *  your code interacting with the simulated world.
  *  A process may be defined as a <em>code</em> with some <em>private data</em>.
- *  Processes must be located on <em>hosts</em> (#msg_host_t), and they exchange data by sending tasks (#msg_task_t) that are similar to envelops containing data.
+ *  Processes must be located on <em>hosts</em> (#msg_host_t), and they exchange data by sending tasks (#msg_task_t)
+ *  that are similar to envelops containing data.
  */
 
 /******************************** Process ************************************/
-
 /**
  * \brief Cleans the MSG data of a process.
  * \param smx_proc a SIMIX process
  */
-void MSG_process_cleanup_from_SIMIX(smx_process_t smx_proc)
+void MSG_process_cleanup_from_SIMIX(smx_actor_t smx_actor)
 {
-  simdata_process_t msg_proc;
+  MsgActorExt* msg_actor;
 
   // get the MSG process from the SIMIX process
-  if (smx_proc == SIMIX_process_self()) {
+  if (smx_actor == SIMIX_process_self()) {
     /* avoid a SIMIX request if this function is called by the process itself */
-    msg_proc = (simdata_process_t) SIMIX_process_self_get_data(smx_proc);
-    SIMIX_process_self_set_data(smx_proc, NULL);
-  }
-  else {
-    msg_proc = (simdata_process_t) simcall_process_get_data(smx_proc);
-    simcall_process_set_data(smx_proc, NULL);
+    msg_actor = (MsgActorExt*)SIMIX_process_self_get_data();
+    SIMIX_process_self_set_data(nullptr);
+  } else {
+    msg_actor = (MsgActorExt*)smx_actor->data;
+    simcall_process_set_data(smx_actor, nullptr);
   }
 
-  TRACE_msg_process_destroy(smx_proc->name, smx_proc->pid);
+  TRACE_msg_process_destroy(smx_actor->name.c_str(), smx_actor->pid);
   // free the data if a function was provided
-  if (msg_proc && msg_proc->data && msg_global->process_data_cleanup) {
-    msg_global->process_data_cleanup(msg_proc->data);
+  if (msg_actor && msg_actor->data && msg_global->process_data_cleanup) {
+    msg_global->process_data_cleanup(msg_actor->data);
   }
 
   // free the MSG process
-  xbt_free(msg_proc);
-  SIMIX_process_cleanup(smx_proc);
+  xbt_free(msg_actor);
+  SIMIX_process_cleanup(smx_actor);
 }
 
 /* This function creates a MSG process. It has the prototype enforced by SIMIX_function_register_process_create */
-smx_process_t MSG_process_create_from_SIMIX(const char *name,
-                                   xbt_main_func_t code, void *data,
-                                   const char *hostname, double kill_time,
-                                   int argc, char **argv,
-                                   xbt_dict_t properties, int auto_restart,
-                                   smx_process_t parent_process)
+smx_actor_t MSG_process_create_from_SIMIX(
+  const char *name, std::function<void()> code, void *data, sg_host_t host,
+  double kill_time, xbt_dict_t properties,
+  int auto_restart, smx_actor_t parent_process)
 {
-  msg_host_t host = MSG_host_by_name(hostname);
-  msg_process_t p = MSG_process_create_with_environment(name, code, data,
-                                                      host, argc, argv,
-                                                      properties);
+  msg_process_t p = MSG_process_create_with_environment(name, std::move(code), data, host, properties);
   if (p) {
     MSG_process_set_kill_time(p,kill_time);
     MSG_process_auto_restart_set(p,auto_restart);
@@ -75,36 +66,28 @@ smx_process_t MSG_process_create_from_SIMIX(const char *name,
 /** \ingroup m_process_management
  * \brief Creates and runs a new #msg_process_t.
  *
- * Does exactly the same as #MSG_process_create_with_arguments but without
  providing standard arguments (\a argc, \a argv, \a start_time, \a kill_time).
+ * Does exactly the same as #MSG_process_create_with_arguments but without providing standard arguments
* (\a argc, \a argv, \a start_time, \a kill_time).
  * \sa MSG_process_create_with_arguments
  */
-msg_process_t MSG_process_create(const char *name,
-                               xbt_main_func_t code, void *data,
-                               msg_host_t host)
+msg_process_t MSG_process_create(const char *name, xbt_main_func_t code, void *data, msg_host_t host)
 {
-  return MSG_process_create_with_environment(name, code, data, host, 0,
-                                             NULL, NULL);
+  return MSG_process_create_with_environment(name, code, data, host, 0, nullptr, nullptr);
 }
 
 /** \ingroup m_process_management
  * \brief Creates and runs a new #msg_process_t.
 
- * A constructor for #msg_process_t taking four arguments and returning the
- * corresponding object. The structure (and the corresponding thread) is
- * created, and put in the list of ready process.
- * \param name a name for the object. It is for user-level information
-   and can be NULL.
- * \param code is a function describing the behavior of the process. It
-   should then only use functions described in \ref
-   m_process_management (to create a new #msg_process_t for example),
-   in \ref m_host_management (only the read-only functions i.e. whose
-   name contains the word get), in \ref m_task_management (to create
-   or destroy some #msg_task_t for example) and in \ref
-   msg_task_usage (to handle file transfers and task processing).
- * \param data a pointer to any data one may want to attach to the new
-   object.  It is for user-level information and can be NULL. It can
-   be retrieved with the function \ref MSG_process_get_data.
+ * A constructor for #msg_process_t taking four arguments and returning the corresponding object. The structure (and
+ * the corresponding thread) is created, and put in the list of ready process.
+ * \param name a name for the object. It is for user-level information and can be nullptr.
+ * \param code is a function describing the behavior of the process. It should then only use functions described
+ * in \ref m_process_management (to create a new #msg_process_t for example),
+   in \ref m_host_management (only the read-only functions i.e. whose name contains the word get),
+   in \ref m_task_management (to create or destroy some #msg_task_t for example) and
+   in \ref msg_task_usage (to handle file transfers and task processing).
+ * \param data a pointer to any data one may want to attach to the new object.  It is for user-level information and
+ *        can be nullptr. It can be retrieved with the function \ref MSG_process_get_data.
  * \param host the location where the new process is executed.
  * \param argc first argument passed to \a code
  * \param argv second argument passed to \a code
@@ -112,83 +95,115 @@ msg_process_t MSG_process_create(const char *name,
  * \return The new corresponding object.
  */
 
-msg_process_t MSG_process_create_with_arguments(const char *name,
-                                              xbt_main_func_t code,
-                                              void *data, msg_host_t host,
+msg_process_t MSG_process_create_with_arguments(const char *name, xbt_main_func_t code, void *data, msg_host_t host,
                                               int argc, char **argv)
 {
-  return MSG_process_create_with_environment(name, code, data, host,
-                                             argc, argv, NULL);
+  return MSG_process_create_with_environment(name, code, data, host, argc, argv, nullptr);
 }
 
 /** \ingroup m_process_management
  * \brief Creates and runs a new #msg_process_t.
 
- * A constructor for #msg_process_t taking four arguments and returning the
- * corresponding object. The structure (and the corresponding thread) is
- * created, and put in the list of ready process.
- * \param name a name for the object. It is for user-level information
-   and can be NULL.
- * \param code is a function describing the behavior of the process. It
-   should then only use functions described in \ref
-   m_process_management (to create a new #msg_process_t for example),
-   in \ref m_host_management (only the read-only functions i.e. whose
-   name contains the word get), in \ref m_task_management (to create
-   or destroy some #msg_task_t for example) and in \ref
-   msg_task_usage (to handle file transfers and task processing).
- * \param data a pointer to any data one may want to attach to the new
-   object.  It is for user-level information and can be NULL. It can
-   be retrieved with the function \ref MSG_process_get_data.
+ * A constructor for #msg_process_t taking four arguments and returning the corresponding object. The structure (and
+ * the corresponding thread) is created, and put in the list of ready process.
+ * \param name a name for the object. It is for user-level information and can be nullptr.
+ * \param code is a function describing the behavior of the process. It should then only use functions described
+ * in \ref m_process_management (to create a new #msg_process_t for example),
+   in \ref m_host_management (only the read-only functions i.e. whose name contains the word get),
+   in \ref m_task_management (to create or destroy some #msg_task_t for example) and
+   in \ref msg_task_usage (to handle file transfers and task processing).
+ * \param data a pointer to any data one may want to attach to the new object.  It is for user-level information and
+ *        can be nullptr. It can be retrieved with the function \ref MSG_process_get_data.
  * \param host the location where the new process is executed.
  * \param argc first argument passed to \a code
- * \param argv second argument passed to \a code. WARNING, these strings are freed by the SimGrid kernel when the process exits, so they cannot be static nor shared between several processes.
+ * \param argv second argument passed to \a code. WARNING, these strings are freed by the SimGrid kernel when the
+ *             process exits, so they cannot be static nor shared between several processes.
  * \param properties list a properties defined for this process
  * \see msg_process_t
  * \return The new corresponding object.
  */
-msg_process_t MSG_process_create_with_environment(const char *name,
-                                                xbt_main_func_t code,
-                                                void *data, msg_host_t host,
-                                                int argc, char **argv,
-                                                xbt_dict_t properties)
+msg_process_t MSG_process_create_with_environment(const char *name, xbt_main_func_t code, void *data, msg_host_t host,
+                                                  int argc, char **argv, xbt_dict_t properties)
+{
+  std::function<void()> function;
+  if (code)
+    function = simgrid::xbt::wrapMain(code, argc, const_cast<const char*const*>(argv));
+  msg_process_t res = MSG_process_create_with_environment(name,
+    std::move(function), data, host, properties);
+  for (int i = 0; i != argc; ++i)
+    xbt_free(argv[i]);
+  xbt_free(argv);
+  return res;
+}
+
+msg_process_t MSG_process_create_with_environment(
+  const char *name, std::function<void()> code, void *data,
+  msg_host_t host, xbt_dict_t properties)
 {
-  xbt_assert(code != NULL && host != NULL, "Invalid parameters: host and code params must not be NULL");
-  simdata_process_t simdata = xbt_new0(s_simdata_process_t, 1);
-  msg_process_t process;
+  xbt_assert(code != nullptr && host != nullptr, "Invalid parameters: host and code params must not be nullptr");
+  MsgActorExt* simdata = new MsgActorExt();
 
   /* Simulator data for MSG */
-  simdata->waiting_action = NULL;
-  simdata->waiting_task = NULL;
   simdata->m_host = host;
-  simdata->argc = argc;
-  simdata->argv = argv;
   simdata->data = data;
   simdata->last_errno = MSG_OK;
 
-  /* The TRACE process is created before the SIMIX one
-   * to avoid issues when SIMIX decides to start the new
-   * process right now (before returning the flow control). */
-  unsigned int next_pid = SIMIX_process_get_maxpid();
-  TRACE_msg_process_create(name, next_pid, host);
-
   /* Let's create the process: SIMIX may decide to start it right now,
    * even before returning the flow control to us */
-  process = simcall_process_create(name, code, simdata, sg_host_get_name(host), -1,
-                           argc, argv, properties,0);
+  msg_process_t process = simcall_process_create(name, std::move(code), simdata, host, -1, properties, 0);
 
   if (!process) {
     /* Undo everything we have just changed */
     xbt_free(simdata);
-    return NULL;
-    // FIXME: is the TRACE process destroyed in this case?
+    return nullptr;
   }
   else {
-    xbt_assert(next_pid == process->pid, "Bad trace pid hack");
     simcall_process_on_exit(process,(int_f_pvoid_pvoid_t)TRACE_msg_process_kill,process);
   }
   return process;
 }
 
+static int MSG_maestro(int argc, char** argv)
+{
+  int res = MSG_main();
+  return res;
+}
+
+/* Become a process in the simulation
+ *
+ * Currently this can only be called by the main thread (once) and only work with some thread factories
+ * (currently ThreadContextFactory).
+ *
+ * In the future, it might be extended in order to attach other threads created by a third party library.
+ */
+msg_process_t MSG_process_attach(const char *name, void *data, msg_host_t host, xbt_dict_t properties)
+{
+  xbt_assert(host != nullptr, "Invalid parameters: host and code params must not be nullptr");
+  MsgActorExt* msgExt = new MsgActorExt();
+
+  /* Simulator data for MSG */
+  msgExt->m_host = host;
+  msgExt->data   = data;
+
+  /* Let's create the process: SIMIX may decide to start it right now, even before returning the flow control to us */
+  msg_process_t process = SIMIX_process_attach(name, msgExt, host->cname(), properties, nullptr);
+  if (!process)
+    xbt_die("Could not attach");
+  simcall_process_on_exit(process,(int_f_pvoid_pvoid_t)TRACE_msg_process_kill,process);
+  return process;
+}
+
+/** Detach a process attached with `MSG_process_attach()`
+ *
+ *  This is called when the current process has finished its job.
+ *  Used in the main thread, it waits for the simulation to finish before  returning. When it returns, the other
+ *  simulated processes and the maestro are destroyed.
+ */
+void MSG_process_detach()
+{
+  SIMIX_process_detach();
+}
+
 /** \ingroup m_process_management
  * \param process poor victim
  *
@@ -196,15 +211,7 @@ msg_process_t MSG_process_create_with_environment(const char *name,
  */
 void MSG_process_kill(msg_process_t process)
 {
-//  /* FIXME: why do we only cancel communication actions? is this useful? */
-//  simdata_process_t p_simdata = simcall_process_get_data(process);
-//  if (p_simdata->waiting_task && p_simdata->waiting_task->simdata->comm) {
-//    simcall_comm_cancel(p_simdata->waiting_task->simdata->comm);
-//  }
-
   simcall_process_kill(process);
-
-  return;
 }
 
 /**
@@ -221,89 +228,88 @@ msg_error_t MSG_process_join(msg_process_t process, double timeout){
 /** \ingroup m_process_management
  * \brief Migrates a process to another location.
  *
- * This function checks whether \a process and \a host are valid pointers
  and change the value of the #msg_host_t on which \a process is running.
+ * This function checks whether \a process and \a host are valid pointers and change the value of the #msg_host_t on
* which \a process is running.
  */
 msg_error_t MSG_process_migrate(msg_process_t process, msg_host_t host)
 {
-  simdata_process_t simdata = (simdata_process_t) simcall_process_get_data(process);
-  simdata->m_host = host;
-  msg_host_t now = simdata->m_host;
+  MsgActorExt* msgExt = (MsgActorExt*)process->data;
+  msgExt->m_host      = host;
+  msg_host_t now      = msgExt->m_host;
   TRACE_msg_process_change_host(process, now, host);
   simcall_process_set_host(process, host);
   return MSG_OK;
 }
 
+/** Yield the current actor; let the other actors execute first */
+void MSG_process_yield()
+{
+  simgrid::simix::kernelImmediate([] { /* do nothing*/ });
+}
+
 /** \ingroup m_process_management
  * \brief Returns the user data of a process.
  *
- * This function checks whether \a process is a valid pointer or not
-   and returns the user data associated to this process.
+ * This function checks whether \a process is a valid pointer and returns the user data associated to this process.
  */
 void* MSG_process_get_data(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: first parameter must not be NULL!");
+  xbt_assert(process != nullptr, "Invalid parameter: first parameter must not be nullptr!");
 
   /* get from SIMIX the MSG process data, and then the user data */
-  simdata_process_t simdata = (simdata_process_t) simcall_process_get_data(process);
-  if (simdata)
-    return simdata->data;
+  MsgActorExt* msgExt = (MsgActorExt*)process->data;
+  if (msgExt)
+    return msgExt->data;
   else
-    return NULL;
+    return nullptr;
 }
 
 /** \ingroup m_process_management
  * \brief Sets the user data of a process.
  *
- * This function checks whether \a process is a valid pointer or not
-   and sets the user data associated to this process.
+ * This function checks whether \a process is a valid pointer and sets the user data associated to this process.
  */
 msg_error_t MSG_process_set_data(msg_process_t process, void *data)
 {
-  xbt_assert(process != NULL, "Invalid parameter: first parameter must not be NULL!");
+  xbt_assert(process != nullptr, "Invalid parameter: first parameter must not be nullptr!");
 
-  simdata_process_t simdata =
-    (simdata_process_t) simcall_process_get_data(process);
-  simdata->data = data;
+  MsgActorExt* msgExt = (MsgActorExt*)process->data;
+  msgExt->data        = data;
 
   return MSG_OK;
 }
 
 /** \ingroup m_process_management
- * \brief Sets a cleanup function to be called to free the userdata of a
- * process when a process is destroyed.
- * \param data_cleanup a cleanup function for the userdata of a process,
- * or NULL to call no function
+ * \brief Sets a cleanup function to be called to free the userdata of a process when a process is destroyed.
+ * \param data_cleanup a cleanup function for the userdata of a process, or nullptr to call no function
  */
 XBT_PUBLIC(void) MSG_process_set_data_cleanup(void_f_pvoid_t data_cleanup) {
-
   msg_global->process_data_cleanup = data_cleanup;
 }
 
 /** \ingroup m_process_management
  * \brief Return the location on which a process is running.
- * \param process a process (NULL means the current one)
- * \return the msg_host_t corresponding to the location on which \a
- * process is running.
+ * \param process a process (nullptr means the current one)
+ * \return the msg_host_t corresponding to the location on which \a process is running.
  */
 msg_host_t MSG_process_get_host(msg_process_t process)
 {
-  simdata_process_t simdata;
-  if (process == NULL) {
-    simdata = (simdata_process_t) SIMIX_process_self_get_data(SIMIX_process_self());
+  MsgActorExt* msgExt;
+  if (process == nullptr) {
+    msgExt = (MsgActorExt*)SIMIX_process_self_get_data();
   }
   else {
-    simdata = (simdata_process_t) simcall_process_get_data(process);
+    msgExt = (MsgActorExt*)process->data;
   }
-  return simdata ? simdata->m_host : NULL;
+  return msgExt ? msgExt->m_host : nullptr;
 }
 
 /** \ingroup m_process_management
  *
  * \brief Return a #msg_process_t given its PID.
  *
- * This function search in the list of all the created msg_process_t for a msg_process_t
  whose PID is equal to \a PID. If no host is found, \c NULL is returned.
+ * This function search in the list of all the created msg_process_t for a msg_process_t  whose PID is equal to \a PID.
* If no host is found, \c nullptr is returned.
    Note that the PID are uniq in the whole simulation, not only on a given host.
  */
 msg_process_t MSG_process_from_PID(int PID)
@@ -312,12 +318,12 @@ msg_process_t MSG_process_from_PID(int PID)
 }
 
 /** @brief returns a list of all currently existing processes */
-xbt_dynar_t MSG_processes_as_dynar(void) {
+xbt_dynar_t MSG_processes_as_dynar() {
   return SIMIX_processes_as_dynar();
 }
-/** @brief Return the current number MSG processes.
- */
-int MSG_process_get_number(void)
+
+/** @brief Return the current number MSG processes. */
+int MSG_process_get_number()
 {
   return SIMIX_process_count();
 }
@@ -337,44 +343,36 @@ msg_error_t MSG_process_set_kill_time(msg_process_t process, double kill_time)
 /** \ingroup m_process_management
  * \brief Returns the process ID of \a process.
  *
- * This function checks whether \a process is a valid pointer or not
-   and return its PID (or 0 in case of problem).
+ * This function checks whether \a process is a valid pointer and return its PID (or 0 in case of problem).
  */
 int MSG_process_get_PID(msg_process_t process)
 {
   /* 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 */
-  if (process == NULL) {
+  if (process == nullptr)
     return 0;
-  }
-  return simcall_process_get_PID(process);
+  return process->pid;
 }
 
 /** \ingroup m_process_management
  * \brief Returns the process ID of the parent of \a process.
  *
- * This function checks whether \a process is a valid pointer or not
-   and return its PID. Returns -1 if the process has not been created by
-   any other process.
+ * This function checks whether \a process is a valid pointer and return its PID.
+ * Returns -1 if the process has not been created by any other process.
  */
 int MSG_process_get_PPID(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
-
-  return simcall_process_get_PPID(process);
+  return process->ppid;
 }
 
 /** \ingroup m_process_management
  * \brief Return the name of a process.
  *
- * This function checks whether \a process is a valid pointer or not
-   and return its name.
+ * This function checks whether \a process is a valid pointer and return its name.
  */
 const char *MSG_process_get_name(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
-
-  return simcall_process_get_name(process);
+  return process->name.c_str();
 }
 
 /** \ingroup m_process_management
@@ -382,10 +380,9 @@ const char *MSG_process_get_name(msg_process_t process)
  *
  * \param process a process
  * \param name a property name
- * \return value of a property (or NULL if the property is not set)
+ * \return value of a property (or nullptr if the property is not set)
  */
-const char *MSG_process_get_property_value(msg_process_t process,
-                                           const char *name)
+const char *MSG_process_get_property_value(msg_process_t process, const char *name)
 {
   return (char*) xbt_dict_get_or_null(MSG_process_get_properties(process), name);
 }
@@ -397,10 +394,8 @@ const char *MSG_process_get_property_value(msg_process_t process,
  */
 xbt_dict_t MSG_process_get_properties(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
-
+  xbt_assert(process != nullptr, "Invalid parameter: First argument must not be nullptr");
   return simcall_process_get_properties(process);
-
 }
 
 /** \ingroup m_process_management
@@ -408,7 +403,7 @@ xbt_dict_t MSG_process_get_properties(msg_process_t process)
  *
  * This function returns the PID of the currently running #msg_process_t.
  */
-int MSG_process_self_PID(void)
+int MSG_process_self_PID()
 {
   return MSG_process_get_PID(MSG_process_self());
 }
@@ -416,10 +411,9 @@ int MSG_process_self_PID(void)
 /** \ingroup m_process_management
  * \brief Return the PPID of the current process.
  *
- * This function returns the PID of the parent of the currently
- * running #msg_process_t.
+ * This function returns the PID of the parent of the currently running #msg_process_t.
  */
-int MSG_process_self_PPID(void)
+int MSG_process_self_PPID()
 {
   return MSG_process_get_PPID(MSG_process_self());
 }
@@ -429,7 +423,7 @@ int MSG_process_self_PPID(void)
  *
  * This function returns the currently running #msg_process_t.
  */
-msg_process_t MSG_process_self(void)
+msg_process_t MSG_process_self()
 {
   return SIMIX_process_self();
 }
@@ -437,42 +431,39 @@ msg_process_t MSG_process_self(void)
 /** \ingroup m_process_management
  * \brief Suspend the process.
  *
- * This function suspends the process by suspending the task on which
- * it was waiting for the completion.
+ * This function suspends the process by suspending the task on which it was waiting for the completion.
  */
 msg_error_t MSG_process_suspend(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
+  xbt_assert(process != nullptr, "Invalid parameter: First argument must not be nullptr");
 
   TRACE_msg_process_suspend(process);
   simcall_process_suspend(process);
-  MSG_RETURN(MSG_OK);
+  return MSG_OK;
 }
 
 /** \ingroup m_process_management
  * \brief Resume a suspended process.
  *
- * This function resumes a suspended process by resuming the task on
- * which it was waiting for the completion.
+ * This function resumes a suspended process by resuming the task on which it was waiting for the completion.
  */
 msg_error_t MSG_process_resume(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
+  xbt_assert(process != nullptr, "Invalid parameter: First argument must not be nullptr");
 
   TRACE_msg_process_resume(process);
   simcall_process_resume(process);
-  MSG_RETURN(MSG_OK);
+  return MSG_OK;
 }
 
 /** \ingroup m_process_management
  * \brief Returns true if the process is suspended .
  *
- * This checks whether a process is suspended or not by inspecting the
- * task on which it was waiting for the completion.
+ * This checks whether a process is suspended or not by inspecting the task on which it was waiting for the completion.
  */
 int MSG_process_is_suspended(msg_process_t process)
 {
-  xbt_assert(process != NULL, "Invalid parameter: First argument must not be NULL");
+  xbt_assert(process != nullptr, "Invalid parameter: First argument must not be nullptr");
   return simcall_process_is_suspended(process);
 }
 
@@ -491,8 +482,7 @@ void MSG_process_on_exit(int_f_pvoid_pvoid_t fun, void *data) {
 /**
  * \ingroup m_process_management
  * \brief Sets the "auto-restart" flag of the process.
- * If the flag is set to 1, the process will be automatically restarted when
- * its host comes back up.
+ * If the flag is set to 1, the process will be automatically restarted when its host comes back up.
  */
 XBT_PUBLIC(void) MSG_process_auto_restart_set(msg_process_t process, int auto_restart) {
   simcall_process_auto_restart_set(process,auto_restart);