diff options
author | Andreas Hansson <andreas.hansson@arm.com> | 2013-05-30 12:54:02 -0400 |
---|---|---|
committer | Andreas Hansson <andreas.hansson@arm.com> | 2013-05-30 12:54:02 -0400 |
commit | 7e13c4d046ea8335fa5681e8c035c3839f0749f9 (patch) | |
tree | 1a6102adf5c6167deed7e8fa21ce5da06f98fcc2 /src/mem/port.hh | |
parent | 2308f812ef848d027b99dd52c0900aed11260d87 (diff) | |
download | gem5-7e13c4d046ea8335fa5681e8c035c3839f0749f9.tar.xz |
mem: Make returning snoop responses occupy response layer
This patch introduces a mirrored internal snoop port to facilitate
easy addition of flow control for the snoop responses that are turned
into normal responses on their return. To perform this, the slave
ports of the coherent bus are wrapped in internal master ports that
are passed as the source ports to the response layer in question.
As a result of this patch, there is more contention for the response
resources, and as such system performance will decrease slightly.
A consequence of the mirrored internal port is that the port the bus
tells to retry (the internal one) and the port actually retrying (the
mirrored) one are not the same. Thus, the existing check in tryTiming
is not longer correct. In fact, the test is redundant as the layer is
only in the retry state while calling sendRetry on the waiting port,
and if the latter does not immediately call the bus then the retry
state is left. Consequently the check is removed.
Diffstat (limited to 'src/mem/port.hh')
-rw-r--r-- | src/mem/port.hh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mem/port.hh b/src/mem/port.hh index a4e823796..18db800b6 100644 --- a/src/mem/port.hh +++ b/src/mem/port.hh @@ -251,7 +251,7 @@ class MasterPort : public BaseMasterPort * Send a retry to the slave port that previously attempted a * sendTimingResp to this master port and failed. */ - void sendRetry(); + virtual void sendRetry(); /** * Determine if this master port is snooping or not. The default |