VISHNU RAMADAS has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/63971?usp=email )
Change subject: configs, gpu-compute: Add configurable L1 access latencies
......................................................................
configs, gpu-compute: Add configurable L1 access latencies
Previously the L1 request and response latencies were not configurable
in the GPU config scripts. As a result, the simulations used the default
values from GPU.py. This commits adds support to change this value as an
input parameter. The parameters to use are "--mem-req-latency" followed
by the value and "--mem-resp-latency" followed by the value. The default
values are the same as those in GPU.py (which is 50). These new
parameters should be set instead of changing the mandatory queue latency
when configuring the L1 cache.
Change-Id: I812d77758ea12530899953f308c91f4c8b05866d
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/63971
Reviewed-by: Matt Sinclair <mattdsincl...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
Maintainer: Matt Sinclair <mattdsincl...@gmail.com>
---
M configs/example/apu_se.py
1 file changed, 35 insertions(+), 0 deletions(-)
Approvals:
Matt Sinclair: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index 16e3365..b33daa5 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -264,6 +264,18 @@
help="Latency while accessing shared memory",
)
parser.add_argument(
+ "--mem-req-latency",
+ type=int,
+ default=50,
+ help="Latency for requests from the cu to ruby.",
+)
+parser.add_argument(
+ "--mem-resp-latency",
+ type=int,
+ default=50,
+ help="Latency for responses from ruby to the cu.",
+)
+parser.add_argument(
"--TLB-prefetch", type=int, help="prefetch depth for" "TLBs"
)
parser.add_argument(
@@ -449,6 +461,8 @@
countPages=args.countPages,
max_cu_tokens=args.max_cu_tokens,
vrf_lm_bus_latency=args.vrf_lm_bus_latency,
+ mem_req_latency=args.mem_req_latency,
+ mem_resp_latency=args.mem_resp_latency,
localDataStore=LdsState(
banks=args.numLdsBanks,
bankConflictPenalty=args.ldsBankConflictPenalty,
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/63971?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: I812d77758ea12530899953f308c91f4c8b05866d
Gerrit-Change-Number: 63971
Gerrit-PatchSet: 6
Gerrit-Owner: VISHNU RAMADAS <vrama...@wisc.edu>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Matt Sinclair <mattdsinclair.w...@gmail.com>
Gerrit-Reviewer: Matt Sinclair <mattdsincl...@gmail.com>
Gerrit-Reviewer: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: VISHNU RAMADAS <vrama...@wisc.edu>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
