diff options
| author | Rekai Gonzalez-Alberquilla <rekai.gonzalezalberquilla@arm.com> | 2016-09-22 10:49:10 +0100 |
|---|---|---|
| committer | Rekai Gonzalez-Alberquilla <rekai.gonzalezalberquilla@arm.com> | 2016-09-22 10:49:10 +0100 |
| commit | ad296b068cf4a0b043988be518994ac6bd7582f0 (patch) | |
| tree | 584633dffa6806a93d2c2c1a94b90096e3c50d2e /src/cpu/o3/rename_impl.hh | |
| parent | d0ffd2f9b88d6abb29ed861deec12b40bdb7419f (diff) | |
| download | gem5-ad296b068cf4a0b043988be518994ac6bd7582f0.tar.xz | |
cpu: Fix the O3 CPU Drain
The drain did not wait until stages were ready again. Therefore, as a
result of messages in the TimeBuffer being drain, the state after the
drain was not consistent and asserts fired in some places when the
draining happened after a stage got blocked, but before the notification
arrived to the previous stages.
Change-Id: Ib50b3b40b7f745b62c1eba2931dec76860824c71
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Diffstat (limited to 'src/cpu/o3/rename_impl.hh')
| -rw-r--r-- | src/cpu/o3/rename_impl.hh | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/cpu/o3/rename_impl.hh b/src/cpu/o3/rename_impl.hh index b6ab4cc3a..00179faae 100644 --- a/src/cpu/o3/rename_impl.hh +++ b/src/cpu/o3/rename_impl.hh @@ -304,7 +304,8 @@ DefaultRename<Impl>::isDrained() const if (instsInProgress[tid] != 0 || !historyBuffer[tid].empty() || !skidBuffer[tid].empty() || - !insts[tid].empty()) + !insts[tid].empty() || + (renameStatus[tid] != Idle && renameStatus[tid] != Running)) return false; } return true; |