Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | #define LOG_TAG "BufferHubBufferTest" |
| 18 | |
Fan Xu | ca70b7b | 2018-10-31 13:20:12 -0700 | [diff] [blame] | 19 | #include <android/frameworks/bufferhub/1.0/IBufferClient.h> |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 20 | #include <android/frameworks/bufferhub/1.0/IBufferHub.h> |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 21 | #include <android/hardware_buffer.h> |
Fan Xu | 467e08f | 2018-11-09 15:58:51 -0800 | [diff] [blame] | 22 | #include <cutils/native_handle.h> |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 23 | #include <gtest/gtest.h> |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 24 | #include <hidl/ServiceManagement.h> |
| 25 | #include <hwbinder/IPCThreadState.h> |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 26 | #include <ui/BufferHubBuffer.h> |
| 27 | |
| 28 | namespace android { |
| 29 | |
| 30 | namespace { |
| 31 | |
| 32 | const int kWidth = 640; |
| 33 | const int kHeight = 480; |
| 34 | const int kLayerCount = 1; |
| 35 | const int kFormat = HAL_PIXEL_FORMAT_RGBA_8888; |
| 36 | const int kUsage = 0; |
| 37 | const size_t kUserMetadataSize = 0; |
| 38 | |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame^] | 39 | using dvr::BufferHubDefs::AnyClientAcquired; |
| 40 | using dvr::BufferHubDefs::AnyClientGained; |
| 41 | using dvr::BufferHubDefs::AnyClientPosted; |
Tianyu | f669f6a | 2018-10-10 15:34:32 -0700 | [diff] [blame] | 42 | using dvr::BufferHubDefs::IsBufferReleased; |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame^] | 43 | using dvr::BufferHubDefs::IsClientAcquired; |
| 44 | using dvr::BufferHubDefs::IsClientGained; |
| 45 | using dvr::BufferHubDefs::IsClientPosted; |
| 46 | using dvr::BufferHubDefs::IsClientReleased; |
Tianyu Jiang | 63dd7c3 | 2018-10-30 18:35:06 -0700 | [diff] [blame] | 47 | using dvr::BufferHubDefs::kFirstClientBitMask; |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 48 | using dvr::BufferHubDefs::kMetadataHeaderSize; |
Fan Xu | ca70b7b | 2018-10-31 13:20:12 -0700 | [diff] [blame] | 49 | using frameworks::bufferhub::V1_0::BufferHubStatus; |
| 50 | using frameworks::bufferhub::V1_0::IBufferClient; |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 51 | using frameworks::bufferhub::V1_0::IBufferHub; |
| 52 | using hardware::hidl_handle; |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 53 | using hardware::graphics::common::V1_2::HardwareBufferDescription; |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 54 | using hidl::base::V1_0::IBase; |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 55 | using pdx::LocalChannelHandle; |
| 56 | |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 57 | class BufferHubBufferTest : public ::testing::Test { |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame^] | 58 | protected: |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 59 | void SetUp() override { android::hardware::ProcessState::self()->startThreadPool(); } |
| 60 | }; |
| 61 | |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame^] | 62 | class BufferHubBufferStateTransitionTest : public BufferHubBufferTest { |
| 63 | protected: |
| 64 | void SetUp() override { |
| 65 | BufferHubBufferTest::SetUp(); |
| 66 | CreateTwoClientsOfABuffer(); |
| 67 | } |
| 68 | |
| 69 | std::unique_ptr<BufferHubBuffer> b1; |
| 70 | uint64_t b1ClientMask = 0ULL; |
| 71 | std::unique_ptr<BufferHubBuffer> b2; |
| 72 | uint64_t b2ClientMask = 0ULL; |
| 73 | |
| 74 | private: |
| 75 | // Creates b1 and b2 as the clients of the same buffer for testing. |
| 76 | void CreateTwoClientsOfABuffer(); |
| 77 | }; |
| 78 | |
| 79 | void BufferHubBufferStateTransitionTest::CreateTwoClientsOfABuffer() { |
| 80 | b1 = BufferHubBuffer::Create(kWidth, kHeight, kLayerCount, kFormat, kUsage, kUserMetadataSize); |
| 81 | b1ClientMask = b1->client_state_mask(); |
| 82 | ASSERT_NE(b1ClientMask, 0ULL); |
| 83 | auto statusOrHandle = b1->Duplicate(); |
| 84 | ASSERT_TRUE(statusOrHandle); |
| 85 | LocalChannelHandle h2 = statusOrHandle.take(); |
| 86 | b2 = BufferHubBuffer::Import(std::move(h2)); |
| 87 | b2ClientMask = b2->client_state_mask(); |
| 88 | ASSERT_NE(b2ClientMask, 0ULL); |
| 89 | ASSERT_NE(b2ClientMask, b1ClientMask); |
| 90 | } |
| 91 | |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 92 | TEST_F(BufferHubBufferTest, CreateBufferHubBufferFails) { |
| 93 | // Buffer Creation will fail: BLOB format requires height to be 1. |
| 94 | auto b1 = BufferHubBuffer::Create(kWidth, /*height=*/2, kLayerCount, |
| 95 | /*format=*/HAL_PIXEL_FORMAT_BLOB, kUsage, kUserMetadataSize); |
| 96 | |
| 97 | EXPECT_FALSE(b1->IsConnected()); |
| 98 | EXPECT_FALSE(b1->IsValid()); |
| 99 | |
| 100 | // Buffer Creation will fail: user metadata size too large. |
| 101 | auto b2 = BufferHubBuffer::Create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
| 102 | /*userMetadataSize=*/std::numeric_limits<size_t>::max()); |
| 103 | |
| 104 | EXPECT_FALSE(b2->IsConnected()); |
| 105 | EXPECT_FALSE(b2->IsValid()); |
| 106 | |
| 107 | // Buffer Creation will fail: user metadata size too large. |
| 108 | const size_t userMetadataSize = std::numeric_limits<size_t>::max() - kMetadataHeaderSize; |
| 109 | auto b3 = BufferHubBuffer::Create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
| 110 | userMetadataSize); |
| 111 | |
| 112 | EXPECT_FALSE(b3->IsConnected()); |
| 113 | EXPECT_FALSE(b3->IsValid()); |
| 114 | } |
| 115 | |
| 116 | TEST_F(BufferHubBufferTest, CreateBufferHubBuffer) { |
| 117 | auto b1 = BufferHubBuffer::Create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
| 118 | kUserMetadataSize); |
| 119 | EXPECT_TRUE(b1->IsConnected()); |
| 120 | EXPECT_TRUE(b1->IsValid()); |
| 121 | EXPECT_NE(b1->id(), 0); |
| 122 | } |
| 123 | |
| 124 | TEST_F(BufferHubBufferTest, DuplicateBufferHubBuffer) { |
| 125 | auto b1 = BufferHubBuffer::Create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
| 126 | kUserMetadataSize); |
| 127 | int id1 = b1->id(); |
Tianyu Jiang | 8f10b75 | 2018-10-30 17:24:51 -0700 | [diff] [blame] | 128 | uint64_t bufferStateMask1 = b1->client_state_mask(); |
| 129 | EXPECT_NE(bufferStateMask1, 0ULL); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 130 | EXPECT_TRUE(b1->IsValid()); |
| 131 | EXPECT_EQ(b1->user_metadata_size(), kUserMetadataSize); |
| 132 | |
| 133 | auto statusOrHandle = b1->Duplicate(); |
| 134 | EXPECT_TRUE(statusOrHandle); |
| 135 | |
| 136 | // The detached buffer should still be valid. |
| 137 | EXPECT_TRUE(b1->IsConnected()); |
| 138 | EXPECT_TRUE(b1->IsValid()); |
| 139 | |
| 140 | // Gets the channel handle for the duplicated buffer. |
| 141 | LocalChannelHandle h2 = statusOrHandle.take(); |
| 142 | EXPECT_TRUE(h2.valid()); |
| 143 | |
| 144 | std::unique_ptr<BufferHubBuffer> b2 = BufferHubBuffer::Import(std::move(h2)); |
| 145 | EXPECT_FALSE(h2.valid()); |
| 146 | ASSERT_TRUE(b2 != nullptr); |
| 147 | EXPECT_TRUE(b2->IsValid()); |
| 148 | EXPECT_EQ(b2->user_metadata_size(), kUserMetadataSize); |
| 149 | |
| 150 | int id2 = b2->id(); |
Tianyu Jiang | 8f10b75 | 2018-10-30 17:24:51 -0700 | [diff] [blame] | 151 | uint64_t bufferStateMask2 = b2->client_state_mask(); |
| 152 | EXPECT_NE(bufferStateMask2, 0ULL); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 153 | |
| 154 | // These two buffer instances are based on the same physical buffer under the |
| 155 | // hood, so they should share the same id. |
| 156 | EXPECT_EQ(id1, id2); |
Tianyu Jiang | 7e204b7 | 2018-10-26 15:39:18 -0700 | [diff] [blame] | 157 | // We use client_state_mask() to tell those two instances apart. |
Tianyu Jiang | 8f10b75 | 2018-10-30 17:24:51 -0700 | [diff] [blame] | 158 | EXPECT_NE(bufferStateMask1, bufferStateMask2); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 159 | |
Tianyu | f669f6a | 2018-10-10 15:34:32 -0700 | [diff] [blame] | 160 | // Both buffer instances should be in released state currently. |
| 161 | EXPECT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 162 | EXPECT_TRUE(IsBufferReleased(b2->buffer_state())); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 163 | |
| 164 | // TODO(b/112338294): rewrite test after migration |
| 165 | return; |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 166 | } |
| 167 | |
Fan Xu | a7422fe | 2018-11-19 15:21:32 -0800 | [diff] [blame] | 168 | TEST_F(BufferHubBufferTest, AllocateAndFreeBuffer) { |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 169 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 170 | sp<IBufferHub> bufferHub = IBufferHub::getService(); |
| 171 | ASSERT_NE(nullptr, bufferHub.get()); |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 172 | |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 173 | // Stride is an output, rfu0 and rfu1 are reserved data slot for future use. |
| 174 | AHardwareBuffer_Desc aDesc = {kWidth, kHeight, kLayerCount, kFormat, |
| 175 | kUsage, /*stride=*/0UL, /*rfu0=*/0UL, /*rfu1=*/0ULL}; |
| 176 | HardwareBufferDescription desc; |
| 177 | memcpy(&desc, &aDesc, sizeof(HardwareBufferDescription)); |
| 178 | |
Fan Xu | a7422fe | 2018-11-19 15:21:32 -0800 | [diff] [blame] | 179 | sp<IBufferClient> client; |
| 180 | BufferHubStatus ret; |
| 181 | IBufferHub::allocateBuffer_cb callback = [&](const auto& outClient, const auto& outStatus) { |
| 182 | client = outClient; |
| 183 | ret = outStatus; |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 184 | }; |
| 185 | EXPECT_TRUE(bufferHub->allocateBuffer(desc, kUserMetadataSize, callback).isOk()); |
Fan Xu | a7422fe | 2018-11-19 15:21:32 -0800 | [diff] [blame] | 186 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 187 | ASSERT_NE(nullptr, client.get()); |
| 188 | |
| 189 | EXPECT_EQ(BufferHubStatus::NO_ERROR, client->close()); |
| 190 | EXPECT_EQ(BufferHubStatus::CLIENT_CLOSED, client->close()); |
| 191 | } |
| 192 | |
| 193 | TEST_F(BufferHubBufferTest, DuplicateFreedBuffer) { |
| 194 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 195 | sp<IBufferHub> bufferHub = IBufferHub::getService(); |
| 196 | ASSERT_NE(nullptr, bufferHub.get()); |
| 197 | |
| 198 | // Stride is an output, rfu0 and rfu1 are reserved data slot for future use. |
| 199 | AHardwareBuffer_Desc aDesc = {kWidth, kHeight, kLayerCount, kFormat, |
| 200 | kUsage, /*stride=*/0UL, /*rfu0=*/0UL, /*rfu1=*/0ULL}; |
| 201 | HardwareBufferDescription desc; |
| 202 | memcpy(&desc, &aDesc, sizeof(HardwareBufferDescription)); |
| 203 | |
| 204 | sp<IBufferClient> client; |
| 205 | BufferHubStatus ret; |
| 206 | IBufferHub::allocateBuffer_cb callback = [&](const auto& outClient, const auto& outStatus) { |
| 207 | client = outClient; |
| 208 | ret = outStatus; |
| 209 | }; |
| 210 | EXPECT_TRUE(bufferHub->allocateBuffer(desc, kUserMetadataSize, callback).isOk()); |
| 211 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 212 | ASSERT_NE(nullptr, client.get()); |
| 213 | |
| 214 | EXPECT_EQ(BufferHubStatus::NO_ERROR, client->close()); |
| 215 | |
| 216 | hidl_handle token; |
| 217 | IBufferClient::duplicate_cb dup_cb = [&](const auto& outToken, const auto& status) { |
| 218 | token = outToken; |
| 219 | ret = status; |
| 220 | }; |
| 221 | EXPECT_TRUE(client->duplicate(dup_cb).isOk()); |
| 222 | EXPECT_EQ(ret, BufferHubStatus::CLIENT_CLOSED); |
| 223 | EXPECT_EQ(token.getNativeHandle(), nullptr); |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 224 | } |
| 225 | |
Fan Xu | 467e08f | 2018-11-09 15:58:51 -0800 | [diff] [blame] | 226 | TEST_F(BufferHubBufferTest, DuplicateAndImportBuffer) { |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 227 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 228 | sp<IBufferHub> bufferhub = IBufferHub::getService(); |
| 229 | ASSERT_NE(nullptr, bufferhub.get()); |
| 230 | |
| 231 | // Stride is an output, rfu0 and rfu1 are reserved data slot for future use. |
| 232 | AHardwareBuffer_Desc aDesc = {kWidth, kHeight, kLayerCount, kFormat, |
| 233 | kUsage, /*stride=*/0UL, /*rfu0=*/0UL, /*rfu1=*/0ULL}; |
| 234 | HardwareBufferDescription desc; |
| 235 | memcpy(&desc, &aDesc, sizeof(HardwareBufferDescription)); |
| 236 | |
| 237 | sp<IBufferClient> client; |
Fan Xu | c32c172 | 2018-11-16 14:38:29 -0800 | [diff] [blame] | 238 | BufferHubStatus ret; |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 239 | IBufferHub::allocateBuffer_cb alloc_cb = [&](const auto& outClient, const auto& status) { |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 240 | client = outClient; |
Fan Xu | c32c172 | 2018-11-16 14:38:29 -0800 | [diff] [blame] | 241 | ret = status; |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 242 | }; |
| 243 | ASSERT_TRUE(bufferhub->allocateBuffer(desc, kUserMetadataSize, alloc_cb).isOk()); |
Fan Xu | c32c172 | 2018-11-16 14:38:29 -0800 | [diff] [blame] | 244 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 245 | ASSERT_NE(nullptr, client.get()); |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 246 | |
Fan Xu | c32c172 | 2018-11-16 14:38:29 -0800 | [diff] [blame] | 247 | hidl_handle token; |
| 248 | IBufferClient::duplicate_cb dup_cb = [&](const auto& outToken, const auto& status) { |
| 249 | token = outToken; |
| 250 | ret = status; |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 251 | }; |
Fan Xu | 467e08f | 2018-11-09 15:58:51 -0800 | [diff] [blame] | 252 | ASSERT_TRUE(client->duplicate(dup_cb).isOk()); |
Fan Xu | c32c172 | 2018-11-16 14:38:29 -0800 | [diff] [blame] | 253 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 254 | ASSERT_NE(token.getNativeHandle(), nullptr); |
| 255 | EXPECT_EQ(token->numInts, 1); |
| 256 | EXPECT_EQ(token->numFds, 0); |
Fan Xu | 467e08f | 2018-11-09 15:58:51 -0800 | [diff] [blame] | 257 | |
| 258 | sp<IBufferClient> client2; |
| 259 | IBufferHub::importBuffer_cb import_cb = [&](const auto& outClient, const auto& status) { |
| 260 | ret = status; |
| 261 | client2 = outClient; |
| 262 | }; |
| 263 | ASSERT_TRUE(bufferhub->importBuffer(token, import_cb).isOk()); |
| 264 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 265 | EXPECT_NE(nullptr, client2.get()); |
| 266 | // TODO(b/116681016): once BufferNode.id() is exposed via BufferHubBuffer, check origin.id = |
| 267 | // improted.id here. |
| 268 | } |
| 269 | |
| 270 | // nullptr must not crash the service |
| 271 | TEST_F(BufferHubBufferTest, ImportNullToken) { |
| 272 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 273 | sp<IBufferHub> bufferhub = IBufferHub::getService(); |
| 274 | ASSERT_NE(nullptr, bufferhub.get()); |
| 275 | |
| 276 | hidl_handle nullToken; |
| 277 | sp<IBufferClient> client; |
| 278 | BufferHubStatus ret; |
| 279 | IBufferHub::importBuffer_cb import_cb = [&](const auto& outClient, const auto& status) { |
| 280 | client = outClient; |
| 281 | ret = status; |
| 282 | }; |
| 283 | ASSERT_TRUE(bufferhub->importBuffer(nullToken, import_cb).isOk()); |
| 284 | EXPECT_EQ(ret, BufferHubStatus::INVALID_TOKEN); |
| 285 | EXPECT_EQ(nullptr, client.get()); |
| 286 | } |
| 287 | |
| 288 | // This test has a very little chance to fail (number of existing tokens / 2 ^ 32) |
| 289 | TEST_F(BufferHubBufferTest, ImportInvalidToken) { |
| 290 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 291 | sp<IBufferHub> bufferhub = IBufferHub::getService(); |
| 292 | ASSERT_NE(nullptr, bufferhub.get()); |
| 293 | |
| 294 | native_handle_t* tokenHandle = native_handle_create(/*numFds=*/0, /*numInts=*/1); |
| 295 | tokenHandle->data[0] = 0; |
| 296 | |
| 297 | hidl_handle invalidToken(tokenHandle); |
| 298 | sp<IBufferClient> client; |
| 299 | BufferHubStatus ret; |
| 300 | IBufferHub::importBuffer_cb import_cb = [&](const auto& outClient, const auto& status) { |
| 301 | client = outClient; |
| 302 | ret = status; |
| 303 | }; |
| 304 | ASSERT_TRUE(bufferhub->importBuffer(invalidToken, import_cb).isOk()); |
| 305 | EXPECT_EQ(ret, BufferHubStatus::INVALID_TOKEN); |
| 306 | EXPECT_EQ(nullptr, client.get()); |
| 307 | |
| 308 | native_handle_delete(tokenHandle); |
Fan Xu | 18d90ea | 2018-11-06 15:46:44 -0800 | [diff] [blame] | 309 | } |
| 310 | |
Fan Xu | a7422fe | 2018-11-19 15:21:32 -0800 | [diff] [blame] | 311 | TEST_F(BufferHubBufferTest, ImportFreedBuffer) { |
| 312 | // TODO(b/116681016): directly test on BufferHubBuffer instead of the service. |
| 313 | sp<IBufferHub> bufferhub = IBufferHub::getService(); |
| 314 | ASSERT_NE(nullptr, bufferhub.get()); |
| 315 | |
| 316 | // Stride is an output, rfu0 and rfu1 are reserved data slot for future use. |
| 317 | AHardwareBuffer_Desc aDesc = {kWidth, kHeight, kLayerCount, kFormat, |
| 318 | kUsage, /*stride=*/0UL, /*rfu0=*/0UL, /*rfu1=*/0ULL}; |
| 319 | HardwareBufferDescription desc; |
| 320 | memcpy(&desc, &aDesc, sizeof(HardwareBufferDescription)); |
| 321 | |
| 322 | sp<IBufferClient> client; |
| 323 | BufferHubStatus ret; |
| 324 | IBufferHub::allocateBuffer_cb alloc_cb = [&](const auto& outClient, const auto& status) { |
| 325 | client = outClient; |
| 326 | ret = status; |
| 327 | }; |
| 328 | ASSERT_TRUE(bufferhub->allocateBuffer(desc, kUserMetadataSize, alloc_cb).isOk()); |
| 329 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 330 | ASSERT_NE(nullptr, client.get()); |
| 331 | |
| 332 | hidl_handle token; |
| 333 | IBufferClient::duplicate_cb dup_cb = [&](const auto& outToken, const auto& status) { |
| 334 | token = outToken; |
| 335 | ret = status; |
| 336 | }; |
| 337 | ASSERT_TRUE(client->duplicate(dup_cb).isOk()); |
| 338 | EXPECT_EQ(ret, BufferHubStatus::NO_ERROR); |
| 339 | ASSERT_NE(token.getNativeHandle(), nullptr); |
| 340 | EXPECT_EQ(token->numInts, 1); |
| 341 | EXPECT_EQ(token->numFds, 0); |
| 342 | |
| 343 | // Close the client. Now the token should be invalid. |
| 344 | client->close(); |
| 345 | |
| 346 | sp<IBufferClient> client2; |
| 347 | IBufferHub::importBuffer_cb import_cb = [&](const auto& outClient, const auto& status) { |
| 348 | client2 = outClient; |
| 349 | ret = status; |
| 350 | }; |
| 351 | EXPECT_TRUE(bufferhub->importBuffer(token, import_cb).isOk()); |
| 352 | EXPECT_EQ(ret, BufferHubStatus::INVALID_TOKEN); |
| 353 | EXPECT_EQ(nullptr, client2.get()); |
| 354 | } |
| 355 | |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame^] | 356 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromReleasedState) { |
| 357 | ASSERT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 358 | |
| 359 | // Successful gaining the buffer should change the buffer state bit of b1 to |
| 360 | // gained state, other client state bits to released state. |
| 361 | EXPECT_EQ(b1->Gain(), 0); |
| 362 | EXPECT_TRUE(IsClientGained(b1->buffer_state(), b1ClientMask)); |
| 363 | } |
| 364 | |
| 365 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromGainedState) { |
| 366 | ASSERT_EQ(b1->Gain(), 0); |
| 367 | auto current_buffer_state = b1->buffer_state(); |
| 368 | ASSERT_TRUE(IsClientGained(current_buffer_state, b1ClientMask)); |
| 369 | |
| 370 | // Gaining from gained state by the same client should not return error. |
| 371 | EXPECT_EQ(b1->Gain(), 0); |
| 372 | |
| 373 | // Gaining from gained state by another client should return error. |
| 374 | EXPECT_EQ(b2->Gain(), -EBUSY); |
| 375 | } |
| 376 | |
| 377 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromAcquiredState) { |
| 378 | ASSERT_EQ(b1->Gain(), 0); |
| 379 | ASSERT_EQ(b1->Post(), 0); |
| 380 | ASSERT_EQ(b2->Acquire(), 0); |
| 381 | ASSERT_TRUE(AnyClientAcquired(b1->buffer_state())); |
| 382 | |
| 383 | // Gaining from acquired state should fail. |
| 384 | EXPECT_EQ(b1->Gain(), -EBUSY); |
| 385 | EXPECT_EQ(b2->Gain(), -EBUSY); |
| 386 | } |
| 387 | |
| 388 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromOtherClientInPostedState) { |
| 389 | ASSERT_EQ(b1->Gain(), 0); |
| 390 | ASSERT_EQ(b1->Post(), 0); |
| 391 | ASSERT_TRUE(AnyClientPosted(b1->buffer_state())); |
| 392 | |
| 393 | // Gaining a buffer who has other posted client should succeed. |
| 394 | EXPECT_EQ(b1->Gain(), 0); |
| 395 | } |
| 396 | |
| 397 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromSelfInPostedState) { |
| 398 | ASSERT_EQ(b1->Gain(), 0); |
| 399 | ASSERT_EQ(b1->Post(), 0); |
| 400 | ASSERT_TRUE(AnyClientPosted(b1->buffer_state())); |
| 401 | |
| 402 | // A posted client should be able to gain the buffer when there is no other clients in |
| 403 | // acquired state. |
| 404 | EXPECT_EQ(b2->Gain(), 0); |
| 405 | } |
| 406 | |
| 407 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromOtherInGainedState) { |
| 408 | ASSERT_EQ(b1->Gain(), 0); |
| 409 | ASSERT_TRUE(IsClientGained(b1->buffer_state(), b1ClientMask)); |
| 410 | |
| 411 | EXPECT_EQ(b2->Post(), -EBUSY); |
| 412 | } |
| 413 | |
| 414 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromSelfInGainedState) { |
| 415 | ASSERT_EQ(b1->Gain(), 0); |
| 416 | ASSERT_TRUE(IsClientGained(b1->buffer_state(), b1ClientMask)); |
| 417 | |
| 418 | EXPECT_EQ(b1->Post(), 0); |
| 419 | auto current_buffer_state = b1->buffer_state(); |
| 420 | EXPECT_TRUE(IsClientReleased(current_buffer_state, b1ClientMask)); |
| 421 | EXPECT_TRUE(IsClientPosted(current_buffer_state, b2ClientMask)); |
| 422 | } |
| 423 | |
| 424 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromPostedState) { |
| 425 | ASSERT_EQ(b1->Gain(), 0); |
| 426 | ASSERT_EQ(b1->Post(), 0); |
| 427 | ASSERT_TRUE(AnyClientPosted(b1->buffer_state())); |
| 428 | |
| 429 | // Post from posted state should fail. |
| 430 | EXPECT_EQ(b1->Post(), -EBUSY); |
| 431 | EXPECT_EQ(b2->Post(), -EBUSY); |
| 432 | } |
| 433 | |
| 434 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromAcquiredState) { |
| 435 | ASSERT_EQ(b1->Gain(), 0); |
| 436 | ASSERT_EQ(b1->Post(), 0); |
| 437 | ASSERT_EQ(b2->Acquire(), 0); |
| 438 | ASSERT_TRUE(AnyClientAcquired(b1->buffer_state())); |
| 439 | |
| 440 | // Posting from acquired state should fail. |
| 441 | EXPECT_EQ(b1->Post(), -EBUSY); |
| 442 | EXPECT_EQ(b2->Post(), -EBUSY); |
| 443 | } |
| 444 | |
| 445 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromReleasedState) { |
| 446 | ASSERT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 447 | |
| 448 | // Posting from released state should fail. |
| 449 | EXPECT_EQ(b1->Post(), -EBUSY); |
| 450 | EXPECT_EQ(b2->Post(), -EBUSY); |
| 451 | } |
| 452 | |
| 453 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromSelfInPostedState) { |
| 454 | ASSERT_EQ(b1->Gain(), 0); |
| 455 | ASSERT_EQ(b1->Post(), 0); |
| 456 | ASSERT_TRUE(IsClientPosted(b1->buffer_state(), b2ClientMask)); |
| 457 | |
| 458 | // Acquire from posted state should pass. |
| 459 | EXPECT_EQ(b2->Acquire(), 0); |
| 460 | } |
| 461 | |
| 462 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromOtherInPostedState) { |
| 463 | ASSERT_EQ(b1->Gain(), 0); |
| 464 | ASSERT_EQ(b1->Post(), 0); |
| 465 | ASSERT_TRUE(IsClientPosted(b1->buffer_state(), b2ClientMask)); |
| 466 | |
| 467 | // Acquire from released state should fail, although there are other clients |
| 468 | // in posted state. |
| 469 | EXPECT_EQ(b1->Acquire(), -EBUSY); |
| 470 | } |
| 471 | |
| 472 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromSelfInAcquiredState) { |
| 473 | ASSERT_EQ(b1->Gain(), 0); |
| 474 | ASSERT_EQ(b1->Post(), 0); |
| 475 | ASSERT_EQ(b2->Acquire(), 0); |
| 476 | auto current_buffer_state = b1->buffer_state(); |
| 477 | ASSERT_TRUE(IsClientAcquired(current_buffer_state, b2ClientMask)); |
| 478 | |
| 479 | // Acquiring from acquired state by the same client should not error out. |
| 480 | EXPECT_EQ(b2->Acquire(), 0); |
| 481 | } |
| 482 | |
| 483 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromReleasedState) { |
| 484 | ASSERT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 485 | |
| 486 | // Acquiring form released state should fail. |
| 487 | EXPECT_EQ(b1->Acquire(), -EBUSY); |
| 488 | EXPECT_EQ(b2->Acquire(), -EBUSY); |
| 489 | } |
| 490 | |
| 491 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromGainedState) { |
| 492 | ASSERT_EQ(b1->Gain(), 0); |
| 493 | ASSERT_TRUE(AnyClientGained(b1->buffer_state())); |
| 494 | |
| 495 | // Acquiring from gained state should fail. |
| 496 | EXPECT_EQ(b1->Acquire(), -EBUSY); |
| 497 | EXPECT_EQ(b2->Acquire(), -EBUSY); |
| 498 | } |
| 499 | |
| 500 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInReleasedState) { |
| 501 | ASSERT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 502 | |
| 503 | EXPECT_EQ(b1->Release(), 0); |
| 504 | } |
| 505 | |
| 506 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInGainedState) { |
| 507 | ASSERT_TRUE(IsBufferReleased(b1->buffer_state())); |
| 508 | ASSERT_EQ(b1->Gain(), 0); |
| 509 | ASSERT_TRUE(AnyClientGained(b1->buffer_state())); |
| 510 | |
| 511 | EXPECT_EQ(b1->Release(), 0); |
| 512 | } |
| 513 | |
| 514 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInPostedState) { |
| 515 | ASSERT_EQ(b1->Gain(), 0); |
| 516 | ASSERT_EQ(b1->Post(), 0); |
| 517 | ASSERT_TRUE(AnyClientPosted(b1->buffer_state())); |
| 518 | |
| 519 | EXPECT_EQ(b2->Release(), 0); |
| 520 | } |
| 521 | |
| 522 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInAcquiredState) { |
| 523 | ASSERT_EQ(b1->Gain(), 0); |
| 524 | ASSERT_EQ(b1->Post(), 0); |
| 525 | ASSERT_EQ(b2->Acquire(), 0); |
| 526 | ASSERT_TRUE(AnyClientAcquired(b1->buffer_state())); |
| 527 | |
| 528 | EXPECT_EQ(b2->Release(), 0); |
| 529 | } |
| 530 | |
Fan Xu | ca70b7b | 2018-10-31 13:20:12 -0700 | [diff] [blame] | 531 | } // namespace |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 532 | } // namespace android |