sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

cpu: o3: remove stat totalCommittedInsts

Browse source 
This patch removes the stat totalCommittedInsts.  This variable was used for
recording the total number of instructions committed across all the threads
of a core.  The instructions committed by each thread are recorded invidually.
The total would now be generated by summing these individual counts.
This commit is contained in:
Nilay Vaish 2014-05-23 06:07:02 -05:00
parent 53dd4497b3
commit 1e26b7ea29
2 changed files with 6 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/cpu/o3/cpu.cc
View file

@ -509,16 +509,14 @@ FullO3CPU<Impl>::regStats()
committedInsts committedInsts
.init(numThreads) .init(numThreads)
.name(name() + ".committedInsts") .name(name() + ".committedInsts")
.desc("Number of Instructions Simulated"); .desc("Number of Instructions Simulated")
.flags(Stats::total);
committedOps committedOps
.init(numThreads) .init(numThreads)
.name(name() + ".committedOps") .name(name() + ".committedOps")
.desc("Number of Ops (including micro ops) Simulated"); .desc("Number of Ops (including micro ops) Simulated")
.flags(Stats::total);
totalCommittedInsts
.name(name() + ".committedInsts_total")
.desc("Number of Instructions Simulated");
cpi cpi
.name(name() + ".cpi") .name(name() + ".cpi")
@ -530,7 +528,7 @@ FullO3CPU<Impl>::regStats()
.name(name() + ".cpi_total") .name(name() + ".cpi_total")
.desc("CPI: Total CPI of All Threads") .desc("CPI: Total CPI of All Threads")
.precision(6); .precision(6);
totalCpi = numCycles / totalCommittedInsts; totalCpi = numCycles / sum(committedInsts);
ipc ipc
.name(name() + ".ipc") .name(name() + ".ipc")
@ -542,7 +540,7 @@ FullO3CPU<Impl>::regStats()
.name(name() + ".ipc_total") .name(name() + ".ipc_total")
.desc("IPC: Total IPC of All Threads") .desc("IPC: Total IPC of All Threads")
.precision(6); .precision(6);
totalIpc = totalCommittedInsts / numCycles; totalIpc = sum(committedInsts) / numCycles;
this->fetch.regStats(); this->fetch.regStats();
this->decode.regStats(); this->decode.regStats();
@ -1601,7 +1599,6 @@ FullO3CPU<Impl>::instDone(ThreadID tid, DynInstPtr &inst)
thread[tid]->numInst++; thread[tid]->numInst++;
thread[tid]->numInsts++; thread[tid]->numInsts++;
committedInsts[tid]++; committedInsts[tid]++;
totalCommittedInsts++;
} }
thread[tid]->numOp++; thread[tid]->numOp++;
thread[tid]->numOps++; thread[tid]->numOps++;

2
src/cpu/o3/cpu.hh
View file

@ -843,8 +843,6 @@ class FullO3CPU : public BaseO3CPU
Stats::Vector committedInsts; Stats::Vector committedInsts;
/** Stat for the number of committed ops (including micro ops) per thread. */ /** Stat for the number of committed ops (including micro ops) per thread. */
Stats::Vector committedOps; Stats::Vector committedOps;
/** Stat for the total number of committed instructions. */
Stats::Scalar totalCommittedInsts;
/** Stat for the CPI per thread. */ /** Stat for the CPI per thread. */
Stats::Formula cpi; Stats::Formula cpi;
/** Stat for the total CPI. */ /** Stat for the total CPI. */