> On Aug. 14, 2015, 10:05 p.m., Joel Hestness wrote:
> > Looks good to me.
> > 
> > Though I haven't often found use for these aggregated stats, do we have an 
> > aim or plan for adding them back? Could we use the Python stats 
> > post-processing capabilities?
> 
> Nilay Vaish wrote:
>     I will be adding some python pre-processing script but it is at the end 
> of the priority list.  First concern is to improve ruby's speed of simulation.
> 
> Joel Hestness wrote:
>     Cool. Also good to have the idea noted. Thanks!

We cannot get rid of these stats.  They are essential for debugging.  I don't 
understand why these stats are significantly impacting simulation, but even if 
they are, we need to keep them.  What is the good of Ruby simulation if you 
don't understand what is going in the protocol.


- Brad


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3013/#review6968
-----------------------------------------------------------


On Aug. 9, 2015, 6:26 p.m., Nilay Vaish wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3013/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2015, 6:26 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 11050:8f8fc7ddc278
> ---------------------------
> ruby: convert global statistical variables to local ones.
> Several statistical quantities collected by ruby where recorded / published
> using global variables.  Since we want to have multiple ruby systems working
> together, this is a hindrance.  Hence, we are convert such global variables to
> local ones.
> 
> 
> Diffs
> -----
> 
>   src/mem/ruby/slicc_interface/AbstractController.hh 863d314f6356 
>   src/mem/ruby/slicc_interface/AbstractController.cc 863d314f6356 
>   src/mem/slicc/symbols/StateMachine.py 863d314f6356 
> 
> Diff: http://reviews.gem5.org/r/3013/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nilay Vaish
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to