From 19323c8bd704f9ad95c639a71e653b30a98d3595 Mon Sep 17 00:00:00 2001 From: Melissa Jost Date: Mon, 20 Mar 2023 00:01:53 -0700 Subject: [PATCH] cpu: Remove duplicated fetch stats This removes numFetchSuspends and duplicates numBranches calls so we only have the updated fetch stats outputting. Change-Id: Ia7a6830ee947f5c67386dd1e6e3db1744a7ee43c Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/69105 Reviewed-by: Bobby Bruce Maintainer: Bobby Bruce Tested-by: kokoro --- src/cpu/minor/execute.cc | 2 -- src/cpu/minor/stats.cc | 2 -- src/cpu/minor/stats.hh | 3 --- src/cpu/simple/base.cc | 2 -- src/cpu/simple/exec_context.hh | 7 ------- 5 files changed, 16 deletions(-) diff --git a/src/cpu/minor/execute.cc b/src/cpu/minor/execute.cc index a65a77e643..0d704c7135 100644 --- a/src/cpu/minor/execute.cc +++ b/src/cpu/minor/execute.cc @@ -1062,8 +1062,6 @@ Execute::commitInst(MinorDynInstPtr inst, bool early_memory_issue, DPRINTF(MinorInterrupt, "Suspending thread: %d from Execute" " inst: %s\n", thread_id, *inst); - // output both old and new stats - cpu.stats.numFetchSuspends++; cpu.fetchStats[thread_id]->numFetchSuspends++; updateBranchData(thread_id, BranchData::SuspendThread, inst, diff --git a/src/cpu/minor/stats.cc b/src/cpu/minor/stats.cc index 64d4c475e0..e9ca562c16 100644 --- a/src/cpu/minor/stats.cc +++ b/src/cpu/minor/stats.cc @@ -52,8 +52,6 @@ MinorStats::MinorStats(BaseCPU *base_cpu) ADD_STAT(numDiscardedOps, statistics::units::Count::get(), "Number of ops (including micro ops) which were discarded before " "commit"), - ADD_STAT(numFetchSuspends, statistics::units::Count::get(), - "Number of times Execute suspended instruction fetching"), ADD_STAT(quiesceCycles, statistics::units::Cycle::get(), "Total number of cycles that CPU has spent quiesced or waiting " "for an interrupt"), diff --git a/src/cpu/minor/stats.hh b/src/cpu/minor/stats.hh index 1ab81f4407..524d20f85d 100644 --- a/src/cpu/minor/stats.hh +++ b/src/cpu/minor/stats.hh @@ -68,9 +68,6 @@ struct MinorStats : public statistics::Group /** Number of ops discarded before committing */ statistics::Scalar numDiscardedOps; - /** Number of times fetch was asked to suspend by Execute */ - statistics::Scalar numFetchSuspends; - /** Number of cycles in quiescent state */ statistics::Scalar quiesceCycles; diff --git a/src/cpu/simple/base.cc b/src/cpu/simple/base.cc index 9e831a25f6..4f2665401f 100644 --- a/src/cpu/simple/base.cc +++ b/src/cpu/simple/base.cc @@ -431,8 +431,6 @@ BaseSimpleCPU::postExecute() } if (curStaticInst->isControl()) { - // output both old and new stats - ++t_info.execContextStats.numBranches; ++fetchStats[t_info.thread->threadId()]->numBranches; } diff --git a/src/cpu/simple/exec_context.hh b/src/cpu/simple/exec_context.hh index 31aa5d44c7..3b6593da9d 100644 --- a/src/cpu/simple/exec_context.hh +++ b/src/cpu/simple/exec_context.hh @@ -152,8 +152,6 @@ class SimpleExecContext : public ExecContext "ICache total stall cycles"), ADD_STAT(dcacheStallCycles, statistics::units::Cycle::get(), "DCache total stall cycles"), - ADD_STAT(numBranches, statistics::units::Count::get(), - "Number of branches fetched"), ADD_STAT(numPredictedBranches, statistics::units::Count::get(), "Number of branches predicted as taken"), ADD_STAT(numBranchMispred, statistics::units::Count::get(), @@ -204,9 +202,6 @@ class SimpleExecContext : public ExecContext numIdleCycles = idleFraction * cpu->baseStats.numCycles; numBusyCycles = notIdleFraction * cpu->baseStats.numCycles; - numBranches - .prereq(numBranches); - numPredictedBranches .prereq(numPredictedBranches); @@ -298,8 +293,6 @@ class SimpleExecContext : public ExecContext statistics::Scalar dcacheStallCycles; /// @{ - /// Total number of branches fetched - statistics::Scalar numBranches; /// Number of branches predicted as taken statistics::Scalar numPredictedBranches; /// Number of misprediced branches