Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
fix 32 bits build (bis, damnit)
[simgrid.git] / src / simdag / sd_task.cpp
index c1b86b4..cc7d15c 100644 (file)
-/* Copyright (c) 2006-2019. The SimGrid Team.
+/* Copyright (c) 2006-2021. 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 "simdag_private.hpp"
+#include "simgrid/kernel/routing/NetPoint.hpp"
 #include "src/surf/HostImpl.hpp"
 #include "src/surf/surf_interface.hpp"
 #include <algorithm>
 
 XBT_LOG_NEW_DEFAULT_SUBCATEGORY(sd_task, sd, "Logging specific to SimDag (task)");
 
-/* Destroys the data memorized by SD_task_schedule. Task state must be SD_SCHEDULED or SD_RUNNABLE. */
-static void __SD_task_destroy_scheduling_data(SD_task_t task)
+namespace simgrid {
+
+template class xbt::Extendable<sd::Task>;
+
+namespace sd {
+
+Task* Task::create(const std::string& name, double amount, void* userdata)
 {
-  if (task->state != SD_SCHEDULED && task->state != SD_RUNNABLE)
-    throw std::invalid_argument(
-        simgrid::xbt::string_printf("Task '%s' must be SD_SCHEDULED or SD_RUNNABLE", SD_task_get_name(task)));
+  auto task = new Task();
+  task->set_name(name);
+  task->set_amount(amount);
+  task->set_data(userdata);
+  task->allocation_ = new std::vector<sg_host_t>();
+  sd_global->initial_tasks.insert(task);
+
+  return task;
+}
+
+Task* Task::create_comm_e2e(const std::string& name, double amount, void* userdata)
+{
+  auto task              = create(name, amount, userdata);
+  task->bytes_amount_    = xbt_new0(double, 4);
+  task->bytes_amount_[2] = amount;
+  task->set_kind(SD_TASK_COMM_E2E);
+
+  return task;
+}
+
+Task* Task::create_comp_seq(const std::string& name, double amount, void* userdata)
+{
+  auto task              = create(name, amount, userdata);
+  task->flops_amount_    = xbt_new0(double, 1);
+  task->flops_amount_[0] = amount;
+  task->set_kind(SD_TASK_COMP_SEQ);
+
+  return task;
+}
+
+Task* Task::create_comp_par_amdahl(const std::string& name, double amount, void* userdata, double alpha)
+{
+  xbt_assert(alpha < 1. && alpha >= 0., "Invalid parameter: alpha must be in [0.;1.[");
+
+  auto task = create(name, amount, userdata);
+  task->set_alpha(alpha);
+  task->set_kind(SD_TASK_COMP_PAR_AMDAHL);
+
+  return task;
+}
+
+Task* Task::create_comm_par_mxn_1d_block(const std::string& name, double amount, void* userdata)
+{
+  auto task = create(name, amount, userdata);
+  task->set_kind(SD_TASK_COMM_PAR_MXN_1D_BLOCK);
+
+  return task;
+}
+
+void Task::distribute_comp_amdahl(int count)
+{
+  xbt_assert(kind_ == SD_TASK_COMP_PAR_AMDAHL,
+             "Task %s is not a SD_TASK_COMP_PAR_AMDAHL typed task."
+             "Cannot use this function.",
+             get_cname());
+  flops_amount_ = xbt_new0(double, count);
+  for (int i = 0; i < count; i++)
+    flops_amount_[i] = (alpha_ + (1 - alpha_) / count) * amount_;
+}
+
+void Task::build_MxN_1D_block_matrix(int src_nb, int dst_nb)
+{
+  xbt_assert(kind_ == SD_TASK_COMM_PAR_MXN_1D_BLOCK,
+             "Task %s is not a SD_TASK_COMM_PAR_MXN_1D_BLOCK typed task."
+             "Cannot use this function.",
+             get_cname());
+  xbt_free(bytes_amount_);
+  bytes_amount_ = xbt_new0(double, allocation_->size() * allocation_->size());
+
+  for (int i = 0; i < src_nb; i++) {
+    double src_start = i * amount_ / src_nb;
+    double src_end   = src_start + amount_ / src_nb;
+    for (int j = 0; j < dst_nb; j++) {
+      double dst_start = j * amount_ / dst_nb;
+      double dst_end   = dst_start + amount_ / dst_nb;
+      XBT_VERB("(%d->%d): (%.2f, %.2f)-> (%.2f, %.2f)", i, j, src_start, src_end, dst_start, dst_end);
+      bytes_amount_[i * (src_nb + dst_nb) + src_nb + j] = 0.0;
+      if ((src_end > dst_start) && (dst_end > src_start)) { /* There is something to send */
+        bytes_amount_[i * (src_nb + dst_nb) + src_nb + j] = std::min(src_end, dst_end) - std::max(src_start, dst_start);
+        XBT_VERB("==> %.2f", bytes_amount_[i * (src_nb + dst_nb) + src_nb + j]);
+      }
+    }
+  }
+}
+
+bool Task::is_parent_of(Task* task) const
+{
+  return (successors_.find(task) != successors_.end() || outputs_.find(task) != outputs_.end());
+}
+
+bool Task::is_child_of(Task* task) const
+{
+  return (inputs_.find(task) != inputs_.end() || predecessors_.find(task) != predecessors_.end());
+}
+
+void Task::set_amount(double amount)
+{
+  amount_ = amount;
+  if (kind_ == SD_TASK_COMP_SEQ)
+    flops_amount_[0] = amount;
+  if (kind_ == SD_TASK_COMM_E2E) {
+    bytes_amount_[2] = amount;
+  }
+}
+
+void Task::set_rate(double rate)
+{
+  xbt_assert(kind_ == SD_TASK_COMM_E2E, "The rate can be modified for end-to-end communications only.");
+  if (state_ < SD_RUNNING) {
+    rate_ = rate;
+  } else {
+    XBT_WARN("Task %p has started. Changing rate is ineffective.", this);
+  }
+}
+void Task::set_state(e_SD_task_state_t new_state)
+{
+  std::set<Task*>::iterator idx;
+  XBT_DEBUG("Set state of '%s' to %d", get_cname(), new_state);
+  if ((new_state == SD_NOT_SCHEDULED || new_state == SD_SCHEDULABLE) && state_ == SD_FAILED) {
+    sd_global->completed_tasks.erase(this);
+    sd_global->initial_tasks.insert(this);
+  }
+
+  if (new_state == SD_SCHEDULED && state_ == SD_RUNNABLE) {
+    sd_global->initial_tasks.insert(this);
+    sd_global->runnable_tasks.erase(this);
+  }
+
+  if (new_state == SD_RUNNABLE) {
+    idx = sd_global->initial_tasks.find(this);
+    if (idx != sd_global->initial_tasks.end()) {
+      sd_global->runnable_tasks.insert(*idx);
+      sd_global->initial_tasks.erase(idx);
+    }
+  }
+
+  if (new_state == SD_RUNNING)
+    sd_global->runnable_tasks.erase(this);
+
+  if (new_state == SD_DONE || new_state == SD_FAILED) {
+    sd_global->completed_tasks.insert(this);
+    start_time_ = surf_action_->get_start_time();
+    if (new_state == SD_DONE) {
+      finish_time_ = surf_action_->get_finish_time();
+#if SIMGRID_HAVE_JEDULE
+      jedule_log_sd_event(this);
+#endif
+    } else
+      finish_time_ = simgrid_get_clock();
+    surf_action_->unref();
+    surf_action_ = nullptr;
+    allocation_->clear();
+  }
+
+  state_ = new_state;
+
+  if (watch_points_ & new_state) {
+    XBT_VERB("Watch point reached with task '%s'!", get_cname());
+    sd_global->watch_point_reached = true;
+    unwatch(new_state); /* remove the watch point */
+  }
+}
+
+double Task::get_alpha() const
+{
+  xbt_assert(kind_ == SD_TASK_COMP_PAR_AMDAHL, "Alpha parameter is not defined for this kind of task");
+  return alpha_;
+}
+
+double Task::get_remaining_amount() const
+{
+  if (surf_action_)
+    return surf_action_->get_remains();
+  else
+    return (state_ == SD_DONE) ? 0 : amount_;
+}
+
+double Task::get_start_time() const
+{
+  if (surf_action_)
+    return surf_action_->get_start_time();
+  else
+    return start_time_;
+}
+
+double Task::get_finish_time() const
+{
+  if (surf_action_) /* should never happen as actions are destroyed right after their completion */
+    return surf_action_->get_finish_time();
+  else
+    return finish_time_;
+}
+
+void Task::set_sender_side_allocation(unsigned long count, const std::vector<s4u::Host*>* sender)
+{
+  for (unsigned long i = 0; i < count; i++)
+    allocation_->push_back(sender->at(i));
+}
+
+void Task::set_receiver_side_allocation(unsigned long count, const std::vector<s4u::Host*>* receiver)
+{
+  for (unsigned long i = 0; i < count; i++)
+    allocation_->insert(allocation_->begin() + i, receiver->at(i));
+}
+
+void Task::watch(e_SD_task_state_t state)
+{
+  if (state & SD_NOT_SCHEDULED)
+    throw std::invalid_argument("Cannot add a watch point for state SD_NOT_SCHEDULED");
+
+  watch_points_ = watch_points_ | state;
+}
+
+void Task::unwatch(e_SD_task_state_t state)
+{
+  xbt_assert(state != SD_NOT_SCHEDULED, "SimDag error: Cannot have a watch point for state SD_NOT_SCHEDULED");
+  watch_points_ = watch_points_ & ~state;
+}
+
+void Task::dump() const
+{
+  XBT_INFO("Displaying task %s", get_cname());
+  if (state_ == SD_RUNNABLE)
+    XBT_INFO("  - state: runnable");
+  else if (state_ < SD_RUNNABLE)
+    XBT_INFO("  - state: %s not runnable", __get_state_name(state_));
+  else
+    XBT_INFO("  - state: not runnable %s", __get_state_name(state_));
+
+  if (kind_ != 0) {
+    switch (kind_) {
+      case SD_TASK_COMM_E2E:
+        XBT_INFO("  - kind: end-to-end communication");
+        break;
+      case SD_TASK_COMP_SEQ:
+        XBT_INFO("  - kind: sequential computation");
+        break;
+      case SD_TASK_COMP_PAR_AMDAHL:
+        XBT_INFO("  - kind: parallel computation following Amdahl's law");
+        break;
+      case SD_TASK_COMM_PAR_MXN_1D_BLOCK:
+        XBT_INFO("  - kind: MxN data redistribution assuming 1D block distribution");
+        break;
+      default:
+        XBT_INFO("  - (unknown kind %d)", kind_);
+    }
+  }
+
+  XBT_INFO("  - amount: %.0f", amount_);
+  if (kind_ == SD_TASK_COMP_PAR_AMDAHL)
+    XBT_INFO("  - alpha: %.2f", alpha_);
+  XBT_INFO("  - Dependencies to satisfy: %lu", has_unsolved_dependencies());
+  if (has_unsolved_dependencies() > 0) {
+    XBT_INFO("  - pre-dependencies:");
+    for (auto const& it : predecessors_)
+      XBT_INFO("    %s", it->get_cname());
+
+    for (auto const& it : inputs_)
+      XBT_INFO("    %s", it->get_cname());
+  }
+  if (is_waited_by() > 0) {
+    XBT_INFO("  - post-dependencies:");
+
+    for (auto const& it : successors_)
+      XBT_INFO("    %s", it->get_cname());
+    for (auto const& it : outputs_)
+      XBT_INFO("    %s", it->get_cname());
+  }
+}
+
+void Task::released_by(Task* pred)
+{
+  predecessors_.erase(pred);
+  inputs_.erase(pred);
+  XBT_DEBUG("Release dependency on %s: %lu remain(s). Becomes schedulable if %zu=0", get_cname(),
+            has_unsolved_dependencies(), predecessors_.size());
+
+  if (state_ == SD_NOT_SCHEDULED && predecessors_.empty())
+    set_state(SD_SCHEDULABLE);
+
+  if (state_ == SD_SCHEDULED && has_unsolved_dependencies() == 0)
+    set_state(SD_RUNNABLE);
+
+  if (state_ == SD_RUNNABLE && not sd_global->watch_point_reached)
+    run();
+}
+
+void Task::produced_by(Task* pred)
+{
+  start_time_ = pred->get_finish_time();
+  predecessors_.erase(pred);
+  if (state_ == SD_SCHEDULED)
+    set_state(SD_RUNNABLE);
+  else
+    set_state(SD_SCHEDULABLE);
+
+  Task* comm_dst = *(successors_.begin());
+  if (comm_dst->get_state() == SD_NOT_SCHEDULED && comm_dst->get_predecessors().empty()) {
+    XBT_DEBUG("%s is a transfer, %s may be ready now if %zu=0", get_cname(), comm_dst->get_cname(),
+              comm_dst->get_predecessors().size());
+    comm_dst->set_state(SD_SCHEDULABLE);
+  }
+  if (state_ == SD_RUNNABLE && not sd_global->watch_point_reached)
+    run();
+}
+
+void Task::do_schedule()
+{
+  if (state_ > SD_SCHEDULABLE)
+    throw std::invalid_argument(simgrid::xbt::string_printf("Task '%s' has already been scheduled", get_cname()));
+
+  if (has_unsolved_dependencies() == 0)
+    set_state(SD_RUNNABLE);
+  else
+    set_state(SD_SCHEDULED);
+}
+
+void Task::schedule(const std::vector<s4u::Host*>& hosts, const double* flops_amount, const double* bytes_amount,
+                    double rate)
+{
+  unsigned long host_count = hosts.size();
+  rate_                    = rate;
+
+  if (flops_amount) {
+    flops_amount_ = static_cast<double*>(xbt_realloc(flops_amount_, sizeof(double) * host_count));
+    memcpy(flops_amount_, flops_amount, sizeof(double) * host_count);
+  } else {
+    xbt_free(flops_amount_);
+    flops_amount_ = nullptr;
+  }
+
+  unsigned long communication_nb = host_count * host_count;
+  if (bytes_amount) {
+    bytes_amount_ = static_cast<double*>(xbt_realloc(bytes_amount_, sizeof(double) * communication_nb));
+    memcpy(bytes_amount_, bytes_amount, sizeof(double) * communication_nb);
+  } else {
+    xbt_free(bytes_amount_);
+    bytes_amount_ = nullptr;
+  }
+
+  for (unsigned long i = 0; i < host_count; i++)
+    allocation_->push_back(hosts[i]);
+
+  do_schedule();
+}
+
+void Task::schedulev(const std::vector<s4u::Host*>& hosts)
+{
+  xbt_assert(kind_ == SD_TASK_COMP_SEQ || kind_ == SD_TASK_COMP_PAR_AMDAHL,
+             "Task %s is not typed. Cannot automatically schedule it.", get_cname());
+
+  for (unsigned long i = 0; i < hosts.size(); i++)
+    allocation_->push_back(hosts[i]);
+
+  XBT_VERB("Schedule computation task %s on %zu host(s)", get_cname(), allocation_->size());
+
+  if (kind_ == SD_TASK_COMP_SEQ) {
+    if (not flops_amount_) { /*This task has failed and is rescheduled. Reset the flops_amount*/
+      flops_amount_    = xbt_new0(double, 1);
+      flops_amount_[0] = amount_;
+    }
+    XBT_VERB("It costs %.f flops", flops_amount_[0]);
+  }
+
+  if (kind_ == SD_TASK_COMP_PAR_AMDAHL) {
+    distribute_comp_amdahl(hosts.size());
+    XBT_VERB("%.f flops will be distributed following Amdahl's Law", flops_amount_[0]);
+  }
+
+  do_schedule();
+
+  /* Iterate over all inputs and outputs to say where I am located (and start them if runnable) */
+  for (auto const& input : inputs_) {
+    unsigned long src_nb = input->get_allocation_size();
+    unsigned long dst_nb = hosts.size();
+    if (src_nb == 0)
+      XBT_VERB("Sender side of '%s' not scheduled. Set receiver side to '%s''s allocation", input->get_cname(),
+               get_cname());
+    input->set_sender_side_allocation(dst_nb, allocation_);
+
+    if (input->get_allocation_size() > allocation_->size()) {
+      if (kind_ == SD_TASK_COMP_PAR_AMDAHL)
+        input->build_MxN_1D_block_matrix(src_nb, dst_nb);
+
+      input->do_schedule();
+      XBT_VERB("Auto-Schedule Communication task '%s'. Send %.f bytes from %lu hosts to %lu hosts.", input->get_cname(),
+               input->get_amount(), src_nb, dst_nb);
+    }
+  }
+
+  for (auto const& output : outputs_) {
+    unsigned long src_nb = hosts.size();
+    unsigned long dst_nb = output->get_allocation_size();
+    if (dst_nb == 0)
+      XBT_VERB("Receiver side of '%s' not scheduled. Set sender side to '%s''s allocation", output->get_cname(),
+               get_cname());
+    output->set_receiver_side_allocation(src_nb, allocation_);
+
+    if (output->get_allocation_size() > allocation_->size()) {
+      if (kind_ == SD_TASK_COMP_PAR_AMDAHL)
+        output->build_MxN_1D_block_matrix(src_nb, dst_nb);
+
+      output->do_schedule();
+      XBT_VERB("Auto-Schedule Communication task %s. Send %.f bytes from %lu hosts to %lu hosts.", output->get_cname(),
+               output->get_amount(), src_nb, dst_nb);
+    }
+  }
+}
+
+void Task::unschedule()
+{
+  if (state_ == SD_NOT_SCHEDULED || state_ == SD_SCHEDULABLE)
+    throw std::invalid_argument(xbt::string_printf(
+        "Task %s: the state must be SD_SCHEDULED, SD_RUNNABLE, SD_RUNNING or SD_FAILED", get_cname()));
+
+  if (state_ == SD_SCHEDULED || state_ == SD_RUNNABLE) /* if the task is scheduled or runnable */ {
+    allocation_->clear();
+    if (kind_ == SD_TASK_COMP_PAR_AMDAHL || kind_ == SD_TASK_COMM_PAR_MXN_1D_BLOCK) {
+      /* Don't free scheduling data for typed tasks */
+      xbt_free(flops_amount_);
+      xbt_free(bytes_amount_);
+      bytes_amount_ = nullptr;
+      flops_amount_ = nullptr;
+    }
+  }
+
+  if (state_ == SD_RUNNING)
+    /* the task should become SD_FAILED */
+    surf_action_->cancel();
+  else {
+    if (has_unsolved_dependencies() == 0)
+      set_state(SD_SCHEDULABLE);
+    else
+      set_state(SD_NOT_SCHEDULED);
+  }
+  start_time_ = -1.0;
+}
+
+void Task::run()
+{
+  xbt_assert(state_ == SD_RUNNABLE, "Task '%s' is not runnable! Task state: %d", get_cname(), (int)state_);
+  xbt_assert(not allocation_->empty(), "Task '%s': host_list is empty!", get_cname());
+
+  XBT_VERB("Executing task '%s'", get_cname());
+
+  /* Beware! The scheduling data are now used by the surf action directly! no copy was done */
+  auto host_model = allocation_->front()->get_netpoint()->get_englobing_zone()->get_host_model();
+  surf_action_    = host_model->execute_parallel(*allocation_, flops_amount_, bytes_amount_, rate_);
+
+  surf_action_->set_data(this);
+
+  XBT_DEBUG("surf_action = %p", surf_action_);
+
+  set_state(SD_RUNNING);
+  sd_global->return_set.insert(this);
+}
+
+void Task::destroy()
+{
+  XBT_DEBUG("Destroying task %s...", get_cname());
+
+  /* First Remove all dependencies associated with the task. */
+  while (not predecessors_.empty())
+    SD_task_dependency_remove(*(predecessors_.begin()), this);
+  while (not inputs_.empty())
+    SD_task_dependency_remove(*(inputs_.begin()), this);
+  while (not successors_.empty())
+    SD_task_dependency_remove(this, *(successors_.begin()));
+  while (not outputs_.empty())
+    SD_task_dependency_remove(this, *(outputs_.begin()));
+
+  if (state_ == SD_SCHEDULED || state_ == SD_RUNNABLE) {
+    xbt_free(flops_amount_);
+    xbt_free(bytes_amount_);
+    bytes_amount_ = nullptr;
+    flops_amount_ = nullptr;
+  }
 
-  xbt_free(task->flops_amount);
-  xbt_free(task->bytes_amount);
-  task->bytes_amount = nullptr;
-  task->flops_amount = nullptr;
+  xbt_free(flops_amount_);
+  xbt_free(bytes_amount_);
+
+  delete allocation_;
+
+  if (surf_action_ != nullptr)
+    surf_action_->unref();
+
+  XBT_DEBUG("Task destroyed.");
 }
+} // namespace sd
+} // namespace simgrid
+
+/* **************************** Public C interface *************************** */
 
 /**
  * @brief Creates a new task.
@@ -33,41 +522,12 @@ static void __SD_task_destroy_scheduling_data(SD_task_t task)
  * @return the new task
  * @see SD_task_destroy()
  */
-SD_task_t SD_task_create(const char *name, void *data, double amount)
+SD_task_t SD_task_create(const char* name, void* data, double amount)
 {
-  SD_task_t task = xbt_new0(s_SD_task_t, 1);
-  task->kind = SD_TASK_NOT_TYPED;
-  task->state= SD_NOT_SCHEDULED;
-  sd_global->initial_tasks.insert(task);
-
-  task->marked = 0;
-  task->start_time = -1.0;
-  task->finish_time = -1.0;
-  task->surf_action = nullptr;
-  task->watch_points = 0;
-
-  task->inputs = new std::set<SD_task_t>();
-  task->outputs = new std::set<SD_task_t>();
-  task->predecessors = new std::set<SD_task_t>();
-  task->successors = new std::set<SD_task_t>();
-
-  task->data = data;
-  task->name = xbt_strdup(name);
-  task->amount = amount;
-  task->allocation = new std::vector<sg_host_t>();
-  task->rate = -1;
-  return task;
-}
-
-static inline SD_task_t SD_task_create_sized(const char *name, void *data, double amount, int count)
-{
-  SD_task_t task = SD_task_create(name, data, amount);
-  task->bytes_amount = xbt_new0(double, count * count);
-  task->flops_amount = xbt_new0(double, count);
-  return task;
+  return simgrid::sd::Task::create(name, amount, data);
 }
 
-/** @brief create a end-to-end communication task that can then be auto-scheduled
+/** @brief create an end-to-end communication task that can then be auto-scheduled
  *
  * Auto-scheduling mean that the task can be used with SD_task_schedulev(). This allows one to specify the task costs at
  * creation, and decouple them from the scheduling process where you just specify which resource should deliver the
@@ -76,13 +536,9 @@ static inline SD_task_t SD_task_create_sized(const char *name, void *data, doubl
  * A end-to-end communication must be scheduled on 2 hosts, and the amount specified at creation is sent from hosts[0]
  * to hosts[1].
  */
-SD_task_t SD_task_create_comm_e2e(const char *name, void *data, double amount)
+SD_task_t SD_task_create_comm_e2e(const char* name, void* data, double amount)
 {
-  SD_task_t res = SD_task_create_sized(name, data, amount, 2);
-  res->bytes_amount[2] = amount;
-  res->kind = SD_TASK_COMM_E2E;
-
-  return res;
+  return simgrid::sd::Task::create_comm_e2e(name, amount, data);
 }
 
 /** @brief create a sequential computation task that can then be auto-scheduled
@@ -98,13 +554,9 @@ SD_task_t SD_task_create_comm_e2e(const char *name, void *data, double amount)
  * @param flops_amount amount of compute work to be done by the task
  * @return the new SD_TASK_COMP_SEQ typed task
  */
-SD_task_t SD_task_create_comp_seq(const char *name, void *data, double flops_amount)
+SD_task_t SD_task_create_comp_seq(const char* name, void* data, double flops_amount)
 {
-  SD_task_t res = SD_task_create_sized(name, data, flops_amount, 1);
-  res->flops_amount[0] = flops_amount;
-  res->kind = SD_TASK_COMP_SEQ;
-
-  return res;
+  return simgrid::sd::Task::create_comp_seq(name, flops_amount, data);
 }
 
 /** @brief create a parallel computation task that can then be auto-scheduled
@@ -122,15 +574,9 @@ SD_task_t SD_task_create_comp_seq(const char *name, void *data, double flops_amo
  * @param alpha purely serial fraction of the work to be done (in [0.;1.[)
  * @return the new task
  */
-SD_task_t SD_task_create_comp_par_amdahl(const char *name, void *data, double flops_amount, double alpha)
+SD_task_t SD_task_create_comp_par_amdahl(const char* name, void* data, double flops_amount, double alpha)
 {
-  xbt_assert(alpha < 1. && alpha >= 0., "Invalid parameter: alpha must be in [0.;1.[");
-
-  SD_task_t res = SD_task_create(name, data, flops_amount);
-  res->alpha = alpha;
-  res->kind = SD_TASK_COMP_PAR_AMDAHL;
-
-  return res;
+  return simgrid::sd::Task::create_comp_par_amdahl(name, flops_amount, data, alpha);
 }
 
 /** @brief create a complex data redistribution task that can then be  auto-scheduled
@@ -147,12 +593,9 @@ SD_task_t SD_task_create_comp_par_amdahl(const char *name, void *data, double fl
  * @param amount amount of data to redistribute by the task
  * @return the new task
  */
-SD_task_t SD_task_create_comm_par_mxn_1d_block(const char *name, void *data, double amount)
+SD_task_t SD_task_create_comm_par_mxn_1d_block(const char* name, void* data, double amount)
 {
-  SD_task_t res = SD_task_create(name, data, amount);
-  res->kind = SD_TASK_COMM_PAR_MXN_1D_BLOCK;
-
-  return res;
+  return simgrid::sd::Task::create_comm_par_mxn_1d_block(name, amount, data);
 }
 
 /**
@@ -165,36 +608,7 @@ SD_task_t SD_task_create_comm_par_mxn_1d_block(const char *name, void *data, dou
  */
 void SD_task_destroy(SD_task_t task)
 {
-  XBT_DEBUG("Destroying task %s...", SD_task_get_name(task));
-
-  /* First Remove all dependencies associated with the task. */
-  while (not task->predecessors->empty())
-    SD_task_dependency_remove(*(task->predecessors->begin()), task);
-  while (not task->inputs->empty())
-    SD_task_dependency_remove(*(task->inputs->begin()), task);
-  while (not task->successors->empty())
-    SD_task_dependency_remove(task, *(task->successors->begin()));
-  while (not task->outputs->empty())
-    SD_task_dependency_remove(task, *(task->outputs->begin()));
-
-  if (task->state == SD_SCHEDULED || task->state == SD_RUNNABLE)
-    __SD_task_destroy_scheduling_data(task);
-
-  xbt_free(task->name);
-
-  if (task->surf_action != nullptr)
-    task->surf_action->unref();
-
-  delete task->allocation;
-  xbt_free(task->bytes_amount);
-  xbt_free(task->flops_amount);
-  delete task->inputs;
-  delete task->outputs;
-  delete task->predecessors;
-  delete task->successors;
-  xbt_free(task);
-
-  XBT_DEBUG("Task destroyed.");
+  task->destroy();
 }
 
 /**
@@ -204,9 +618,9 @@ void SD_task_destroy(SD_task_t task)
  * @return the user data associated with this task (can be @c nullptr)
  * @see SD_task_set_data()
  */
-void *SD_task_get_data(SD_task_t task)
+void* SD_task_get_data(const_SD_task_t task)
 {
-  return task->data;
+  return task->get_data();
 }
 
 /**
@@ -218,9 +632,9 @@ void *SD_task_get_data(SD_task_t task)
  * @param data the new data you want to associate with this task
  * @see SD_task_get_data()
  */
-void SD_task_set_data(SD_task_t task, void *data)
+void SD_task_set_data(SD_task_t task, voiddata)
 {
-  task->data = data;
+  task->set_data(data);
 }
 
 /**
@@ -238,12 +652,7 @@ void SD_task_set_data(SD_task_t task, void *data)
  */
 void SD_task_set_rate(SD_task_t task, double rate)
 {
-  xbt_assert(task->kind == SD_TASK_COMM_E2E, "The rate can be modified for end-to-end communications only.");
-  if(task->state < SD_RUNNING) {
-    task->rate = rate;
-  } else {
-    XBT_WARN("Task %p has started. Changing rate is ineffective.", task);
-  }
+  task->set_rate(rate);
 }
 
 /**
@@ -254,78 +663,25 @@ void SD_task_set_rate(SD_task_t task, double rate)
  * #SD_NOT_SCHEDULED, #SD_SCHEDULED, #SD_RUNNABLE, #SD_RUNNING, #SD_DONE or #SD_FAILED
  * @see e_SD_task_state_t
  */
-e_SD_task_state_t SD_task_get_state(SD_task_t task)
+e_SD_task_state_t SD_task_get_state(const_SD_task_t task)
 {
-  return task->state;
-}
-
-/* Changes the state of a task. Updates the sd_global->watch_point_reached flag.
- */
-void SD_task_set_state(SD_task_t task, e_SD_task_state_t new_state)
-{
-  std::set<SD_task_t>::iterator idx;
-  XBT_DEBUG("Set state of '%s' to %d", task->name, new_state);
-  if ((new_state == SD_NOT_SCHEDULED || new_state == SD_SCHEDULABLE) && task->state == SD_FAILED){
-    sd_global->completed_tasks.erase(task);
-    sd_global->initial_tasks.insert(task);
-  }
-
-  if (new_state == SD_SCHEDULED && task->state == SD_RUNNABLE){
-    sd_global->initial_tasks.insert(task);
-    sd_global->runnable_tasks.erase(task);
-  }
-
-  if (new_state == SD_RUNNABLE){
-    idx = sd_global->initial_tasks.find(task);
-    if (idx != sd_global->initial_tasks.end()) {
-      sd_global->runnable_tasks.insert(*idx);
-      sd_global->initial_tasks.erase(idx);
-    }
-  }
-
-  if (new_state == SD_RUNNING)
-    sd_global->runnable_tasks.erase(task);
-
-  if (new_state == SD_DONE || new_state == SD_FAILED){
-    sd_global->completed_tasks.insert(task);
-    task->start_time = task->surf_action->get_start_time();
-    if (new_state == SD_DONE){
-      task->finish_time = task->surf_action->get_finish_time();
-#if SIMGRID_HAVE_JEDULE
-      jedule_log_sd_event(task);
-#endif
-    } else
-      task->finish_time = surf_get_clock();
-    task->surf_action->unref();
-    task->surf_action = nullptr;
-    task->allocation->clear();
-  }
-
-  task->state = new_state;
-
-  if (task->watch_points & new_state) {
-    XBT_VERB("Watch point reached with task '%s'!", task->name);
-    sd_global->watch_point_reached = true;
-    SD_task_unwatch(task, new_state);   /* remove the watch point */
-  }
+  return task->get_state();
 }
-
 /**
  * @brief Returns the name of a task
  *
  * @param task a task
  * @return the name of this task (can be @c nullptr)
  */
-const char *SD_task_get_name(SD_task_t task)
+const char* SD_task_get_name(const_SD_task_t task)
 {
-  return task->name;
+  return task->get_cname();
 }
 
 /** @brief Allows to change the name of a task */
-void SD_task_set_name(SD_task_t task, const char *name)
+void SD_task_set_name(SD_task_t task, const charname)
 {
-  xbt_free(task->name);
-  task->name = xbt_strdup(name);
+  task->set_name(name);
 }
 
 /** @brief Returns the dynar of the parents of a task
@@ -334,13 +690,13 @@ void SD_task_set_name(SD_task_t task, const char *name)
  * @return a newly allocated dynar comprising the parents of this task
  */
 
-xbt_dynar_t SD_task_get_parents(SD_task_t task)
+xbt_dynar_t SD_task_get_parents(const_SD_task_t task)
 {
   xbt_dynar_t parents = xbt_dynar_new(sizeof(SD_task_t), nullptr);
 
-  for (auto const& it : *task->predecessors)
+  for (auto const& it : task->get_predecessors())
     xbt_dynar_push(parents, &it);
-  for (auto const& it : *task->inputs)
+  for (auto const& it : task->get_inputs())
     xbt_dynar_push(parents, &it);
 
   return parents;
@@ -351,13 +707,13 @@ xbt_dynar_t SD_task_get_parents(SD_task_t task)
  * @param task a task
  * @return a newly allocated dynar comprising the parents of this task
  */
-xbt_dynar_t SD_task_get_children(SD_task_t task)
+xbt_dynar_t SD_task_get_children(const_SD_task_t task)
 {
   xbt_dynar_t children = xbt_dynar_new(sizeof(SD_task_t), nullptr);
 
-  for (auto const& it : *task->successors)
+  for (auto const& it : task->get_successors())
     xbt_dynar_push(children, &it);
-  for (auto const& it : *task->outputs)
+  for (auto const& it : task->get_outputs())
     xbt_dynar_push(children, &it);
 
   return children;
@@ -369,9 +725,9 @@ xbt_dynar_t SD_task_get_children(SD_task_t task)
  * Only call this on already scheduled tasks!
  * @param task a task
  */
-int SD_task_get_workstation_count(SD_task_t task)
+int SD_task_get_workstation_count(const_SD_task_t task)
 {
-  return task->allocation->size();
+  return static_cast<int>(task->get_allocation_size());
 }
 
 /**
@@ -380,9 +736,9 @@ int SD_task_get_workstation_count(SD_task_t task)
  * Only call this on already scheduled tasks!
  * @param task a task
  */
-sg_host_t *SD_task_get_workstation_list(SD_task_t task)
+sg_host_t* SD_task_get_workstation_list(const_SD_task_t task)
 {
-  return task->allocation->data();
+  return task->get_allocation()->data();
 }
 
 /**
@@ -392,9 +748,9 @@ sg_host_t *SD_task_get_workstation_list(SD_task_t task)
  * @return the total amount of work (computation or data transfer) for this task
  * @see SD_task_get_remaining_amount()
  */
-double SD_task_get_amount(SD_task_t task)
+double SD_task_get_amount(const_SD_task_t task)
 {
-  return task->amount;
+  return task->get_amount();
 }
 
 /** @brief Sets the total amount of work of a task
@@ -407,11 +763,7 @@ double SD_task_get_amount(SD_task_t task)
  */
 void SD_task_set_amount(SD_task_t task, double amount)
 {
-  task->amount = amount;
-  if (task->kind == SD_TASK_COMP_SEQ)
-    task->flops_amount[0] = amount;
-  if (task->kind == SD_TASK_COMM_E2E)
-    task->bytes_amount[2] = amount;
+  task->set_amount(amount);
 }
 
 /**
@@ -420,10 +772,9 @@ void SD_task_set_amount(SD_task_t task, double amount)
  * @param task a parallel task assuming Amdahl's law as speedup model
  * @return the alpha parameter (serial part of a task in percent) for this task
  */
-double SD_task_get_alpha(SD_task_t task)
+double SD_task_get_alpha(const_SD_task_t task)
 {
-  xbt_assert(SD_task_get_kind(task) == SD_TASK_COMP_PAR_AMDAHL, "Alpha parameter is not defined for this kind of task");
-  return task->alpha;
+  return task->get_alpha();
 }
 
 /**
@@ -433,92 +784,43 @@ double SD_task_get_alpha(SD_task_t task)
  * @return the remaining amount of work (computation or data transfer) of this task
  * @see SD_task_get_amount()
  */
-double SD_task_get_remaining_amount(SD_task_t task)
+double SD_task_get_remaining_amount(const_SD_task_t task)
 {
-  if (task->surf_action)
-    return task->surf_action->get_remains();
-  else
-    return (task->state == SD_DONE) ? 0 : task->amount;
+  return task->get_remaining_amount();
 }
 
-e_SD_task_kind_t SD_task_get_kind(SD_task_t task)
+e_SD_task_kind_t SD_task_get_kind(const_SD_task_t task)
 {
-  return task->kind;
+  return task->get_kind();
 }
 
 /** @brief Displays debugging information about a task */
-void SD_task_dump(SD_task_t task)
+void SD_task_dump(const_SD_task_t task)
 {
-  XBT_INFO("Displaying task %s", SD_task_get_name(task));
-  if (task->state == SD_RUNNABLE)
-    XBT_INFO("  - state: runnable");
-  else if (task->state < SD_RUNNABLE)
-    XBT_INFO("  - state: %s not runnable", __get_state_name(task->state));
-  else
-    XBT_INFO("  - state: not runnable %s", __get_state_name(task->state));
+  task->dump();
+}
 
-  if (task->kind != 0) {
-    switch (task->kind) {
+/** @brief Dumps the task in dotty formalism into the FILE* passed as second argument */
+void SD_task_dotty(const_SD_task_t task, void* out)
+{
+  auto* fout = static_cast<FILE*>(out);
+  fprintf(fout, "  T%p [label=\"%.20s\"", task, task->get_cname());
+  switch (task->get_kind()) {
     case SD_TASK_COMM_E2E:
-      XBT_INFO("  - kind: end-to-end communication");
+    case SD_TASK_COMM_PAR_MXN_1D_BLOCK:
+      fprintf(fout, ", shape=box");
       break;
     case SD_TASK_COMP_SEQ:
-      XBT_INFO("  - kind: sequential computation");
-      break;
     case SD_TASK_COMP_PAR_AMDAHL:
-      XBT_INFO("  - kind: parallel computation following Amdahl's law");
-      break;
-    case SD_TASK_COMM_PAR_MXN_1D_BLOCK:
-      XBT_INFO("  - kind: MxN data redistribution assuming 1D block distribution");
+      fprintf(fout, ", shape=circle");
       break;
     default:
-      XBT_INFO("  - (unknown kind %d)", task->kind);
-    }
-  }
-
-  XBT_INFO("  - amount: %.0f", SD_task_get_amount(task));
-  if (task->kind == SD_TASK_COMP_PAR_AMDAHL)
-    XBT_INFO("  - alpha: %.2f", task->alpha);
-  XBT_INFO("  - Dependencies to satisfy: %zu", task->inputs->size()+ task->predecessors->size());
-  if ((task->inputs->size()+ task->predecessors->size()) > 0) {
-    XBT_INFO("  - pre-dependencies:");
-    for (auto const& it : *task->predecessors)
-      XBT_INFO("    %s", it->name);
-
-    for (auto const& it : *task->inputs)
-      XBT_INFO("    %s", it->name);
-  }
-  if ((task->outputs->size() + task->successors->size()) > 0) {
-    XBT_INFO("  - post-dependencies:");
-
-    for (auto const& it : *task->successors)
-      XBT_INFO("    %s", it->name);
-    for (auto const& it : *task->outputs)
-      XBT_INFO("    %s", it->name);
-  }
-}
-
-/** @brief Dumps the task in dotty formalism into the FILE* passed as second argument */
-void SD_task_dotty(SD_task_t task, void *out)
-{
-  FILE *fout = static_cast<FILE*>(out);
-  fprintf(fout, "  T%p [label=\"%.20s\"", task, task->name);
-  switch (task->kind) {
-  case SD_TASK_COMM_E2E:
-  case SD_TASK_COMM_PAR_MXN_1D_BLOCK:
-    fprintf(fout, ", shape=box");
-    break;
-  case SD_TASK_COMP_SEQ:
-  case SD_TASK_COMP_PAR_AMDAHL:
-    fprintf(fout, ", shape=circle");
-    break;
-  default:
-    xbt_die("Unknown task type!");
+      xbt_die("Unknown task type!");
   }
   fprintf(fout, "];\n");
-  for (auto const& it : *task->predecessors)
+  for (auto const& it : task->get_predecessors())
     fprintf(fout, " T%p -> T%p;\n", it, task);
-  for (auto const& it : *task->inputs)
+  for (auto const& it : task->get_inputs())
     fprintf(fout, " T%p -> T%p;\n", it, task);
 }
 
@@ -538,39 +840,39 @@ void SD_task_dependency_add(SD_task_t src, SD_task_t dst)
     throw std::invalid_argument(
         simgrid::xbt::string_printf("Cannot add a dependency between task '%s' and itself", SD_task_get_name(src)));
 
-  if (src->state == SD_DONE || src->state == SD_FAILED)
+  if (src->get_state() == SD_DONE || src->get_state() == SD_FAILED)
     throw std::invalid_argument(simgrid::xbt::string_printf(
-        "Task '%s' must be SD_NOT_SCHEDULED, SD_SCHEDULABLE, SD_SCHEDULED, SD_RUNNABLE, or SD_RUNNING", src->name));
+        "Task '%s' must be SD_NOT_SCHEDULED, SD_SCHEDULABLE, SD_SCHEDULED, SD_RUNNABLE, or SD_RUNNING",
+        src->get_cname()));
 
-  if (dst->state == SD_DONE || dst->state == SD_FAILED || dst->state == SD_RUNNING)
+  if (dst->get_state() == SD_DONE || dst->get_state() == SD_FAILED || dst->get_state() == SD_RUNNING)
     throw std::invalid_argument(simgrid::xbt::string_printf(
-        "Task '%s' must be SD_NOT_SCHEDULED, SD_SCHEDULABLE, SD_SCHEDULED, or SD_RUNNABLE", dst->name));
+        "Task '%s' must be SD_NOT_SCHEDULED, SD_SCHEDULABLE, SD_SCHEDULED, or SD_RUNNABLE", dst->get_cname()));
 
-  if (dst->inputs->find(src) != dst->inputs->end() || src->outputs->find(dst) != src->outputs->end() ||
-      src->successors->find(dst) != src->successors->end() || dst->predecessors->find(src) != dst->predecessors->end())
+  if (dst->is_child_of(src) || src->is_parent_of(dst))
     throw std::invalid_argument(simgrid::xbt::string_printf(
-        "A dependency already exists between task '%s' and task '%s'", src->name, dst->name));
+        "A dependency already exists between task '%s' and task '%s'", src->get_cname(), dst->get_cname()));
 
-  XBT_DEBUG("SD_task_dependency_add: src = %s, dst = %s", src->name, dst->name);
+  XBT_DEBUG("SD_task_dependency_add: src = %s, dst = %s", src->get_cname(), dst->get_cname());
 
-  if (src->kind == SD_TASK_COMM_E2E || src->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK){
-    if (dst->kind == SD_TASK_COMP_SEQ || dst->kind == SD_TASK_COMP_PAR_AMDAHL)
-        dst->inputs->insert(src);
+  if (src->get_kind() == SD_TASK_COMM_E2E || src->get_kind() == SD_TASK_COMM_PAR_MXN_1D_BLOCK) {
+    if (dst->get_kind() == SD_TASK_COMP_SEQ || dst->get_kind() == SD_TASK_COMP_PAR_AMDAHL)
+      dst->add_input(src);
     else
-      dst->predecessors->insert(src);
-    src->successors->insert(dst);
+      dst->add_predecessor(src);
+    src->add_successor(dst);
   } else {
-    if (dst->kind == SD_TASK_COMM_E2E|| dst->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK)
-      src->outputs->insert(dst);
+    if (dst->get_kind() == SD_TASK_COMM_E2E || dst->get_kind() == SD_TASK_COMM_PAR_MXN_1D_BLOCK)
+      src->add_output(dst);
     else
-      src->successors->insert(dst);
-    dst->predecessors->insert(src);
+      src->add_successor(dst);
+    dst->add_predecessor(src);
   }
 
   /* if the task was runnable, the task goes back to SD_SCHEDULED because of the new dependency*/
-  if (dst->state == SD_RUNNABLE) {
-    XBT_DEBUG("SD_task_dependency_add: %s was runnable and becomes scheduled!", dst->name);
-    SD_task_set_state(dst, SD_SCHEDULED);
+  if (dst->get_state() == SD_RUNNABLE) {
+    XBT_DEBUG("SD_task_dependency_add: %s was runnable and becomes scheduled!", dst->get_cname());
+    dst->set_state(SD_SCHEDULED);
   }
 }
 
@@ -583,19 +885,17 @@ void SD_task_dependency_add(SD_task_t src, SD_task_t dst)
  * If src is nullptr, checks whether dst has any pre-dependency.
  * If dst is nullptr, checks whether src has any post-dependency.
  */
-int SD_task_dependency_exists(SD_task_t src, SD_task_t dst)
+int SD_task_dependency_exists(const_SD_task_t src, SD_task_t dst)
 {
   xbt_assert(src != nullptr || dst != nullptr, "Invalid parameter: both src and dst are nullptr");
 
-  if (src) {
-    if (dst) {
-      return (src->successors->find(dst) != src->successors->end() || src->outputs->find(dst) != src->outputs->end());
-    } else {
-      return src->successors->size() + src->outputs->size();
-    }
-  } else {
-    return dst->predecessors->size() + dst->inputs->size();
-  }
+  if (src)
+    if (dst)
+      return src->is_parent_of(dst);
+    else
+      return static_cast<int>(src->is_waited_by());
+  else
+    return static_cast<int>(dst->has_unsolved_dependencies());
 }
 
 /**
@@ -607,30 +907,30 @@ int SD_task_dependency_exists(SD_task_t src, SD_task_t dst)
  */
 void SD_task_dependency_remove(SD_task_t src, SD_task_t dst)
 {
-  XBT_DEBUG("SD_task_dependency_remove: src = %s, dst = %s", SD_task_get_name(src), SD_task_get_name(dst));
+  XBT_DEBUG("SD_task_dependency_remove: src = %s, dst = %s", src->get_cname(), dst->get_cname());
 
-  if (src->successors->find(dst) == src->successors->end() && src->outputs->find(dst) == src->outputs->end())
+  if (not src->is_parent_of(dst))
     throw std::invalid_argument(simgrid::xbt::string_printf(
-        "No dependency found between task '%s' and '%s': task '%s' is not a successor of task '%s'", src->name,
-        dst->name, dst->name, src->name));
+        "No dependency found between task '%s' and '%s': task '%s' is not a successor of task '%s'", src->get_cname(),
+        dst->get_cname(), dst->get_cname(), src->get_cname()));
 
-  if (src->kind == SD_TASK_COMM_E2E || src->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK){
-    if (dst->kind == SD_TASK_COMP_SEQ || dst->kind == SD_TASK_COMP_PAR_AMDAHL)
-      dst->inputs->erase(src);
+  if (src->get_kind() == SD_TASK_COMM_E2E || src->get_kind() == SD_TASK_COMM_PAR_MXN_1D_BLOCK) {
+    if (dst->get_kind() == SD_TASK_COMP_SEQ || dst->get_kind() == SD_TASK_COMP_PAR_AMDAHL)
+      dst->rm_input(src);
     else
-      dst->predecessors->erase(src);
-    src->successors->erase(dst);
+      dst->rm_predecessor(src);
+    src->rm_successor(dst);
   } else {
-    if (dst->kind == SD_TASK_COMM_E2E|| dst->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK)
-      src->outputs->erase(dst);
+    if (dst->get_kind() == SD_TASK_COMM_E2E || dst->get_kind() == SD_TASK_COMM_PAR_MXN_1D_BLOCK)
+      src->rm_output(dst);
     else
-      src->successors->erase(dst);
-    dst->predecessors->erase(src);
+      src->rm_successor(dst);
+    dst->rm_predecessor(src);
   }
 
   /* if the task was scheduled and dependencies are satisfied, we can make it runnable */
-  if (dst->predecessors->empty() && dst->inputs->empty() && dst->state == SD_SCHEDULED)
-    SD_task_set_state(dst, SD_RUNNABLE);
+  if (dst->has_unsolved_dependencies() == 0 && dst->get_state() == SD_SCHEDULED)
+    dst->set_state(SD_RUNNABLE);
 }
 
 /**
@@ -645,10 +945,7 @@ void SD_task_dependency_remove(SD_task_t src, SD_task_t dst)
  */
 void SD_task_watch(SD_task_t task, e_SD_task_state_t state)
 {
-  if (state & SD_NOT_SCHEDULED)
-    throw std::invalid_argument("Cannot add a watch point for state SD_NOT_SCHEDULED");
-
-  task->watch_points = task->watch_points | state;
+  task->watch(state);
 }
 
 /**
@@ -660,8 +957,7 @@ void SD_task_watch(SD_task_t task, e_SD_task_state_t state)
  */
 void SD_task_unwatch(SD_task_t task, e_SD_task_state_t state)
 {
-  xbt_assert(state != SD_NOT_SCHEDULED, "SimDag error: Cannot have a watch point for state SD_NOT_SCHEDULED");
-  task->watch_points = task->watch_points & ~state;
+  task->unwatch(state);
 }
 
 /**
@@ -676,7 +972,7 @@ void SD_task_unwatch(SD_task_t task, e_SD_task_state_t state)
  * @param bytes_amount communication amount between each pair of hosts (i.e., a matrix of host_count*host_count doubles)
  * @see SD_schedule()
  */
-double SD_task_get_execution_time(SD_task_t /*task*/, int host_count, const sg_host_t* host_list,
+double SD_task_get_execution_time(const_SD_task_t /*task*/, int host_count, const sg_host_t* host_list,
                                   const double* flops_amount, const double* bytes_amount)
 {
   xbt_assert(host_count > 0, "Invalid parameter");
@@ -691,8 +987,8 @@ double SD_task_get_execution_time(SD_task_t /*task*/, int host_count, const sg_h
     if (bytes_amount != nullptr)
       for (int j = 0; j < host_count; j++)
         if (bytes_amount[i * host_count + j] != 0)
-          time += (sg_host_route_latency(host_list[i], host_list[j]) +
-                   bytes_amount[i * host_count + j] / sg_host_route_bandwidth(host_list[i], host_list[j]));
+          time += (sg_host_get_route_latency(host_list[i], host_list[j]) +
+                   bytes_amount[i * host_count + j] / sg_host_get_route_bandwidth(host_list[i], host_list[j]));
 
     if (time > max_time)
       max_time = time;
@@ -700,18 +996,6 @@ double SD_task_get_execution_time(SD_task_t /*task*/, int host_count, const sg_h
   return max_time;
 }
 
-static inline void SD_task_do_schedule(SD_task_t task)
-{
-  if (SD_task_get_state(task) > SD_SCHEDULABLE)
-    throw std::invalid_argument(
-        simgrid::xbt::string_printf("Task '%s' has already been scheduled", SD_task_get_name(task)));
-
-  if (task->predecessors->empty() && task->inputs->empty())
-    SD_task_set_state(task, SD_RUNNABLE);
-  else
-    SD_task_set_state(task, SD_SCHEDULED);
-}
-
 /**
  * @brief Schedules a task
  *
@@ -726,34 +1010,16 @@ static inline void SD_task_do_schedule(SD_task_t task)
  * @param rate task execution speed rate
  * @see SD_task_unschedule()
  */
-void SD_task_schedule(SD_task_t task, int host_count, const sg_host_t * host_list,
-                      const double *flops_amount, const double *bytes_amount, double rate)
+void SD_task_schedule(SD_task_t task, int host_count, const sg_host_t* host_list, const double* flops_amount,
+                      const doublebytes_amount, double rate)
 {
   xbt_assert(host_count > 0, "host_count must be positive");
+  std::vector<sg_host_t> hosts(host_count);
 
-  task->rate = rate;
+  for (int i = 0; i < host_count; i++)
+    hosts[i] = host_list[i];
 
-  if (flops_amount) {
-    task->flops_amount = static_cast<double*>(xbt_realloc(task->flops_amount, sizeof(double) * host_count));
-    memcpy(task->flops_amount, flops_amount, sizeof(double) * host_count);
-  } else {
-    xbt_free(task->flops_amount);
-    task->flops_amount = nullptr;
-  }
-
-  int communication_nb = host_count * host_count;
-  if (bytes_amount) {
-    task->bytes_amount = static_cast<double*>(xbt_realloc(task->bytes_amount, sizeof(double) * communication_nb));
-    memcpy(task->bytes_amount, bytes_amount, sizeof(double) * communication_nb);
-  } else {
-    xbt_free(task->bytes_amount);
-    task->bytes_amount = nullptr;
-  }
-
-  for(int i =0; i<host_count; i++)
-    task->allocation->push_back(host_list[i]);
-
-  SD_task_do_schedule(task);
+  task->schedule(hosts, flops_amount, bytes_amount, rate);
 }
 
 /**
@@ -768,47 +1034,7 @@ void SD_task_schedule(SD_task_t task, int host_count, const sg_host_t * host_lis
  */
 void SD_task_unschedule(SD_task_t task)
 {
-  if (task->state == SD_NOT_SCHEDULED || task->state == SD_SCHEDULABLE)
-    throw std::invalid_argument(simgrid::xbt::string_printf(
-        "Task %s: the state must be SD_SCHEDULED, SD_RUNNABLE, SD_RUNNING or SD_FAILED", task->name));
-
-  if ((task->state == SD_SCHEDULED || task->state == SD_RUNNABLE) /* if the task is scheduled or runnable */
-      && ((task->kind == SD_TASK_COMP_PAR_AMDAHL) || (task->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK))) {
-          /* Don't free scheduling data for typed tasks */
-    __SD_task_destroy_scheduling_data(task);
-    task->allocation->clear();
-  }
-
-  if (SD_task_get_state(task) == SD_RUNNING)
-    /* the task should become SD_FAILED */
-    task->surf_action->cancel();
-  else {
-    if (task->predecessors->empty() && task->inputs->empty())
-      SD_task_set_state(task, SD_SCHEDULABLE);
-    else
-      SD_task_set_state(task, SD_NOT_SCHEDULED);
-  }
-  task->start_time = -1.0;
-}
-
-/* Runs a task. */
-void SD_task_run(SD_task_t task)
-{
-  xbt_assert(task->state == SD_RUNNABLE, "Task '%s' is not runnable! Task state: %d", task->name, (int) task->state);
-  xbt_assert(task->allocation != nullptr, "Task '%s': host_list is nullptr!", task->name);
-
-  XBT_VERB("Executing task '%s'", task->name);
-
-  /* Beware! The scheduling data are now used by the surf action directly! no copy was done */
-  task->surf_action =
-      surf_host_model->execute_parallel(*task->allocation, task->flops_amount, task->bytes_amount, task->rate);
-
-  task->surf_action->set_data(task);
-
-  XBT_DEBUG("surf_action = %p", task->surf_action);
-
-  SD_task_set_state(task, SD_RUNNING);
-  sd_global->return_set.insert(task);
+  task->unschedule();
 }
 
 /**
@@ -819,12 +1045,9 @@ void SD_task_run(SD_task_t task)
  * @param task: a task
  * @return the start time of this task
  */
-double SD_task_get_start_time(SD_task_t task)
+double SD_task_get_start_time(const_SD_task_t task)
 {
-  if (task->surf_action)
-    return task->surf_action->get_start_time();
-  else
-    return task->start_time;
+  return task->get_start_time();
 }
 
 /**
@@ -837,47 +1060,19 @@ double SD_task_get_start_time(SD_task_t task)
  * @param task: a task
  * @return the start time of this task
  */
-double SD_task_get_finish_time(SD_task_t task)
+double SD_task_get_finish_time(const_SD_task_t task)
 {
-  if (task->surf_action)        /* should never happen as actions are destroyed right after their completion */
-    return task->surf_action->get_finish_time();
-  else
-    return task->finish_time;
+  return task->get_finish_time();
 }
 
 void SD_task_distribute_comp_amdahl(SD_task_t task, int count)
 {
-  xbt_assert(task->kind == SD_TASK_COMP_PAR_AMDAHL, "Task %s is not a SD_TASK_COMP_PAR_AMDAHL typed task."
-              "Cannot use this function.", task->name);
-  task->flops_amount = xbt_new0(double, count);
-  task->bytes_amount = xbt_new0(double, count * count);
-
-  for (int i=0; i<count; i++){
-    task->flops_amount[i] = (task->alpha + (1 - task->alpha)/count) * task->amount;
-  }
+  task->distribute_comp_amdahl(count);
 }
 
-void SD_task_build_MxN_1D_block_matrix(SD_task_t task, int src_nb, int dst_nb){
-  xbt_assert(task->kind == SD_TASK_COMM_PAR_MXN_1D_BLOCK, "Task %s is not a SD_TASK_COMM_PAR_MXN_1D_BLOCK typed task."
-              "Cannot use this function.", task->name);
-  xbt_free(task->bytes_amount);
-  task->bytes_amount = xbt_new0(double,task->allocation->size() * task->allocation->size());
-
-  for (int i=0; i<src_nb; i++) {
-    double src_start = i*task->amount/src_nb;
-    double src_end = src_start + task->amount/src_nb;
-    for (int j=0; j<dst_nb; j++) {
-      double dst_start = j*task->amount/dst_nb;
-      double dst_end = dst_start + task->amount/dst_nb;
-      XBT_VERB("(%d->%d): (%.2f, %.2f)-> (%.2f, %.2f)", i, j, src_start, src_end, dst_start, dst_end);
-      task->bytes_amount[i*(src_nb+dst_nb)+src_nb+j]=0.0;
-      if ((src_end > dst_start) && (dst_end > src_start)) { /* There is something to send */
-        task->bytes_amount[i * (src_nb + dst_nb) + src_nb + j] =
-            std::min(src_end, dst_end) - std::max(src_start, dst_start);
-        XBT_VERB("==> %.2f", task->bytes_amount[i*(src_nb+dst_nb)+src_nb+j]);
-      }
-    }
-  }
+void SD_task_build_MxN_1D_block_matrix(SD_task_t task, int src_nb, int dst_nb)
+{
+  task->build_MxN_1D_block_matrix(src_nb, dst_nb);
 }
 
 /** @brief Auto-schedules a task.
@@ -888,69 +1083,12 @@ void SD_task_build_MxN_1D_block_matrix(SD_task_t task, int src_nb, int dst_nb){
  *
  * To be auto-schedulable, a task must be a typed computation SD_TASK_COMP_SEQ or SD_TASK_COMP_PAR_AMDAHL.
  */
-void SD_task_schedulev(SD_task_t task, int count, const sg_host_t * list)
+void SD_task_schedulev(SD_task_t task, int count, const sg_host_t* host_list)
 {
-  xbt_assert(task->kind == SD_TASK_COMP_SEQ || task->kind == SD_TASK_COMP_PAR_AMDAHL,
-      "Task %s is not typed. Cannot automatically schedule it.", SD_task_get_name(task));
-
-  for(int i =0; i<count; i++)
-    task->allocation->push_back(list[i]);
-
-  XBT_VERB("Schedule computation task %s on %zu host(s)", task->name, task->allocation->size());
-
-  if (task->kind == SD_TASK_COMP_SEQ) {
-    if (not task->flops_amount) { /*This task has failed and is rescheduled. Reset the flops_amount*/
-      task->flops_amount = xbt_new0(double, 1);
-      task->flops_amount[0] = task->amount;
-    }
-    XBT_VERB("It costs %.f flops", task->flops_amount[0]);
-  }
-
-  if (task->kind == SD_TASK_COMP_PAR_AMDAHL) {
-    SD_task_distribute_comp_amdahl(task, count);
-    XBT_VERB("%.f flops will be distributed following Amdahl's Law", task->flops_amount[0]);
-  }
-
-  SD_task_do_schedule(task);
-
-  /* Iterate over all inputs and outputs to say where I am located (and start them if runnable) */
-  for (auto const& input : *task->inputs) {
-    int src_nb = input->allocation->size();
-    int dst_nb = count;
-    if (input->allocation->empty())
-      XBT_VERB("Sender side of '%s' not scheduled. Set receiver side to '%s''s allocation", input->name, task->name);
-
-    for (int i=0; i<count;i++)
-      input->allocation->push_back(task->allocation->at(i));
-
-    if (input->allocation->size () > task->allocation->size()) {
-      if (task->kind == SD_TASK_COMP_PAR_AMDAHL)
-        SD_task_build_MxN_1D_block_matrix(input, src_nb, dst_nb);
-
-      SD_task_do_schedule(input);
-      XBT_VERB ("Auto-Schedule Communication task '%s'. Send %.f bytes from %d hosts to %d hosts.",
-          input->name,input->amount, src_nb, dst_nb);
-    }
-  }
-
-  for (auto const& output : *task->outputs) {
-    int src_nb = count;
-    int dst_nb = output->allocation->size();
-    if (output->allocation->empty())
-      XBT_VERB("Receiver side of '%s' not scheduled. Set sender side to '%s''s allocation", output->name, task->name);
-
-    for (int i=0; i<count;i++)
-      output->allocation->insert(output->allocation->begin()+i, task->allocation->at(i));
-
-    if (output->allocation->size () > task->allocation->size()) {
-      if (task->kind == SD_TASK_COMP_PAR_AMDAHL)
-        SD_task_build_MxN_1D_block_matrix(output, src_nb, dst_nb);
-
-      SD_task_do_schedule(output);
-      XBT_VERB ("Auto-Schedule Communication task %s. Send %.f bytes from %d hosts to %d hosts.",
-                output->name, output->amount, src_nb, dst_nb);
-    }
-  }
+  std::vector<sg_host_t> list(count);
+  for (int i = 0; i < count; i++)
+    list[i] = host_list[i];
+  task->schedulev(list);
 }
 
 /** @brief autoschedule a task on a list of hosts
@@ -961,12 +1099,11 @@ void SD_task_schedulev(SD_task_t task, int count, const sg_host_t * list)
 void SD_task_schedulel(SD_task_t task, int count, ...)
 {
   va_list ap;
-  sg_host_t* list = new sg_host_t[count];
+  std::vector<sg_host_t> list(count);
   va_start(ap, count);
-  for (int i=0; i<count; i++)
+  for (int i = 0; i < count; i++)
     list[i] = va_arg(ap, sg_host_t);
 
   va_end(ap);
-  SD_task_schedulev(task, count, list);
-  delete[] list;
+  task->schedulev(list);
 }