commit 0757bef15d934b22555c396bcbcb91c0a1dffbe5
author David Guillen Fandos <david.guillen@arm.com> Fri Jul 01 17:46:25 2016 +0100
committer Andreas Sandberg <andreas.sandberg@arm.com> Thu Nov 16 16:15:12 2017 +0000
tree 1473b0ad4803cceeac138e22821e53630f1a582c
parent 899ad72060a7a95d6c1d7090c40f23a04699b43c

pwr: Enable multiple power models per component

This patch allows the user to specify more than one
power model for any given Clocked Object. This is
useful and some times necessary to properly model
components that have multiple power sources (or
subcomponents) but gem5 doesn't model them.

The ideal solution would be to have a DictParam
to replace the VectorParam so each model can have
a name and can be identified in the stats file.

Change-Id: I4080a7054a16b56069c44750a7a9ce4e674cdf9d
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/5733
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/sim/ClockedObject.py

diff --git a/src/sim/ClockedObject.py b/src/sim/ClockedObject.py
index c8bf809..2d61718 100644
--- a/src/sim/ClockedObject.py
+++ b/src/sim/ClockedObject.py
@@ -67,7 +67,7 @@
     clk_domain = Param.ClockDomain(Parent.clk_domain, "Clock domain")
 
     # Power model for this ClockedObject
-    power_model = Param.PowerModel(NULL, "Power model")
+    power_model = VectorParam.PowerModel([], "Power models")
 
     # Provide initial power state, should ideally get redefined in startup
     # routine

src/sim/clocked_object.cc

diff --git a/src/sim/clocked_object.cc b/src/sim/clocked_object.cc
index f1e3134..5638911 100644
--- a/src/sim/clocked_object.cc
+++ b/src/sim/clocked_object.cc
@@ -49,8 +49,8 @@
     prvEvalTick(0)
 {
     // Register the power_model with the object
-    if (p->power_model)
-        p->power_model->setClockedObject(this);
+    for (auto & power_model: p->power_model)
+        power_model->setClockedObject(this);
 }
 
 void