diff options
| author | Tony Gutierrez <anthony.gutierrez@amd.com> | 2018-12-10 19:20:06 -0500 |
|---|---|---|
| committer | Anthony Gutierrez <anthony.gutierrez@amd.com> | 2018-12-11 00:40:00 +0000 |
| commit | 866b200c202dded37fdd857a1a42ec149bd109c9 (patch) | |
| tree | b0771c077b40c64ce34a90c2174a750168a1ba75 | |
| parent | 9c1b329bc62ebf693cc4b23ddc7bd2225ef49e6a (diff) | |
| download | gem5-866b200c202dded37fdd857a1a42ec149bd109c9.tar.xz | |
cpu-o3: Fix bug in LSQUnit(uint32_t, uint32_t) ctor
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>
| -rw-r--r-- | src/cpu/o3/lsq_unit_impl.hh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cpu/o3/lsq_unit_impl.hh b/src/cpu/o3/lsq_unit_impl.hh index 5438c4d07..13b148768 100644 --- a/src/cpu/o3/lsq_unit_impl.hh +++ b/src/cpu/o3/lsq_unit_impl.hh @@ -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) |