----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1816/#review4204 -----------------------------------------------------------
Ship it! Ship It! - Nilay Vaish On April 7, 2013, 8:56 p.m., Joel Hestness wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1816/ > ----------------------------------------------------------- > > (Updated April 7, 2013, 8:56 p.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9633:68b31b29f5ad > --------------------------- > Simulate.py: Fix up maxtick and maxtime > > This patch contains a couple fixes: > > 1) Since the global simulator frequency isn't bound until m5.instantiate() > is called, the maxtick resolution needs to happen after this call, since > changes to the global frequency will cause m5.simulate() to misinterpret the > maxtick value. Shuffling this also requires tweaking the checkpoint directory > handling to signal the checkpoint restore tick back to run(). Fixing this > completely and correctly will require storing the simulation frequency into > checkpoints, which is beyond the scope of this patch. > > 2) The maxtick option in Options.py was defaulted to MaxTicks, so the old > code > would always skip over the maxtime part of the conditionals at the beginning > of run(). Change the maxtick default to None, and set the maxtick local > variable in run() appropriately. > > > Diffs > ----- > > configs/common/Options.py fa31189e1fb5 > configs/common/Simulation.py fa31189e1fb5 > > Diff: http://reviews.gem5.org/r/1816/diff/ > > > Testing > ------- > > > Thanks, > > Joel Hestness > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
