python: Always throw TypeError on slave-slave connections
authorNicholas Lindsay <nicholas.lindsay@arm.com>
Mon, 2 Jul 2018 15:07:31 +0000 (16:07 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Fri, 25 Jan 2019 12:46:15 +0000 (12:46 +0000)
params.py checks the validity of memory port-port connections before
they are instantiated in C++. This commit ensures that attempting to
connect two slave ports together will cause a TypeError.

Change-Id: Ia7d0a15df28b96c7bf5e568c4f4917d21a19b824
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/15896
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>

src/python/m5/params.py

index 74bd40b7d6fd429bb44098835401c9e90d5acb53..854c8e3793203565df17fb4a1c263bc939b4915c 100644 (file)
@@ -1883,12 +1883,9 @@ class PortRef(object):
     def ccConnect(self):
         from _m5.pyobject import connectPorts
 
-        if self.role == 'SLAVE':
-            # do nothing and let the master take care of it
-            return
-
         if self.ccConnected: # already done this
             return
+
         peer = self.peer
         if not self.peer: # nothing to connect to
             return
@@ -1899,6 +1896,10 @@ class PortRef(object):
                 "cannot connect '%s' and '%s' due to identical role '%s'" \
                 % (peer, self, self.role)
 
+        if self.role == 'SLAVE':
+            # do nothing and let the master take care of it
+            return
+
         try:
             # self is always the master and peer the slave
             connectPorts(self.simobj.getCCObject(), self.name, self.index,