ext: McPAT interface changes and fixes

This patch includes software engineering changes and some generic bug fixes
Joel Hestness and Yasuko Eckert made to McPAT 0.8. There are still known
issues/concernts we did not have a chance to address in this patch.

High-level changes in this patch include:
 1) Making XML parsing modular and hierarchical:
   - Shift parsing responsibility into the components
   - Read XML in a (mostly) context-free recursive manner so that McPAT input
     files can contain arbitrary component hierarchies
 2) Making power, energy, and area calculations a hierarchical and recursive
    process
   - Components track their subcomponents and recursively call compute
     functions in stages
   - Make C++ object hierarchy reflect inheritance of classes of components
     with similar structures
   - Simplify computeArea() and computeEnergy() functions to eliminate
     successive calls to calculate separate TDP vs. runtime energy
   - Remove Processor component (now unnecessary) and introduce a more abstract
     System component
 3) Standardizing McPAT output across all components
   - Use a single, common data structure for storing and printing McPAT output
   - Recursively call print functions through component hierarchy
 4) For caches, allow splitting data array and tag array reads and writes for
    better accuracy
 5) Improving the usability of CACTI by printing more helpful warning and error
    messages
 6) Minor: Impose more rigorous code style for clarity (more work still to be
    done)
Overall, these changes greatly reduce the amount of replicated code, and they
improve McPAT runtime and decrease memory footprint.

1 files changed, 35 insertions, 51 deletions

diff --git a/ext/mcpat/array.h b/ext/mcpat/array.h
index 8c6124d46..6a4c0b6cb 100644
--- a/ext/mcpat/array.h
+++ b/ext/mcpat/array.h
@@ -2,6 +2,7 @@
  *                                McPAT
  *                      SOFTWARE LICENSE AGREEMENT
  *            Copyright 2012 Hewlett-Packard Development Company, L.P.
+ *            Copyright (c) 2010-2013 Advanced Micro Devices, Inc.
  *                          All Rights Reserved
  *
  * Redistribution and use in source and binary forms, with or without
@@ -25,7 +26,7 @@
  * 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.”
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  ***************************************************************************/
 
@@ -43,59 +44,42 @@
 
 using namespace std;
 
-class ArrayST :public Component{
- public:
-  ArrayST(){};
-  ArrayST(const InputParameter *configure_interface, string _name, enum Device_ty device_ty_, bool opt_local_=true, enum Core_type core_ty_=Inorder,  bool _is_default=true);
-
-  InputParameter l_ip;
-  string         name;
-  enum Device_ty device_ty;
-  bool opt_local;
-  enum Core_type core_ty;
-  bool           is_default;
-  uca_org_t      local_result;
+class ArrayST : public McPATComponent {
+public:
+    static double area_efficiency_threshold;
 
-  statsDef       tdp_stats;
-  statsDef       rtp_stats;
-  statsDef       stats_t;
-  powerDef       power_t;
+    // These are used for the CACTI interface.
+    static int ed;
+    static int delay_wt;
+    static int cycle_time_wt;
+    static int area_wt;
+    static int dynamic_power_wt;
+    static int leakage_power_wt;
+    static int delay_dev;
+    static int cycle_time_dev;
+    static int area_dev;
+    static int dynamic_power_dev;
+    static int leakage_power_dev;
+    static int cycle_time_dev_threshold;
 
-  virtual void optimize_array();
-  virtual void compute_base_power();
-  virtual ~ArrayST();
+    InputParameter l_ip;
+    enum Device_ty device_ty;
+    bool opt_local;
+    enum Core_type core_ty;
+    bool is_default;
+    uca_org_t local_result;
+    statsDef stats_t;
 
-  void leakage_feedback(double temperature);
-};
+    ArrayST(XMLNode* _xml_data, const InputParameter *configure_interface,
+            string _name, enum Device_ty device_ty_, double _clockRate = 0.0f,
+            bool opt_local_ = true,
+            enum Core_type core_ty_ = Inorder, bool _is_default = true);
+    void computeArea();
+    void computeEnergy();
+    void compute_base_power();
+    ~ArrayST();
 
-class InstCache :public Component{
-public:
-  ArrayST* caches;
-  ArrayST* missb;
-  ArrayST* ifb;
-  ArrayST* prefetchb;
-  powerDef power_t;//temp value holder for both (max) power and runtime power
-  InstCache(){caches=0;missb=0;ifb=0;prefetchb=0;};
-  ~InstCache(){
-          if (caches)    {//caches->local_result.cleanup();
-                                          delete caches; caches=0;}
-          if (missb)     {//missb->local_result.cleanup();
-                                          delete missb; missb=0;}
-          if (ifb)       {//ifb->local_result.cleanup();
-                                          delete ifb; ifb=0;}
-          if (prefetchb) {//prefetchb->local_result.cleanup();
-                                          delete prefetchb; prefetchb=0;}
-   };
-};
-
-class DataCache :public InstCache{
-public:
-  ArrayST* wbb;
-  DataCache(){wbb=0;};
-  ~DataCache(){
-          if (wbb) {//wbb->local_result.cleanup();
-                                delete wbb; wbb=0;}
-   };
+    void leakage_feedback(double temperature);
 };
 
-#endif /* TLB_H_ */
+#endif /* ARRAY_H_ */