-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/932/#review1834
-----------------------------------------------------------


How does this relate to the existing work item code?  It seems a little odd 
that the old code kept only per-CPU stats but the new code keeps only per 
system stats.  Are these logically independent things that just happen to use 
the same pseudo-insts?

- Steve


On 2011-12-16 13:36:29, Ali Saidi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/932/
> -----------------------------------------------------------
> 
> (Updated 2011-12-16 13:36:29)
> 
> 
> Review request for Default.
> 
> 
> Summary
> -------
> 
> WorkItems: Enables run time sampling for code marked using pseudo insts.
> 
> Enhances histogram stat to report geometric mean.
> Adds mechanism to collect run time samples for specific portions of a 
> benchmark,
> using work_begin and work_end pseudo instructions.
> 
> 
> Diffs
> -----
> 
>   src/base/statistics.hh ca98021c3f96 
>   src/base/stats/info.hh ca98021c3f96 
>   src/base/stats/text.cc ca98021c3f96 
>   src/python/m5/main.py ca98021c3f96 
>   src/sim/System.py ca98021c3f96 
>   src/sim/pseudo_inst.cc ca98021c3f96 
>   src/sim/system.hh ca98021c3f96 
>   src/sim/system.cc ca98021c3f96 
> 
> Diff: http://reviews.m5sim.org/r/932/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ali
> 
>

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

Reply via email to