VISHNU RAMADAS has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/63652?usp=email )
Change subject: configs,gpu-compute: Add configurable LDS bus latency
......................................................................
configs,gpu-compute: Add configurable LDS bus latency
Previously the LDS bus latency was not configurable in the GPU config
scripts. As a result, the simulations would use the default value from
GPU.py. This commit adds support to change this value as an input
option. The option to use is "--vrf_lm_bus_latency".
Change-Id: I8d8852e6d7b9d03ebec1fe8b392968f396dd3526
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/63652
Maintainer: Matthew Poremba <matthew.pore...@amd.com>
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Matt Sinclair <mattdsinclair.w...@gmail.com>
---
M configs/example/apu_se.py
1 file changed, 25 insertions(+), 0 deletions(-)
Approvals:
Matthew Poremba: Looks good to me, approved
Matt Sinclair: Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index 1a649d3..16e3365 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -258,6 +258,12 @@
help="Number of coalescer tokens per CU",
)
parser.add_argument(
+ "--vrf_lm_bus_latency",
+ type=int,
+ default=1,
+ help="Latency while accessing shared memory",
+)
+parser.add_argument(
"--TLB-prefetch", type=int, help="prefetch depth for" "TLBs"
)
parser.add_argument(
@@ -442,6 +448,7 @@
localMemBarrier=args.LocalMemBarrier,
countPages=args.countPages,
max_cu_tokens=args.max_cu_tokens,
+ vrf_lm_bus_latency=args.vrf_lm_bus_latency,
localDataStore=LdsState(
banks=args.numLdsBanks,
bankConflictPenalty=args.ldsBankConflictPenalty,
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/63652?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: I8d8852e6d7b9d03ebec1fe8b392968f396dd3526
Gerrit-Change-Number: 63652
Gerrit-PatchSet: 5
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: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: VISHNU RAMADAS <vrama...@wisc.edu>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-CC: Jason Lowe-Power <power...@gmail.com>
Gerrit-CC: Matt Sinclair <mattdsincl...@gmail.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org