Revert to 74c1e6513bd0 (sim: Thermal support for Linux)
authorAndreas Sandberg <andreas.sandberg@arm.com>
Thu, 7 Apr 2016 09:42:07 +0000 (10:42 +0100)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Thu, 7 Apr 2016 09:42:07 +0000 (10:42 +0100)
24 files changed:
src/arch/alpha/tlb.cc
src/arch/mips/tlb.cc
src/arch/power/tlb.cc
src/cpu/testers/memtest/memtest.cc
src/dev/arm/flash_device.cc
src/dev/arm/hdlcd.cc
src/dev/arm/ufs_device.cc
src/mem/probes/stack_dist.cc
src/mem/ruby/network/garnet/BaseGarnetNetwork.cc
src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc
src/mem/ruby/network/simple/SimpleNetwork.cc
src/mem/ruby/network/simple/Switch.cc
src/mem/ruby/slicc_interface/AbstractController.cc
src/mem/ruby/structures/CacheMemory.cc
src/mem/ruby/structures/Prefetcher.cc
src/mem/ruby/system/RubySystem.hh
src/mem/ruby/system/Sequencer.cc
src/mem/snoop_filter.cc
src/sim/ClockedObject.py
src/sim/SConscript
src/sim/clock_domain.cc
src/sim/clocked_object.cc [deleted file]
src/sim/clocked_object.hh
src/sim/voltage_domain.cc

index fcd2b518b71605a6f6c33f67c7f8319782e18150..3360b34c563ba17f4d233023b4da1c9b26e58542 100644 (file)
@@ -76,8 +76,6 @@ TLB::~TLB()
 void
 TLB::regStats()
 {
-    BaseTLB::regStats();
-
     fetch_hits
         .name(name() + ".fetch_hits")
         .desc("ITB hits");
index 340c830216603cc406ec6c525004600786ce0587..d2aa5ad709fb9900c002ea3cac037dd6cd2c6958 100644 (file)
@@ -226,8 +226,6 @@ TLB::unserialize(CheckpointIn &cp)
 void
 TLB::regStats()
 {
-    BaseTLB::regStats();
-
     read_hits
         .name(name() + ".read_hits")
         .desc("DTB read hits")
index 90a341d857a9ce06fb1b93cd360003c898be6982..edfb4f453dd4983c5ef94d2c87f1b0a3c00d4ab2 100644 (file)
@@ -223,8 +223,6 @@ TLB::unserialize(CheckpointIn &cp)
 void
 TLB::regStats()
 {
-    BaseTLB::regStats();
-
     read_hits
         .name(name() + ".read_hits")
         .desc("DTB read hits")
index 2235320883e35600d56dfab71ec8c364ba9cb77b..b0dde6d27ada7f624e2474acb8efc2610b0008fa 100644 (file)
@@ -197,8 +197,6 @@ MemTest::completeRequest(PacketPtr pkt, bool functional)
 void
 MemTest::regStats()
 {
-    MemObject::regStats();
-
     using namespace Stats;
 
     numReadsStat
index 297354b650a5136cdbf7dbe3cd77011820b0575f..60c910626ece05a2897913c7c8d4b912e7c8ebc8 100644 (file)
@@ -472,8 +472,6 @@ FlashDevice::getUnknownPages(uint32_t index)
 void
 FlashDevice::regStats()
 {
-    AbstractNVM::regStats();
-
     using namespace Stats;
 
     std::string fd_name = name() + ".FlashDevice";
index 0f63f23fc0aae6df3326e79a4b43af6ee37e694b..b04de21bfcb648286a4a73fec5b4b688ea128e3c 100644 (file)
@@ -97,8 +97,6 @@ HDLcd::~HDLcd()
 void
 HDLcd::regStats()
 {
-    AmbaDmaDevice::regStats();
-
     using namespace Stats;
 
     stats.underruns
index fe05b3279a72c674edcc8da352262c71775cdad9..07d50903b1987a1a3ede3c96160f41cdb4587659 100644 (file)
@@ -774,8 +774,6 @@ UFSHostDeviceParams::create()
 void
 UFSHostDevice::regStats()
 {
-    DmaDevice::regStats();
-
     using namespace Stats;
 
     std::string UFSHost_name = name() + ".UFSDiskHost";
index b12c81e2ce4051eeb84c1f36dede98e05c95d79f..a447f49e5ed8f8c519849a9b12e9fcfc19607f3b 100644 (file)
@@ -57,8 +57,6 @@ StackDistProbe::StackDistProbe(StackDistProbeParams *p)
 void
 StackDistProbe::regStats()
 {
-    BaseMemProbe::regStats();
-
     const StackDistProbeParams *p(
         dynamic_cast<const StackDistProbeParams *>(params()));
     assert(p);
index 2bd2acb9ff27c3e4c6e8fda80718a48dde7af4a8..1213073e949b48786043a2b244a7821be0fb8bb2 100644 (file)
@@ -69,8 +69,6 @@ BaseGarnetNetwork::init()
 void
 BaseGarnetNetwork::regStats()
 {
-    Network::regStats();
-
     m_flits_received
         .init(m_virtual_networks)
         .name(name() + ".flits_received")
index dab9b7dda4ac569c8255c6e70692b18f068962e0..97bc1abdd3d9e987158383714710fbed3c4f85fe 100644 (file)
@@ -158,8 +158,6 @@ Router_d::update_sw_winner(int inport, flit_d *t_flit)
 void
 Router_d::regStats()
 {
-    BasicRouter::regStats();
-
     m_buffer_reads
         .name(name() + ".buffer_reads")
         .flags(Stats::nozero)
index 2fc7b6440b0c40e2d47d4c54f06434bed7b758be..25d0b6f4bd16f84162556a8b7a27ff23d5c9245e 100644 (file)
@@ -132,8 +132,6 @@ SimpleNetwork::makeInternalLink(SwitchID src, SwitchID dest, BasicLink* link,
 void
 SimpleNetwork::regStats()
 {
-    Network::regStats();
-
     for (MessageSizeType type = MessageSizeType_FIRST;
          type < MessageSizeType_NUM; ++type) {
         m_msg_counts[(unsigned int) type]
index 78f5b609c5c8a26a7c0cdce609d4b7b556daa150..747884f165024146fa86ca60786ca2aa08304186 100644 (file)
@@ -112,8 +112,6 @@ Switch::getThrottle(LinkID link_number) const
 void
 Switch::regStats()
 {
-    BasicRouter::regStats();
-
     for (int link = 0; link < m_throttles.size(); link++) {
         m_throttles[link]->regStats(name());
     }
index be48628e98eea3a0e4107d58aa3b3763287a405e..2a53e53beb96efc51c66c8b1fce51038ba90f03d 100644 (file)
@@ -76,8 +76,6 @@ AbstractController::resetStats()
 void
 AbstractController::regStats()
 {
-    MemObject::regStats();
-
     m_fully_busy_cycles
         .name(name() + ".fully_busy_cycles")
         .desc("cycles for which number of transistions == max transitions")
index 36d109769914ebaaf2b0c6b6eec6cfa1e922393f..f7c1961195886ed68d1ae91e7ddf479ef242a404 100644 (file)
@@ -488,8 +488,6 @@ CacheMemory::isLocked(Addr address, int context)
 void
 CacheMemory::regStats()
 {
-    SimObject::regStats();
-
     m_demand_hits
         .name(name() + ".demand_hits")
         .desc("Number of cache demand hits")
index eef51dcf7dbd73af75524c15014323a2edca58f5..ce6d36c048f26295c4f7fae5fbda8114a80f6811 100644 (file)
@@ -86,8 +86,6 @@ Prefetcher::~Prefetcher()
 void
 Prefetcher::regStats()
 {
-    SimObject::regStats();
-
     numMissObserved
         .name(name() + ".miss_observed")
         .desc("number of misses observed")
index 8ebd3494ab63fe731e7844ca811e6e7fd17bbaf7..62330e19da8fb8848337ba5b2cb6d4b919a93386 100644 (file)
@@ -89,10 +89,7 @@ class RubySystem : public ClockedObject
         return m_profiler;
     }
 
-    void regStats() override {
-        ClockedObject::regStats();
-        m_profiler->regStats(name());
-    }
+    void regStats() override { m_profiler->regStats(name()); }
     void collateStats() { m_profiler->collateStats(); }
     void resetStats() override;
 
index fbaad84071c7aa5f9b8c30ddc86f393478be7c40..dedade3cf8f4a9e5a6a64f71a3338895412d6032 100644 (file)
@@ -688,8 +688,6 @@ Sequencer::evictionCallback(Addr address)
 void
 Sequencer::regStats()
 {
-    RubyPort::regStats();
-
     m_store_waiting_on_load
         .name(name() + ".store_waiting_on_load")
         .desc("Number of times a store aliased with a pending load")
index 9e8f8afb87dab5484ec4c419ac54aa510d4a0c99..9d02ed249023d4ffb7c2fd8e2316ac835e9e0424 100755 (executable)
@@ -351,8 +351,6 @@ SnoopFilter::updateResponse(const Packet* cpkt, const SlavePort& slave_port)
 void
 SnoopFilter::regStats()
 {
-    SimObject::regStats();
-
     totRequests
         .name(name() + ".tot_requests")
         .desc("Total number of requests made to the snoop filter.");
index b933ea07a5700cd7170e31d15a2d29101ff2b1fe..2562f1f01ff9f53e00731e21c72534d247caf331 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (c) 2012, 2015 ARM Limited
+# Copyright (c) 2012 ARM Limited
 # All rights reserved.
 #
 # The license below extends only to copyright in the software and shall
@@ -39,24 +39,6 @@ from m5.SimObject import SimObject
 from m5.params import *
 from m5.proxy import *
 
-# Enumerate set of allowed power states that can be used by a clocked object.
-# The list is kept generic to express a base minimal set.
-# State definition :-
-#   Undefined: Invalid state, no power state derived information is available.
-#   On: The logic block is actively running and consuming dynamic and leakage
-#       energy depending on the amount of processing required.
-#   Clk_gated: The clock circuity within the block is gated to save dynamic
-#              energy, the power supply to the block is still on and leakage
-#              energy is being consumed by the block.
-#   Sram_retention: The SRAMs within the logic blocks are pulled into retention
-#                   state to reduce leakage energy further.
-#   Off: The logic block is power gated and is not consuming any energy.
-class PwrState(Enum): vals = ['UNDEFINED',
-                             'ON',
-                             'CLK_GATED',
-                             'SRAM_RETENTION',
-                             'OFF']
-
 class ClockedObject(SimObject):
     type = 'ClockedObject'
     abstract = True
@@ -65,12 +47,3 @@ class ClockedObject(SimObject):
     # The clock domain this clocked object belongs to, inheriting the
     # parent's clock domain by default
     clk_domain = Param.ClockDomain(Parent.clk_domain, "Clock domain")
-
-    # Provide initial power state, should ideally get redefined in startup
-    # routine
-    default_p_state = Param.PwrState("UNDEFINED", "Default Power State")
-
-    p_state_clk_gate_min = Param.Latency('1ns', "Min value of the distribution")
-    p_state_clk_gate_max = Param.Latency('1s', "Max value of the distribution")
-    p_state_clk_gate_bins = Param.Unsigned('20',
-        "# bins in clk gated distribution")
index e40c43f0c679ea59096742535e5ac3c1910bd83d..3238301ed01fe1dccf0a2ec9201cf4fdd37ca106 100644 (file)
@@ -69,7 +69,6 @@ Source('voltage_domain.cc')
 Source('linear_solver.cc')
 Source('system.cc')
 Source('dvfs_handler.cc')
-Source('clocked_object.cc')
 
 if env['TARGET_ISA'] != 'null':
     SimObject('InstTracer.py')
index 9865c4d116df110d2d650c3d026b6e7f10d079f0..1ccee7f1da5f774615e92f4b81c9c749131334e6 100644 (file)
@@ -56,8 +56,6 @@
 void
 ClockDomain::regStats()
 {
-    SimObject::regStats();
-
     using namespace Stats;
 
     // Expose the current clock period as a stat for observability in
diff --git a/src/sim/clocked_object.cc b/src/sim/clocked_object.cc
deleted file mode 100644 (file)
index 9a682a4..0000000
+++ /dev/null
@@ -1,177 +0,0 @@
-/*
- * 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: Akash Bagdia
- *          David Guillen Fandos
- */
-
-#include "sim/clocked_object.hh"
-
-#include "base/misc.hh"
-
-void
-ClockedObject::serialize(CheckpointOut &cp) const
-{
-    unsigned int currPwrState = (unsigned int)_currPwrState;
-
-    SERIALIZE_SCALAR(currPwrState);
-    SERIALIZE_SCALAR(prvEvalTick);
-}
-
-void
-ClockedObject::unserialize(CheckpointIn &cp)
-{
-    unsigned int currPwrState;
-
-    UNSERIALIZE_SCALAR(currPwrState);
-    UNSERIALIZE_SCALAR(prvEvalTick);
-
-    _currPwrState = Enums::PwrState(currPwrState);
-}
-
-void
-ClockedObject::pwrState(Enums::PwrState p)
-{
-    // Function should ideally be called only when there is a state change
-    if (_currPwrState == p) {
-        warn("ClockedObject: Already in the requested power state, request "\
-             "ignored");
-        return;
-    }
-
-    // No need to compute stats if in the same tick, update state
-    // though. This can happen in cases like a) during start of the
-    // simulation multiple state changes happens in init/startup phase,
-    // b) one takes a decision to migrate state but decides to reverts
-    // back to the original state in the same tick if other conditions
-    // are not met elsewhere. Any state change related stats would have
-    // been recorded on previous call to the pwrState() function.
-    if (prvEvalTick == curTick()) {
-        warn("ClockedObject: More than one power state change request "\
-             "encountered within the same simulation tick");
-        _currPwrState = p;
-        return;
-    }
-
-    // Record stats for previous state.
-    computeStats();
-
-    _currPwrState = p;
-
-    numPwrStateTransitions++;
-}
-
-void
-ClockedObject::computeStats()
-{
-    // Calculate time elapsed from last (valid) state change
-    Tick elapsed_time = curTick() - prvEvalTick;
-
-    pwrStateResidencyTicks[_currPwrState] += elapsed_time;
-
-    // Time spent in CLK_GATED state, this might change depending on
-    // transition to other low power states in respective simulation
-    // objects.
-    if (_currPwrState == Enums::PwrState::CLK_GATED) {
-        pwrStateClkGateDist.sample(elapsed_time);
-    }
-
-    prvEvalTick = curTick();
-}
-
-std::vector<double>
-ClockedObject::pwrStateWeights() const
-{
-    // Get residency stats
-    std::vector<double> ret;
-    Stats::VCounter residencies;
-    pwrStateResidencyTicks.value(residencies);
-
-    // Account for current state too!
-    Tick elapsed_time = curTick() - prvEvalTick;
-    residencies[_currPwrState] += elapsed_time;
-
-    ret.resize(Enums::PwrState::Num_PwrState);
-    for (unsigned i = 0; i < Enums::PwrState::Num_PwrState; i++)
-        ret[i] = residencies[i] /
-                     (pwrStateResidencyTicks.total() + elapsed_time);
-
-    return ret;
-}
-
-void
-ClockedObject::regStats()
-{
-    SimObject::regStats();
-
-    using namespace Stats;
-
-    numPwrStateTransitions
-        .name(params()->name + ".numPwrStateTransitions")
-        .desc("Number of power state transitions")
-        ;
-
-    // Each sample is time in ticks
-    unsigned num_bins = std::max(params()->p_state_clk_gate_bins, 10U);
-    pwrStateClkGateDist
-        .init(params()->p_state_clk_gate_min, params()->p_state_clk_gate_max,
-          (params()->p_state_clk_gate_max / num_bins))
-        .name(params()->name + ".pwrStateClkGateDist")
-        .desc("Distribution of time spent in the clock gated state")
-        .flags(pdf)
-        ;
-
-    pwrStateResidencyTicks
-        .init(Enums::PwrState::Num_PwrState)
-        .name(params()->name + ".pwrStateResidencyTicks")
-        .desc("Cumulative time (in ticks) in various power states")
-        ;
-    for (int i = 0; i < Enums::PwrState::Num_PwrState; i++) {
-        pwrStateResidencyTicks.subname(i, Enums::PwrStateStrings[i]);
-    }
-
-    numPwrStateTransitions = 0;
-
-    /**
-     * For every stats dump, the power state residency and other distribution
-     * stats should be computed just before the dump to ensure correct stats
-     * value being reported for current dump window. It avoids things like
-     * having any unreported time spent in a power state to be forwarded to the
-     * next dump window which might have rather unpleasant effects (like
-     * perturbing the distribution stats).
-     */
-    registerDumpCallback(new ClockedObjectDumpCallback(this));
-}
index 1ba5ca6176cfa92f604932d2b625f12fd871e368..b9a2481ec3ae6a3ae1eab6129bfa9f6baa31ef67 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012-2013, 2015 ARM Limited
+ * Copyright (c) 2012-2013 ARM Limited
  * Copyright (c) 2013 Cornell University
  * All rights reserved
  *
@@ -37,8 +37,6 @@
  *
  * Authors: Andreas Hansson
  *          Christopher Torng
- *          Akash Bagdia
- *          David Guillen Fandos
  */
 
 /**
 #ifndef __SIM_CLOCKED_OBJECT_HH__
 #define __SIM_CLOCKED_OBJECT_HH__
 
-#include "base/callback.hh"
 #include "base/intmath.hh"
 #include "base/misc.hh"
-#include "enums/PwrState.hh"
 #include "params/ClockedObject.hh"
 #include "sim/core.hh"
 #include "sim/clock_domain.hh"
@@ -237,58 +233,7 @@ class ClockedObject
 {
   public:
     ClockedObject(const ClockedObjectParams *p)
-        : SimObject(p), Clocked(*p->clk_domain),
-        _currPwrState(p->default_p_state),
-        prvEvalTick(0)
-    { }
-
-    /** Parameters of ClockedObject */
-    typedef ClockedObjectParams Params;
-    const Params* params() const
-    { return reinterpret_cast<const Params*>(_params); }
-
-    void serialize(CheckpointOut &cp) const override;
-    void unserialize(CheckpointIn &cp) override;
-
-    inline Enums::PwrState pwrState() const
-    { return _currPwrState; }
-
-    inline std::string pwrStateName() const
-    { return Enums::PwrStateStrings[_currPwrState]; }
-
-    /** Returns the percentage residency for each power state */
-    std::vector<double> pwrStateWeights() const;
-
-    /**
-     * Record stats values like state residency by computing the time
-     * difference from previous update. Also, updates the previous
-     * evaluation tick once all stats are recorded.
-     * Usually called on power state change and stats dump callback.
-     */
-    void computeStats();
-
-    void pwrState(Enums::PwrState);
-    void regStats();
-
-  protected:
-
-    /** To keep track of the current power state */
-    Enums::PwrState _currPwrState;
-
-    Tick prvEvalTick;
-
-    Stats::Scalar numPwrStateTransitions;
-    Stats::Distribution pwrStateClkGateDist;
-    Stats::Vector pwrStateResidencyTicks;
-
-};
-
-class ClockedObjectDumpCallback : public Callback
-{
-    ClockedObject *co;
-  public:
-    ClockedObjectDumpCallback(ClockedObject *co_t) : co(co_t) {}
-    virtual void process() { co->computeStats(); };
+        : SimObject(p), Clocked(*p->clk_domain) { }
 };
 
 #endif //__SIM_CLOCKED_OBJECT_HH__
index 61715dfbcc9f4297e6a6a27c31b6ad4e4a1d98f7..b82efda33012d3d01854ef624364de639c0ae011 100644 (file)
@@ -128,8 +128,6 @@ VoltageDomain::startup() {
 void
 VoltageDomain::regStats()
 {
-    SimObject::regStats();
-
     currentVoltage
         .method(this, &VoltageDomain::voltage)
         .name(params()->name + ".voltage")