mem-garnet: Fix default value of network bridge
authorSrikant Bharadwaj <srikant.bharadwaj@amd.com>
Thu, 10 Sep 2020 06:31:43 +0000 (02:31 -0400)
committerSrikant Bharadwaj <srikant.bharadwaj@amd.com>
Fri, 11 Sep 2020 02:00:13 +0000 (02:00 +0000)
Initializing the network bridge with NULL causes it to have
an class error when instatiating a link. The bridge is only
needed whne either a CDC or SerDes is enabled. This is handled
later during construction of the GarnetLink.

Change-Id: If19a21a6d9bf49449b9c390467d08d3422ae991a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/34257
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/mem/ruby/network/garnet2.0/GarnetLink.py

index 33a3d3149afe8ca7b4234e338328acd2da4e2143..9f7fe8b14e23dc0fa8083772c9e7b2c9bce98ee9 100644 (file)
@@ -140,14 +140,14 @@ class GarnetExtLink(BasicExtLink):
     # The network bridge encapsulates both the CDC and Ser-Des
     # units in HeteroGarnet. This is automatically enabled when
     # either CDC or Ser-Des is enabled.
-    ext_net_bridge = VectorParam.NetworkBridge(NULL,
+    ext_net_bridge = VectorParam.NetworkBridge([],
         "Network Bridge at external end")
-    ext_cred_bridge = VectorParam.NetworkBridge(NULL,
+    ext_cred_bridge = VectorParam.NetworkBridge([],
         "Credit Bridge at external end")
-    int_net_bridge = VectorParam.NetworkBridge(NULL,
+    int_net_bridge = VectorParam.NetworkBridge([],
         "Network Bridge at internal end")
-    int_cred_bridge = VectorParam.NetworkBridge(NULL,
-        "Credit Bridge at intternal end")
+    int_cred_bridge = VectorParam.NetworkBridge([],
+        "Credit Bridge at internal end")
 
 
     width = Param.UInt32(Parent.ni_flit_size,