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

Ship it!


Thanks again for doing this!

- Jason Lowe-Power


On Oct. 12, 2016, 2:30 p.m., Andreas Hansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3647/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2016, 2:30 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 11670:aeab4c3c27ef
> ---------------------------
> ruby: Fix regressions and make Ruby configs Python packages
> 
> This patch moves the addition of network options into the Ruby module
> to avoid the regressions all having to add it explicitly. Doing this
> exposes an issue in our current config system though, namely the fact
> that addtoPath is relative to the Python script being executed. Since
> both example and regression scripts use the Ruby module we would end
> up with two different (relative) paths being added. Instead we take a
> first step at turning the config modules into Python packages, simply
> by adding a __init__.py in the configs/ruby, configs/topologies and
> configs/network subdirectories.
> 
> As a result, we can now add the top-level configs directory to the
> Python search path, and then use the package names in the various
> modules. The example scripts are also updated, and the messy
> path-deducing variations in the scripts are unified.
> 
> 
> Diffs
> -----
> 
>   tests/configs/simple-timing-ruby.py 220fa4099b9a 
>   tests/configs/gpu-ruby.py 220fa4099b9a 
>   tests/configs/memtest-ruby.py 220fa4099b9a 
>   tests/configs/o3-timing-mp-ruby.py 220fa4099b9a 
>   tests/configs/o3-timing-ruby.py 220fa4099b9a 
>   tests/configs/pc-simple-timing-ruby.py 220fa4099b9a 
>   tests/configs/rubytest-ruby.py 220fa4099b9a 
>   tests/configs/simple-atomic-mp-ruby.py 220fa4099b9a 
>   tests/configs/simple-timing-mp-ruby.py 220fa4099b9a 
>   configs/ruby/AMD_Base_Constructor.py 220fa4099b9a 
>   configs/ruby/GPU_RfO.py 220fa4099b9a 
>   configs/ruby/GPU_VIPER.py 220fa4099b9a 
>   configs/ruby/GPU_VIPER_Baseline.py 220fa4099b9a 
>   configs/ruby/GPU_VIPER_Region.py 220fa4099b9a 
>   configs/ruby/MOESI_AMD_Base.py 220fa4099b9a 
>   configs/ruby/Ruby.py 220fa4099b9a 
>   configs/ruby/__init__.py PRE-CREATION 
>   configs/topologies/__init__.py PRE-CREATION 
>   tests/configs/gpu-randomtest-ruby.py 220fa4099b9a 
>   configs/example/apu_se.py 220fa4099b9a 
>   configs/example/fs.py 220fa4099b9a 
>   configs/example/garnet_synth_traffic.py 220fa4099b9a 
>   configs/example/ruby_direct_test.py 220fa4099b9a 
>   configs/example/ruby_gpu_random_test.py 220fa4099b9a 
>   configs/example/ruby_mem_test.py 220fa4099b9a 
>   configs/example/ruby_random_test.py 220fa4099b9a 
>   configs/example/se.py 220fa4099b9a 
>   configs/network/__init__.py PRE-CREATION 
> 
> Diff: http://reviews.gem5.org/r/3647/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Hansson
> 
>

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

Reply via email to