Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
cosmetics in the VM/energy code
authorMartin Quinson <martin.quinson@loria.fr>
Sat, 14 Nov 2015 21:47:32 +0000 (22:47 +0100)
committerMartin Quinson <martin.quinson@loria.fr>
Sat, 14 Nov 2015 21:56:37 +0000 (22:56 +0100)
src/surf/cpu_interface.cpp
src/surf/cpu_interface.hpp
src/surf/plugins/energy.cpp
src/surf/vm_hl13.cpp

index 345ad6c..1493337 100644 (file)
@@ -228,7 +228,7 @@ void Cpu::setState(e_surf_resource_state_t state)
   surf_callback_emit(cpuStateChangedCallbacks, this, old, state);
 }
 
-void Cpu::setVirtual(Cpu *physCpu)
+void Cpu::setPhysicalCPU(Cpu *physCpu)
 {
   if (!surf_energy)
          return;
@@ -239,7 +239,7 @@ void Cpu::setVirtual(Cpu *physCpu)
   (*surf_energy)[this] = cpu_energy_it->second;
 }
 
-Cpu* Cpu::isVirtual(void)
+Cpu* Cpu::getPhysicalCPU(void)
 {
    return physCpu;
 }
index 6ac0ad9..e5c82bd 100644 (file)
@@ -207,8 +207,8 @@ public:
   // Adrien - Oct 2015. Additional code related to VM energy extensions
   /* TODO: 
      1./ create a VirtualCpuModel that inherits from CpuModel or create
-     a VirtualCpuCas01Model that inherist from CpuCas01Model
-         => Question: not sure which one is really usefull :( The objective is to
+     a VirtualCpuCas01Model that inherit from CpuCas01Model
+         => Question: not sure which one is really useful :( The objective is to
      be able to create a VirtualCpu instead of a Cpu (i.e. a CpuCas01 to be exact). 
      2. create VirtualCpu class that inherits from Cpu or create
      a VirtualCpuCas01 class that inherits from CpuCas01 (same issue as the one described above).      
@@ -221,19 +221,11 @@ public:
   */
   Cpu *physCpu = NULL;
 
-       /**
-        * @brief Set the current Cpu as virtual or physical
-        *
-        * @param isVirtual true to indicate this Cpu is virtual, false otherwise
-        */
-       void setVirtual(Cpu *physCpu);
-
-       /**
-        * @brief Learn if this Cpu is physical or virtual
-        *
-        * @return phys CPU  if the current Cpu is a VM, null otherwise
-        */
-       Cpu* isVirtual(void);
+       /** @brief Specify the physical CPU of the current CPU (which is a vCPU) */
+       void setPhysicalCPU(Cpu *physCpu);
+
+       /** @brief Get the physical if the current one is a vCPU, or NULL if it's already a physical CPU */
+       Cpu* getPhysicalCPU(void);
    
    // Adrien - Oct 2015. End of additional code related to VM energy extensions
    // ////
index d1d871f..d3f8c68 100644 (file)
@@ -87,9 +87,8 @@ static void energyCpuDestructedCallback(Cpu *cpu){
   CpuEnergy *cpu_energy = cpu_energy_it->second;
   update_consumption(cpu, cpu_energy);
 
-  // Adrien - October 2015, Changes related to VM energy extensions
-  // Only report/delete and erase if the cpu_energy is related to a physical one
-  if(cpu->isVirtual() == NULL){
+  // Do nothing if that's a virtual CPU, only act for physical CPUs
+  if(cpu->getPhysicalCPU() == NULL){
     XBT_INFO("Total energy of host %s: %f Joules", cpu->getName(), cpu_energy->getConsumedEnergy());
     delete cpu_energy_it->second;
     surf_energy->erase(cpu_energy_it);
index 10dd53e..f35a2c7 100644 (file)
@@ -208,7 +208,7 @@ VMHL13::VMHL13(VMModel *model, const char* name, xbt_dict_t props,
       NULL,                       // host->state_trace,
       NULL);                       // host->properties,
 
-       p_cpu->setVirtual(sub_cpu);
+       p_cpu->setPhysicalCPU(sub_cpu);
 
   /* We create cpu_action corresponding to a VM process on the host operating system. */
   /* FIXME: TODO: we have to periodically input GUESTOS_NOISE to the system? how ? */