Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Kill unused simgrid::surf::HostAction
[simgrid.git] / src / surf / host_interface.cpp
index 737583b..a9f48dd 100644 (file)
@@ -6,6 +6,8 @@
 
 #include "host_interface.hpp"
 
+#include <simgrid/Host.hpp>
+
 #include "src/simix/smx_private.h"
 #include "cpu_cas01.hpp"
 #include "simgrid/sg_config.h"
@@ -28,11 +30,8 @@ void host_add_traces(){
 
 namespace simgrid {
 namespace surf {
-  
-surf_callback(void, simgrid::surf::Host*) hostCreatedCallbacks;
-surf_callback(void, simgrid::surf::Host*) hostDestructedCallbacks;
-surf_callback(void, simgrid::surf::Host*, e_surf_resource_state_t, e_surf_resource_state_t) hostStateChangedCallbacks;
-surf_callback(void, simgrid::surf::HostAction*, e_surf_action_state_t, e_surf_action_state_t) hostActionStateChangedCallbacks;
+
+simgrid::xbt::Extension<simgrid::Host, Host> Host::EXTENSION_ID;
 
 /*********
  * Model *
@@ -76,9 +75,26 @@ void HostModel::adjustWeightOfDummyCpuActions()
 /************
  * Resource *
  ************/
+simgrid::surf::signal<void(simgrid::surf::Host*)> Host::onCreation;
+simgrid::surf::signal<void(simgrid::surf::Host*)> Host::onDestruction;
+simgrid::surf::signal<void(simgrid::surf::Host*, e_surf_resource_state_t, e_surf_resource_state_t)> Host::onStateChange;
+
+static void host_destroy(void *h){
+       static_cast<simgrid::surf::Host*>(h)->destroy();
+}
+
+void Host::classInit()
+{
+  if (!EXTENSION_ID.valid()) {
+    EXTENSION_ID = simgrid::Host::extension_create<simgrid::surf::Host>(host_destroy);
+    SURF_HOST_LEVEL = EXTENSION_ID.id(); // FIXME: KILLME
+  }
+}
+
 Host::Host(simgrid::surf::Model *model, const char *name, xbt_dict_t props,
                                 xbt_dynar_t storage, RoutingEdge *netElm, Cpu *cpu)
- : Resource(model, name, props)
+ : Resource(model, name)
+ , PropertyHolder(props)
  , p_storage(storage), p_netElm(netElm), p_cpu(cpu)
 {
   p_params.ramsize = 0;
@@ -86,27 +102,57 @@ Host::Host(simgrid::surf::Model *model, const char *name, xbt_dict_t props,
 
 Host::Host(simgrid::surf::Model *model, const char *name, xbt_dict_t props, lmm_constraint_t constraint,
                                         xbt_dynar_t storage, RoutingEdge *netElm, Cpu *cpu)
- : Resource(model, name, props, constraint)
+ : Resource(model, name, constraint)
+ , PropertyHolder(props)
  , p_storage(storage), p_netElm(netElm), p_cpu(cpu)
 {
   p_params.ramsize = 0;
 }
 
-Host::~Host(){
-  surf_callback_emit(hostDestructedCallbacks, this);
+/** @brief use destroy() instead of this destructor */
+Host::~Host()
+{
+       xbt_assert(currentlyDestroying_, "Don't delete Hosts directly. Call destroy() instead.");
+}
+/** @brief Fire the require callbacks and destroy the object
+ *
+ * Don't delete directly an Host, call h->destroy() instead.
+ */
+void Host::destroy()
+{
+       if (!currentlyDestroying_) {
+               currentlyDestroying_ = true;
+               onDestruction(this);
+               delete this;
+       }
+}
+
+void Host::attach(simgrid::Host* host)
+{
+  if (p_host != nullptr)
+    xbt_die("Already attached to host %s", host->id().c_str());
+  host->extension_set(this);
+  p_host = host;
+  onCreation(this);
 }
 
+e_surf_resource_state_t Host::getState() {
+  return p_cpu->getState();
+}
 void Host::setState(e_surf_resource_state_t state){
   e_surf_resource_state_t old = Resource::getState();
   Resource::setState(state);
-  surf_callback_emit(hostStateChangedCallbacks, this, old, state);
+  onStateChange(this, old, state);
   p_cpu->setState(state);
 }
 
-xbt_dict_t Host::getProperties()
-{
-  return p_cpu->getProperties();
+Action *Host::execute(double size) {
+  return p_cpu->execute(size);
 }
+Action *Host::sleep(double duration) {
+  return p_cpu->sleep(duration);
+}
+
 
 simgrid::surf::Storage *Host::findStorageOnMountList(const char* mount)
 {
@@ -345,15 +391,5 @@ void Host::setParams(vm_params_t params)
   p_params = *params;
 }
 
-/**********
- * Action *
- **********/
-
-void HostAction::setState(e_surf_action_state_t state){
-  e_surf_action_state_t old = getState();
-  Action::setState(state);
-  surf_callback_emit(hostActionStateChangedCallbacks, this, old, state);
-}
-
 }
 }