Anthony Gutierrez has submitted this change. ( )

Change subject: gpu-compute: remove recvToken from GM pipe exec

gpu-compute: remove recvToken from GM pipe exec

Tokens were previously acquired in GM pipe exec but has been moved to
acqCoalescerToken. This removes the extraneous code which was acquiring
tokens twice, causing them to be depleted and triggering an assertion.

Change-Id: Ic92de8f06cc85828b29c69790bdadde057ef1777
Reviewed-by: Anthony Gutierrez <>
Reviewed-by: Matthew Poremba <>
Maintainer: Anthony Gutierrez <>
Tested-by: kokoro <>
M src/gpu-compute/
1 file changed, 0 insertions(+), 6 deletions(-)

  Anthony Gutierrez: Looks good to me, approved; Looks good to me, approved
  Matthew Poremba: Looks good to me, approved
  kokoro: Regressions pass

diff --git a/src/gpu-compute/ b/src/gpu-compute/
index 0bbacc4..c73184a 100644
--- a/src/gpu-compute/
+++ b/src/gpu-compute/
@@ -190,12 +190,6 @@

         DPRINTF(GPUCoalescer, "initiateAcc for %s seqNum %d\n",
                 mp->disassemble(), mp->seqNum());
-        // Memfences will not return tokens and must be issued so we should
- // not request one as this will deplete the token count until deadlock
-        if (!mp->isMemSync()) {
-            assert(mp->computeUnit()->getTokenManager()->haveTokens(1));
-            mp->computeUnit()->getTokenManager()->acquireTokens(1);
-        }

if (((mp->isMemSync() && !mp->isEndOfKernel()) | | !mp->isMemSync())) {

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ic92de8f06cc85828b29c69790bdadde057ef1777
Gerrit-Change-Number: 29916
Gerrit-PatchSet: 6
Gerrit-Owner: Anthony Gutierrez <>
Gerrit-Reviewer: Anthony Gutierrez <>
Gerrit-Reviewer: Matthew Poremba <>
Gerrit-Reviewer: Tony Gutierrez <>
Gerrit-Reviewer: Tuan Ta <>
Gerrit-Reviewer: kokoro <>
Gerrit-MessageType: merged
gem5-dev mailing list --
To unsubscribe send an email to

Reply via email to