From af8d107191cc69a77624e2af34f108dc9c1ff03f Mon Sep 17 00:00:00 2001 From: Giacomo Travaglini Date: Wed, 19 Jun 2019 13:03:38 +0100 Subject: 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 Reviewed-by: Nikos Nikoleris Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19268 Reviewed-by: Matthew Poremba Reviewed-by: Jason Lowe-Power Maintainer: Jason Lowe-Power Tested-by: kokoro --- configs/common/MemConfig.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configs/common/MemConfig.py b/configs/common/MemConfig.py index d09869285..3910cacbd 100644 --- a/configs/common/MemConfig.py +++ b/configs/common/MemConfig.py @@ -165,6 +165,7 @@ def config_mem(options, system): 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 @@ def config_mem(options, system): # 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' -- cgit v1.2.3