derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

configs: Update apu_se.py argparse support

Browse Source 
There was a merge error caused by new options being added to this script
while all scripts were being converted from optparse. This fixes the
error.

This also removes the mostly unused setOption / getOption as you can
directly assign a value to an argument after parsing

Change-Id: Ic8aaa0728a43936cd4c6e1ed590e01ba5f0fbf5b
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44785
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Reviewed-by: Matt Sinclair <mattdsinclair@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Matthew Poremba
2021-04-23 11:23:35 -05:00
parent f403edf6b4
commit 4f7d15a70b
1 changed files with 7 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
configs/example/apu_se.py
View File

@@ -49,26 +49,6 @@ from common import GPUTLBOptions, GPUTLBConfig
import hsaTopology
from common import FileSystemConfig
########################## Script Options ########################
def setOption(parser, opt_str, value = 1):
# check to make sure the option actually exists
if not parser.has_option(opt_str):
raise Exception("cannot find %s in list of possible options" % opt_str)
opt = parser.get_option(opt_str)
# set the value
exec("parser.values.%s = %s" % (opt.dest, value))
def getOption(parser, opt_str):
# check to make sure the option actually exists
if not parser.has_option(opt_str):
raise Exception("cannot find %s in list of possible options" % opt_str)
opt = parser.get_option(opt_str)
# get the value
exec("return_value = parser.values.%s" % opt.dest)
return return_value
# Adding script options
parser = argparse.ArgumentParser()
@@ -186,12 +166,12 @@ parser.add_argument("--num-hw-queues", type=int, default=10,
parser.add_argument("--reg-alloc-policy", type=str, default="simple",
help="register allocation policy (simple/dynamic)")
parser.add_option("--dgpu", action="store_true", default=False,
help="Configure the system as a dGPU instead of an APU. "
"The dGPU config has its own local memory pool and is not "
"coherent with the host through hardware. Data is "
"transfered from host to device memory using runtime calls "
"that copy data over a PCIe-like IO bus.")
parser.add_argument("--dgpu", action="store_true", default=False,
help="Configure the system as a dGPU instead of an APU. "
"The dGPU config has its own local memory pool and is not "
"coherent with the host through hardware. Data is "
"transfered from host to device memory using runtime calls "
"that copy data over a PCIe-like IO bus.")
Ruby.define_options(parser)
@@ -201,7 +181,7 @@ GPUTLBOptions.tlb_options(parser)
args = parser.parse_args()
# The GPU cache coherence protocols only work with the backing store
setOption(parser, "--access-backing-store")
args.access_backing_store = True
# if benchmark root is specified explicitly, that overrides the search path
if args.benchmark_root: