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/commit_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/commit_impl.hh')
| -rw-r--r-- | src/cpu/o3/commit_impl.hh | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/cpu/o3/commit_impl.hh b/src/cpu/o3/commit_impl.hh index 776968f8c..c7b255123 100644 --- a/src/cpu/o3/commit_impl.hh +++ b/src/cpu/o3/commit_impl.hh @@ -60,6 +60,7 @@ #include "debug/CommitRate.hh" #include "debug/Drain.hh" #include "debug/ExecFaulting.hh" +#include "debug/O3PipeView.hh" #include "params/DerivO3CPU.hh" #include "sim/faults.hh" #include "sim/full_system.hh" @@ -1267,7 +1268,9 @@ DefaultCommit<Impl>::commitHead(DynInstPtr &head_inst, unsigned inst_num) rob->retireHead(tid); #if TRACING_ON - head_inst->commitTick = curTick() - head_inst->fetchTick; + if (DTRACE(O3PipeView)) { + head_inst->commitTick = curTick() - head_inst->fetchTick; + } #endif // If this was a store, record it for this cycle. |