From d2bfc31592dc63d922f2aa3962000e9f9c9f913b Mon Sep 17 00:00:00 2001 From: Hansung Kim Date: Tue, 31 Oct 2023 23:06:35 -0700 Subject: [PATCH] Fix store opcode assertion in AOpcodeIsStore Now we support PutPartialData for narrow write requests that doesn't have all-1 mask. --- src/main/scala/tilelink/Coalescing.scala | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/main/scala/tilelink/Coalescing.scala b/src/main/scala/tilelink/Coalescing.scala index 51d490a..864f2e7 100644 --- a/src/main/scala/tilelink/Coalescing.scala +++ b/src/main/scala/tilelink/Coalescing.scala @@ -56,7 +56,7 @@ object DefaultInFlightTableSizeEnum extends InFlightTableSizeEnum { case class CoalescerConfig( enable: Boolean, // globally enable or disable coalescing numLanes: Int, // number of lanes (or threads) in a warp - reqQueueDepth: Int, // request window per lane + reqQueueDepth: Int, // request window per lane waitTimeout: Int, // max cycles to wait before forced fifo dequeue, per lane addressWidth: Int, // assume <= 32 dataBusWidth: Int, // memory-side downstream TileLink data bus size. Nominally, this has @@ -1366,7 +1366,8 @@ object TLUtils { // 0: PutFullData, 1: PutPartialData, 4: Get when(checkOpcode) { assert( - opcode === TLMessages.PutFullData || opcode === TLMessages.Get, + opcode === TLMessages.PutFullData || opcode === TLMessages.PutPartialData || + opcode === TLMessages.Get, "unhandled TL A opcode found" ) }