python: Replace the Master/Slave Ports with Request/Response ports.
authorGabe Black <gabeblack@google.com>
Tue, 16 Apr 2019 03:47:48 +0000 (20:47 -0700)
committerGabe Black <gabeblack@google.com>
Sat, 27 Apr 2019 20:26:31 +0000 (20:26 +0000)
These are a little more descriptive and a little less potentially
offensive.

Change-Id: I84c30f783f0a4c242cb4f54ab272b6fdf1e9eec7
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18174
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Anthony Gutierrez <anthony.gutierrez@amd.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/python/m5/params.py

index 60d4a14b99b05e7a912e97c7eed56fad49bbe118..8b0a14af34c55fff0d725ee57fb399eaec0eb50c 100644 (file)
@@ -2099,17 +2099,18 @@ class Port(object):
     def cxx_decl(self, code):
         code('unsigned int port_${{self.name}}_connection_count;')
 
-Port.compat('MASTER', 'SLAVE')
+Port.compat('GEM5 REQUESTER', 'GEM5 RESPONDER')
 
-class MasterPort(Port):
-    # MasterPort("description")
+class RequestPort(Port):
+    # RequestPort("description")
     def __init__(self, desc):
-        super(MasterPort, self).__init__('MASTER', desc, is_source=True)
+        super(RequestPort, self).__init__(
+                'GEM5 REQUESTER', desc, is_source=True)
 
-class SlavePort(Port):
-    # SlavePort("description")
+class ResponsePort(Port):
+    # ResponsePort("description")
     def __init__(self, desc):
-        super(SlavePort, self).__init__('SLAVE', desc)
+        super(ResponsePort, self).__init__('GEM5 RESPONDER', desc)
 
 # VectorPort description object.  Like Port, but represents a vector
 # of connections (e.g., as on a XBar).
@@ -2122,15 +2123,22 @@ class VectorPort(Port):
     def makeRef(self, simobj):
         return VectorPortRef(simobj, self.name, self.role, self.is_source)
 
-class VectorMasterPort(VectorPort):
-    # VectorMasterPort("description")
+class VectorRequestPort(VectorPort):
+    # VectorRequestPort("description")
     def __init__(self, desc):
-        super(VectorMasterPort, self).__init__('MASTER', desc, is_source=True)
+        super(VectorRequestPort, self).__init__(
+                'GEM5 REQUESTER', desc, is_source=True)
 
-class VectorSlavePort(VectorPort):
-    # VectorSlavePort("description")
+class VectorResponsePort(VectorPort):
+    # VectorResponsePort("description")
     def __init__(self, desc):
-        super(VectorSlavePort, self).__init__('SLAVE', desc)
+        super(VectorResponsePort, self).__init__('GEM5 RESPONDER', desc)
+
+# Old names, maintained for compatibility.
+MasterPort = RequestPort
+SlavePort = ResponsePort
+VectorMasterPort = VectorRequestPort
+VectorSlavePort = VectorResponsePort
 
 # 'Fake' ParamDesc for Port references to assign to the _pdesc slot of
 # proxy objects (via set_param_desc()) so that proxy error messages
@@ -2164,5 +2172,6 @@ __all__ = ['Param', 'VectorParam',
            'MaxAddr', 'MaxTick', 'AllMemory',
            'Time',
            'NextEthernetAddr', 'NULL',
-           'Port', 'MasterPort', 'SlavePort',
-           'VectorPort', 'VectorMasterPort', 'VectorSlavePort']
+           'Port', 'RequestPort', 'ResponsePort', 'MasterPort', 'SlavePort',
+           'VectorPort', 'VectorRequestPort', 'VectorResponsePort',
+           'VectorMasterPort', 'VectorSlavePort']