Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/13463
Change subject: gpu-compute: Explicitly use little endian packet accessors.
......................................................................
gpu-compute: Explicitly use little endian packet accessors.
The gpu ISA doesn't have a well defined endianness, but it really
should. It seems that the GPU is only used with x86, and in that
context it would be little endian.
Change-Id: I1620906564a77f44553fbf6d788866e017b6054b
---
M src/gpu-compute/dispatcher.cc
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/gpu-compute/dispatcher.cc b/src/gpu-compute/dispatcher.cc
index 41e632d..9bebaa3 100644
--- a/src/gpu-compute/dispatcher.cc
+++ b/src/gpu-compute/dispatcher.cc
@@ -139,7 +139,7 @@
assert(pkt->getSize() == 8);
uint64_t retval = dispatchActive;
- pkt->set(retval);
+ pkt->setLE(retval);
} else {
offset -= 8;
assert(offset + pkt->getSize() < sizeof(HsaQueueEntry));
@@ -166,16 +166,16 @@
switch (pkt->getSize()) {
case 1:
- data_val = pkt->get<uint8_t>();
+ data_val = pkt->getLE<uint8_t>();
break;
case 2:
- data_val = pkt->get<uint16_t>();
+ data_val = pkt->getRaw<uint16_t>();
break;
case 4:
- data_val = pkt->get<uint32_t>();
+ data_val = pkt->getRaw<uint32_t>();
break;
case 8:
- data_val = pkt->get<uint64_t>();
+ data_val = pkt->getRaw<uint64_t>();
break;
default:
DPRINTF(GPUDisp, "bad size %d\n", pkt->getSize());
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/13463
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I1620906564a77f44553fbf6d788866e017b6054b
Gerrit-Change-Number: 13463
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
