Matthew Poremba has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/51852 )
Change subject: mem: Add system request flag for dGPUs
......................................................................
mem: Add system request flag for dGPUs
dGPUs can translate a virtual address and will not know if the address
resides in system/host memory or device/dGPU memory until the
translation is complete. In order to mark requests as going to either
system memory or device memory we add a field to the Request class.
Change-Id: Ib1e80e8d03ecdfeb11c24d979ccc4b912ce07f91
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51852
Reviewed-by: Matt Sinclair <[email protected]>
Maintainer: Matt Sinclair <[email protected]>
Tested-by: kokoro <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
---
M src/mem/request.hh
1 file changed, 29 insertions(+), 0 deletions(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved
Matt Sinclair: Looks good to me, but someone else must approve; Looks
good to me, approved
kokoro: Regressions pass
diff --git a/src/mem/request.hh b/src/mem/request.hh
index 85a9bed..b68edd2 100644
--- a/src/mem/request.hh
+++ b/src/mem/request.hh
@@ -436,6 +436,12 @@
*/
uint32_t _substreamId = 0;
+ /**
+ * For fullsystem GPU simulation, this determines if a requests
+ * destination is system (host) memory or dGPU (device) memory.
+ */
+ bool _systemReq = 0;
+
/** The virtual address of the request. */
Addr _vaddr = MaxAddr;
@@ -862,6 +868,10 @@
return _contextId;
}
+ /* For GPU fullsystem mark this request is not to device memory. */
+ void setSystemReq(bool sysReq) { _systemReq = sysReq; }
+ bool systemReq() const { return _systemReq; }
+
bool
hasStreamId() const
{
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/51852
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: Ib1e80e8d03ecdfeb11c24d979ccc4b912ce07f91
Gerrit-Change-Number: 51852
Gerrit-PatchSet: 6
Gerrit-Owner: Matthew Poremba <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Kyle Roarty <[email protected]>
Gerrit-Reviewer: Matt Sinclair <[email protected]>
Gerrit-Reviewer: Matthew Poremba <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
