----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2028/#review4757 -----------------------------------------------------------
Ship it! Ship It! - Andreas Hansson On Sept. 28, 2013, 2:18 a.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2028/ > ----------------------------------------------------------- > > (Updated Sept. 28, 2013, 2:18 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9896:dae3245884f3 > --------------------------- > ruby: eliminate non-determinism from ruby.stats output > > Get rid of non-deterministic "stats" in ruby.stats output > such as time & date of run, elapsed & CPU time used, > and memory usage. These values cause spurious > miscomparisons when looking at output diffs (though > they don't affect regressions, since the regressions > pass/fail status currently ignores ruby.stats entirely). > > Most of this information is already captured in other > places (time & date in stdout, elapsed time & mem usage > in stats.txt), where the regression script is smart > enough to filter it out. It seems easier to get rid of > the redundant output rather than teaching the > regression tester to ignore the same information in > two different places. > > > Diffs > ----- > > src/mem/ruby/profiler/Profiler.cc d1a5e147e72db4c64f7d489bb14069c62215471a > src/mem/ruby/system/System.cc d1a5e147e72db4c64f7d489bb14069c62215471a > > Diff: http://reviews.gem5.org/r/2028/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
