From 2b99bc70831300b4218a6c0696b5aca9e0563fcc Mon Sep 17 00:00:00 2001 From: Giacomo Travaglini Date: Tue, 21 Sep 2021 17:54:32 +0100 Subject: [PATCH] configs: Remove lpae and virtualisation options lpae is already defaulting to True in the ArmSystem, so the check is useless (args.lpae cannot clear system.lpae) If you want to run an hypervisor, you shouldn't be using fs.py in the first place Change-Id: I4d631561d62399b946106a90c14454115040c206 Signed-off-by: Giacomo Travaglini Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51007 Reviewed-by: Jason Lowe-Power Maintainer: Jason Lowe-Power Tested-by: kokoro --- configs/common/Options.py | 3 --- configs/example/fs.py | 6 ------ 2 files changed, 9 deletions(-) diff --git a/configs/common/Options.py b/configs/common/Options.py index 7d72d4ad30..4e696ef923 100644 --- a/configs/common/Options.py +++ b/configs/common/Options.py @@ -260,9 +260,6 @@ def addCommonOptions(parser): Elastic Trace probe in a capture simulation and Trace CPU in a replay simulation""", default="") - parser.add_argument("-l", "--lpae", action="store_true") - parser.add_argument("-V", "--virtualisation", action="store_true") - # dist-gem5 options parser.add_argument("--dist", action="store_true", help="Parallel distributed gem5 simulation.") diff --git a/configs/example/fs.py b/configs/example/fs.py index dab28b199d..cdb2a494dc 100644 --- a/configs/example/fs.py +++ b/configs/example/fs.py @@ -132,12 +132,6 @@ def build_test_system(np): if args.script is not None: test_sys.readfile = args.script - if args.lpae: - test_sys.have_lpae = True - - if args.virtualisation: - test_sys.have_virtualization = True - test_sys.init_param = args.init_param # For now, assign all the CPUs to the same clock domain