Melissa Jost has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/67396?usp=email )
Change subject: cpu-kvm: Implement IPC and CPI base stats for KVM CPU
......................................................................
cpu-kvm: Implement IPC and CPI base stats for KVM CPU
Replaced committedInsts stats of KVM CPU with commitStats.numInsts
of BaseCPU. This results in IPC and CPI printing in stats.txt for
KVM simulations.
Change-Id: I02395630fc50a69adebf11f4ed39d9cefb852e1f
---
M src/cpu/kvm/base.cc
M src/cpu/kvm/base.hh
2 files changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/cpu/kvm/base.cc b/src/cpu/kvm/base.cc
index b76bddc..e22e162 100644
--- a/src/cpu/kvm/base.cc
+++ b/src/cpu/kvm/base.cc
@@ -261,8 +261,6 @@
BaseKvmCPU::StatGroup::StatGroup(statistics::Group *parent)
: statistics::Group(parent),
- ADD_STAT(committedInsts, statistics::units::Count::get(),
- "Number of instructions committed"),
ADD_STAT(numVMExits, statistics::units::Count::get(),
"total number of KVM exits"),
ADD_STAT(numVMHalfEntries, statistics::units::Count::get(),
@@ -778,7 +776,8 @@
/* Update statistics */
baseStats.numCycles += simCyclesExecuted;;
- stats.committedInsts += instsExecuted;
+ commitStats[thread->threadId()]->numInsts += instsExecuted;
+ baseStats.numInsts += instsExecuted;
ctrInsts += instsExecuted;
DPRINTF(KvmRun,
diff --git a/src/cpu/kvm/base.hh b/src/cpu/kvm/base.hh
index 6b4b88a..958b86c 100644
--- a/src/cpu/kvm/base.hh
+++ b/src/cpu/kvm/base.hh
@@ -804,7 +804,6 @@
struct StatGroup : public statistics::Group
{
StatGroup(statistics::Group *parent);
- statistics::Scalar committedInsts;
statistics::Scalar numVMExits;
statistics::Scalar numVMHalfEntries;
statistics::Scalar numExitSignal;
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/67396?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I02395630fc50a69adebf11f4ed39d9cefb852e1f
Gerrit-Change-Number: 67396
Gerrit-PatchSet: 1
Gerrit-Owner: Melissa Jost <melissakj...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
