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

Ship it!


Ship It!

- Ali Saidi


On Dec. 6, 2012, 8:38 p.m., Andreas Hansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/1589/
> -----------------------------------------------------------
> 
> (Updated Dec. 6, 2012, 8:38 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Description
> -------
> 
> Changeset 9423:e78361f34a72
> ---------------------------
> mem: Add DDR3 and LPDDR2 DRAM controller configurations
> 
> This patch moves the default DRAM parameters from the SimpleDRAM class
> to two different subclasses, one for DDR3 and one for LPDDR2. More can
> be added as we go forward.
> 
> The regressions that previously used the SimpleDRAM are now using
> SimpleDDR3 as this is the most similar configuration.
> 
> 
> Diffs
> -----
> 
>   configs/common/FSConfig.py 844f9e724343 
>   src/mem/SimpleDRAM.py 844f9e724343 
>   tests/configs/inorder-timing.py 844f9e724343 
>   tests/configs/o3-timing-checker.py 844f9e724343 
>   tests/configs/o3-timing-mp.py 844f9e724343 
>   tests/configs/o3-timing.py 844f9e724343 
>   tests/configs/tgen-simple-dram.py 844f9e724343 
> 
> Diff: http://reviews.gem5.org/r/1589/diff/
> 
> 
> Testing
> -------
> 
> util/regress all passing (disregarding t1000 and eio)
> 
> 
> Thanks,
> 
> Andreas Hansson
> 
>

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

Reply via email to