Matthew Poremba has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/65991?usp=email )

Change subject: configs: Set CPU vendor to M5 Simulator in apu_se.py
......................................................................

configs: Set CPU vendor to M5 Simulator in apu_se.py

Other vendor strings causes, for some reason, bad addresses to be
computed when running the GPU model. This change reverts back to M5
Simulator only for apu_se.py.

Change-Id: I5992b4e31569f5c0e5e49e523908c8fa0602f845
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/65991
Tested-by: kokoro <[email protected]>
Reviewed-by: Matt Sinclair <[email protected]>
Maintainer: Matt Sinclair <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
---
M configs/example/apu_se.py
1 file changed, 23 insertions(+), 0 deletions(-)

Approvals:
  kokoro: Regressions pass
Matt Sinclair: Looks good to me, but someone else must approve; Looks good to me, approved
  Jason Lowe-Power: Looks good to me, approved




diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index 39def02..8e8bc60 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -757,6 +757,11 @@
         (cpu_list[i], future_cpu_list[i]) for i in range(args.num_cpus)
     ]

+# Other CPU strings cause bad addresses in ROCm. Revert back to M5 Simulator.
+for (i, cpu) in enumerate(cpu_list):
+    for j in range(len(cpu)):
+        cpu.isa[j].vendor_string = "M5 Simulator"
+
 # Full list of processing cores in the system.
 cpu_list = cpu_list + [shader] + cp_list


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/65991?usp=email 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: I5992b4e31569f5c0e5e49e523908c8fa0602f845
Gerrit-Change-Number: 65991
Gerrit-PatchSet: 2
Gerrit-Owner: Matthew Poremba <[email protected]>
Gerrit-Reviewer: Bobby Bruce <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Matt Sinclair <[email protected]>
Gerrit-Reviewer: Matthew Poremba <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to