diff options
author | Gabe Black <gabeblack@google.com> | 2019-10-13 21:43:36 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2019-10-15 01:40:19 +0000 |
commit | 0f1efb84a33d737175ed66e977239fbebc06ca82 (patch) | |
tree | 8c034657c924b5b18d0b02d5ea9a3de46b83d137 /src/cpu/simple/base.cc | |
parent | 3b58400b27629779a386ffb21e62473044d88680 (diff) | |
download | gem5-0f1efb84a33d737175ed66e977239fbebc06ca82.tar.xz |
sim,cpu: Get rid of the unused instEventQueue.
This queue was set up to allow triggering events based on the total
number of instructions executed at the system level, and was added in
a change which added a number of things to support McPAT. No code
checked into gem5 actually schedules an event on that queue, and no
code in McPAT (which seems to have gone dormant) either downloadable
from github or found in ext modify gem5 in a way that makes it use
the instEventQueue.
Also, the KVM CPU does not interact with the instEventQueue correctly.
While it does check the per-thread instruction event queue when
deciding how long to run, it does not check the instEventQueue. It will
poke it to run events when it stops for other reasons, but it may (and
likely will) have run beyond the point where it was supposed to stop.
Since this queue doesn't seem to actually be used for anything, isn't
being used properly in all cases anyway, and adds overhead to all the
CPU models, this change eliminates it.
Change-Id: I0e126df14788c37a6d58ca9e1bb2686b70e60d88
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/21783
Maintainer: Gabe Black <gabeblack@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Tiago Mück <tiago.muck@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/cpu/simple/base.cc')
-rw-r--r-- | src/cpu/simple/base.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/cpu/simple/base.cc b/src/cpu/simple/base.cc index 816add707..b93ae0912 100644 --- a/src/cpu/simple/base.cc +++ b/src/cpu/simple/base.cc @@ -500,7 +500,6 @@ BaseSimpleCPU::preExecute() // check for instruction-count-based events comInstEventQueue[curThread]->serviceEvents(t_info.numInst); - system->instEventQueue.serviceEvents(system->totalNumInsts); // decode the instruction inst = gtoh(inst); |