----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/690/#review1226 -----------------------------------------------------------
I guess I should have said something, but there are several more diffs to come down the road. I just wanted to get some out of the way now. By the end, there will be no output code in C++ at all. src/python/m5/stats/__init__.py <http://reviews.m5sim.org/r/690/#comment1691> If you have a periodic dumping of stats because of an event and you get a dumpstats from an m5op or from kill -USR1, you don't want to dump twice in the same cycle because stuff just doesn't work well. - Nathan On 2011-05-10 06:08:36, Nathan Binkert wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/690/ > ----------------------------------------------------------- > > (Updated 2011-05-10 06:08:36) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > stats: move code that loops over all stats into python > > > Diffs > ----- > > src/base/SConscript 44f8c2507d85 > src/base/statistics.hh 44f8c2507d85 > src/base/statistics.cc 44f8c2507d85 > src/base/stats/info.hh 44f8c2507d85 > src/base/stats/mysql.hh 44f8c2507d85 > src/base/stats/mysql.cc 44f8c2507d85 > src/base/stats/output.hh 44f8c2507d85 > src/base/stats/output.cc 44f8c2507d85 > src/base/stats/text.hh 44f8c2507d85 > src/base/stats/text.cc 44f8c2507d85 > src/base/stats/visit.hh 44f8c2507d85 > src/base/stats/visit.cc 44f8c2507d85 > src/python/m5/simulate.py 44f8c2507d85 > src/python/m5/stats/__init__.py PRE-CREATION > src/python/swig/stats.i 44f8c2507d85 > > Diff: http://reviews.m5sim.org/r/690/diff > > > Testing > ------- > > quick regressions pass (though most recently run with review 689 and 691) > > > Thanks, > > Nathan > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
