> On Nov. 3, 2015, 2:07 p.m., Joel Hestness wrote: > > src/mem/ruby/profiler/Profiler.hh, line 121 > > <http://reviews.gem5.org/r/3192/diff/1/?file=51173#file51173line121> > > > > I don't see where this guy is used. Can you please verify that this is > > used in another patch, so we don't add dead code? > > John Alsop wrote: > The histograms added will be populated when using GPUCoalescer objects > added in patch 3189. Is that what you're asking? > > Joel Hestness wrote: > Apologies for the confusion. In the first diff, it appeared that either > the old histogram or one of the new ones was unnecessary. That is clarified > in the new diff.
No worries- yeah, the new changes were needed to make the stats work with GPUCoalescer. Did you mean to give this a ship-it or are you still reviewing the newer diff? - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3192/#review7463 ----------------------------------------------------------- On Nov. 9, 2015, 1:16 p.m., Tony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3192/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2015, 1:16 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11199:5d5bd1b0d332 > --------------------------- > ruby: split CPU and GPU latency stats > > > Diffs > ----- > > src/mem/ruby/profiler/Profiler.hh 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/profiler/Profiler.cc 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/slicc_interface/AbstractController.hh > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/ruby/slicc_interface/AbstractController.cc > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > src/mem/slicc/symbols/StateMachine.py > 4daf60db14d794e2344a6c86a93bdd8273bc5bb6 > > Diff: http://reviews.gem5.org/r/3192/diff/ > > > Testing > ------- > > > Thanks, > > Tony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
