> On Nov. 6, 2015, 7:50 a.m., Nilay Vaish wrote: > > configs/example/etrace_replay.py, lines 107-109 > > <http://reviews.gem5.org/r/3031/diff/2/?file=51462#file51462line107> > > > > I think these are the only lines that extra > > over what se.py has. Why not just move these to se.py?
There were a few code blocks where cpu or workload was being configured for se and fs. That code did not apply for a trace-based cpu. So, in the previous version of this patch I had inserted a check a all these points - if cpu type is not trace then do these code blocks else do trace cpu counterpart or do nothing. > On Nov. 6, 2015, 7:50 a.m., Nilay Vaish wrote: > > src/cpu/trace/trace_cpu.hh, lines 183-190 > > <http://reviews.gem5.org/r/3031/diff/2/?file=51466#file51466line183> > > > > Merge with patch on trace cpu. The plan is to merge all the elastic traces patches and commit in a single changeset. - Radhika ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3031/#review7520 ----------------------------------------------------------- On Nov. 5, 2015, 9:13 p.m., Curtis Dunham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3031/ > ----------------------------------------------------------- > > (Updated Nov. 5, 2015, 9:13 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > This patch adds changes to the configuration scripts to support elastic > tracing and replay. > > The patch adds a command line option to enable elastic tracing in SE mode > and FS mode. When enabled the Elastic Trace cpu probe is attached to O3CPU > and a few O3 CPU parameters are tuned. The Elastic Trace probe writes out > both instruction fetch and data dependency traces. The patch also enables > configuring the TraceCPU to replay traces using the SE and FS script. > > The replay run is designed to resume from checkpoint using atomic cpu to > restore state keeping it consistent with FS run flow. It then switches to > TraceCPU to replay the input traces. > > > Diffs > ----- > > src/cpu/trace/TraceCPU.py PRE-CREATION > src/cpu/trace/trace_cpu.hh PRE-CREATION > src/cpu/trace/trace_cpu.cc PRE-CREATION > configs/example/fs.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/example/se.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/common/CpuConfig.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/common/MemConfig.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/common/Options.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/common/Simulation.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/dram/sweep.py 854e61d5390e3bee8864848465fab311e95b9759 > configs/example/etrace_replay.py PRE-CREATION > configs/common/CacheConfig.py 854e61d5390e3bee8864848465fab311e95b9759 > > Diff: http://reviews.gem5.org/r/3031/diff/ > > > Testing > ------- > > > Thanks, > > Curtis Dunham > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev