From 457d70df626a8cb0a7fa0ce63b3d3e0886a2bbda Mon Sep 17 00:00:00 2001 From: Melissa Jost Date: Thu, 19 Jan 2023 02:12:49 -0800 Subject: [PATCH] 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 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67396 Reviewed-by: Andreas Sandberg Maintainer: Andreas Sandberg Tested-by: kokoro --- src/cpu/kvm/base.cc | 5 ++--- src/cpu/kvm/base.hh | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/cpu/kvm/base.cc b/src/cpu/kvm/base.cc index b76bddc2fd..e22e1628d2 100644 --- a/src/cpu/kvm/base.cc +++ b/src/cpu/kvm/base.cc @@ -261,8 +261,6 @@ BaseKvmCPU::restartEqThread() 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 @@ BaseKvmCPU::kvmRun(Tick ticks) /* 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 2d81c7c7eb..7bbf393f9b 100644 --- a/src/cpu/kvm/base.hh +++ b/src/cpu/kvm/base.hh @@ -804,7 +804,6 @@ class BaseKvmCPU : public BaseCPU struct StatGroup : public statistics::Group { StatGroup(statistics::Group *parent); - statistics::Scalar committedInsts; statistics::Scalar numVMExits; statistics::Scalar numVMHalfEntries; statistics::Scalar numExitSignal;