From a3842f877a8c86299266be550dcb6f42375cf48d Mon Sep 17 00:00:00 2001 From: vramadas95 Date: Thu, 29 Sep 2022 13:39:30 -0500 Subject: [PATCH] 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 Tested-by: kokoro Maintainer: Matt Sinclair --- configs/example/apu_se.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py index 16e336573b..b33daa5b39 100644 --- a/configs/example/apu_se.py +++ b/configs/example/apu_se.py @@ -263,6 +263,18 @@ parser.add_argument( default=1, 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" ) @@ -449,6 +461,8 @@ for i in range(n_cu): 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,