-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1643/
-----------------------------------------------------------
Review request for Default.
Description
-------
Changeset 9496:d3f96871d795
---------------------------
stats: Optimise the text-based stats output to improve speed
The text-based stats output is slow than the original C++ output. This patch
addresses some of those issues by removing:
Asserts that are not required
stats.ValueProxy, as it is no longer required
Unnecessary calls to __getattr__ and __hasattr__
It also changes the formula calculation to only calculate the values and totals
of formulas if they have not been passed in from the C++. This allows other
formulas to be added, whilst ensuring that the stats output is sufficiently
fast.
Testing has shown a 6x - 8x increase in stats output speed.
Diffs
-----
src/python/m5/stats/__init__.py 5532a1642108
src/python/m5/stats/display.py PRE-CREATION
src/python/m5/stats/info.py PRE-CREATION
Diff: http://reviews.gem5.org/r/1643/diff/
Testing
-------
Ensure that the output files are identical
Thanks,
Andreas Hansson
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
