Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Misc code simplifications guided by Sonar smells.
[simgrid.git] / src / surf / HostImpl.cpp
index 160224e..51f7f98 100644 (file)
-/* Copyright (c) 2013-2016. The SimGrid Team. All rights reserved.          */
+/* Copyright (c) 2013-2022. 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 "src/plugins/vm/VirtualMachineImpl.hpp"
-#include "src/surf/FileImpl.hpp"
-#include <string>
+#include <simgrid/kernel/routing/NetPoint.hpp>
+#include <simgrid/s4u/Engine.hpp>
+#include <simgrid/s4u/Host.hpp>
+
+#include "src/kernel/EngineImpl.hpp"
+#include "src/kernel/resource/VirtualMachineImpl.hpp"
 
-XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_host, surf, "Logging specific to the SURF host module");
+#include <string>
 
-simgrid::surf::HostModel *surf_host_model = nullptr;
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(res_host, ker_resource, "Host resources agregate CPU, networking and I/O features");
 
 /*************
- * Callbacks *
+ * Callbacks *t
  *************/
 
 namespace simgrid {
-namespace surf {
+namespace kernel {
+namespace resource {
 
 /*********
  * Model *
  *********/
+/************
+ * Resource *
+ ************/
+HostImpl::HostImpl(const std::string& name) : piface_(this), name_(name)
+{
+  xbt_assert(s4u::Host::by_name_or_null(name_) == nullptr, "Refusing to create a second host named '%s'.", get_cname());
+}
 
-/* Each VM has a dummy CPU action on the PM layer. This CPU action works as the
- * constraint (capacity) of the VM in the PM layer. If the VM does not have any
- * active task, the dummy CPU action must be deactivated, so that the VM does
- * not get any CPU share in the PM layer. */
-void HostModel::ignoreEmptyVmInPmLMM()
-{
-  /* iterate for all virtual machines */
-  for (s4u::VirtualMachine* ws_vm : vm::VirtualMachineImpl::allVms_) {
-    Cpu* cpu = ws_vm->pimpl_cpu;
-    int active_tasks = lmm_constraint_get_variable_amount(cpu->constraint());
-
-    /* The impact of the VM over its PM is the min between its vCPU amount and the amount of tasks it contains */
-    int impact = std::min(active_tasks, ws_vm->pimpl_vm_->coreAmount());
-
-    XBT_DEBUG("set the weight of the dummy CPU action of VM%p on PM to %d (#tasks: %d)", ws_vm, impact, active_tasks);
-    if (impact > 0)
-      ws_vm->pimpl_vm_->action_->setSharingWeight(1. / impact);
-    else
-      ws_vm->pimpl_vm_->action_->setSharingWeight(0.);
+HostImpl::~HostImpl()
+{
+  /* All actors should be gone when the host is turned off (by the end of the simulation). */
+  if (not actor_list_.empty()) {
+    const char* msg = "Shutting down host, but it's not empty";
+    try {
+      std::string actors;
+      for (auto const& actor : actor_list_)
+        actors += "\n\t" + std::string(actor.get_name());
+
+      EngineImpl::get_instance()->display_all_actor_status();
+      xbt_die("%s:%s", msg, actors.c_str());
+    } catch (const std::bad_alloc& ba) {
+      xbt_die("%s (cannot print actor list: %s)", msg, ba.what());
+    }
   }
+  for (auto const& arg : actors_at_boot_)
+    delete arg;
+  actors_at_boot_.clear();
+
+  for (auto const& [_, d] : disks_)
+    d->destroy();
+
+  for (auto const& [_, vm] : vms_)
+    vm->vm_destroy();
 }
 
-/* Helper function for executeParallelTask */
-static inline double has_cost(double* array, int pos)
+/** @brief Fire the required callbacks and destroy the object
+ *
+ * Don't delete directly a Host, call h->destroy() instead.
+ */
+void HostImpl::destroy()
 {
-  if (array)
-    return array[pos];
-  else
-    return -1.0;
+  s4u::Host::on_destruction(*this->get_iface());
+  delete this;
 }
 
-Action* HostModel::executeParallelTask(int host_nb, simgrid::s4u::Host** host_list, double* flops_amount,
-    double* bytes_amount, double rate)
+/** Re-starts all the actors that are marked as restartable.
+ *
+ * Weird things will happen if you turn on a host that is already on. S4U is fool-proof, not this.
+ */
+void HostImpl::turn_on() const
 {
-  Action* action = nullptr;
-  if ((host_nb == 1) && (has_cost(bytes_amount, 0) <= 0)) {
-    action = host_list[0]->pimpl_cpu->execution_start(flops_amount[0]);
-  } else if ((host_nb == 1) && (has_cost(flops_amount, 0) <= 0)) {
-    action = surf_network_model->communicate(host_list[0], host_list[0], bytes_amount[0], rate);
-  } else if ((host_nb == 2) && (has_cost(flops_amount, 0) <= 0) && (has_cost(flops_amount, 1) <= 0)) {
-    int nb = 0;
-    double value = 0.0;
+  for (auto const& arg : actors_at_boot_) {
+    XBT_DEBUG("Booting Actor %s(%s) right now", arg->name.c_str(), arg->host->get_cname());
+    actor::ActorImplPtr actor = actor::ActorImpl::create(arg);
+  }
+}
 
-    for (int i = 0; i < host_nb * host_nb; i++) {
-      if (has_cost(bytes_amount, i) > 0.0) {
-        nb++;
-        value = has_cost(bytes_amount, i);
+/** Kill all actors hosted here */
+void HostImpl::turn_off(const actor::ActorImpl* issuer)
+{
+  /* turn_off VMs running on host */
+  for (const auto& [_, vm] : vms_) {
+    // call s4u functions to generate the good on_state_change signal, maybe one day this wont be necessary
+    vm->get_iface()->shutdown();
+    vm->get_iface()->turn_off();
+  }
+  for (auto& actor : actor_list_) {
+    XBT_DEBUG("Killing Actor %s@%s on behalf of %s which turned off that host.", actor.get_cname(),
+              actor.get_host()->get_cname(), issuer->get_cname());
+    issuer->kill(&actor);
+  }
+  for (const auto& activity : EngineImpl::get_instance()->get_maestro()->activities_) {
+    auto* exec = dynamic_cast<activity::ExecImpl*>(activity.get());
+    if (exec != nullptr) {
+      auto hosts = exec->get_hosts();
+      if (std::find(hosts.begin(), hosts.end(), &piface_) != hosts.end()) {
+        exec->cancel();
+        exec->set_state(activity::State::FAILED);
       }
     }
-    if (nb == 1) {
-      action = surf_network_model->communicate(host_list[0], host_list[1], value, rate);
-    } else if (nb == 0) {
-      xbt_die("Cannot have a communication with no flop to exchange in this model. You should consider using the "
-          "ptask model");
-    } else {
-      xbt_die("Cannot have a communication that is not a simple point-to-point in this model. You should consider "
-          "using the ptask model");
-    }
-  } else
-    xbt_die(
-        "This model only accepts one of the following. You should consider using the ptask model for the other cases.\n"
-        " - execution with one host only and no communication\n"
-        " - Self-comms with one host only\n"
-        " - Communications with two hosts and no computation");
-  xbt_free(host_list);
-  return action;
+  }
+  // When a host is turned off, we want to keep only the actors that should restart for when it will boot again.
+  // Then get rid of the others.
+  auto elm = remove_if(begin(actors_at_boot_), end(actors_at_boot_), [](const actor::ProcessArg* arg) {
+    if (arg->auto_restart)
+      return false;
+    delete arg;
+    return true;
+  });
+  actors_at_boot_.erase(elm, end(actors_at_boot_));
 }
 
-/************
- * Resource *
- ************/
-HostImpl::HostImpl(s4u::Host* host) : piface_(host)
+HostImpl* HostImpl::set_englobing_zone(routing::NetZoneImpl* englobing_zone)
 {
-  /* The VM wants to reinstall a new HostImpl, but we don't want to leak the previously existing one */
-  delete piface_->pimpl_;
-  piface_->pimpl_ = this;
+  englobing_zone_ = englobing_zone;
+  return this;
 }
 
-simgrid::surf::StorageImpl* HostImpl::findStorageOnMountList(const char* mount)
+std::vector<s4u::ActorPtr> HostImpl::get_all_actors()
+{
+  std::vector<s4u::ActorPtr> res;
+  for (auto& actor : actor_list_)
+    res.emplace_back(actor.get_ciface());
+  return res;
+}
+size_t HostImpl::get_actor_count() const
 {
-  XBT_DEBUG("Search for storage name '%s' on '%s'", mount, piface_->getCname());
-  if (storage_.find(mount) == storage_.end())
-    xbt_die("Can't find mount '%s' for '%s'", mount, piface_->getCname());
+  return actor_list_.size();
+}
 
-  return storage_.at(mount);
+std::vector<s4u::Disk*> HostImpl::get_disks() const
+{
+  std::vector<s4u::Disk*> disks;
+  for (auto const& [_, d] : disks_)
+    disks.push_back(d->get_iface());
+  return disks;
 }
 
-void HostImpl::getAttachedStorageList(std::vector<const char*>* storages)
+s4u::VirtualMachine* HostImpl::create_vm(const std::string& name, int core_amount, size_t ramsize)
 {
-  for (auto s : storage_)
-    if (s.second->attach_ == piface_->getCname())
-      storages->push_back(s.second->piface_.getName());
+  auto* host_vm = new kernel::resource::VirtualMachineImpl(name, get_iface(), core_amount, ramsize);
+  auto* vm      = new s4u::VirtualMachine(host_vm);
+  host_vm->set_piface(vm);
+  return create_vm(name, vm);
 }
 
-Action* HostImpl::read(surf_file_t fd, sg_size_t size)
+s4u::VirtualMachine* HostImpl::create_vm(const std::string& name, s4u::VirtualMachine* vm)
 {
-  simgrid::surf::StorageImpl* st = findStorageOnMountList(fd->mount());
-  XBT_DEBUG("READ %s on disk '%s'", fd->cname(), st->cname());
-  if (fd->tell() + size > fd->size()) {
-    if (fd->tell() > fd->size()) {
-      size = 0;
-    } else {
-      size = fd->size() - fd->tell();
-    }
-    fd->setPosition(fd->size());
-  } else
-    fd->incrPosition(size);
+  vms_[name] = vm->get_vm_impl();
 
-  return st->read(size);
+  // Create a VCPU for this VM
+  std::vector<double> speeds;
+  for (unsigned long i = 0; i < get_iface()->get_pstate_count(); i++)
+    speeds.push_back(get_iface()->get_pstate_speed(i));
+
+  auto* cpu =
+      englobing_zone_->get_cpu_vm_model()->create_cpu(vm, speeds)->set_core_count(vm->get_vm_impl()->get_core_amount());
+
+  if (get_iface()->get_pstate() != 0)
+    cpu->set_pstate(get_iface()->get_pstate());
+
+  cpu->seal();
+
+  /* Currently, a VM uses the network resource of its physical host */
+  vm->set_netpoint(get_iface()->get_netpoint());
+
+  vm->seal();
+
+  return vm;
+}
+
+void HostImpl::move_vm(VirtualMachineImpl* vm, HostImpl* destination)
+{
+  xbt_assert(vm && destination);
+
+  vms_.erase(vm->get_name());
+  destination->vms_[vm->get_name()] = vm;
 }
 
-Action* HostImpl::write(surf_file_t fd, sg_size_t size)
+void HostImpl::destroy_vm(const std::string& name)
 {
-  simgrid::surf::StorageImpl* st = findStorageOnMountList(fd->mount());
-  XBT_DEBUG("WRITE %s on disk '%s'. size '%llu/%llu'", fd->cname(), st->cname(), size, fd->size());
+  auto* vm = vms_[name];
+  vms_.erase(name);
+  vm->vm_destroy();
+}
+
+VirtualMachineImpl* HostImpl::get_vm_by_name_or_null(const std::string& name) const
+{
+  auto vm_it = vms_.find(name);
+  return vm_it == vms_.end() ? nullptr : vm_it->second;
+}
 
-  StorageAction* action = st->write(size);
-  action->file_         = fd;
-  /* Substract the part of the file that might disappear from the used sized on the storage element */
-  st->usedSize_ -= (fd->size() - fd->tell());
-  // If the storage is full before even starting to write
-  if (st->usedSize_ >= st->size_) {
-    action->setState(Action::State::failed);
+std::vector<s4u::VirtualMachine*> HostImpl::get_vms() const
+{
+  std::vector<s4u::VirtualMachine*> vms;
+  for (const auto& [_, vm] : vms_) {
+    vms.push_back(vm->get_iface());
   }
-  return action;
+  return vms;
 }
 
+s4u::Disk* HostImpl::create_disk(const std::string& name, double read_bandwidth, double write_bandwidth)
+{
+  auto disk = piface_.get_netpoint()->get_englobing_zone()->get_disk_model()->create_disk(name, read_bandwidth,
+                                                                                          write_bandwidth);
+  return disk->set_host(&piface_)->get_iface();
 }
+
+void HostImpl::add_disk(const s4u::Disk* disk)
+{
+  disks_[disk->get_name()] = disk->get_impl();
+}
+
+void HostImpl::remove_disk(const std::string& name)
+{
+  disks_.erase(name);
+}
+
+void HostImpl::seal()
+{
+  if (sealed_) {
+    return;
+  }
+  // seals host's CPU
+  get_iface()->get_cpu()->seal();
+  sealed_ = true;
+
+  /* seal its disks */
+  for (auto const& [_, disk] : disks_)
+    disk->seal();
+
+  /* seal its VMs */
+  for (auto const& [_, vm] : vms_)
+    vm->seal();
 }
+} // namespace resource
+} // namespace kernel
+} // namespace simgrid