power: Avoid forward declarations that confuse wrappers
authorAndreas Sandberg <andreas.sandberg@arm.com>
Mon, 27 Feb 2017 13:17:51 +0000 (13:17 +0000)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Tue, 7 Mar 2017 11:44:32 +0000 (11:44 +0000)
The Python wrappers get confused by the forward declarations in the
power framework. This changeset restructures the code slightly to
avoid the troublesome forward declarations.

Change-Id: Id8c93224f1988edb5fdf9d3abc6237f2f688c02d
Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Andreas Hansson <andreas.hansson@arm.com>
Reviewed-by: Curtis Dunham <curtis.dunham@arm.com>
Reviewed-by: Sascha Bischoff <sascha.bischoff@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/2227
Reviewed-by: Pierre-Yves PĂ©neau <pierre-yves.peneau@lirmm.fr>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/sim/power/SConscript
src/sim/power/ThermalModel.py
src/sim/power/thermal_domain.hh
src/sim/power/thermal_entity.hh
src/sim/power/thermal_model.cc
src/sim/power/thermal_model.hh
src/sim/power/thermal_node.cc [new file with mode: 0644]
src/sim/power/thermal_node.hh [new file with mode: 0644]

index 2f0eb4fc19e2ac7b6c23c001fb90da82e56474b5..b90d93045b501a10e8227aec9e4874aea2b1702f 100644 (file)
@@ -40,5 +40,6 @@ Source('power_model.cc')
 Source('mathexpr_powermodel.cc')
 Source('thermal_domain.cc')
 Source('thermal_model.cc')
+Source('thermal_node.cc')
 
 DebugFlag('ThermalDomain')
index 4a2f5d9310f33ae7b26bce8d66a482bb6f692a90..4c397119da8948a07d7fb8e92fca2040921caf41 100644 (file)
@@ -45,7 +45,7 @@ from m5.objects import ThermalDomain
 # Represents a thermal node
 class ThermalNode(SimObject):
     type = 'ThermalNode'
-    cxx_header = "sim/power/thermal_model.hh"
+    cxx_header = "sim/power/thermal_node.hh"
 
 # Represents a thermal resistor
 class ThermalResistor(SimObject):
index f385994f8e5e182275b6c8008a42108f1ab5e80c..af59af5ccd0045aace677731275640035a59e1b5 100644 (file)
 #include "base/statistics.hh"
 #include "params/ThermalDomain.hh"
 #include "sim/power/thermal_entity.hh"
+#include "sim/power/thermal_node.hh"
 #include "sim/sim_object.hh"
+#include "sim/sub_system.hh"
 
-class SubSystem;
-class ThermalNode;
 template <class T> class ProbePointArg;
 
 /**
index fb703961fd5e2ee5d62c9495a0480e84b7717385..5c0ab30450dbcee971bee7634bb03010b0d25988 100644 (file)
@@ -40,6 +40,8 @@
 #ifndef __SIM_THERMAL_ENTITY_HH__
 #define __SIM_THERMAL_ENTITY_HH__
 
+#include "sim/sim_object.hh"
+
 class LinearEquation;
 class ThermalNode;
 
index 13242f666ef320ada5326e6c6d3171f30eb9d811..05ce42bc90de0f21e9abe1231e3558ead7eabb7e 100644 (file)
@@ -41,7 +41,6 @@
 
 #include "base/statistics.hh"
 #include "params/ThermalCapacitor.hh"
-#include "params/ThermalNode.hh"
 #include "params/ThermalReference.hh"
 #include "params/ThermalResistor.hh"
 #include "sim/clocked_object.hh"
 #include "sim/power/thermal_domain.hh"
 #include "sim/sim_object.hh"
 
-/**
- * ThermalNode
- */
-ThermalNode::ThermalNode(const Params *p)
-    : SimObject(p), id(-1), isref(false), temp(0.0f)
-{
-}
-
-ThermalNode *
-ThermalNodeParams::create()
-{
-    return new ThermalNode(this);
-}
-
 /**
  * ThermalReference
  */
index 32f34f109f39249bf1df1da666777771a9db4c03..7ee4ffdc3ce5ee630fa102d19a488b5bf0de2ef3 100644 (file)
 #include "params/ThermalReference.hh"
 #include "params/ThermalResistor.hh"
 #include "sim/clocked_object.hh"
+#include "sim/power/thermal_domain.hh"
 #include "sim/power/thermal_entity.hh"
+#include "sim/power/thermal_node.hh"
 #include "sim/sim_object.hh"
 
-class ThermalDomain;
-
-
-/**
- * A ThermalNode is used to connect thermal entities, such as
- * resistors, capacitors, references and domains. It is the circuital
- * equivalent to a voltage node.
- */
-class ThermalNode : public SimObject
-{
-  public:
-    typedef SimObjectParams Params;
-    ThermalNode(const Params *p);
-
-    int id;
-    bool isref;
-    double temp;
-};
 
 /**
  * A ThermalResistor is used to model a thermal resistance between two
diff --git a/src/sim/power/thermal_node.cc b/src/sim/power/thermal_node.cc
new file mode 100644 (file)
index 0000000..9209b0b
--- /dev/null
@@ -0,0 +1,57 @@
+/*
+ * Copyright (c) 2015 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder.  You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: David Guillen Fandos
+ */
+
+
+#include "sim/power/thermal_node.hh"
+
+#include "params/ThermalNode.hh"
+
+/**
+ * ThermalNode
+ */
+ThermalNode::ThermalNode(const ThermalNodeParams *p)
+    : SimObject(p), id(-1), isref(false), temp(0.0f)
+{
+}
+
+ThermalNode *
+ThermalNodeParams::create()
+{
+    return new ThermalNode(this);
+}
diff --git a/src/sim/power/thermal_node.hh b/src/sim/power/thermal_node.hh
new file mode 100644 (file)
index 0000000..46a3537
--- /dev/null
@@ -0,0 +1,62 @@
+/*
+ * Copyright (c) 2015 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder.  You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: David Guillen Fandos
+ */
+
+#ifndef __SIM_THERMAL_NODE_HH__
+#define __SIM_THERMAL_NODE_HH__
+
+#include "sim/sim_object.hh"
+
+struct ThermalNodeParams;
+
+/**
+ * A ThermalNode is used to connect thermal entities, such as
+ * resistors, capacitors, references and domains. It is the circuital
+ * equivalent to a voltage node.
+ */
+class ThermalNode : public SimObject
+{
+  public:
+    ThermalNode(const ThermalNodeParams *p);
+
+    int id;
+    bool isref;
+    double temp;
+};
+
+#endif