Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Don't auto-destroy the MSG_comm_t in MSG_comm_wait(), it's the
[simgrid.git] / src / msg / gos.c
index 7b6367d..899b135 100644 (file)
@@ -1,14 +1,12 @@
-/*     $Id$      */
-
-/* Copyright (c) 2002-2007 Arnaud Legrand.                                  */
-/* Copyright (c) 2007 Bruno Donassolo.                                      */
-/* All rights reserved.                                                     */
+/* Copyright (c) 2004, 2005, 2006, 2007, 2008, 2009, 2010. 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 "msg/private.h"
+#include "private.h"
 #include "xbt/sysdep.h"
+#include "mc/mc.h"
 #include "xbt/log.h"
 #include "mailbox.h"
 
@@ -32,7 +30,7 @@ MSG_error_t MSG_get_errno(void)
  * This function is used for describing the behavior of an agent. It
  * takes only one parameter.
  * \param task a #m_task_t to execute on the location on which the
  agent is running.
+ agent is running.
  * \return #MSG_FATAL if \a task is not properly initialized and
  * #MSG_OK otherwise.
  */
@@ -40,71 +38,71 @@ MSG_error_t MSG_task_execute(m_task_t task)
 {
   simdata_task_t simdata = NULL;
   m_process_t self = MSG_process_self();
-  e_surf_action_state_t state = SURF_ACTION_NOT_IN_THE_SYSTEM;
   CHECK_HOST();
-#ifdef HAVE_TRACING
-  TRACE_msg_task_execute_start (task);
-#endif
 
   simdata = task->simdata;
 
+  xbt_assert0(simdata->host_nb == 0,
+              "This is a parallel task. Go to hell.");
+
+#ifdef HAVE_TRACING
+  TRACE_msg_task_execute_start(task);
+#endif
+
   xbt_assert1((!simdata->compute) && (task->simdata->refcount == 1),
-              "This task is executed somewhere else. Go fix your code! %d", task->simdata->refcount);
+              "This task is executed somewhere else. Go fix your code! %d",
+              task->simdata->refcount);
 
   DEBUG1("Computing on %s", MSG_process_self()->simdata->m_host->name);
 
   if (simdata->computation_amount == 0) {
 #ifdef HAVE_TRACING
-    TRACE_msg_task_execute_end (task);
+    TRACE_msg_task_execute_end(task);
 #endif
     return MSG_OK;
   }
   simdata->refcount++;
-  SIMIX_mutex_lock(simdata->mutex);
   simdata->compute =
-    SIMIX_action_execute(SIMIX_host_self(), task->name,
-                         simdata->computation_amount);
-  SIMIX_action_set_priority(simdata->compute, simdata->priority);
+      SIMIX_req_host_execute(task->name, SIMIX_host_self(),
+                           simdata->computation_amount);
+  SIMIX_req_host_execution_set_priority(simdata->compute, simdata->priority);
+#ifdef HAVE_TRACING
+  SIMIX_req_set_category(simdata->compute, task->category);
+#endif
 
-  /* changed to waiting action since we are always waiting one action (execute, communicate or sleep) */
   self->simdata->waiting_action = simdata->compute;
-  SIMIX_register_action_to_condition(simdata->compute, simdata->cond);
-  do {
-    SIMIX_cond_wait(simdata->cond, simdata->mutex);
-    state = SIMIX_action_get_state(simdata->compute);
-  } while (state == SURF_ACTION_READY || state == SURF_ACTION_RUNNING);
-  SIMIX_unregister_action_to_condition(simdata->compute, simdata->cond);
+  SIMIX_req_host_execution_wait(simdata->compute);
   self->simdata->waiting_action = NULL;
 
-  SIMIX_mutex_unlock(simdata->mutex);
   simdata->refcount--;
 
-  if (SIMIX_action_get_state(task->simdata->compute) == SURF_ACTION_DONE) {
+  DEBUG2("Execution task '%s' finished in state %d", task->name, SIMIX_req_host_execution_get_state(task->simdata->compute));
+  if (SIMIX_req_host_execution_get_state(task->simdata->compute) == SIMIX_DONE) {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->computation_amount = 0.0;
     simdata->comm = NULL;
     simdata->compute = NULL;
 #ifdef HAVE_TRACING
-    TRACE_msg_task_execute_end (task);
+    TRACE_msg_task_execute_end(task);
 #endif
     MSG_RETURN(MSG_OK);
-  } else if (SIMIX_host_get_state(SIMIX_host_self()) == 0) {
+  } else if (SIMIX_req_host_get_state(SIMIX_host_self()) == 0) {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->comm = NULL;
     simdata->compute = NULL;
 #ifdef HAVE_TRACING
-    TRACE_msg_task_execute_end (task);
+    TRACE_msg_task_execute_end(task);
 #endif
     MSG_RETURN(MSG_HOST_FAILURE);
   } else {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->comm = NULL;
     simdata->compute = NULL;
 #ifdef HAVE_TRACING
-    TRACE_msg_task_execute_end (task);
+    TRACE_msg_task_execute_end(task);
 #endif
     MSG_RETURN(MSG_TASK_CANCELLED);
   }
@@ -114,18 +112,18 @@ MSG_error_t MSG_task_execute(m_task_t task)
  * \brief Creates a new #m_task_t (a parallel one....).
  *
  * A constructor for #m_task_t taking six arguments and returning the
  corresponding object.
+ corresponding object.
  * \param name a name for the object. It is for user-level information
  and can be NULL.
+ and can be NULL.
  * \param host_nb the number of hosts implied in the parallel task.
  * \param host_list an array of \p host_nb m_host_t.
  * \param computation_amount an array of \p host_nb
  doubles. computation_amount[i] is the total number of operations
  that have to be performed on host_list[i].
+ doubles. computation_amount[i] is the total number of operations
+ that have to be performed on host_list[i].
  * \param communication_amount an array of \p host_nb* \p host_nb doubles.
  * \param data a pointer to any data 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_task_get_data.
+ object.  It is for user-level information and can be NULL. It can
+ be retrieved with the function \ref MSG_task_get_data.
  * \see m_task_t
  * \return The new corresponding object.
  */
@@ -147,8 +145,6 @@ MSG_parallel_task_create(const char *name, int host_nb,
   /* Simulator Data */
   simdata->computation_amount = 0;
   simdata->message_size = 0;
-  simdata->cond = SIMIX_cond_init();
-  simdata->mutex = SIMIX_mutex_init();
   simdata->compute = NULL;
   simdata->comm = NULL;
   simdata->rate = -1.0;
@@ -172,7 +168,6 @@ MSG_error_t MSG_parallel_task_execute(m_task_t task)
 {
   simdata_task_t simdata = NULL;
   m_process_t self = MSG_process_self();
-  e_surf_action_state_t state = SURF_ACTION_NOT_IN_THE_SYSTEM;
   CHECK_HOST();
 
   simdata = task->simdata;
@@ -181,53 +176,48 @@ MSG_error_t MSG_parallel_task_execute(m_task_t task)
               && (task->simdata->refcount == 1),
               "This task is executed somewhere else. Go fix your code!");
 
-  xbt_assert0(simdata->host_nb, "This is not a parallel task. Go to hell.");
+  xbt_assert0(simdata->host_nb,
+              "This is not a parallel task. Go to hell.");
 
-  DEBUG1("Computing on %s", MSG_process_self()->simdata->m_host->name);
+  DEBUG1("Parallel computing on %s", MSG_process_self()->simdata->m_host->name);
 
   simdata->refcount++;
 
-  SIMIX_mutex_lock(simdata->mutex);
   simdata->compute =
-    SIMIX_action_parallel_execute(task->name, simdata->host_nb,
-                                  simdata->host_list, simdata->comp_amount,
+      SIMIX_req_host_parallel_execute(task->name, simdata->host_nb,
+                                  simdata->host_list,
+                                  simdata->comp_amount,
                                   simdata->comm_amount, 1.0, -1.0);
+  DEBUG1("Parallel execution action created: %p", simdata->compute);
 
   self->simdata->waiting_action = simdata->compute;
-  SIMIX_register_action_to_condition(simdata->compute, simdata->cond);
-  do {
-    SIMIX_cond_wait(simdata->cond, simdata->mutex);
-    state = SIMIX_action_get_state(task->simdata->compute);
-  } while (state == SURF_ACTION_READY || state == SURF_ACTION_RUNNING);
-
-  SIMIX_unregister_action_to_condition(simdata->compute, simdata->cond);
+  SIMIX_req_host_execution_wait(simdata->compute);
   self->simdata->waiting_action = NULL;
 
+  DEBUG2("Finished waiting for execution of action %p, state = %d", simdata->compute, SIMIX_req_host_execution_get_state(task->simdata->compute));
 
-  SIMIX_mutex_unlock(simdata->mutex);
   simdata->refcount--;
 
-  if (SIMIX_action_get_state(task->simdata->compute) == SURF_ACTION_DONE) {
+  if (SIMIX_req_host_execution_get_state(task->simdata->compute) == SIMIX_DONE) {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->computation_amount = 0.0;
     simdata->comm = NULL;
     simdata->compute = NULL;
     MSG_RETURN(MSG_OK);
-  } else if (SIMIX_host_get_state(SIMIX_host_self()) == 0) {
+  } else if (SIMIX_req_host_get_state(SIMIX_host_self()) == 0) {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->comm = NULL;
     simdata->compute = NULL;
     MSG_RETURN(MSG_HOST_FAILURE);
   } else {
     /* action ended, set comm and compute = NULL, the actions is already destroyed in the main function */
-    SIMIX_action_destroy(task->simdata->compute);
+    SIMIX_req_host_execution_destroy(task->simdata->compute);
     simdata->comm = NULL;
     simdata->compute = NULL;
     MSG_RETURN(MSG_TASK_CANCELLED);
   }
-
 }
 
 
@@ -240,60 +230,37 @@ MSG_error_t MSG_parallel_task_execute(m_task_t task)
  */
 MSG_error_t MSG_process_sleep(double nb_sec)
 {
-  smx_action_t act_sleep;
-  m_process_t proc = MSG_process_self();
-  e_surf_action_state_t state = SURF_ACTION_NOT_IN_THE_SYSTEM;
-  smx_mutex_t mutex;
-  smx_cond_t cond;
+  e_smx_state_t state;
+  /*m_process_t proc = MSG_process_self();*/
 
 #ifdef HAVE_TRACING
-  TRACE_msg_process_sleep_in (MSG_process_self());
+  TRACE_msg_process_sleep_in(MSG_process_self());
 #endif
 
   /* create action to sleep */
-  act_sleep =
-    SIMIX_action_sleep(SIMIX_process_get_host(proc->simdata->s_process),
-                       nb_sec);
-
-  mutex = SIMIX_mutex_init();
-  SIMIX_mutex_lock(mutex);
-
-  /* create conditional and register action to it */
-  cond = SIMIX_cond_init();
-
-  proc->simdata->waiting_action = act_sleep;
-  SIMIX_register_action_to_condition(act_sleep, cond);
-  do {
-    SIMIX_cond_wait(cond, mutex);
-    state = SIMIX_action_get_state(act_sleep);
-  } while (state == SURF_ACTION_READY || state == SURF_ACTION_RUNNING);
-  proc->simdata->waiting_action = NULL;
-  SIMIX_unregister_action_to_condition(act_sleep, cond);
-  SIMIX_mutex_unlock(mutex);
-
-  /* remove variables */
-  SIMIX_cond_destroy(cond);
-  SIMIX_mutex_destroy(mutex);
-
-  if (SIMIX_action_get_state(act_sleep) == SURF_ACTION_DONE) {
-    if (SIMIX_host_get_state(SIMIX_host_self()) == SURF_RESOURCE_OFF) {
-      SIMIX_action_destroy(act_sleep);
+  state = SIMIX_req_process_sleep(nb_sec);
+
+  /*proc->simdata->waiting_action = act_sleep;
+
+  FIXME: check if not setting the waiting_action breaks something on msg
+  
+  proc->simdata->waiting_action = NULL;*/
+  
+  if (state == SIMIX_DONE) {
+    if (SIMIX_req_host_get_state(SIMIX_host_self()) == SURF_RESOURCE_OFF) {
 #ifdef HAVE_TRACING
-      TRACE_msg_process_sleep_out (MSG_process_self());
+      TRACE_msg_process_sleep_out(MSG_process_self());
 #endif
       MSG_RETURN(MSG_HOST_FAILURE);
     }
   } else {
-    SIMIX_action_destroy(act_sleep);
 #ifdef HAVE_TRACING
-    TRACE_msg_process_sleep_out (MSG_process_self());
+    TRACE_msg_process_sleep_out(MSG_process_self());
 #endif
     MSG_RETURN(MSG_HOST_FAILURE);
   }
-
-  SIMIX_action_destroy(act_sleep);
 #ifdef HAVE_TRACING
-  TRACE_msg_process_sleep_out (MSG_process_self());
+  TRACE_msg_process_sleep_out(MSG_process_self());
 #endif
   MSG_RETURN(MSG_OK);
 }
@@ -303,15 +270,15 @@ MSG_error_t MSG_process_sleep(double nb_sec)
  *
  * It takes three parameters.
  * \param task a memory location for storing a #m_task_t. It will
  hold a task when this function will return. Thus \a task should not
  be equal to \c NULL and \a *task should be equal to \c NULL. If one of
  those two condition does not hold, there will be a warning message.
+ hold a task when this function will return. Thus \a task should not
+ be equal to \c NULL and \a *task should be equal to \c NULL. If one of
+ those two condition does not hold, there will be a warning message.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \param host the host that is to be watched.
  * \return #MSG_FATAL if \a task is equal to \c NULL, #MSG_WARNING
  if \a *task is not equal to \c NULL, and #MSG_OK otherwise.
+ if \a *task is not equal to \c NULL, and #MSG_OK otherwise.
  */
 MSG_error_t
 MSG_task_get_from_host(m_task_t * task, m_channel_t channel, m_host_t host)
@@ -324,12 +291,12 @@ MSG_task_get_from_host(m_task_t * task, m_channel_t channel, m_host_t host)
  *
  * It takes two parameters.
  * \param task a memory location for storing a #m_task_t. It will
  hold a task when this function will return. Thus \a task should not
  be equal to \c NULL and \a *task should be equal to \c NULL. If one of
  those two condition does not hold, there will be a warning message.
+ hold a task when this function will return. Thus \a task should not
+ be equal to \c NULL and \a *task should be equal to \c NULL. If one of
+ those two condition does not hold, there will be a warning message.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \return #MSG_FATAL if \a task is equal to \c NULL, #MSG_WARNING
  * if \a *task is not equal to \c NULL, and #MSG_OK otherwise.
  */
@@ -343,18 +310,18 @@ MSG_error_t MSG_task_get(m_task_t * task, m_channel_t channel)
  *
  * It takes three parameters.
  * \param task a memory location for storing a #m_task_t. It will
  hold a task when this function will return. Thus \a task should not
  be equal to \c NULL and \a *task should be equal to \c NULL. If one of
  those two condition does not hold, there will be a warning message.
+ hold a task when this function will return. Thus \a task should not
+ be equal to \c NULL and \a *task should be equal to \c NULL. If one of
+ those two condition does not hold, there will be a warning message.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \param max_duration the maximum time to wait for a task before giving
   up. In such a case, #MSG_TRANSFER_FAILURE will be returned, \a task
   will not be modified and will still be
   equal to \c NULL when returning.
+ up. In such a case, #MSG_TRANSFER_FAILURE will be returned, \a task
+ will not be modified and will still be
+ equal to \c NULL when returning.
  * \return #MSG_FATAL if \a task is equal to \c NULL, #MSG_WARNING
  if \a *task is not equal to \c NULL, and #MSG_OK otherwise.
+ if \a *task is not equal to \c NULL, and #MSG_OK otherwise.
  */
 MSG_error_t
 MSG_task_get_with_timeout(m_task_t * task, m_channel_t channel,
@@ -377,12 +344,14 @@ MSG_task_get_ext(m_task_t * task, m_channel_t channel, double timeout,
               channel);
 
   return
-    MSG_mailbox_get_task_ext(MSG_mailbox_get_by_channel
-                             (MSG_host_self(), channel), task, host, timeout);
+      MSG_mailbox_get_task_ext(MSG_mailbox_get_by_channel
+                               (MSG_host_self(), channel), task, host,
+                               timeout);
 }
 
 MSG_error_t
-MSG_task_receive_from_host(m_task_t * task, const char *alias, m_host_t host)
+MSG_task_receive_from_host(m_task_t * task, const char *alias,
+                           m_host_t host)
 {
   return MSG_task_receive_ext(task, alias, -1, host);
 }
@@ -403,10 +372,199 @@ MSG_error_t
 MSG_task_receive_ext(m_task_t * task, const char *alias, double timeout,
                      m_host_t host)
 {
-  return MSG_mailbox_get_task_ext(MSG_mailbox_get_by_alias(alias), task, host,
-                                  timeout);
+  DEBUG1
+      ("MSG_task_receive_ext: Trying to receive a message on mailbox '%s'",
+       alias);
+  return MSG_mailbox_get_task_ext(MSG_mailbox_get_by_alias(alias), task,
+                                  host, timeout);
 }
 
+/** \ingroup msg_gos_functions
+ * \brief Send a task on a channel.
+ *
+ * This function takes two parameter.
+ * \param task a #m_task_t to send on another location.
+ * \param alias the channel on which the agent should put this
+ task. This value has to be >=0 and < than the maximal number of
+ channels fixed with MSG_set_channel_number().
+ * \return the msg_comm_t communication.
+ */
+msg_comm_t MSG_task_isend(m_task_t task, const char *alias)
+{
+  simdata_task_t t_simdata = NULL;
+  m_process_t process = MSG_process_self();
+  msg_mailbox_t mailbox = MSG_mailbox_get_by_alias(alias);
+
+  CHECK_HOST();
+
+  /* FIXME: these functions are not tracable */
+
+  /* Prepare the task to send */
+  t_simdata = task->simdata;
+  t_simdata->sender = process;
+  t_simdata->source = MSG_host_self();
+
+  xbt_assert0(t_simdata->refcount == 1,
+              "This task is still being used somewhere else. You cannot send it now. Go fix your code!");
+
+  t_simdata->refcount++;
+  msg_global->sent_msg++;
+  process->simdata->waiting_task = task;
+
+  /* Send it by calling SIMIX network layer */
+
+  return SIMIX_req_comm_isend(mailbox, t_simdata->message_size,
+                             t_simdata->rate, task, sizeof(void *),
+                             &t_simdata->comm);
+}
+
+/** \ingroup msg_gos_functions
+ * \brief Listen on a channel for receiving a task from an asynchronous communication.
+ *
+ * It takes two parameters.
+ * \param task a memory location for storing a #m_task_t.
+ * \param alias the channel on which the agent should be
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
+ * \return the msg_comm_t communication.
+ */
+msg_comm_t MSG_task_irecv(m_task_t * task, const char *alias)
+{
+  smx_rdv_t rdv = MSG_mailbox_get_by_alias(alias);
+
+  CHECK_HOST();
+
+  /* FIXME: these functions are not tracable */
+
+  /* Sanity check */
+  xbt_assert0(task, "Null pointer for the task storage");
+
+  if (*task)
+    CRITICAL0
+        ("MSG_task_get() was asked to write in a non empty task struct.");
+
+  /* Try to receive it by calling SIMIX network layer */
+  return SIMIX_req_comm_irecv(rdv, task, NULL);
+}
+
+/** \ingroup msg_gos_functions
+ * \brief Test the status of a communication.
+ *
+ * It takes one parameter.
+ * \param comm the communication to test.
+ * \return the status of the communication:
+ *             TRUE : the communication is completed
+ *             FALSE: the communication is incompleted
+ * If the status is FALSE, don't forget to use MSG_process_sleep() after the test.
+ */
+int MSG_comm_test(msg_comm_t comm)
+{
+  return SIMIX_req_comm_test(comm);
+}
+
+/** \ingroup msg_gos_functions
+ * \brief After received TRUE to MSG_comm_test(), the communication must be destroyed.
+ *
+ * It takes one parameter.
+ * \param comm the communication to destroy.
+ */
+void MSG_comm_destroy(msg_comm_t comm)
+{
+  if (SIMIX_req_comm_get_src_proc(comm) != SIMIX_process_self()) {
+    m_task_t task;
+    task = (m_task_t) SIMIX_req_comm_get_src_buff(comm);
+    task->simdata->refcount--;
+  }
+  SIMIX_req_comm_destroy(comm);
+}
+
+/** \ingroup msg_gos_functions
+ * \brief Wait for the completion of a communication.
+ *
+ * It takes two parameters.
+ * \param comm the communication to wait.
+ * \param timeout Wait until the communication terminates or the timeout occurs
+ * \return MSG_error_t
+ */
+MSG_error_t MSG_comm_wait(msg_comm_t comm, double timeout)
+{
+  xbt_ex_t e;
+  MSG_error_t res = MSG_OK;
+  TRY {
+    SIMIX_req_comm_wait(comm, timeout);
+
+    if (SIMIX_req_comm_get_src_proc(comm) != SIMIX_process_self()) {
+      m_task_t task;
+      task = (m_task_t) SIMIX_req_comm_get_src_buff(comm);
+      task->simdata->refcount--;
+    }
+
+    /* FIXME: these functions are not tracable */
+  }
+  CATCH(e) {
+    switch (e.category) {
+    case host_error:
+      res = MSG_HOST_FAILURE;
+      break;
+    case network_error:
+      res = MSG_TRANSFER_FAILURE;
+      break;
+    case timeout_error:
+      res = MSG_TIMEOUT;
+      break;
+    default:
+      xbt_die(bprintf("Unhandled SIMIX network exception: %s", e.msg));
+    }
+    xbt_ex_free(e);
+  }
+  return res;
+}
+
+/** \ingroup msg_gos_functions
+* \brief This function is called by a sender and permit to wait for each communication
+*
+* It takes three parameters.
+* \param comm a vector of communication
+* \param nb_elem is the size of the comm vector
+* \param timeout for each call of  MSG_comm_wait
+*/
+void MSG_comm_waitall(msg_comm_t * comm, int nb_elem, double timeout)
+{
+  int i = 0;
+  for (i = 0; i < nb_elem; i++) {
+    MSG_comm_wait(comm[i], timeout);
+  }
+}
+
+/** \ingroup msg_gos_functions
+* \brief This function wait for the first completed communication
+*
+* It takes on parameter.
+* \param comms a vector of communication
+* \return the position of the completed communication from the xbt_dynar_t.
+*/
+int MSG_comm_waitany(xbt_dynar_t comms)
+{
+  return SIMIX_req_comm_waitany(comms);
+}
+
+/** \ingroup msg_gos_functions
+* \brief This function wait for the first completed communication
+*
+* It takes on parameter.
+* \param comms a vector of communication
+* \return the position of the completed communication, if any, or -1 if none was completed
+*/
+int MSG_comm_testany(xbt_dynar_t comms)
+{
+  return SIMIX_req_comm_testany(comms);
+}
+
+m_task_t MSG_comm_get_task(msg_comm_t comm)
+{
+  xbt_assert0(comm, "Invalid parameters");
+  return (m_task_t) SIMIX_req_comm_get_src_buff(comm);
+}
 
 /** \ingroup msg_gos_functions
  * \brief Put a task on a channel of an host and waits for the end of the
@@ -415,19 +573,19 @@ MSG_task_receive_ext(m_task_t * task, const char *alias, double timeout,
  * This function is used for describing the behavior of an agent. It
  * takes three parameter.
  * \param task a #m_task_t to send on another location. This task
  will not be usable anymore when the function will return. There is
  no automatic task duplication and you have to save your parameters
  before calling this function. Tasks are unique and once it has been
  sent to another location, you should not access it anymore. You do
  not need to call MSG_task_destroy() but to avoid using, as an
  effect of inattention, this task anymore, you definitely should
  renitialize it with #MSG_TASK_UNINITIALIZED. Note that this task
  can be transfered iff it has been correctly created with
  MSG_task_create().
+ will not be usable anymore when the function will return. There is
+ no automatic task duplication and you have to save your parameters
+ before calling this function. Tasks are unique and once it has been
+ sent to another location, you should not access it anymore. You do
+ not need to call MSG_task_destroy() but to avoid using, as an
+ effect of inattention, this task anymore, you definitely should
+ renitialize it with #MSG_TASK_UNINITIALIZED. Note that this task
+ can be transfered iff it has been correctly created with
+ MSG_task_create().
  * \param dest the destination of the message
  * \param channel the channel on which the agent should put this
  task. This value has to be >=0 and < than the maximal number of
  channels fixed with MSG_set_channel_number().
+ task. This value has to be >=0 and < than the maximal number of
+ channels fixed with MSG_set_channel_number().
  * \return #MSG_FATAL if \a task is not properly initialized and
  * #MSG_OK otherwise. Returns #MSG_HOST_FAILURE if the host on which
  * this function was called was shut down. Returns
@@ -460,43 +618,45 @@ MSG_task_put_bounded(m_task_t task, m_host_t dest, m_channel_t channel,
  * This function is used for describing the behavior of an agent. It
  * takes four parameter.
  * \param task a #m_task_t to send on another location. This task
  will not be usable anymore when the function will return. There is
  no automatic task duplication and you have to save your parameters
  before calling this function. Tasks are unique and once it has been
  sent to another location, you should not access it anymore. You do
  not need to call MSG_task_destroy() but to avoid using, as an
  effect of inattention, this task anymore, you definitely should
  renitialize it with #MSG_TASK_UNINITIALIZED. Note that this task
  can be transfered iff it has been correctly created with
  MSG_task_create().
+ will not be usable anymore when the function will return. There is
+ no automatic task duplication and you have to save your parameters
+ before calling this function. Tasks are unique and once it has been
+ sent to another location, you should not access it anymore. You do
+ not need to call MSG_task_destroy() but to avoid using, as an
+ effect of inattention, this task anymore, you definitely should
+ renitialize it with #MSG_TASK_UNINITIALIZED. Note that this task
+ can be transfered iff it has been correctly created with
+ MSG_task_create().
  * \param dest the destination of the message
  * \param channel the channel on which the agent should put this
  task. This value has to be >=0 and < than the maximal number of
  channels fixed with MSG_set_channel_number().
+ task. This value has to be >=0 and < than the maximal number of
+ channels fixed with MSG_set_channel_number().
  * \param timeout the maximum time to wait for a task before giving
   up. In such a case, #MSG_TRANSFER_FAILURE will be returned, \a task
   will not be modified
+ up. In such a case, #MSG_TRANSFER_FAILURE will be returned, \a task
+ will not be modified
  * \return #MSG_FATAL if \a task is not properly initialized and
-   #MSG_OK otherwise. Returns #MSG_HOST_FAILURE if the host on which
-   this function was called was shut down. Returns
-   #MSG_TRANSFER_FAILURE if the transfer could not be properly done
-   (network failure, dest failure, timeout...)
+#MSG_OK otherwise. Returns #MSG_HOST_FAILURE if the host on which
+this function was called was shut down. Returns
+#MSG_TRANSFER_FAILURE if the transfer could not be properly done
+(network failure, dest failure, timeout...)
  */
 MSG_error_t
-MSG_task_put_with_timeout(m_task_t task, m_host_t dest, m_channel_t channel,
-                          double timeout)
+MSG_task_put_with_timeout(m_task_t task, m_host_t dest,
+                          m_channel_t channel, double timeout)
 {
   xbt_assert1((channel >= 0)
               && (channel < msg_global->max_channel), "Invalid channel %d",
               channel);
 
+  DEBUG1("MSG_task_put_with_timout: Trying to send a task to '%s'", dest->name);
   return
-    MSG_mailbox_put_with_timeout(MSG_mailbox_get_by_channel(dest, channel),
-                                 task, timeout);
+      MSG_mailbox_put_with_timeout(MSG_mailbox_get_by_channel
+                                   (dest, channel), task, timeout);
 }
 
 MSG_error_t MSG_task_send(m_task_t task, const char *alias)
 {
+  DEBUG1("MSG_task_send: Trying to send a message on mailbox '%s'", alias);
   return MSG_task_send_with_timeout(task, alias, -1);
 }
 
@@ -510,10 +670,11 @@ MSG_task_send_bounded(m_task_t task, const char *alias, double maxrate)
 
 
 MSG_error_t
-MSG_task_send_with_timeout(m_task_t task, const char *alias, double timeout)
+MSG_task_send_with_timeout(m_task_t task, const char *alias,
+                           double timeout)
 {
-  return MSG_mailbox_put_with_timeout(MSG_mailbox_get_by_alias(alias), task,
-                                      timeout);
+  return MSG_mailbox_put_with_timeout(MSG_mailbox_get_by_alias(alias),
+                                      task, timeout);
 }
 
 int MSG_task_listen(const char *alias)
@@ -528,8 +689,8 @@ int MSG_task_listen(const char *alias)
  *
  * It takes one parameter.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \return 1 if there is a pending communication and 0 otherwise
  */
 int MSG_task_Iprobe(m_channel_t channel)
@@ -541,22 +702,22 @@ int MSG_task_Iprobe(m_channel_t channel)
   CHECK_HOST();
 
   return
-    !MSG_mailbox_is_empty(MSG_mailbox_get_by_channel
-                          (MSG_host_self(), channel));
+      !MSG_mailbox_is_empty(MSG_mailbox_get_by_channel
+                            (MSG_host_self(), channel));
 }
 
 /** \ingroup msg_gos_functions
 
  * \brief Return the number of tasks waiting to be received on a \a
  channel and sent by \a host.
+ channel and sent by \a host.
  *
  * It takes two parameters.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \param host the host that is to be watched.
  * \return the number of tasks waiting to be received on \a channel
  and sent by \a host.
+ and sent by \a host.
  */
 int MSG_task_probe_from_host(int channel, m_host_t host)
 {
@@ -567,9 +728,9 @@ int MSG_task_probe_from_host(int channel, m_host_t host)
   CHECK_HOST();
 
   return
-    MSG_mailbox_get_count_host_waiting_tasks(MSG_mailbox_get_by_channel
-                                             (MSG_host_self(), channel),
-                                             host);
+      MSG_mailbox_get_count_host_waiting_tasks(MSG_mailbox_get_by_channel
+                                               (MSG_host_self(), channel),
+                                               host);
 
 }
 
@@ -578,8 +739,8 @@ int MSG_task_listen_from_host(const char *alias, m_host_t host)
   CHECK_HOST();
 
   return
-    MSG_mailbox_get_count_host_waiting_tasks(MSG_mailbox_get_by_alias(alias),
-                                             host);
+      MSG_mailbox_get_count_host_waiting_tasks(MSG_mailbox_get_by_alias
+                                               (alias), host);
 }
 
 /** \ingroup msg_gos_functions
@@ -587,8 +748,8 @@ int MSG_task_listen_from_host(const char *alias, m_host_t host)
  *
  * It takes one parameter.
  * \param channel the channel on which the agent should be
  listening. This value has to be >=0 and < than the maximal
  number of channels fixed with MSG_set_channel_number().
+ listening. This value has to be >=0 and < than the maximal
+ number of channels fixed with MSG_set_channel_number().
  * \return -1 if there is no pending communication and the PID of the process who sent it otherwise
  */
 int MSG_task_probe_from(m_channel_t channel)
@@ -616,169 +777,9 @@ int MSG_task_listen_from(const char *alias)
 
   CHECK_HOST();
 
-  if (NULL == (task = MSG_mailbox_get_head(MSG_mailbox_get_by_alias(alias))))
+  if (NULL ==
+      (task = MSG_mailbox_get_head(MSG_mailbox_get_by_alias(alias))))
     return -1;
 
   return MSG_process_get_PID(task->simdata->sender);
 }
-
-/** \ingroup msg_gos_functions
- * \brief Wait for at most \a max_duration second for a task reception
-   on \a channel.
-
- * \a PID is updated with the PID of the first process that triggered this event if any.
- *
- * It takes three parameters:
- * \param channel the channel on which the agent should be
-   listening. This value has to be >=0 and < than the maximal.
-   number of channels fixed with MSG_set_channel_number().
- * \param PID a memory location for storing an int.
- * \param timeout the maximum time to wait for a task before
-    giving up. In the case of a reception, *\a PID will be updated
-    with the PID of the first process to send a task.
- * \return #MSG_HOST_FAILURE if the host is shut down in the meantime
-   and #MSG_OK otherwise.
- */
-MSG_error_t
-MSG_channel_select_from(m_channel_t channel, double timeout, int *PID)
-{
-  m_host_t h = NULL;
-  simdata_host_t h_simdata = NULL;
-  m_task_t t;
-  int first_time = 1;
-  smx_cond_t cond;
-  msg_mailbox_t mailbox;
-
-  xbt_assert1((channel >= 0)
-              && (channel < msg_global->max_channel), "Invalid channel %d",
-              channel);
-
-  if (PID) {
-    *PID = -1;
-  }
-
-  if (timeout == 0.0) {
-    *PID = MSG_task_probe_from(channel);
-    MSG_RETURN(MSG_OK);
-  } else {
-    CHECK_HOST();
-    h = MSG_host_self();
-    h_simdata = h->simdata;
-
-    mailbox = MSG_mailbox_get_by_channel(MSG_host_self(), channel);
-
-    while (MSG_mailbox_is_empty(mailbox)) {
-      if (timeout > 0) {
-        if (!first_time) {
-          MSG_RETURN(MSG_OK);
-        }
-      }
-
-      SIMIX_mutex_lock(h_simdata->mutex);
-
-      xbt_assert1(!MSG_mailbox_get_cond(mailbox),
-                  "A process is already blocked on this channel %d", channel);
-
-      cond = SIMIX_cond_init();
-
-      MSG_mailbox_set_cond(mailbox, cond);
-
-      if (timeout > 0) {
-        SIMIX_cond_wait_timeout(cond, h_simdata->mutex, timeout);
-      } else {
-        SIMIX_cond_wait(cond, h_simdata->mutex);
-      }
-
-      SIMIX_cond_destroy(cond);
-      SIMIX_mutex_unlock(h_simdata->mutex);
-
-      if (SIMIX_host_get_state(h_simdata->smx_host) == 0) {
-        MSG_RETURN(MSG_HOST_FAILURE);
-      }
-
-      MSG_mailbox_set_cond(mailbox, NULL);
-      first_time = 0;
-    }
-
-    if (NULL == (t = MSG_mailbox_get_head(mailbox)))
-      MSG_RETURN(MSG_OK);
-
-
-    if (PID) {
-      *PID = MSG_process_get_PID(t->simdata->sender);
-    }
-
-    MSG_RETURN(MSG_OK);
-  }
-}
-
-
-MSG_error_t MSG_alias_select_from(const char *alias, double timeout, int *PID)
-{
-  m_host_t h = NULL;
-  simdata_host_t h_simdata = NULL;
-  m_task_t t;
-  int first_time = 1;
-  smx_cond_t cond;
-  msg_mailbox_t mailbox;
-
-  if (PID) {
-    *PID = -1;
-  }
-
-  if (timeout == 0.0) {
-    *PID = MSG_task_listen_from(alias);
-    MSG_RETURN(MSG_OK);
-  } else {
-    CHECK_HOST();
-    h = MSG_host_self();
-    h_simdata = h->simdata;
-
-    DEBUG2("Probing on alias %s (%s)", alias, h->name);
-
-    mailbox = MSG_mailbox_get_by_alias(alias);
-
-    while (MSG_mailbox_is_empty(mailbox)) {
-      if (timeout > 0) {
-        if (!first_time) {
-          MSG_RETURN(MSG_OK);
-        }
-      }
-
-      SIMIX_mutex_lock(h_simdata->mutex);
-
-      xbt_assert1(!MSG_mailbox_get_cond(mailbox),
-                  "A process is already blocked on this alias %s", alias);
-
-      cond = SIMIX_cond_init();
-
-      MSG_mailbox_set_cond(mailbox, cond);
-
-      if (timeout > 0) {
-        SIMIX_cond_wait_timeout(cond, h_simdata->mutex, timeout);
-      } else {
-        SIMIX_cond_wait(cond, h_simdata->mutex);
-      }
-
-      SIMIX_cond_destroy(cond);
-      SIMIX_mutex_unlock(h_simdata->mutex);
-
-      if (SIMIX_host_get_state(h_simdata->smx_host) == 0) {
-        MSG_RETURN(MSG_HOST_FAILURE);
-      }
-
-      MSG_mailbox_set_cond(mailbox, NULL);
-      first_time = 0;
-    }
-
-    if (NULL == (t = MSG_mailbox_get_head(mailbox)))
-      MSG_RETURN(MSG_OK);
-
-
-    if (PID) {
-      *PID = MSG_process_get_PID(t->simdata->sender);
-    }
-
-    MSG_RETURN(MSG_OK);
-  }
-}