> On 2011-02-02 13:03:26, Nathan Binkert wrote:
> > Is it necessary to preserve this from run to run?  The code looks right, I 
> > just wonder if it's overkill to do this.  We need to think about whether 
> > people want or not want to change this and why?

I'm ok with putting it in or leaving it out. I realized these settings would be 
lost across a checkpoint and wanted to put this out there as a solution, but I 
think I even mentioned in an email a while ago I wasn't sure how this stuff 
would interact with checkpointing.


- Gabe


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/459/#review830
-----------------------------------------------------------


On 2011-01-30 21:18:51, Gabe Black wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/459/
> -----------------------------------------------------------
> 
> (Updated 2011-01-30 21:18:51)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> Root: Serialize the time sync state.
> 
> 
> Diffs
> -----
> 
>   src/sim/root.hh 31a04e5ac4be 
>   src/sim/root.cc 31a04e5ac4be 
> 
> Diff: http://reviews.m5sim.org/r/459/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gabe
> 
>

_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to