From: Lisa Hsu <Lisa.Hsu@amd.com>
Date: Tue, 23 Feb 2010 17:33:18 +0000 (-0800)
Subject: stats: this makes some fixes to AverageStat and AverageVector.
X-Git-Tag: stable_2012_02_02~1505
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=be4cf50c5a6a5761f6474fb9f85a9c241101f3ce;p=gem5.git

stats: this makes some fixes to AverageStat and AverageVector.
Also, make Formulas work on AverageVector.  First, Stat::Average (and thus
Stats::AverageVector) was broken when coming out of a checkpoint and on resets,
this fixes that.  Formulas also didn't work with AverageVector, but added
support for that.
---

diff --git a/src/base/statistics.hh b/src/base/statistics.hh
index 0f001dccb..06a9d465d 100644
--- a/src/base/statistics.hh
+++ b/src/base/statistics.hh
@@ -492,6 +492,8 @@ class AvgStor
   private:
     /** The current count. */
     Counter current;
+    /** The tick of the last reset */
+    Tick lastReset;
     /** The total count for all tick. */
     mutable Result total;
     /** The tick that current last changed. */
@@ -505,7 +507,7 @@ class AvgStor
      * Build and initializes this stat storage.
      */
     AvgStor(Info *info)
-        : current(0), total(0), last(0)
+        : current(0), lastReset(0), total(0), last(0)
     { }
 
     /**
@@ -547,7 +549,7 @@ class AvgStor
     result() const
     {
         assert(last == curTick);
-        return (Result)(total + current) / (Result)(curTick + 1);
+        return (Result)(total + current) / (Result)(curTick - lastReset + 1);
     }
 
     /**
@@ -573,6 +575,7 @@ class AvgStor
     {
         total = 0.0;
         last = curTick;
+        lastReset = curTick;
     }
 
 };
@@ -2551,6 +2554,10 @@ class Temp
         : node(new VectorStatNode(s.info()))
     { }
 
+    Temp(const AverageVector &s)
+        : node(new VectorStatNode(s.info()))
+    { }
+
     /**
      *
      */
diff --git a/src/python/m5/simulate.py b/src/python/m5/simulate.py
index 092bd0339..291fdc7b7 100644
--- a/src/python/m5/simulate.py
+++ b/src/python/m5/simulate.py
@@ -148,6 +148,7 @@ def restoreCheckpoint(root, dir):
     print "Restoring from checkpoint"
     internal.core.unserializeAll(dir)
     need_resume.append(root)
+    stats.reset()
 
 def changeToAtomic(system):
     if not isinstance(system, (objects.Root, objects.System)):