Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 1 | // |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 2 | // Copyright © 2019 Arm Ltd and Contributors. All rights reserved. |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 3 | // SPDX-License-Identifier: MIT |
| 4 | // |
| 5 | |
Jim Flynn | 3e9bc19 | 2022-03-23 23:01:26 +0000 | [diff] [blame] | 6 | #include <client/src/BufferManager.hpp> |
| 7 | #include <client/src/PacketBuffer.hpp> |
| 8 | #include <client/src/ProfilingUtils.hpp> |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 9 | |
Jim Flynn | bbfe603 | 2020-07-20 16:57:44 +0100 | [diff] [blame] | 10 | #include <common/include/SwTrace.hpp> |
| 11 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 12 | #include <doctest/doctest.h> |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 13 | |
Cathal Corbett | 5aa9fd7 | 2022-02-25 15:33:28 +0000 | [diff] [blame] | 14 | using namespace arm::pipe; |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 15 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 16 | TEST_SUITE("BufferTests") |
| 17 | { |
| 18 | TEST_CASE("PacketBufferTest0") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 19 | { |
Matteo Martincigh | 2ffcc41 | 2019-11-05 11:47:40 +0000 | [diff] [blame] | 20 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(512); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 21 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 22 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 23 | |
| 24 | // Write data to the buffer |
| 25 | WriteUint32(packetBuffer, 0, 10); |
| 26 | WriteUint32(packetBuffer, 4, 20); |
| 27 | WriteUint32(packetBuffer, 8, 30); |
| 28 | WriteUint32(packetBuffer, 12, 40); |
| 29 | |
| 30 | // Commit |
| 31 | packetBuffer->Commit(16); |
| 32 | |
| 33 | // Size of buffer is equal to committed data |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 34 | CHECK(packetBuffer->GetSize() == 16); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 35 | |
| 36 | // Read data from the buffer |
| 37 | auto readBuffer = packetBuffer->GetReadableData(); |
| 38 | uint32_t readData0 = ReadUint32(readBuffer, 0); |
| 39 | uint32_t readData1 = ReadUint32(readBuffer, 4); |
| 40 | uint32_t readData2 = ReadUint32(readBuffer, 8); |
| 41 | uint32_t readData3 = ReadUint32(readBuffer, 12); |
| 42 | |
| 43 | // Check that data is correct |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 44 | CHECK(readData0 == 10); |
| 45 | CHECK(readData1 == 20); |
| 46 | CHECK(readData2 == 30); |
| 47 | CHECK(readData3 == 40); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 48 | |
| 49 | // Mark read |
| 50 | packetBuffer->MarkRead(); |
| 51 | |
| 52 | // Size of buffer become 0 after marked read |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 53 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 54 | } |
| 55 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 56 | TEST_CASE("PacketBufferTest1") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 57 | { |
Matteo Martincigh | 2ffcc41 | 2019-11-05 11:47:40 +0000 | [diff] [blame] | 58 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(512); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 59 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 60 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 61 | |
| 62 | // Write data to the buffer using GetWritableData |
| 63 | auto writeBuffer = packetBuffer->GetWritableData(); |
| 64 | WriteUint32(writeBuffer, 0, 10); |
| 65 | WriteUint32(writeBuffer, 4, 20); |
| 66 | WriteUint32(writeBuffer, 8, 30); |
| 67 | WriteUint32(writeBuffer, 12, 40); |
| 68 | |
| 69 | packetBuffer->Commit(16); |
| 70 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 71 | CHECK(packetBuffer->GetSize() == 16); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 72 | |
| 73 | // Read data from the buffer |
| 74 | auto readBuffer = packetBuffer->GetReadableData(); |
| 75 | uint32_t readData0 = ReadUint32(readBuffer, 0); |
| 76 | uint32_t readData1 = ReadUint32(readBuffer, 4); |
| 77 | uint32_t readData2 = ReadUint32(readBuffer, 8); |
| 78 | uint32_t readData3 = ReadUint32(readBuffer, 12); |
| 79 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 80 | CHECK(readData0 == 10); |
| 81 | CHECK(readData1 == 20); |
| 82 | CHECK(readData2 == 30); |
| 83 | CHECK(readData3 == 40); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 84 | |
| 85 | packetBuffer->MarkRead(); |
| 86 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 87 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 88 | } |
| 89 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 90 | TEST_CASE("PacketBufferReleaseTest") |
| 91 | { |
Matteo Martincigh | 2ffcc41 | 2019-11-05 11:47:40 +0000 | [diff] [blame] | 92 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(512); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 93 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 94 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 95 | |
| 96 | auto writeBuffer = packetBuffer->GetWritableData(); |
| 97 | |
| 98 | WriteUint32(writeBuffer, 0, 10); |
| 99 | WriteUint32(writeBuffer, 4, 20); |
| 100 | WriteUint32(writeBuffer, 8, 30); |
| 101 | WriteUint32(writeBuffer, 12, 40); |
| 102 | |
| 103 | packetBuffer->Release(); |
| 104 | |
| 105 | // Size of buffer become 0 after release |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 106 | CHECK(packetBuffer->GetSize() == 0); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 107 | } |
| 108 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 109 | TEST_CASE("PacketBufferCommitErrorTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 110 | { |
Matteo Martincigh | 2ffcc41 | 2019-11-05 11:47:40 +0000 | [diff] [blame] | 111 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(8); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 112 | |
| 113 | // Cannot commit data bigger than the max size of the buffer |
Jim Flynn | f9db3ef | 2022-03-08 21:23:44 +0000 | [diff] [blame] | 114 | CHECK_THROWS_AS(packetBuffer->Commit(16);, arm::pipe::ProfilingException); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 115 | } |
| 116 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 117 | TEST_CASE("BufferReserveTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 118 | { |
| 119 | BufferManager bufferManager(1, 512); |
| 120 | unsigned int reservedSize = 0; |
| 121 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 122 | |
| 123 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 124 | CHECK(reservedSize == 512); |
| 125 | CHECK(packetBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 126 | } |
| 127 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 128 | TEST_CASE("BufferReserveExceedingSpaceTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 129 | { |
| 130 | BufferManager bufferManager(1, 512); |
| 131 | unsigned int reservedSize = 0; |
| 132 | |
| 133 | // Cannot reserve buffer bigger than maximum buffer size |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 134 | auto reservedBuffer = bufferManager.Reserve(1024, reservedSize); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 135 | CHECK(reservedSize == 0); |
| 136 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 137 | } |
| 138 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 139 | TEST_CASE("BufferExhaustionTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 140 | { |
| 141 | BufferManager bufferManager(1, 512); |
| 142 | unsigned int reservedSize = 0; |
| 143 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 144 | |
| 145 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 146 | CHECK(reservedSize == 512); |
| 147 | CHECK(packetBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 148 | |
| 149 | // Cannot reserve buffer when buffer is not available |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 150 | // NOTE: because the buffer manager now has surge capacity of |
| 151 | // initial size * 3 we should be able to reserve three |
| 152 | // buffers before exhaustion |
| 153 | packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 154 | |
| 155 | // Successfully reserved the second buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 156 | CHECK(reservedSize == 512); |
| 157 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 158 | |
| 159 | packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 160 | |
| 161 | // Successfully reserved the third buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 162 | CHECK(reservedSize == 512); |
| 163 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 164 | |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 165 | auto reservedBuffer = bufferManager.Reserve(512, reservedSize); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 166 | CHECK(reservedSize == 0); |
| 167 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 168 | } |
| 169 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 170 | TEST_CASE("BufferReserveMultipleTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 171 | { |
| 172 | BufferManager bufferManager(3, 512); |
| 173 | unsigned int reservedSize0 = 0; |
| 174 | auto packetBuffer0 = bufferManager.Reserve(512, reservedSize0); |
| 175 | |
| 176 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 177 | CHECK(reservedSize0 == 512); |
| 178 | CHECK(packetBuffer0.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 179 | |
| 180 | unsigned int reservedSize1 = 0; |
| 181 | auto packetBuffer1 = bufferManager.Reserve(128, reservedSize1); |
| 182 | |
| 183 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 184 | CHECK(reservedSize1 == 128); |
| 185 | CHECK(packetBuffer1.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 186 | |
| 187 | unsigned int reservedSize2 = 0; |
| 188 | auto packetBuffer2 = bufferManager.Reserve(512, reservedSize2); |
| 189 | |
| 190 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 191 | CHECK(reservedSize2 == 512); |
| 192 | CHECK(packetBuffer2.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 193 | |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 194 | // NOTE: the buffer now has a surge capacity of initial size * 3 |
| 195 | // so we can grab 9 of them prior to exhaustion now |
| 196 | for (unsigned int i = 0; i < 6 ; ++i) |
| 197 | { |
| 198 | // grab another six buffers to exhaust the surge capacity |
| 199 | unsigned int reservedSize = 0; |
| 200 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 201 | |
| 202 | // Successfully reserved the third buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 203 | CHECK(reservedSize == 512); |
| 204 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 205 | } |
| 206 | |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 207 | // Cannot reserve when buffer is not available |
| 208 | unsigned int reservedSize3 = 0; |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 209 | auto reservedBuffer = bufferManager.Reserve(512, reservedSize3); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 210 | CHECK(reservedSize3 == 0); |
| 211 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 212 | } |
| 213 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 214 | TEST_CASE("BufferReleaseTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 215 | { |
| 216 | BufferManager bufferManager(2, 512); |
| 217 | unsigned int reservedSize0 = 0; |
| 218 | auto packetBuffer0 = bufferManager.Reserve(512, reservedSize0); |
| 219 | |
| 220 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 221 | CHECK(reservedSize0 == 512); |
| 222 | CHECK(packetBuffer0.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 223 | |
| 224 | unsigned int reservedSize1 = 0; |
| 225 | auto packetBuffer1 = bufferManager.Reserve(128, reservedSize1); |
| 226 | |
| 227 | // Successfully reserved the buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 228 | CHECK(reservedSize1 == 128); |
| 229 | CHECK(packetBuffer1.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 230 | |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 231 | // NOTE: now that we have a surge capacity of up to |
| 232 | // initial size * 3 we need to allocate four more |
| 233 | // buffers to exhaust the manager |
| 234 | for (unsigned int i = 0; i < 4 ; ++i) |
| 235 | { |
| 236 | // grab another six buffers to exhaust the surge capacity |
| 237 | unsigned int reservedSize = 0; |
| 238 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 239 | |
| 240 | // Successfully reserved the third buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 241 | CHECK(reservedSize == 512); |
| 242 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 243 | } |
| 244 | |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 245 | // Cannot reserve when buffer is not available |
| 246 | unsigned int reservedSize2 = 0; |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 247 | auto reservedBuffer = bufferManager.Reserve(512, reservedSize2); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 248 | CHECK(reservedSize2 == 0); |
| 249 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 250 | |
| 251 | bufferManager.Release(packetBuffer0); |
| 252 | |
| 253 | // Buffer should become available after release |
| 254 | auto packetBuffer2 = bufferManager.Reserve(128, reservedSize2); |
| 255 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 256 | CHECK(reservedSize2 == 128); |
| 257 | CHECK(packetBuffer2.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 258 | } |
| 259 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 260 | TEST_CASE("BufferCommitTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 261 | { |
| 262 | BufferManager bufferManager(2, 512); |
| 263 | unsigned int reservedSize0 = 0; |
| 264 | auto packetBuffer0 = bufferManager.Reserve(512, reservedSize0); |
| 265 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 266 | CHECK(reservedSize0 == 512); |
| 267 | CHECK(packetBuffer0.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 268 | |
| 269 | unsigned int reservedSize1 = 0; |
| 270 | auto packetBuffer1 = bufferManager.Reserve(128, reservedSize1); |
| 271 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 272 | CHECK(reservedSize1 == 128); |
| 273 | CHECK(packetBuffer1.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 274 | |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 275 | // NOTE: now that we have a surge capacity of up to |
| 276 | // initial size * 3 we need to allocate four more |
| 277 | // buffers to exhaust the manager |
| 278 | for (unsigned int i = 0; i < 4 ; ++i) |
| 279 | { |
| 280 | // grab another six buffers to exhaust the surge capacity |
| 281 | unsigned int reservedSize = 0; |
| 282 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 283 | |
| 284 | // Successfully reserved the third buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 285 | CHECK(reservedSize == 512); |
| 286 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 287 | } |
| 288 | |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 289 | unsigned int reservedSize2 = 0; |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 290 | auto reservedBuffer = bufferManager.Reserve(512, reservedSize2); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 291 | CHECK(reservedSize2 == 0); |
| 292 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 293 | |
| 294 | bufferManager.Commit(packetBuffer0, 256); |
| 295 | |
| 296 | // Buffer should become readable after commit |
| 297 | auto packetBuffer2 = bufferManager.GetReadableBuffer(); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 298 | CHECK(packetBuffer2.get()); |
| 299 | CHECK(packetBuffer2->GetSize() == 256); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 300 | |
| 301 | // Buffer not set back to available list after commit |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 302 | unsigned int reservedSize = 0; |
| 303 | reservedBuffer = bufferManager.Reserve(512, reservedSize); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 304 | CHECK(reservedSize == 0); |
| 305 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 306 | } |
| 307 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 308 | TEST_CASE("BufferMarkReadTest") |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 309 | { |
| 310 | BufferManager bufferManager(2, 512); |
| 311 | unsigned int reservedSize0 = 0; |
| 312 | auto packetBuffer0 = bufferManager.Reserve(512, reservedSize0); |
| 313 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 314 | CHECK(reservedSize0 == 512); |
| 315 | CHECK(packetBuffer0.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 316 | |
| 317 | unsigned int reservedSize1 = 0; |
| 318 | auto packetBuffer1 = bufferManager.Reserve(128, reservedSize1); |
| 319 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 320 | CHECK(reservedSize1 == 128); |
| 321 | CHECK(packetBuffer1.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 322 | |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 323 | // NOTE: now that we have a surge capacity of up to |
| 324 | // initial size * 3 we need to allocate four more |
| 325 | // buffers to exhaust the manager |
| 326 | for (unsigned int i = 0; i < 4 ; ++i) |
| 327 | { |
| 328 | // grab another six buffers to exhaust the surge capacity |
| 329 | unsigned int reservedSize = 0; |
| 330 | auto packetBuffer = bufferManager.Reserve(512, reservedSize); |
| 331 | |
| 332 | // Successfully reserved the third buffer with requested size |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 333 | CHECK(reservedSize == 512); |
| 334 | CHECK(packetBuffer.get()); |
Jim Flynn | 0204f09 | 2020-06-22 20:41:43 +0100 | [diff] [blame] | 335 | } |
| 336 | |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 337 | // Cannot reserve when buffer is not available |
| 338 | unsigned int reservedSize2 = 0; |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 339 | auto reservedBuffer = bufferManager.Reserve(512, reservedSize2); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 340 | CHECK(reservedSize2 == 0); |
| 341 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 342 | |
| 343 | bufferManager.Commit(packetBuffer0, 256); |
| 344 | |
| 345 | // Buffer should become readable after commit |
| 346 | auto packetBuffer2 = bufferManager.GetReadableBuffer(); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 347 | CHECK(packetBuffer2.get()); |
| 348 | CHECK(packetBuffer2->GetSize() == 256); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 349 | |
| 350 | // Buffer not set back to available list after commit |
Narumol Prangnawarat | 0ec068f | 2019-09-30 16:20:20 +0100 | [diff] [blame] | 351 | reservedBuffer = bufferManager.Reserve(512, reservedSize2); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 352 | CHECK(reservedSize2 == 0); |
| 353 | CHECK(!reservedBuffer.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 354 | |
| 355 | bufferManager.MarkRead(packetBuffer2); |
| 356 | |
| 357 | //Buffer should set back to available list after marked read and can be reserved |
| 358 | auto readBuffer = bufferManager.GetReadableBuffer(); |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 359 | CHECK(!readBuffer); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 360 | unsigned int reservedSize3 = 0; |
| 361 | auto packetBuffer3 = bufferManager.Reserve(56, reservedSize3); |
| 362 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 363 | CHECK(reservedSize3 == 56); |
| 364 | CHECK(packetBuffer3.get()); |
Narumol Prangnawarat | 7be47ef | 2019-09-27 18:00:11 +0100 | [diff] [blame] | 365 | } |
| 366 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 367 | TEST_CASE("ReadSwTraceMessageExceptionTest0") |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 368 | { |
| 369 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(512); |
| 370 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 371 | CHECK(packetBuffer->GetSize() == 0); |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 372 | |
| 373 | // Write zero data to the buffer |
| 374 | WriteUint32(packetBuffer, 0, 0); |
| 375 | WriteUint32(packetBuffer, 4, 0); |
| 376 | WriteUint32(packetBuffer, 8, 0); |
| 377 | WriteUint32(packetBuffer, 12, 0); |
| 378 | |
| 379 | // Commit |
| 380 | packetBuffer->Commit(16); |
| 381 | |
| 382 | unsigned int uint32_t_size = sizeof(uint32_t); |
| 383 | unsigned int offset = uint32_t_size; |
Cathal Corbett | 5aa9fd7 | 2022-02-25 15:33:28 +0000 | [diff] [blame] | 384 | CHECK_THROWS_AS(ReadSwTraceMessage(packetBuffer->GetReadableData(), offset, packetBuffer->GetSize()), |
Jim Flynn | f9db3ef | 2022-03-08 21:23:44 +0000 | [diff] [blame] | 385 | arm::pipe::ProfilingException); |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 386 | |
| 387 | } |
| 388 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 389 | TEST_CASE("ReadSwTraceMessageExceptionTest1") |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 390 | { |
| 391 | IPacketBufferPtr packetBuffer = std::make_unique<PacketBuffer>(512); |
| 392 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 393 | CHECK(packetBuffer->GetSize() == 0); |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 394 | |
| 395 | // Write data to the buffer |
| 396 | WriteUint32(packetBuffer, 0, 10); |
| 397 | WriteUint32(packetBuffer, 4, 20); |
| 398 | WriteUint32(packetBuffer, 8, 30); |
| 399 | WriteUint32(packetBuffer, 12, 40); |
| 400 | |
| 401 | // Commit |
| 402 | packetBuffer->Commit(16); |
| 403 | |
| 404 | unsigned int uint32_t_size = sizeof(uint32_t); |
| 405 | unsigned int offset = uint32_t_size; |
Cathal Corbett | 5aa9fd7 | 2022-02-25 15:33:28 +0000 | [diff] [blame] | 406 | CHECK_THROWS_AS(ReadSwTraceMessage(packetBuffer->GetReadableData(), offset, packetBuffer->GetSize()), |
Jim Flynn | f9db3ef | 2022-03-08 21:23:44 +0000 | [diff] [blame] | 407 | arm::pipe::ProfilingException); |
Kevin May | c135179 | 2020-07-28 11:29:04 +0100 | [diff] [blame] | 408 | |
| 409 | } |
| 410 | |
Sadik Armagan | 1625efc | 2021-06-10 18:24:34 +0100 | [diff] [blame] | 411 | } |