Anthony Gutierrez has submitted this change and it was merged. (
https://gem5-review.googlesource.com/7101 )
Change subject: gpu-compute: call createThreads() on cpu objs in apu_se.py
......................................................................
gpu-compute: call createThreads() on cpu objs in apu_se.py
commit 8ad26e2688b8736f9290086bb4026cc7500429e9
cpu: Don't override ISA if provided by user
removed the default ISA from the BaseCPU, and instead relies on
createThreads() to initiate a default ISA if none is specified. the
apu_se.py
script, however does not call creatThreads() leading to a fatal when
constructing CPU objects. this patch adds the appropriate calls to
createThreads() inside apu_se.py.
Change-Id: I16a5929454c59d68a3f1b7b3858c48a70cb76412
Reviewed-on: https://gem5-review.googlesource.com/7101
Reviewed-by: Jason Lowe-Power <[email protected]>
Reviewed-by: Michael LeBeane <[email protected]>
Maintainer: Anthony Gutierrez <[email protected]>
---
M configs/example/apu_se.py
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved
Michael LeBeane: Looks good to me, approved
Anthony Gutierrez: Looks good to me, approved
diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index f93442d..012fdd3 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -392,6 +392,7 @@
# OpenCL driver
driver = ClDriver(filename="hsa", codefile=kernel_files)
for cpu in cpu_list:
+ cpu.createThreads()
cpu.workload = Process(executable = executable,
cmd = [options.cmd] + options.options.split(),
drivers = [driver])
@@ -401,6 +402,7 @@
if fast_forward:
for i in xrange(len(future_cpu_list)):
future_cpu_list[i].workload = cpu_list[i].workload
+ future_cpu_list[i].createThreads()
########################## Create the overall system
########################
# List of CPUs that must be switched when moving between KVM and simulation
--
To view, visit https://gem5-review.googlesource.com/7101
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I16a5929454c59d68a3f1b7b3858c48a70cb76412
Gerrit-Change-Number: 7101
Gerrit-PatchSet: 2
Gerrit-Owner: Anthony Gutierrez <[email protected]>
Gerrit-Reviewer: Alexandru Duțu <[email protected]>
Gerrit-Reviewer: Anthony Gutierrez <[email protected]>
Gerrit-Reviewer: Brandon Potter <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Michael LeBeane <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
