----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/635/#review1102 -----------------------------------------------------------
src/mem/ruby/network/simple/PerfectSwitch.cc <http://reviews.m5sim.org/r/635/#comment1467> If DPRINTF() prints the time, is this piece of code required? src/mem/ruby/system/Sequencer.cc <http://reviews.m5sim.org/r/635/#comment1468> Again, do we need curTick() here? src/mem/ruby/system/Sequencer.cc <http://reviews.m5sim.org/r/635/#comment1469> Same comment as before. - Nilay On 2011-04-05 11:19:26, Korey Sewell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/635/ > ----------------------------------------------------------- > > (Updated 2011-04-05 11:19:26) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > ruby: dbg: use system ticks instead of cycles > It's easier to debug simulations (find the exact point to rerun a trace) when > the output is in > the system ticks instead of the Ruby cycle time > > > Diffs > ----- > > src/mem/ruby/buffers/MessageBuffer.cc 54a65799e4c1 > src/mem/ruby/network/simple/PerfectSwitch.cc 54a65799e4c1 > src/mem/ruby/system/Sequencer.cc 54a65799e4c1 > src/mem/slicc/symbols/StateMachine.py 54a65799e4c1 > src/mem/slicc/symbols/Type.py 54a65799e4c1 > > Diff: http://reviews.m5sim.org/r/635/diff > > > Testing > ------- > > > Thanks, > > Korey > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev