Matthew Poremba has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44825 )

Change subject: configs: apu_se.py hotfix
......................................................................

configs: apu_se.py hotfix

Missed two optparse -> argparse changes. Square runs.

Change-Id: I3a652380e4c4202a376413602fa3698a28ff9206
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44825
Maintainer: Matthew Poremba <matthew.pore...@amd.com>
Reviewed-by: Daniel Carvalho <oda...@yahoo.com.br>
Tested-by: kokoro <noreply+kok...@google.com>
---
M configs/example/apu_se.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Daniel Carvalho: Looks good to me, approved
  Matthew Poremba: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index b0da8df..f779df3 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -181,14 +181,14 @@
 #--     0   1   x   UC_L2   (Uncached_GL2)
 #--     0   0   x   UC_All  (Uncached_All_Load)
 # default value: 5/C_RO_S (only allow caching in GL2 for read. Shared)
-parser.add_argument("--m-type", type='int', default=5,
+parser.add_argument("--m-type", type=int, default=5,
help="Default Mtype for GPU memory accesses. This is the " "value used for all memory accesses on an APU and is the " "default mode for dGPU unless explicitly overwritten by "
                     "the driver on a per-page basis.  Valid values are "
                     "between 0-7")

-parser.add_argument("--gfx-version", type="string", default='gfx801',
+parser.add_argument("--gfx-version", type=str, default='gfx801',
                     help="Gfx version for gpu: gfx801, gfx803, gfx900")

 Ruby.define_options(parser)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44825
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I3a652380e4c4202a376413602fa3698a28ff9206
Gerrit-Change-Number: 44825
Gerrit-PatchSet: 2
Gerrit-Owner: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to