Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Misc code simplifications guided by Sonar smells.
[simgrid.git] / src / surf / HostImpl.cpp
index 4fe0ac9..51f7f98 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2013-2021. 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. */
@@ -28,35 +28,36 @@ namespace resource {
 /************
  * Resource *
  ************/
-HostImpl::HostImpl(const std::string& name, s4u::Host* piface) : piface_(this), name_(name)
-{
-  xbt_assert(s4u::Host::by_name_or_null(name_) == nullptr, "Refusing to create a second host named '%s'.", get_cname());
-  s4u::Engine::get_instance()->host_register(name_, piface);
-}
-
 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());
-  s4u::Engine::get_instance()->host_register(name_, &piface_);
 }
 
 HostImpl::~HostImpl()
 {
   /* All actors should be gone when the host is turned off (by the end of the simulation). */
   if (not actor_list_.empty()) {
-    std::string msg = "Shutting down host, but it's not empty:";
-    for (auto const& actor : actor_list_)
-      msg += "\n\t" + std::string(actor.get_name());
-
-    EngineImpl::get_instance()->display_all_actor_status();
-    xbt_die("%s", msg.c_str());
+    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.second->destroy();
+  for (auto const& [_, d] : disks_)
+    d->destroy();
+
+  for (auto const& [_, vm] : vms_)
+    vm->vm_destroy();
 }
 
 /** @brief Fire the required callbacks and destroy the object
@@ -66,7 +67,6 @@ HostImpl::~HostImpl()
 void HostImpl::destroy()
 {
   s4u::Host::on_destruction(*this->get_iface());
-  s4u::Engine::get_instance()->host_unregister(std::string(name_));
   delete this;
 }
 
@@ -78,34 +78,31 @@ void HostImpl::turn_on() const
 {
   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->name, arg->code, nullptr, arg->host, nullptr);
-    actor->set_properties(arg->properties);
-    if (arg->on_exit)
-      *actor->on_exit = *arg->on_exit;
-    if (arg->kill_time >= 0)
-      actor->set_kill_time(arg->kill_time);
-    if (arg->auto_restart)
-      actor->set_auto_restart(arg->auto_restart);
-    if (arg->daemon_)
-      actor->daemonize();
+    actor::ActorImplPtr actor = actor::ActorImpl::create(arg);
   }
 }
 
 /** 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 (auto& activity : EngineImpl::get_instance()->get_maestro()->activities_) {
+  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->state_ = activity::State::FAILED;
+        exec->set_state(activity::State::FAILED);
       }
     }
   }
@@ -120,6 +117,12 @@ void HostImpl::turn_off(const actor::ActorImpl* issuer)
   actors_at_boot_.erase(elm, end(actors_at_boot_));
 }
 
+HostImpl* HostImpl::set_englobing_zone(routing::NetZoneImpl* englobing_zone)
+{
+  englobing_zone_ = englobing_zone;
+  return this;
+}
+
 std::vector<s4u::ActorPtr> HostImpl::get_all_actors()
 {
   std::vector<s4u::ActorPtr> res;
@@ -135,11 +138,74 @@ size_t HostImpl::get_actor_count() const
 std::vector<s4u::Disk*> HostImpl::get_disks() const
 {
   std::vector<s4u::Disk*> disks;
-  for (auto const& d : disks_)
-    disks.push_back(d.second->get_iface());
+  for (auto const& [_, d] : disks_)
+    disks.push_back(d->get_iface());
   return disks;
 }
 
+s4u::VirtualMachine* HostImpl::create_vm(const std::string& name, int core_amount, size_t ramsize)
+{
+  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);
+}
+
+s4u::VirtualMachine* HostImpl::create_vm(const std::string& name, s4u::VirtualMachine* vm)
+{
+  vms_[name] = vm->get_vm_impl();
+
+  // 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;
+}
+
+void HostImpl::destroy_vm(const std::string& name)
+{
+  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;
+}
+
+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 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,
@@ -167,8 +233,12 @@ void HostImpl::seal()
   sealed_ = true;
 
   /* seal its disks */
-  for (auto const& disk : disks_)
-    disk.second->seal();
+  for (auto const& [_, disk] : disks_)
+    disk->seal();
+
+  /* seal its VMs */
+  for (auto const& [_, vm] : vms_)
+    vm->seal();
 }
 } // namespace resource
 } // namespace kernel