cpu-o3: Fix bug in LSQUnit(uint32_t, uint32_t) ctor
authorTony Gutierrez <anthony.gutierrez@amd.com>
Tue, 11 Dec 2018 00:20:06 +0000 (19:20 -0500)
committerAnthony Gutierrez <anthony.gutierrez@amd.com>
Tue, 11 Dec 2018 00:40:00 +0000 (00:40 +0000)
Change 9af1214 added a new ctor to the LSQUnit, however
there is a typo/bug because it sizes the SQEntries
member variable to lqEntries + 1, as opposed to
sqEntries + 1. This change corrects the issue by
using sqEntries.

Change-Id: I19dfaa5c0e335bd7b84343a92034147d7c5d914e
Reviewed-on: https://gem5-review.googlesource.com/c/15015
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/cpu/o3/lsq_unit_impl.hh

index 5438c4d07df68a5c01d6d3fa0c74232fec29c7a7..13b1487684bc7112b5b24888928c60aab7587d19 100644 (file)
@@ -144,7 +144,7 @@ LSQUnit<Impl>::completeDataAccess(PacketPtr pkt)
 template <class Impl>
 LSQUnit<Impl>::LSQUnit(uint32_t lqEntries, uint32_t sqEntries)
     : lsqID(-1), storeQueue(sqEntries+1), loadQueue(lqEntries+1),
-      LQEntries(lqEntries+1), SQEntries(lqEntries+1),
+      LQEntries(lqEntries+1), SQEntries(sqEntries+1),
       loads(0), stores(0), storesToWB(0), cacheBlockMask(0), stalled(false),
       isStoreBlocked(false), storeInFlight(false), hasPendingPkt(false),
       pendingPkt(nullptr)