From 4a5ec70e0822da711ce0c4a1f5e33bf63114885d Mon Sep 17 00:00:00 2001
From: BujSet <ranganath1000@gmail.com>
Date: Wed, 15 Nov 2023 09:20:07 -0600
Subject: [PATCH] gpu-compute: Minor edits for atomic no returns and stores
 (#565)

Since returned data is not needed for AtomicNoReturn and Store memory
requests, the coalescer need not spend time writing in dummy data for
packets of these types.

Change-Id: Ie669e8c2a3bf44b5b0c290f62c49c5d4876a9a6a
---
 src/mem/ruby/system/GPUCoalescer.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mem/ruby/system/GPUCoalescer.cc b/src/mem/ruby/system/GPUCoalescer.cc
index 9b1b537618..f9d071ba62 100644
--- a/src/mem/ruby/system/GPUCoalescer.cc
+++ b/src/mem/ruby/system/GPUCoalescer.cc
@@ -600,8 +600,8 @@ GPUCoalescer::hitCallback(CoalescedRequest* crequest,
                 // data response is not needed.
                 case RubyRequestType_ATOMIC_NO_RETURN:
                     assert(pkt->isAtomicOp());
+                    break;
                 case RubyRequestType_ST:
-                    data.setData(pkt->getPtr<uint8_t>(), offset, pkt_size);
                     break;
                 case RubyRequestType_LD:
                     pkt->setData(data.getData(offset, pkt_size));