X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/40dc178ab282a23a717380eedcfc0336134a1dc5..7073caeb8a32a2a18ad9bef9143d42cbbceee47f:/src/plugins/host_energy.cpp diff --git a/src/plugins/host_energy.cpp b/src/plugins/host_energy.cpp index b68edc8aee..81f1162a12 100644 --- a/src/plugins/host_energy.cpp +++ b/src/plugins/host_energy.cpp @@ -1,4 +1,4 @@ -/* Copyright (c) 2010-2020. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2010-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. */ @@ -131,6 +131,7 @@ class HostEnergy { simgrid::s4u::Host* host_ = nullptr; /*< List of (idle_power, epsilon_power, max_power) tuple corresponding to each cpu pstate */ std::vector power_range_watts_list_; + bool has_pstate_power_values_ = false; /*< Whether power consumption values were provided for all pstates */ /* We need to keep track of what pstate has been used, as we will sometimes be notified only *after* a pstate has been * used (but we need to update the energy consumption with the old pstate!) @@ -155,19 +156,26 @@ public: explicit HostEnergy(simgrid::s4u::Host* ptr); ~HostEnergy(); + bool has_pstate_power_values() const; + double get_current_watts_value(); - double get_current_watts_value(double cpu_load); + double get_current_watts_value(double cpu_load) const; double get_consumed_energy(); - double get_watt_idle_at(int pstate); - double get_watt_min_at(int pstate); - double get_watt_max_at(int pstate); - double get_power_range_slope_at(int pstate); - double get_last_update_time() { return last_updated_; } + double get_watt_idle_at(int pstate) const; + double get_watt_min_at(int pstate) const; + double get_watt_max_at(int pstate) const; + double get_power_range_slope_at(int pstate) const; + double get_last_update_time() const { return last_updated_; } void update(); }; simgrid::xbt::Extension HostEnergy::EXTENSION_ID; +/* Returns whether power consumption values were provided for all pstates. */ +bool HostEnergy::has_pstate_power_values() const { + return has_pstate_power_values_; +} + /* Computes the consumption so far. Called lazily on need. */ void HostEnergy::update() { @@ -206,15 +214,6 @@ HostEnergy::HostEnergy(simgrid::s4u::Host* ptr) : host_(ptr) init_watts_range_list(); const char* off_power_str = host_->get_property("wattage_off"); - if (off_power_str == nullptr) { - off_power_str = host_->get_property("watt_off"); - - static bool warned = false; - if (off_power_str != nullptr && not warned) { - warned = true; - XBT_WARN("Please use 'wattage_off' instead of 'watt_off' to define the idle wattage of hosts in your XML."); - } - } if (off_power_str != nullptr) { try { this->watts_off_ = std::stod(std::string(off_power_str)); @@ -228,31 +227,31 @@ HostEnergy::HostEnergy(simgrid::s4u::Host* ptr) : host_(ptr) HostEnergy::~HostEnergy() = default; -double HostEnergy::get_watt_idle_at(int pstate) +double HostEnergy::get_watt_idle_at(int pstate) const { - xbt_assert(not power_range_watts_list_.empty(), "No power range properties specified for host %s", - host_->get_cname()); + if (not has_pstate_power_values_) + return 0.0; return power_range_watts_list_[pstate].idle_; } -double HostEnergy::get_watt_min_at(int pstate) +double HostEnergy::get_watt_min_at(int pstate) const { - xbt_assert(not power_range_watts_list_.empty(), "No power range properties specified for host %s", - host_->get_cname()); + if (not has_pstate_power_values_) + return 0.0; return power_range_watts_list_[pstate].epsilon_; } -double HostEnergy::get_watt_max_at(int pstate) +double HostEnergy::get_watt_max_at(int pstate) const { - xbt_assert(not power_range_watts_list_.empty(), "No power range properties specified for host %s", - host_->get_cname()); + if (not has_pstate_power_values_) + return 0.0; return power_range_watts_list_[pstate].max_; } -double HostEnergy::get_power_range_slope_at(int pstate) +double HostEnergy::get_power_range_slope_at(int pstate) const { - xbt_assert(not power_range_watts_list_.empty(), "No power range properties specified for host %s", - host_->get_cname()); + if (not has_pstate_power_values_) + return 0.0; return power_range_watts_list_[pstate].slope_; } @@ -292,10 +291,10 @@ double HostEnergy::get_current_watts_value() * * Whether the host is ON or OFF is not taken into account. */ -double HostEnergy::get_current_watts_value(double cpu_load) +double HostEnergy::get_current_watts_value(double cpu_load) const { - xbt_assert(not power_range_watts_list_.empty(), "No power range properties specified for host %s", - host_->get_cname()); + if (not has_pstate_power_values_) + return 0.0; /* Return watts_off if pstate == pstate_off (ie, if the host is off) */ if (this->pstate_ == pstate_off_) { @@ -340,70 +339,9 @@ double HostEnergy::get_consumed_energy() void HostEnergy::init_watts_range_list() { - const char* old_prop = host_->get_property("watt_per_state"); - if (old_prop != nullptr) { - std::vector all_power_values; - boost::split(all_power_values, old_prop, boost::is_any_of(",")); - - xbt_assert(all_power_values.size() == (unsigned)host_->get_pstate_count(), - "Invalid XML file. Found %zu energetic profiles for %d pstates", all_power_values.size(), - host_->get_pstate_count()); - - // XBT_ATTRIB_DEPRECATED_v328: putting this macro name here so that we find it during the deprecation cleanups - std::string msg = std::string("DEPRECATION WARNING: Property 'watt_per_state' will only work until v3.28.\n"); - msg += std::string("The old syntax 'Idle:OneCore:AllCores' must be converted into 'Idle:Epsilon:AllCores' to " - "properly model the consumption of non-whole tasks on mono-core hosts. Here are the values to " - "use for host '") + - host_->get_cname() + "' in your XML file:\n"; - msg += " current_power_values; - boost::split(current_power_values, current_power_values_str, boost::is_any_of(":")); - double p_idle = xbt_str_parse_double((current_power_values.at(0)).c_str(), - "Invalid obsolete XML file. Fix your watt_per_state property."); - double p_full; - double p_epsilon; - - if (current_power_values.size() == 3) { - double p_one_core = xbt_str_parse_double((current_power_values.at(1)).c_str(), - "Invalid obsolete XML file. Fix your watt_per_state property."); - p_full = xbt_str_parse_double((current_power_values.at(2)).c_str(), - "Invalid obsolete XML file. Fix your watt_per_state property."); - if (host_->get_core_count() == 1) { - p_epsilon = p_full; - } else { - p_epsilon = p_one_core - ((p_full - p_one_core) / (host_->get_core_count() - 1)); - } - } else { // consumption given with idle and full only - p_full = xbt_str_parse_double((current_power_values.at(1)).c_str(), - "Invalid obsolete XML file. Fix your watt_per_state property."); - if (host_->get_core_count() == 1) { - p_epsilon = p_full; - } else { - p_epsilon = p_idle; - } - } - - PowerRange range(p_idle, p_epsilon, p_full); - power_range_watts_list_.push_back(range); - - msg += std::to_string(p_idle) + ":" + std::to_string(p_epsilon) + ":" + std::to_string(p_full); - msg += ","; - } - msg.pop_back(); // Remove the extraneous ',' - msg += "\" />"; - XBT_WARN("%s", msg.c_str()); - return; - } - const char* all_power_values_str = host_->get_property("wattage_per_state"); if (all_power_values_str == nullptr) { - /* If no power values are given, we assume it's 0 everywhere */ - XBT_DEBUG("No energetic profiles given for host %s, using 0 W by default.", host_->get_cname()); - for (int i = 0; i < host_->get_pstate_count(); ++i) { - PowerRange range(0,0,0); - power_range_watts_list_.push_back(range); - } + XBT_WARN("No energetic profiles (wattage_per_state) given for host %s, using 0 W by default. Direct request of power/energy consumption of this host will fail.", host_->get_cname()); return; } @@ -430,28 +368,27 @@ void HostEnergy::init_watts_range_list() double epsilon_power; double max_power; - char* msg_idle = bprintf("Invalid Idle value for pstate %d on host %s: %%s", i, host_->get_cname()); - char* msg_epsilon = bprintf("Invalid Epsilon value for pstate %d on host %s: %%s", i, host_->get_cname()); - char* msg_max = bprintf("Invalid AllCores value for pstate %d on host %s: %%s", i, host_->get_cname()); + auto msg_idle = xbt::string_printf("Invalid Idle value for pstate %d on host %s", i, host_->get_cname()); + auto msg_epsilon = xbt::string_printf("Invalid Epsilon value for pstate %d on host %s", i, host_->get_cname()); + auto msg_max = xbt::string_printf("Invalid AllCores value for pstate %d on host %s", i, host_->get_cname()); - idle_power = xbt_str_parse_double((current_power_values.at(0)).c_str(), msg_idle); + idle_power = xbt_str_parse_double((current_power_values.at(0)).c_str(), msg_idle.c_str()); if (current_power_values.size() == 2) { // Case: Idle:AllCores - epsilon_power = xbt_str_parse_double((current_power_values.at(0)).c_str(), msg_idle); - max_power = xbt_str_parse_double((current_power_values.at(1)).c_str(), msg_max); + epsilon_power = xbt_str_parse_double((current_power_values.at(0)).c_str(), msg_idle.c_str()); + max_power = xbt_str_parse_double((current_power_values.at(1)).c_str(), msg_max.c_str()); } else { // Case: Idle:Epsilon:AllCores - epsilon_power = xbt_str_parse_double((current_power_values.at(1)).c_str(), msg_epsilon); - max_power = xbt_str_parse_double((current_power_values.at(2)).c_str(), msg_max); + epsilon_power = xbt_str_parse_double((current_power_values.at(1)).c_str(), msg_epsilon.c_str()); + max_power = xbt_str_parse_double((current_power_values.at(2)).c_str(), msg_max.c_str()); } XBT_DEBUG("Creating PowerRange for host %s. Idle:%f, Epsilon:%f, AllCores:%f.", host_->get_cname(), idle_power, epsilon_power, max_power); PowerRange range(idle_power, epsilon_power, max_power); power_range_watts_list_.push_back(range); - xbt_free(msg_idle); - xbt_free(msg_epsilon); - xbt_free(msg_max); ++i; } + + has_pstate_power_values_ = true; } } // namespace plugin } // namespace simgrid @@ -472,8 +409,8 @@ static void on_creation(simgrid::s4u::Host& host) static void on_action_state_change(simgrid::kernel::resource::CpuAction const& action, simgrid::kernel::resource::Action::State /*previous*/) { - for (simgrid::kernel::resource::Cpu* const& cpu : action.cpus()) { - simgrid::s4u::Host* host = cpu->get_host(); + for (simgrid::kernel::resource::CpuImpl* const& cpu : action.cpus()) { + simgrid::s4u::Host* host = cpu->get_iface(); if (host != nullptr) { // If it's a VM, take the corresponding PM const simgrid::s4u::VirtualMachine* vm = dynamic_cast(host); @@ -481,7 +418,7 @@ static void on_action_state_change(simgrid::kernel::resource::CpuAction const& a host = vm->get_pm(); // Get the host_energy extension for the relevant host - HostEnergy* host_energy = host->extension(); + auto* host_energy = host->extension(); if (host_energy->get_last_update_time() < surf_get_clock()) host_energy->update(); @@ -496,7 +433,7 @@ static void on_host_change(simgrid::s4u::Host const& host) if (dynamic_cast(&host)) // Ignore virtual machines return; - HostEnergy* host_energy = host.extension(); + auto* host_energy = host.extension(); host_energy->update(); } @@ -515,7 +452,7 @@ static void on_simulation_end() double total_energy = 0.0; // Total energy consumption (whole platform) double used_hosts_energy = 0.0; // Energy consumed by hosts that computed something for (simgrid::s4u::Host const* host : simgrid::s4u::Engine::get_instance()->get_all_hosts()) { - if (dynamic_cast(host) == nullptr) { // Ignore virtual machines + if (host && dynamic_cast(host) == nullptr) { // Ignore virtual machines double energy = host->extension()->get_consumed_energy(); total_energy += energy; if (host->extension()->host_was_used_) @@ -551,7 +488,7 @@ void sg_host_energy_plugin_init() // that the next trigger would be the 2nd compute, hence ignoring the idle time // during the recv call. By updating at the beginning of a compute, we can // fix that. (If the cpu is not idle, this is not required.) - simgrid::s4u::Exec::on_start.connect([](simgrid::s4u::Actor const&, simgrid::s4u::Exec const& activity) { + simgrid::s4u::Exec::on_start.connect([](simgrid::s4u::Exec const& activity) { if (activity.get_host_number() == 1) { // We only run on one host simgrid::s4u::Host* host = activity.get_host(); const simgrid::s4u::VirtualMachine* vm = dynamic_cast(host); @@ -598,7 +535,10 @@ static void ensure_plugin_inited() double sg_host_get_consumed_energy(const_sg_host_t host) { ensure_plugin_inited(); - return host->extension()->get_consumed_energy(); + auto host_energy = host->extension(); + xbt_assert(host_energy->has_pstate_power_values(), "No power range properties specified for host %s", + host->get_cname()); + return host_energy->get_consumed_energy(); } /** @ingroup plugin_host_energy @@ -649,5 +589,8 @@ double sg_host_get_power_range_slope_at(const_sg_host_t host, int pstate) double sg_host_get_current_consumption(const_sg_host_t host) { ensure_plugin_inited(); - return host->extension()->get_current_watts_value(); + auto host_energy = host->extension(); + xbt_assert(host_energy->has_pstate_power_values(), "No power range properties specified for host %s", + host->get_cname()); + return host_energy->get_current_watts_value(); }