commit ed0f02e1f68e8771f4de514716f34c3de32b3045
author Anouk Van Laer <anouk.vanlaer@arm.com> Wed Mar 01 17:05:18 2017 +0000
committer Anouk Van Laer <anouk.vanlaer@arm.com> Wed Feb 28 21:55:35 2018 +0000
tree b9dafb62a24ee3e7c876c255b245123a736c2ab2
parent fbe63074e3a8128bdbe1a5e8f6509c565a3abbd4

sim: Added model type to power model

Static, dynamic or all to differentiate between types of power models
so for example static models will not be asked for a dynamic power

Change-Id: I3a0385821f7c671aedddaebeb038c677367faa81
Reviewed-by: Sascha Bischoff <sascha.bischoff@arm.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/8601
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/sim/power/PowerModel.py

diff --git a/src/sim/power/PowerModel.py b/src/sim/power/PowerModel.py
index ecb45b4..3425690 100644
--- a/src/sim/power/PowerModel.py
+++ b/src/sim/power/PowerModel.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2016 ARM Limited
+# Copyright (c) 2016-2018 ARM Limited
 # All rights reserved.
 #
 # The license below extends only to copyright in the software and shall
@@ -39,6 +39,9 @@
 from m5.params import *
 from m5.proxy import Parent
 
+# Enum for a type of  power model
+class PMType(Enum) : vals = ['All', 'Static', 'Dynamic']
+
 # Represents a power model for a simobj
 # The model itself is also a SimObject so we can make use some
 # nice features available such as Parent.any
@@ -57,3 +60,6 @@
     # Need a reference to the system so we can query the thermal domain
     # about temperature (temperature is needed for leakage calculation)
     subsystem = Param.SubSystem(Parent.any, "subsystem")
+
+    # Type of power model
+    pm_type = Param.PMType("All", "Type of power model")

src/sim/power/power_model.cc

diff --git a/src/sim/power/power_model.cc b/src/sim/power/power_model.cc
index 5f810fe..e8c522c 100644
--- a/src/sim/power/power_model.cc
+++ b/src/sim/power/power_model.cc
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016 ARM Limited
+ * Copyright (c) 2016-2018 ARM Limited
  * All rights reserved
  *
  * The license below extends only to copyright in the software and shall
@@ -52,7 +52,7 @@
 
 PowerModel::PowerModel(const Params *p)
     : SimObject(p), states_pm(p->pm), subsystem(p->subsystem),
-      clocked_object(NULL)
+      clocked_object(NULL), power_model_type(p->pm_type)
 {
     panic_if(subsystem == NULL,
              "Subsystem is NULL! This is not acceptable for a PowerModel!\n");
@@ -94,6 +94,10 @@
 {
     assert(clocked_object);
 
+    if (power_model_type == Enums::PMType::Static) {
+        // This power model only collects static data
+        return 0;
+    }
     std::vector<double> w = clocked_object->pwrStateWeights();
 
     // Same number of states (excluding UNDEFINED)
@@ -118,6 +122,11 @@
 
     std::vector<double> w = clocked_object->pwrStateWeights();
 
+    if (power_model_type == Enums::PMType::Dynamic) {
+        // This power model only collects dynamic data
+        return 0;
+    }
+
     // Same number of states (excluding UNDEFINED)
     assert(w.size() - 1 == states_pm.size());
 

src/sim/power/power_model.hh

diff --git a/src/sim/power/power_model.hh b/src/sim/power/power_model.hh
index 976c054..e482b80 100644
--- a/src/sim/power/power_model.hh
+++ b/src/sim/power/power_model.hh
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016 ARM Limited
+ * Copyright (c) 2016, 2018 ARM Limited
  * All rights reserved
  *
  * The license below extends only to copyright in the software and shall
@@ -41,6 +41,7 @@
 #define __SIM_POWER_POWER_MODEL_HH__
 
 #include "base/statistics.hh"
+#include "enums/PMType.hh"
 #include "params/PowerModel.hh"
 #include "params/PowerModelState.hh"
 #include "sim/probe/probe.hh"
@@ -184,6 +185,9 @@
 
     /** The clocked object we belong to */
     ClockedObject * clocked_object;
+
+    /** The type of power model - collects all power, static or dynamic only */
+    Enums::PMType power_model_type;
 };
 
 #endif