----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2005/#review4811 -----------------------------------------------------------
Seems like it would be simpler just to move the stats.reset() call inside the 'if need_startup:' block, right? - Steve Reinhardt On Sept. 9, 2013, 5:35 p.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2005/ > ----------------------------------------------------------- > > (Updated Sept. 9, 2013, 5:35 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9868:112a9778300a > --------------------------- > sim: reset stats after startup > Currently statistics are reset after the initial / checkpoint state > has been loaded. But ruby does some checkpoint processing in its > startup() function. So the stats need to be reset after the startup() > function has been called. This patch moves the class to stats.reset() > to achieve this change in functionality. > > > Diffs > ----- > > src/python/m5/simulate.py cc5797147e1c > > Diff: http://reviews.gem5.org/r/2005/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
