X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/ecd5f7562caf1d443bf22788fa5f4fac408776ec..f5b4648bcb7aa3c1174e25c9f3cd28867b3b2871:/src/surf/cpu_interface.cpp diff --git a/src/surf/cpu_interface.cpp b/src/surf/cpu_interface.cpp index 6708124afe..840c9a814c 100644 --- a/src/surf/cpu_interface.cpp +++ b/src/surf/cpu_interface.cpp @@ -1,78 +1,47 @@ -/* Copyright (c) 2013-2017. The SimGrid Team. - * All rights reserved. */ +/* Copyright (c) 2013-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. */ #include "cpu_interface.hpp" -#include "src/instr/instr_private.hpp" // TRACE_is_enabled(). FIXME: remove by subscribing tracing to the surf signals -#include +#include "cpu_ti.hpp" +#include "src/kernel/resource/profile/Profile.hpp" +#include "src/surf/surf_interface.hpp" +#include "surf/surf.hpp" -XBT_LOG_EXTERNAL_CATEGORY(surf_kernel); -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_cpu, surf, "Logging specific to the SURF cpu module"); - -simgrid::surf::CpuModel *surf_cpu_model_pm; -simgrid::surf::CpuModel *surf_cpu_model_vm; +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(res_cpu, ker_resource, "CPU resource, fueling execution activites"); namespace simgrid { -namespace surf { +namespace kernel { +namespace resource { /********* * Model * *********/ -void CpuModel::updateActionsStateLazy(double now, double /*delta*/) +void CpuModel::update_actions_state_lazy(double now, double /*delta*/) { - while (not actionHeapIsEmpty() && double_equals(actionHeapTopDate(), now, sg_surf_precision)) { - - CpuAction* action = static_cast(actionHeapPop()); - XBT_CDEBUG(surf_kernel, "Something happened to action %p", action); - if (TRACE_is_enabled()) { - Cpu* cpu = static_cast(action->getVariable()->get_constraint(0)->get_id()); - TRACE_surf_host_set_utilization(cpu->getCname(), action->getCategory(), action->getVariable()->get_value(), - action->getLastUpdate(), now - action->getLastUpdate()); - } + while (not get_action_heap().empty() && double_equals(get_action_heap().top_date(), now, sg_surf_precision)) { + auto* action = static_cast(get_action_heap().pop()); + XBT_DEBUG("Something happened to action %p", action); - action->finish(Action::State::done); - XBT_CDEBUG(surf_kernel, "Action %p finished", action); - - /* set the remains to 0 due to precision problems when updating the remaining amount */ - action->setRemains(0); - } - if (TRACE_is_enabled()) { - //defining the last timestamp that we can safely dump to trace file - //without losing the event ascending order (considering all CPU's) - double smaller = -1; - for (Action const& action : *getRunningActionSet()) { - if (smaller < 0 || action.getLastUpdate() < smaller) - smaller = action.getLastUpdate(); - } - if (smaller > 0) { - TRACE_last_timestamp_to_dump = smaller; - } + action->finish(kernel::resource::Action::State::FINISHED); + XBT_DEBUG("Action %p finished", action); } } -void CpuModel::updateActionsStateFull(double now, double delta) +void CpuModel::update_actions_state_full(double /*now*/, double delta) { - for (auto it = std::begin(*getRunningActionSet()); it != std::end(*getRunningActionSet());) { - CpuAction& action = static_cast(*it); + for (auto it = std::begin(*get_started_action_set()); it != std::end(*get_started_action_set());) { + auto& action = static_cast(*it); ++it; // increment iterator here since the following calls to action.finish() may invalidate it - if (TRACE_is_enabled()) { - Cpu* cpu = static_cast(action.getVariable()->get_constraint(0)->get_id()); - TRACE_surf_host_set_utilization(cpu->getCname(), action.getCategory(), action.getVariable()->get_value(), - now - delta, delta); - TRACE_last_timestamp_to_dump = now - delta; - } - - action.updateRemains(action.getVariable()->get_value() * delta); - if (action.getMaxDuration() != NO_MAX_DURATION) - action.updateMaxDuration(delta); + action.update_remains(action.get_variable()->get_value() * delta); + action.update_max_duration(delta); - if (((action.getRemainsNoUpdate() <= 0) && (action.getVariable()->get_weight() > 0)) || - ((action.getMaxDuration() != NO_MAX_DURATION) && (action.getMaxDuration() <= 0))) { - action.finish(Action::State::done); + if (((action.get_remains_no_update() <= 0) && (action.get_variable()->get_penalty() > 0)) || + ((action.get_max_duration() != NO_MAX_DURATION) && (action.get_max_duration() <= 0))) { + action.finish(Action::State::FINISHED); } } } @@ -80,159 +49,143 @@ void CpuModel::updateActionsStateFull(double now, double delta) /************ * Resource * ************/ -Cpu::Cpu(Model *model, simgrid::s4u::Host *host, std::vector *speedPerPstate, int core) - : Cpu(model, host, nullptr/*constraint*/, speedPerPstate, core) +Cpu::Cpu(s4u::Host* host, const std::vector& speed_per_pstate) + : Resource_T(host->get_cname()), piface_(host), speed_per_pstate_(speed_per_pstate) { -} - -Cpu::Cpu(Model* model, simgrid::s4u::Host* host, lmm_constraint_t constraint, std::vector* speedPerPstate, - int core) - : Resource(model, host->getCname(), constraint), coresAmount_(core), host_(host) -{ - xbt_assert(core > 0, "Host %s must have at least one core, not 0.", host->getCname()); - - speed_.peak = speedPerPstate->front(); speed_.scale = 1; + speed_.peak = speed_per_pstate_.front(); + // FIXME[donassolo]: don't set here.. everytime I take half an hour to discover where it comes host->pimpl_cpu = this; - xbt_assert(speed_.scale > 0, "Speed of host %s must be >0", host->getCname()); - - // Copy the power peak array: - for (double const& value : *speedPerPstate) { - speedPerPstate_.push_back(value); - } } -Cpu::~Cpu() = default; - -int Cpu::getNbPStates() +void Cpu::reset_vcpu(Cpu* that) { - return speedPerPstate_.size(); + this->pstate_ = that->pstate_; + this->speed_ = that->speed_; + this->speed_per_pstate_.clear(); + this->speed_per_pstate_.assign(that->speed_per_pstate_.begin(), that->speed_per_pstate_.end()); } -void Cpu::setPState(int pstate_index) +void Cpu::set_pstate(int pstate_index) { - xbt_assert(pstate_index <= static_cast(speedPerPstate_.size()), + xbt_assert(pstate_index <= static_cast(speed_per_pstate_.size()), "Invalid parameters for CPU %s (pstate %d > length of pstates %d). Please fix your platform file, or your " "call to change the pstate.", - getCname(), pstate_index, static_cast(speedPerPstate_.size())); + get_cname(), pstate_index, static_cast(speed_per_pstate_.size())); - double new_peak_speed = speedPerPstate_[pstate_index]; + double new_peak_speed = speed_per_pstate_[pstate_index]; pstate_ = pstate_index; speed_.peak = new_peak_speed; - onSpeedChange(); + on_speed_change(); } -int Cpu::getPState() +double Cpu::get_pstate_peak_speed(int pstate_index) const { - return pstate_; -} - -double Cpu::getPstateSpeed(int pstate_index) -{ - xbt_assert((pstate_index <= static_cast(speedPerPstate_.size())), "Invalid parameters (pstate index out of bounds)"); + xbt_assert((pstate_index <= static_cast(speed_per_pstate_.size())), + "Invalid parameters (pstate index out of bounds)"); - return speedPerPstate_[pstate_index]; + return speed_per_pstate_[pstate_index]; } -double Cpu::getSpeed(double load) +void Cpu::on_speed_change() { - return load * speed_.peak; + s4u::Host::on_speed_change(*piface_); } -double Cpu::getAvailableSpeed() +Cpu* Cpu::set_core_count(int core_count) { -/* number between 0 and 1 */ - return speed_.scale; -} + xbt_assert(not is_sealed(), "Core count cannot be changed once CPU has been sealed"); + xbt_assert(core_count > 0, "Host %s must have at least one core, not 0.", piface_->get_cname()); + if (dynamic_cast(get_model()) != nullptr) + xbt_assert(core_count == 1, "Multi-core not handled by this model yet"); -void Cpu::onSpeedChange() { - TRACE_surf_host_set_speed(surf_get_clock(), getCname(), coresAmount_ * speed_.scale * speed_.peak); - s4u::Host::onSpeedChange(*host_); + core_count_ = core_count; + return this; } -int Cpu::coreCount() +int Cpu::get_core_count() { - return coresAmount_; + return core_count_; } -void Cpu::setStateTrace(tmgr_trace_t trace) +void Cpu::set_speed_profile(kernel::profile::Profile* profile) { - xbt_assert(stateEvent_ == nullptr, "Cannot set a second state trace to Host %s", host_->getCname()); + xbt_assert(speed_.event == nullptr, "Cannot set a second speed trace to Host %s", piface_->get_cname()); - stateEvent_ = future_evt_set->add_trace(trace, this); + speed_.event = profile->schedule(&profile::future_evt_set, this); } -void Cpu::setSpeedTrace(tmgr_trace_t trace) -{ - xbt_assert(speed_.event == nullptr, "Cannot set a second speed trace to Host %s", host_->getCname()); - speed_.event = future_evt_set->add_trace(trace, this); +void Cpu::seal() +{ + lmm::System* lmm = get_model()->get_maxmin_system(); + if (dynamic_cast(get_model()) == nullptr) + this->set_constraint(lmm->constraint_new(this, core_count_ * speed_per_pstate_.front())); + Resource::seal(); } - /********** * Action * **********/ -void CpuAction::updateRemainingLazy(double now) +void CpuAction::update_remains_lazy(double now) { - xbt_assert(getStateSet() == getModel()->getRunningActionSet(), "You're updating an action that is not running."); - xbt_assert(getPriority() > 0, "You're updating an action that seems suspended."); + xbt_assert(get_state_set() == get_model()->get_started_action_set(), + "You're updating an action that is not running."); + xbt_assert(get_sharing_penalty() > 0, "You're updating an action that seems suspended."); - double delta = now - getLastUpdate(); + double delta = now - get_last_update(); - if (getRemainsNoUpdate() > 0) { - XBT_CDEBUG(surf_kernel, "Updating action(%p): remains was %f, last_update was: %f", this, getRemainsNoUpdate(), - getLastUpdate()); - updateRemains(getLastValue() * delta); + if (get_remains_no_update() > 0) { + XBT_DEBUG("Updating action(%p): remains was %f, last_update was: %f", this, get_remains_no_update(), + get_last_update()); + update_remains(get_last_value() * delta); - if (TRACE_is_enabled()) { - Cpu* cpu = static_cast(getVariable()->get_constraint(0)->get_id()); - TRACE_surf_host_set_utilization(cpu->getCname(), getCategory(), getLastValue(), getLastUpdate(), - now - getLastUpdate()); - } - XBT_CDEBUG(surf_kernel, "Updating action(%p): remains is now %f", this, getRemainsNoUpdate()); + XBT_DEBUG("Updating action(%p): remains is now %f", this, get_remains_no_update()); } - refreshLastUpdate(); - setLastValue(getVariable()->get_value()); + set_last_update(); + set_last_value(get_variable()->get_value()); } -simgrid::xbt::signal CpuAction::onStateChange; +xbt::signal CpuAction::on_state_change; void CpuAction::suspend(){ - Action::State previous = getState(); - onStateChange(this, previous); + Action::State previous = get_state(); + on_state_change(*this, previous); Action::suspend(); } void CpuAction::resume(){ - Action::State previous = getState(); - onStateChange(this, previous); + Action::State previous = get_state(); + on_state_change(*this, previous); Action::resume(); } -void CpuAction::setState(Action::State state){ - Action::State previous = getState(); - Action::setState(state); - onStateChange(this, previous); +void CpuAction::set_state(Action::State state) +{ + Action::State previous = get_state(); + Action::set_state(state); + on_state_change(*this, previous); } + /** @brief returns a list of all CPUs that this action is using */ -std::list CpuAction::cpus() { +std::list CpuAction::cpus() const +{ std::list retlist; - int llen = getVariable()->get_number_of_constraint(); + int llen = get_variable()->get_number_of_constraint(); for (int i = 0; i < llen; i++) { /* Beware of composite actions: ptasks put links and cpus together */ // extra pb: we cannot dynamic_cast from void*... - Resource* resource = static_cast(getVariable()->get_constraint(i)->get_id()); - Cpu* cpu = dynamic_cast(resource); + Resource* resource = get_variable()->get_constraint(i)->get_id(); + auto* cpu = dynamic_cast(resource); if (cpu != nullptr) retlist.push_back(cpu); } return retlist; } - -} -} +} // namespace resource +} // namespace kernel +} // namespace simgrid