changeset fbe79534d024 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=fbe79534d024
description:
        ruby: remove several unused variables in Profiler
        This patch removes per processor cycle count, histogram for filter 
stats,
        histogram for multicasts, histogram for prefetch wait, some function
        prototypes that do not have definitions.

diffstat:

 src/mem/ruby/profiler/Profiler.cc |  25 -------------------------
 src/mem/ruby/profiler/Profiler.hh |  15 ---------------
 2 files changed, 0 insertions(+), 40 deletions(-)

diffs (99 lines):

diff -r 7d235b709425 -r fbe79534d024 src/mem/ruby/profiler/Profiler.cc
--- a/src/mem/ruby/profiler/Profiler.cc Sun Jun 09 07:29:59 2013 -0500
+++ b/src/mem/ruby/profiler/Profiler.cc Sun Jun 09 07:30:00 2013 -0500
@@ -265,18 +265,6 @@
         out << endl;
     }
 
-    vector<int64_t> perProcCycleCount(m_num_of_sequencers);
-
-    for (int i = 0; i < m_num_of_sequencers; i++) {
-        perProcCycleCount[i] =
-            g_system_ptr->curCycle() - m_cycles_executed_at_start[i] + 1;
-        // The +1 allows us to avoid division by zero
-    }
-
-    out << "ruby_cycles_executed: " << perProcCycleCount << endl;
-
-    out << endl;
-
     if (!short_stats) {
         out << "Busy Controller Counts:" << endl;
         for (uint32_t i = 0; i < MachineType_NUM; i++) {
@@ -403,8 +391,6 @@
 
         printRequestProfile(out);
 
-        out << "filter_action: " << m_filter_action_histogram << endl;
-
         if (!m_all_instructions) {
             m_address_profiler_ptr->printStats(out);
         }
@@ -447,11 +433,6 @@
     m_ruby_start = g_system_ptr->curCycle();
     m_real_time_start_time = time(NULL);
 
-    m_cycles_executed_at_start.resize(m_num_of_sequencers);
-    for (int i = 0; i < m_num_of_sequencers; i++) {
-        m_cycles_executed_at_start[i] = g_system_ptr->curCycle();
-    }
-
     m_busyBankCount = 0;
 
     m_missLatencyHistograms.resize(RubyRequestType_NUM);
@@ -544,12 +525,6 @@
 }
 
 void
-Profiler::profilePFWait(Cycles waitTime)
-{
-    m_prefetchWaitHistogram.add(waitTime);
-}
-
-void
 Profiler::bankBusy()
 {
     m_busyBankCount++;
diff -r 7d235b709425 -r fbe79534d024 src/mem/ruby/profiler/Profiler.hh
--- a/src/mem/ruby/profiler/Profiler.hh Sun Jun 09 07:29:59 2013 -0500
+++ b/src/mem/ruby/profiler/Profiler.hh Sun Jun 09 07:30:00 2013 -0500
@@ -108,12 +108,6 @@
         m_average_latency_estimate.add(latency);
     }
 
-    void recordPrediction(bool wasGood, bool wasPredicted);
-
-    void startTransaction(int cpu);
-    void endTransaction(int cpu);
-    void profilePFWait(Cycles waitTime);
-
     void controllerBusy(MachineID machID);
     void bankBusy();
 
@@ -156,17 +150,10 @@
     AddressProfiler* m_address_profiler_ptr;
     AddressProfiler* m_inst_profiler_ptr;
 
-    std::vector<int64> m_instructions_executed_at_start;
-    std::vector<int64> m_cycles_executed_at_start;
-
     Cycles m_ruby_start;
     time_t m_real_time_start_time;
 
     int64_t m_busyBankCount;
-    Histogram m_multicast_retry_histogram;
-
-    Histogram m_filter_action_histogram;
-    Histogram m_tbeProfile;
 
     Histogram m_read_sharing_histogram;
     Histogram m_write_sharing_histogram;
@@ -174,8 +161,6 @@
     int64 m_cache_to_cache;
     int64 m_memory_to_cache;
 
-    Histogram m_prefetchWaitHistogram;
-
     std::vector<Histogram> m_missLatencyHistograms;
     std::vector<Histogram> m_machLatencyHistograms;
     std::vector< std::vector<Histogram> > m_missMachLatencyHistograms;
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to