sim,cpu: Get rid of the unused instEventQueue.
authorGabe Black <gabeblack@google.com>
Mon, 14 Oct 2019 04:43:36 +0000 (21:43 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 15 Oct 2019 01:40:19 +0000 (01:40 +0000)
commit0f1efb84a33d737175ed66e977239fbebc06ca82
tree8c034657c924b5b18d0b02d5ea9a3de46b83d137
parent3b58400b27629779a386ffb21e62473044d88680
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>
src/cpu/kvm/base.cc
src/cpu/minor/execute.cc
src/cpu/o3/cpu.cc
src/cpu/simple/base.cc
src/sim/system.cc
src/sim/system.hh