From f4d83eaf52926aa379292a9f75ba6b36eb04c52d Mon Sep 17 00:00:00 2001 From: Anouk Van Laer Date: Mon, 21 Aug 2017 16:02:45 +0100 Subject: [PATCH] sim, power: Temperature used for power calculations The temperature used for the power calculations was fixed at 0 degrees, unless a thermal model was setup. This commit allows the user to set the temperature that needs to be used by the power calculation during gem5 configuration. This value will be overwritten if there are thermal models present. Change-Id: I7ca8fa6766bdcba9d362c12fc75d1e1f74385f35 Reviewed-by: Sascha Bischoff Reviewed-by: Andreas Sandberg Reviewed-on: https://gem5-review.googlesource.com/8602 Reviewed-by: Jason Lowe-Power Maintainer: Andreas Sandberg --- src/sim/power/PowerModel.py | 3 +++ src/sim/power/power_model.cc | 6 ++++++ 2 files changed, 9 insertions(+) diff --git a/src/sim/power/PowerModel.py b/src/sim/power/PowerModel.py index 342569013..e3c0fb943 100644 --- a/src/sim/power/PowerModel.py +++ b/src/sim/power/PowerModel.py @@ -63,3 +63,6 @@ class PowerModel(SimObject): # Type of power model pm_type = Param.PMType("All", "Type of power model") + + # Ambient temperature to be used when no thermal model is present + ambient_temp = Param.Float(25.0, "Ambient temperature") diff --git a/src/sim/power/power_model.cc b/src/sim/power/power_model.cc index e8c522cad..6e065d6d6 100644 --- a/src/sim/power/power_model.cc +++ b/src/sim/power/power_model.cc @@ -57,6 +57,12 @@ PowerModel::PowerModel(const Params *p) panic_if(subsystem == NULL, "Subsystem is NULL! This is not acceptable for a PowerModel!\n"); subsystem->registerPowerProducer(this); + // The temperature passed here will be overwritten, if there is + // a thermal model present + for (auto & pms: states_pm){ + pms->setTemperature(p->ambient_temp); + } + } void -- 2.30.2