cpu-o3: branchPred: add ftq end reason stats (#232)

OpenXiangShan · Jan 16, 2025 · 49749b6 · 49749b6
1 parent 28ab3ea
commit 49749b6
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/src/cpu/pred/ftb/decoupled_bpred.cc b/src/cpu/pred/ftb/decoupled_bpred.cc
@@ -566,7 +566,8 @@ DecoupledBPUWithFTB::DBPFTBStats::DBPFTBStats(statistics::Group* parent, unsigne
     ADD_STAT(controlCommitSquashOfUncond, statistics::units::Count::get(), "control squash of uncond branch at commit"),
     ADD_STAT(controlCommitSquashOfUncondDirect, statistics::units::Count::get(), "control squash of uncond direct branch at commit"),
     ADD_STAT(controlCommitSquashOfUncondIndirect, statistics::units::Count::get(), "control squash of indirect branch at commit"),
-    ADD_STAT(controlCommitSquashOfUncondReturn, statistics::units::Count::get(), "control squash of return branch at commit")
+    ADD_STAT(controlCommitSquashOfUncondReturn, statistics::units::Count::get(), "control squash of return branch at commit"),
+    ADD_STAT(ftqEndReasonDist, statistics::units::Count::get(), "distribution of ftq end reason")
 {
     predsOfEachStage.init(numStages);
     commitPredsFromEachStage.init(numStages+1);
@@ -582,6 +583,11 @@ DecoupledBPUWithFTB::DBPFTBStats::DBPFTBStats(statistics::Group* parent, unsigne
     commitTrapSquashLatencyDist.init(1,16,1);
     commitNonControlSquashLatencyDist.init(1,16,1);
     updateLatencyDist.init(1,64,2);
+    ftqEndReasonDist.init(static_cast<int>(FTQEndReason::NUM_REASONS)).flags(statistics::pdf);
+    ftqEndReasonDist.subname(static_cast<int>(FTQEndReason::NOT_END), "not_end");
+    ftqEndReasonDist.subname(static_cast<int>(FTQEndReason::TAKEN), "taken");
+    ftqEndReasonDist.subname(static_cast<int>(FTQEndReason::SIZE_LIMIT), "size_limit");
+    ftqEndReasonDist.subname(static_cast<int>(FTQEndReason::LOOP_END), "loop_end");
 }
 
 DecoupledBPUWithFTB::BpTrace::BpTrace(FetchStream &stream, const DynInstPtr &inst, bool mispred)
@@ -903,6 +909,7 @@ DecoupledBPUWithFTB::decoupledPredict(const StaticInstPtr &inst,
         if (raw_taken) {
             if (current_loop_iter >= loop_iter - 1) {
                 run_out_of_this_entry = true;
+                dbpFtbStats.ftqEndReasonDist[static_cast<int>(FTQEndReason::LOOP_END)]++;
                 if (loop_exit) {
                     taken = false;
                     lb.tryUnpin();
@@ -914,6 +921,7 @@ DecoupledBPUWithFTB::decoupledPredict(const StaticInstPtr &inst,
     } else {
         if (taken) {
             run_out_of_this_entry = true;
+            dbpFtbStats.ftqEndReasonDist[static_cast<int>(FTQEndReason::TAKEN)]++;
         }
     }
 
@@ -931,6 +939,7 @@ DecoupledBPUWithFTB::decoupledPredict(const StaticInstPtr &inst,
         inst->advancePC(*target);
         if (target->instAddr() >= end) {
             run_out_of_this_entry = true;
+            dbpFtbStats.ftqEndReasonDist[static_cast<int>(FTQEndReason::SIZE_LIMIT)]++;
         }
     }
         DPRINTF(DecoupleBP, "Predict it %staken to %#lx\n", taken ? "" : "not ",

diff --git a/src/cpu/pred/ftb/decoupled_bpred.hh b/src/cpu/pred/ftb/decoupled_bpred.hh
@@ -8,6 +8,7 @@
 #include <vector>
 
 #include "arch/generic/pcstate.hh"
+#include "base/statistics.hh"
 #include "config/the_isa.hh"
 // #include "cpu/base.hh"
 #include "cpu/o3/cpu_def.hh"
@@ -532,11 +533,19 @@ class DecoupledBPUWithFTB : public BPredUnit
         statistics::Scalar controlCommitSquashOfUncondIndirect;
         statistics::Scalar controlCommitSquashOfUncondReturn;
 
-
+        statistics::Vector ftqEndReasonDist;
 
         DBPFTBStats(statistics::Group* parent, unsigned numStages, unsigned fsqSize);
     } dbpFtbStats;
 
+    enum class FTQEndReason {
+        NOT_END,
+        TAKEN,
+        SIZE_LIMIT,
+        LOOP_END,
+        NUM_REASONS
+    };
+
   public:
     void tick();
     void ideal_tick();