On Wed, March 6, 2013 11:05 pm, Beckmann, Brad wrote: > Thanks Nilay for checking this in. I know this is a small patch, but > there was a lot of previous effort on your part to make it happen.
Would not have been possible with out your help. > > Going forward, I do think it is important to make the backing store > optional and not remove it completely. I'm fine if we maintain the > default as off, but there are a few times when it is nice to leverage the > backing store for simple boundary experiments. So if we want to enable > the backing store for se mode, is it as simple as setting access_phys_mem > to true and setting the system.physmem.null to true? Can we modify the > se.py/Ruby.py to do this using just one command line parameter? While we can add an option for keeping the functional copy of the memory, I am not keen on it. I am of the view that once we support pio accesses in ruby, we should remove the functional memory completely. > > Nilay, I'm still in the midst of a significant rebase of our tree due to > your clock/time changes. Please hold off on checking any more Ruby > changes until I can get our patches out the door. > I was hoping these patches would not interfere with yours. Are you facing any issues in merging your patches with the clock/time changes? I think I made some errors in those patches. They manifest while trying to create a checkpoint or restore from one. I have some changes that correct should do away with some of the errors. -- Nilay _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
