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