commit af8d107191cc69a77624e2af34f108dc9c1ff03f
author Giacomo Travaglini <giacomo.travaglini@arm.com> Wed Jun 19 13:03:38 2019 +0100
committer Giacomo Travaglini <giacomo.travaglini@arm.com> Thu Jun 20 08:30:14 2019 +0000
tree 3d5264cc59fd1462a0ac36353e3eecac7677bdaa
parent a2f0167b6e951e3a293f3c8d2c11188eb5c59012

configs: Fix NULL dram-lowp regressions

The problem arises since there are some scripts (like NULL dram
regressions) which are making use of MemConfig.py without using
Opions.py so they won't have the new enable-dram-powerdown option

Change-Id: Id9769cce2e8a25b57da76f07eeebd279a6e00440
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19268
Reviewed-by: Matthew Poremba <matthew.poremba@amd.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Tested-by: kokoro <noreply+kokoro@google.com>

configs/common/MemConfig.py

diff --git a/configs/common/MemConfig.py b/configs/common/MemConfig.py
index d098692..3910cac 100644
--- a/configs/common/MemConfig.py
+++ b/configs/common/MemConfig.py
@@ -165,6 +165,7 @@
                                          None)
     opt_elastic_trace_en = getattr(options, "elastic_trace_en", False)
     opt_mem_ranks = getattr(options, "mem_ranks", None)
+    opt_dram_powerdown = getattr(options, "enable_dram_powerdown", None)
 
     if opt_mem_type == "HMC_2500_1x32":
         HMChost = HMC.config_hmc_host_ctrl(options, system)
@@ -226,8 +227,7 @@
 
             # Enable low-power DRAM states if option is set
             if issubclass(cls, m5.objects.DRAMCtrl):
-                mem_ctrl.enable_dram_powerdown = \
-                        options.enable_dram_powerdown
+                mem_ctrl.enable_dram_powerdown = opt_dram_powerdown
 
             if opt_elastic_trace_en:
                 mem_ctrl.latency = '1ns'