configs: Generalize FileSystemConfig for non se.py
authorJason Lowe-Power <jason@lowepower.com>
Fri, 17 May 2019 02:09:56 +0000 (19:09 -0700)
committerJason Lowe-Power <jason@lowepower.com>
Fri, 17 May 2019 17:16:07 +0000 (17:16 +0000)
commite2656006df442a995bf80ee03fa9700d6ec14537
tree5a53ae690b6ca9e222c5f9cb26be1c4d62f9c8a2
parent279501816a5fbff6dffe2a1a7c57bd26ae50eb62
configs: Generalize FileSystemConfig for non se.py

This patch updates the FileSystemConfig so it works with more kinds of
config scripts (e.g., the Learning gem5 scripts).

There are 4 main changes:
- Added system as a parameter to the config_filesystem function so the
function can search the system for the number of CPUs instead of relying
on options from Options.py
- Instead of calling redirect_paths everywhere config_filesystem is
used, now it is implicitly called.
- Cleaned up the Ruby scripts a bit to remove redundant calls to
config_filesystem
- Added a config_filesystem call to the Ruby Learning gem5 script
(currently the only Learning gem5 script that requires it).

In the future, I think it would be better to move the config_filesystem
call into simulate.py, probably into the instantiate function. I tried to
use the per-CPU configuration parameters instead of options from
Options.py, but that's not possible until after the SimObject params
have been finalized in instantiate.

Change-Id: Ie6501a7435cfb3ac9d2b45be3722388b34063b1e
Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18848
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
Tested-by: kokoro <noreply+kokoro@google.com>
configs/common/FileSystemConfig.py
configs/example/se.py
configs/learning_gem5/part3/simple_ruby.py
configs/ruby/GPU_VIPER.py
configs/ruby/MESI_Three_Level.py
configs/ruby/MOESI_AMD_Base.py
configs/ruby/MOESI_hammer.py
configs/ruby/Ruby.py