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

Ship it!


Good to go from my side after addressing the minor comment. But I think a few 
more people should review it too.


configs/common/HMC.py (lines 110 - 121)
<http://reviews.gem5.org/r/3373/#comment7314>

    Change the arch option names.


- Radhika Jagtap


On June 20, 2016, 6:26 p.m., Abdul Mutaal Ahmad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3373/
> -----------------------------------------------------------
> 
> (Updated June 20, 2016, 6:26 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> In this new hmc configuration we have used the existing components in gem5 
> mainly [SerialLink] [NoncoherentXbar]& [DRAMCtrl] to define 3 different 
> architecture for HMC.
> ===========
> Highlights
> ===========
> 1- It explores 3 different HMC architectures 
> 2- It creates 4-HMC crossbars and attaches 16 vault controllers with it. This 
>  will connect vaults to serial links 
> 3- From the previous version, HMCController with round robin funtionality is 
> being removed and all the serial links are being accessible directly from 
> user ports 
> 4- Latency incorporated by HMCController (in previous version) is being added 
> to SerialLink
> 
> 
> Diffs
> -----
> 
>   src/mem/serial_link.cc UNKNOWN 
>   src/mem/serial_link.hh UNKNOWN 
>   src/mem/SerialLink.py UNKNOWN 
>   configs/common/HMC.py UNKNOWN 
>   configs/common/MemConfig.py UNKNOWN 
> 
> Diff: http://reviews.gem5.org/r/3373/diff/
> 
> 
> Testing
> -------
> 
> Current version only works for traffic generator setup {can be extended for 
> full-system and system-emulation} 
> ===============
> How to run it
> ===============
> from you gem5 home directory run the following commands
> 
> build/ARM/gem5.opt configs/example/hmctest.py --mem-type=HMC_2500_x32 
> --mode=RANDOM  --arch=same
> build/ARM/gem5.opt configs/example/hmctest.py --mem-type=HMC_2500_x32 
> --mode=RANDOM  --arch=distributed
> build/ARM/gem5.opt configs/example/hmctest.py --mem-type=HMC_2500_x32 
> --mode=RANDOM  --arch=mixed
> 
> Dependency
> ===========
> This patch required another tester patch to be applied before running the 
> test 
> 
> http://reviews.gem5.org/r/3374/
> 
> 
> Thanks,
> 
> Abdul Mutaal Ahmad
> 
>

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

Reply via email to