diff options
author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2019-01-30 14:46:22 +0100 |
---|---|---|
committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2019-04-19 16:34:00 +0000 |
commit | 9f32d74db86c8977ab16f2671830f6cdfe3c068b (patch) | |
tree | b7dd3a867cbe1f7fc29fa68d8173160632523fdf /src/mem/cache/write_queue_entry.hh | |
parent | f699e91fe53870b6a7f0e9fe61cc92fc44b1033b (diff) | |
download | gem5-9f32d74db86c8977ab16f2671830f6cdfe3c068b.tar.xz |
mem-cache: Move Target to QueueEntry
WriteQueueEntry's target has 100% functionality overlap with MSHR's,
therefore make it base to MSHR::Target.
Change-Id: I48614e78179d708bd91bbe75a752e5a05146e8eb
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17534
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Diffstat (limited to 'src/mem/cache/write_queue_entry.hh')
-rw-r--r-- | src/mem/cache/write_queue_entry.hh | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/src/mem/cache/write_queue_entry.hh b/src/mem/cache/write_queue_entry.hh index f4ccb1a8c..59714d948 100644 --- a/src/mem/cache/write_queue_entry.hh +++ b/src/mem/cache/write_queue_entry.hh @@ -76,21 +76,6 @@ class WriteQueueEntry : public QueueEntry, public Printable friend class WriteQueue; public: - - class Target { - public: - - const Tick recvTime; //!< Time when request was received (for stats) - const Tick readyTime; //!< Time when request is ready to be serviced - const Counter order; //!< Global order (for memory consistency mgmt) - const PacketPtr pkt; //!< Pending request packet. - - Target(PacketPtr _pkt, Tick _readyTime, Counter _order) - : recvTime(curTick()), readyTime(_readyTime), order(_order), - pkt(_pkt) - {} - }; - class TargetList : public std::list<Target> { public: @@ -165,7 +150,7 @@ class WriteQueueEntry : public QueueEntry, public Printable * Returns a reference to the first target. * @return A pointer to the first target. */ - Target *getTarget() + Target *getTarget() override { assert(hasTargets()); return &targets.front(); |