Gabriel B. has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/67660?usp=email )
(
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
)Change subject: configs: Fix default CustomMesh for use with Garnet
......................................................................
configs: Fix default CustomMesh for use with Garnet
Garnet routers do not support 0 latency switches. Use 1 instead if the
network is garnet.
Change-Id: I09841a01eaf413bee0a1629307ecff0ae2bda948
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/67660
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
Maintainer: Jason Lowe-Power <power...@gmail.com>
---
M configs/topologies/CustomMesh.py
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/topologies/CustomMesh.py
b/configs/topologies/CustomMesh.py
index c62b39a..21fa2dc 100644
--- a/configs/topologies/CustomMesh.py
+++ b/configs/topologies/CustomMesh.py
@@ -171,7 +171,9 @@
def _createRNFRouter(self, mesh_router):
# Create a zero-latency router bridging node controllers
# and the mesh router
- node_router = self._Router(router_id=len(self._routers), latency=0)
+ node_router = self._Router(
+ router_id=len(self._routers), latency=self.node_router_latency
+ )
self._routers.append(node_router)
# connect node_router <-> mesh router
@@ -270,6 +272,7 @@
self._ExtLink = ExtLink
self._Router = Router
+ self.node_router_latency = 1 if options.network == "garnet" else 0
if hasattr(options, "router_link_latency"):
self._router_link_latency = options.router_link_latency
self._node_link_latency = options.node_link_latency
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/67660?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I09841a01eaf413bee0a1629307ecff0ae2bda948
Gerrit-Change-Number: 67660
Gerrit-PatchSet: 7
Gerrit-Owner: Gabriel B. <gabriel.bus...@arteris.com>
Gerrit-Reviewer: Gabriel B. <gabriel.bus...@arteris.com>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-CC: kokoro <noreply+kok...@google.com>
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
