diff options
author | Matt Horsnell <Matt.Horsnell@arm.com> | 2013-02-15 17:40:09 -0500 |
---|---|---|
committer | Matt Horsnell <Matt.Horsnell@arm.com> | 2013-02-15 17:40:09 -0500 |
commit | e88e7d88b9a9876ee040dad96acf3deabebe1fa7 (patch) | |
tree | 54322d578272a21cd721788f25746cfbd88d94bb /src/cpu/o3/rename_impl.hh | |
parent | 64599080697a8db49b7e28609927bb4c1ed3c05e (diff) | |
download | gem5-e88e7d88b9a9876ee040dad96acf3deabebe1fa7.tar.xz |
o3: fix tick used for renaming and issue with range selection
Fixes the tick used from rename:
- previously this gathered the tick on leaving rename which was always 1 less
than the dispatch. This conflated the decode ticks when back pressure built
in the pipeline.
- now picks up tick on entry.
Added --store_completions flag:
- will additionally display the store completion tail in the viewer.
- this highlights periods when large numbers of stores are outstanding (>16 LSQ
blocking)
Allows selection by tick range (previously this caused an infinite loop)
Diffstat (limited to 'src/cpu/o3/rename_impl.hh')
-rw-r--r-- | src/cpu/o3/rename_impl.hh | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/cpu/o3/rename_impl.hh b/src/cpu/o3/rename_impl.hh index 15a4ebc13..e27c66dcd 100644 --- a/src/cpu/o3/rename_impl.hh +++ b/src/cpu/o3/rename_impl.hh @@ -49,6 +49,7 @@ #include "cpu/o3/rename.hh" #include "debug/Activity.hh" #include "debug/Rename.hh" +#include "debug/O3PipeView.hh" #include "params/DerivO3CPU.hh" using namespace std; @@ -658,9 +659,6 @@ DefaultRename<Impl>::renameInsts(ThreadID tid) ++renamed_insts; -#if TRACING_ON - inst->renameTick = curTick() - inst->fetchTick; -#endif // Put instruction in rename queue. toIEW->insts[toIEWIndex] = inst; @@ -736,6 +734,11 @@ DefaultRename<Impl>::sortInsts() for (int i = 0; i < insts_from_decode; ++i) { DynInstPtr inst = fromDecode->insts[i]; insts[inst->threadNumber].push_back(inst); +#if TRACING_ON + if (DTRACE(O3PipeView)) { + inst->renameTick = curTick() - inst->fetchTick; + } +#endif } } |