-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1150/#review2554
-----------------------------------------------------------



configs/common/Options.py
<http://reviews.gem5.org/r/1150/#comment2963>

    What if there are multiple disks?



configs/common/Options.py
<http://reviews.gem5.org/r/1150/#comment2964>

    What if there are multiple memories? What about memories of different 
characteristics?
    
    (the comment also says disk image...)



configs/example/fs.py
<http://reviews.gem5.org/r/1150/#comment2965>

    Oversimplification when using distributed memories of different size/type.



configs/example/fs.py
<http://reviews.gem5.org/r/1150/#comment2967>

    Same as before.



configs/example/ruby_fs.py
<http://reviews.gem5.org/r/1150/#comment2966>

    Same as before.


- Andreas Hansson


On April 16, 2012, 11:53 a.m., Jayneel Gandhi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/1150/
> -----------------------------------------------------------
> 
> (Updated April 16, 2012, 11:53 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Description
> -------
> 
> Added the options to Options.py for FS mode with backward compatibility. It 
> is good to provide an option to specify the disk image and the memory size 
> from command line since a lot of disk images are created to support different 
> benchmark suites as well as per user needs. Change in program also leads to 
> change in memory requirements. These options provide the interface to provide 
> both disk image and memory size from the command line and gives more 
> flexibility. 
> 
> 
> Diffs
> -----
> 
>   configs/common/Options.py a6830d615effc2e644fd3ebd2553c569c1d4607f 
>   configs/example/fs.py a6830d615effc2e644fd3ebd2553c569c1d4607f 
>   configs/example/ruby_fs.py a6830d615effc2e644fd3ebd2553c569c1d4607f 
> 
> Diff: http://reviews.gem5.org/r/1150/diff/
> 
> 
> Testing
> -------
> 
> Testing done to confirm backward compatibility
> 
> 
> Thanks,
> 
> Jayneel Gandhi
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to