----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3005/#review6917 -----------------------------------------------------------
I don't quite follow this patch. You say "This is to avoid calling clockEdge() sometimes.", is there another patch coming that removes some calls to clockEdge()? Or am I just missing them in this patch. It doesn't seem very helpful to me to change this function, but maybe I'm missing something. src/mem/ruby/network/garnet/flexible-pipeline/Router.cc (line 302) <http://reviews.gem5.org/r/3005/#comment5958> Misspelling: currTick->curTick - Jason Power On Aug. 9, 2015, 6:24 p.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3005/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2015, 6:24 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11032:95ddf324be85 > --------------------------- > ruby: consumer: pass on the current tick to wakeup() function > > This is to avoid calling clockEdge() sometimes. > > > Diffs > ----- > > src/mem/ruby/network/garnet/fixed-pipeline/VCallocator_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/VCallocator_d.cc 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkInterface.hh > 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkInterface.cc > 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.hh 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.cc 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/Router.hh 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/Router.cc 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/VCarbiter.hh 863d314f6356 > src/mem/ruby/network/garnet/flexible-pipeline/VCarbiter.cc 863d314f6356 > src/mem/ruby/network/simple/PerfectSwitch.hh 863d314f6356 > src/mem/ruby/network/simple/PerfectSwitch.cc 863d314f6356 > src/mem/ruby/network/simple/Throttle.hh 863d314f6356 > src/mem/ruby/network/simple/Throttle.cc 863d314f6356 > src/mem/ruby/slicc_interface/AbstractController.hh 863d314f6356 > src/mem/ruby/structures/RubyMemoryControl.hh 863d314f6356 > src/mem/ruby/structures/RubyMemoryControl.cc 863d314f6356 > src/mem/slicc/symbols/StateMachine.py 863d314f6356 > src/mem/ruby/common/Consumer.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/InputUnit_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/InputUnit_d.cc 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkInterface_d.hh > 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkInterface_d.cc > 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.cc 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/OutputUnit_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/OutputUnit_d.cc 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/SWallocator_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/SWallocator_d.cc 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/Switch_d.hh 863d314f6356 > src/mem/ruby/network/garnet/fixed-pipeline/Switch_d.cc 863d314f6356 > > Diff: http://reviews.gem5.org/r/3005/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
