Kyle Roarty has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/34677 )
Change subject: gpu-compute: Fix DPRINTFs causing segfault in wavefront.cc
......................................................................
gpu-compute: Fix DPRINTFs causing segfault in wavefront.cc
In 2 DPRINTFs, a variable's value was being dereferenced instead of
its address. Fixed by taking the address of the variable.
Change-Id: Id5d1863942848dd7a9e5e17e8180c33adbc72f15
---
M src/gpu-compute/wavefront.cc
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gpu-compute/wavefront.cc b/src/gpu-compute/wavefront.cc
index 0e737db..c659873 100644
--- a/src/gpu-compute/wavefront.cc
+++ b/src/gpu-compute/wavefront.cc
@@ -311,7 +311,7 @@
"Setting KernargSegPtr: s[%d] = %x\n",
computeUnit->cu_id, simdId,
wfSlotId, wfDynId, physSgprIdx,
- ((uint32_t*)kernarg_addr)[0]);
+ ((uint32_t*)&kernarg_addr)[0]);
physSgprIdx =
computeUnit->registerManager->mapSgpr(this,
regInitIdx);
@@ -321,7 +321,7 @@
"Setting KernargSegPtr: s[%d] = %x\n",
computeUnit->cu_id, simdId,
wfSlotId, wfDynId, physSgprIdx,
- ((uint32_t*)kernarg_addr)[1]);
+ ((uint32_t*)&kernarg_addr)[1]);
++regInitIdx;
break;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/34677
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: Id5d1863942848dd7a9e5e17e8180c33adbc72f15
Gerrit-Change-Number: 34677
Gerrit-PatchSet: 1
Gerrit-Owner: Kyle Roarty <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
