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 | |
Tianyu Jiang | ce23274 | 2019-01-29 10:17:31 -0800 | [diff] [blame] | 19 | #include <errno.h> |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 20 | #include <sys/epoll.h> |
| 21 | |
Fan Xu | 93c9490 | 2018-11-01 12:22:05 -0700 | [diff] [blame] | 22 | #include <android/hardware_buffer.h> |
Fan Xu | 467e08f | 2018-11-09 15:58:51 -0800 | [diff] [blame] | 23 | #include <cutils/native_handle.h> |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 24 | #include <gmock/gmock.h> |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 25 | #include <gtest/gtest.h> |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 26 | #include <hidl/ServiceManagement.h> |
| 27 | #include <hwbinder/IPCThreadState.h> |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 28 | #include <ui/BufferHubBuffer.h> |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 29 | #include <ui/BufferHubEventFd.h> |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 30 | |
| 31 | namespace android { |
| 32 | |
| 33 | namespace { |
| 34 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 35 | using ::android::BufferHubDefs::isAnyClientAcquired; |
| 36 | using ::android::BufferHubDefs::isAnyClientGained; |
| 37 | using ::android::BufferHubDefs::isAnyClientPosted; |
| 38 | using ::android::BufferHubDefs::isClientAcquired; |
| 39 | using ::android::BufferHubDefs::isClientGained; |
| 40 | using ::android::BufferHubDefs::isClientPosted; |
| 41 | using ::android::BufferHubDefs::isClientReleased; |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 42 | using ::android::BufferHubDefs::kMetadataHeaderSize; |
| 43 | using ::testing::IsNull; |
| 44 | using ::testing::NotNull; |
| 45 | |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 46 | const int kWidth = 640; |
| 47 | const int kHeight = 480; |
| 48 | const int kLayerCount = 1; |
| 49 | const int kFormat = HAL_PIXEL_FORMAT_RGBA_8888; |
| 50 | const int kUsage = 0; |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 51 | const AHardwareBuffer_Desc kDesc = {kWidth, kHeight, kLayerCount, kFormat, |
| 52 | kUsage, /*stride=*/0UL, /*rfu0=*/0UL, /*rfu1=*/0ULL}; |
| 53 | const size_t kUserMetadataSize = 1; |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 54 | |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 55 | class BufferHubBufferTest : public ::testing::Test { |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 56 | protected: |
Jiwen 'Steve' Cai | d9f2abe | 2018-10-20 17:03:13 -0700 | [diff] [blame] | 57 | void SetUp() override { android::hardware::ProcessState::self()->startThreadPool(); } |
| 58 | }; |
| 59 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 60 | bool cmpAHardwareBufferDesc(const AHardwareBuffer_Desc& desc, const AHardwareBuffer_Desc& other) { |
| 61 | // Not comparing stride because it's unknown before allocation |
| 62 | return desc.format == other.format && desc.height == other.height && |
| 63 | desc.layers == other.layers && desc.usage == other.usage && desc.width == other.width; |
| 64 | } |
| 65 | |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 66 | class BufferHubBufferStateTransitionTest : public BufferHubBufferTest { |
| 67 | protected: |
| 68 | void SetUp() override { |
| 69 | BufferHubBufferTest::SetUp(); |
| 70 | CreateTwoClientsOfABuffer(); |
| 71 | } |
| 72 | |
| 73 | std::unique_ptr<BufferHubBuffer> b1; |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 74 | uint32_t b1ClientMask = 0U; |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 75 | std::unique_ptr<BufferHubBuffer> b2; |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 76 | uint32_t b2ClientMask = 0U; |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 77 | |
| 78 | private: |
| 79 | // Creates b1 and b2 as the clients of the same buffer for testing. |
| 80 | void CreateTwoClientsOfABuffer(); |
| 81 | }; |
| 82 | |
| 83 | void BufferHubBufferStateTransitionTest::CreateTwoClientsOfABuffer() { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 84 | b1 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, kUserMetadataSize); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 85 | ASSERT_THAT(b1, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 86 | b1ClientMask = b1->clientStateMask(); |
Tianyu Jiang | a99f911 | 2018-12-13 18:23:07 -0800 | [diff] [blame] | 87 | ASSERT_NE(b1ClientMask, 0U); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 88 | |
Tianyu Jiang | 8a7e765 | 2019-02-11 13:41:20 -0800 | [diff] [blame] | 89 | sp<NativeHandle> token = b1->duplicate(); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 90 | ASSERT_THAT(token, NotNull()); |
| 91 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 92 | b2 = BufferHubBuffer::import(token); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 93 | ASSERT_THAT(b2, NotNull()); |
| 94 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 95 | b2ClientMask = b2->clientStateMask(); |
Tianyu Jiang | a99f911 | 2018-12-13 18:23:07 -0800 | [diff] [blame] | 96 | ASSERT_NE(b2ClientMask, 0U); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 97 | ASSERT_NE(b2ClientMask, b1ClientMask); |
| 98 | } |
| 99 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 100 | TEST_F(BufferHubBufferTest, CreateBufferFails) { |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 101 | // Buffer Creation will fail: BLOB format requires height to be 1. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 102 | auto b1 = BufferHubBuffer::create(kWidth, /*height=*/2, kLayerCount, |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 103 | /*format=*/HAL_PIXEL_FORMAT_BLOB, kUsage, kUserMetadataSize); |
| 104 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 105 | EXPECT_THAT(b1, IsNull()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 106 | |
| 107 | // Buffer Creation will fail: user metadata size too large. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 108 | auto b2 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 109 | /*userMetadataSize=*/std::numeric_limits<size_t>::max()); |
| 110 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 111 | EXPECT_THAT(b2, IsNull()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 112 | |
| 113 | // Buffer Creation will fail: user metadata size too large. |
| 114 | const size_t userMetadataSize = std::numeric_limits<size_t>::max() - kMetadataHeaderSize; |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 115 | auto b3 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 116 | userMetadataSize); |
| 117 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 118 | EXPECT_THAT(b3, IsNull()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 119 | } |
| 120 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 121 | TEST_F(BufferHubBufferTest, CreateBuffer) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 122 | auto b1 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 123 | kUserMetadataSize); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 124 | ASSERT_THAT(b1, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 125 | EXPECT_TRUE(b1->isConnected()); |
| 126 | EXPECT_TRUE(b1->isValid()); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 127 | EXPECT_TRUE(cmpAHardwareBufferDesc(b1->desc(), kDesc)); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 128 | EXPECT_EQ(b1->userMetadataSize(), kUserMetadataSize); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 129 | } |
| 130 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 131 | TEST_F(BufferHubBufferTest, DuplicateAndImportBuffer) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 132 | auto b1 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 133 | kUserMetadataSize); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 134 | ASSERT_THAT(b1, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 135 | EXPECT_TRUE(b1->isValid()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 136 | |
Tianyu Jiang | 8a7e765 | 2019-02-11 13:41:20 -0800 | [diff] [blame] | 137 | sp<NativeHandle> token = b1->duplicate(); |
| 138 | ASSERT_THAT(token, NotNull()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 139 | |
| 140 | // The detached buffer should still be valid. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 141 | EXPECT_TRUE(b1->isConnected()); |
| 142 | EXPECT_TRUE(b1->isValid()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 143 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 144 | std::unique_ptr<BufferHubBuffer> b2 = BufferHubBuffer::import(token); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 145 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 146 | ASSERT_THAT(b2, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 147 | EXPECT_TRUE(b2->isValid()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 148 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 149 | EXPECT_TRUE(cmpAHardwareBufferDesc(b1->desc(), b2->desc())); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 150 | EXPECT_EQ(b1->userMetadataSize(), b2->userMetadataSize()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 151 | |
| 152 | // These two buffer instances are based on the same physical buffer under the |
| 153 | // hood, so they should share the same id. |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 154 | EXPECT_EQ(b1->id(), b2->id()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 155 | // We use clientStateMask() to tell those two instances apart. |
| 156 | EXPECT_NE(b1->clientStateMask(), b2->clientStateMask()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 157 | |
Tianyu | f669f6a | 2018-10-10 15:34:32 -0700 | [diff] [blame] | 158 | // Both buffer instances should be in released state currently. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 159 | EXPECT_TRUE(b1->isReleased()); |
| 160 | EXPECT_TRUE(b2->isReleased()); |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 161 | |
| 162 | // The event fd should behave like duped event fds. |
| 163 | const BufferHubEventFd& eventFd1 = b1->eventFd(); |
Tianyu Jiang | ce23274 | 2019-01-29 10:17:31 -0800 | [diff] [blame] | 164 | ASSERT_GE(eventFd1.get(), 0); |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 165 | const BufferHubEventFd& eventFd2 = b2->eventFd(); |
Tianyu Jiang | ce23274 | 2019-01-29 10:17:31 -0800 | [diff] [blame] | 166 | ASSERT_GE(eventFd2.get(), 0); |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 167 | |
| 168 | base::unique_fd epollFd(epoll_create(64)); |
| 169 | ASSERT_GE(epollFd.get(), 0); |
| 170 | |
| 171 | // Add eventFd1 to epoll set, and signal eventFd2. |
| 172 | epoll_event e = {.events = EPOLLIN | EPOLLET, .data = {.u32 = 0}}; |
Tianyu Jiang | ce23274 | 2019-01-29 10:17:31 -0800 | [diff] [blame] | 173 | ASSERT_EQ(epoll_ctl(epollFd.get(), EPOLL_CTL_ADD, eventFd1.get(), &e), 0) << strerror(errno); |
Fan Xu | 5cf47bc | 2019-01-15 15:02:15 -0800 | [diff] [blame] | 174 | |
| 175 | std::array<epoll_event, 1> events; |
| 176 | EXPECT_EQ(epoll_wait(epollFd.get(), events.data(), events.size(), 0), 0); |
| 177 | |
| 178 | eventFd2.signal(); |
| 179 | EXPECT_EQ(epoll_wait(epollFd.get(), events.data(), events.size(), 0), 1); |
| 180 | |
| 181 | // The epoll fd is edge triggered, so it only responds to the eventFd once. |
| 182 | EXPECT_EQ(epoll_wait(epollFd.get(), events.data(), events.size(), 0), 0); |
| 183 | |
| 184 | eventFd2.signal(); |
| 185 | eventFd2.clear(); |
| 186 | EXPECT_EQ(epoll_wait(epollFd.get(), events.data(), events.size(), 0), 0); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 187 | } |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 188 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 189 | TEST_F(BufferHubBufferTest, ImportFreedBuffer) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 190 | auto b1 = BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 191 | kUserMetadataSize); |
| 192 | ASSERT_THAT(b1, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 193 | EXPECT_TRUE(b1->isValid()); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 194 | |
Tianyu Jiang | 8a7e765 | 2019-02-11 13:41:20 -0800 | [diff] [blame] | 195 | sp<NativeHandle> token = b1->duplicate(); |
| 196 | ASSERT_THAT(token, NotNull()); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 197 | |
| 198 | // Explicitly destroy b1. Backend buffer should be freed and token becomes invalid |
| 199 | b1.reset(); |
| 200 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 201 | std::unique_ptr<BufferHubBuffer> b2 = BufferHubBuffer::import(token); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 202 | |
| 203 | // Import should fail with INVALID_TOKEN |
| 204 | EXPECT_THAT(b2, IsNull()); |
| 205 | } |
| 206 | |
| 207 | // nullptr must not crash the service |
| 208 | TEST_F(BufferHubBufferTest, ImportNullToken) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 209 | auto b1 = BufferHubBuffer::import(nullptr); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 210 | EXPECT_THAT(b1, IsNull()); |
| 211 | } |
| 212 | |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 213 | TEST_F(BufferHubBufferTest, ImportInvalidToken) { |
| 214 | native_handle_t* token = native_handle_create(/*numFds=*/0, /*numInts=*/1); |
| 215 | token->data[0] = 0; |
| 216 | |
Tianyu Jiang | 750c887 | 2019-02-14 11:08:08 -0800 | [diff] [blame^] | 217 | sp<NativeHandle> tokenHandle = NativeHandle::create(token, /*ownHandle=*/true); |
| 218 | auto b1 = BufferHubBuffer::import(tokenHandle); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 219 | |
| 220 | EXPECT_THAT(b1, IsNull()); |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 221 | } |
| 222 | |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 223 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromReleasedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 224 | ASSERT_TRUE(b1->isReleased()); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 225 | |
| 226 | // Successful gaining the buffer should change the buffer state bit of b1 to |
| 227 | // gained state, other client state bits to released state. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 228 | EXPECT_EQ(b1->gain(), 0); |
| 229 | EXPECT_TRUE(isClientGained(b1->bufferState(), b1ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 230 | } |
| 231 | |
| 232 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromGainedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 233 | ASSERT_EQ(b1->gain(), 0); |
| 234 | auto currentBufferState = b1->bufferState(); |
| 235 | ASSERT_TRUE(isClientGained(currentBufferState, b1ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 236 | |
| 237 | // Gaining from gained state by the same client should not return error. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 238 | EXPECT_EQ(b1->gain(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 239 | |
| 240 | // Gaining from gained state by another client should return error. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 241 | EXPECT_EQ(b2->gain(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 242 | } |
| 243 | |
| 244 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromAcquiredState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 245 | ASSERT_EQ(b1->gain(), 0); |
| 246 | ASSERT_EQ(b1->post(), 0); |
| 247 | ASSERT_EQ(b2->acquire(), 0); |
| 248 | ASSERT_TRUE(isAnyClientAcquired(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 249 | |
| 250 | // Gaining from acquired state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 251 | EXPECT_EQ(b1->gain(), -EBUSY); |
| 252 | EXPECT_EQ(b2->gain(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 253 | } |
| 254 | |
| 255 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromOtherClientInPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 256 | ASSERT_EQ(b1->gain(), 0); |
| 257 | ASSERT_EQ(b1->post(), 0); |
| 258 | ASSERT_TRUE(isAnyClientPosted(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 259 | |
| 260 | // Gaining a buffer who has other posted client should succeed. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 261 | EXPECT_EQ(b1->gain(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 262 | } |
| 263 | |
| 264 | TEST_F(BufferHubBufferStateTransitionTest, GainBuffer_fromSelfInPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 265 | ASSERT_EQ(b1->gain(), 0); |
| 266 | ASSERT_EQ(b1->post(), 0); |
| 267 | ASSERT_TRUE(isAnyClientPosted(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 268 | |
| 269 | // A posted client should be able to gain the buffer when there is no other clients in |
| 270 | // acquired state. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 271 | EXPECT_EQ(b2->gain(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 272 | } |
| 273 | |
| 274 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromOtherInGainedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 275 | ASSERT_EQ(b1->gain(), 0); |
| 276 | ASSERT_TRUE(isClientGained(b1->bufferState(), b1ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 277 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 278 | EXPECT_EQ(b2->post(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 279 | } |
| 280 | |
| 281 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromSelfInGainedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 282 | ASSERT_EQ(b1->gain(), 0); |
| 283 | ASSERT_TRUE(isClientGained(b1->bufferState(), b1ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 284 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 285 | EXPECT_EQ(b1->post(), 0); |
| 286 | auto currentBufferState = b1->bufferState(); |
| 287 | EXPECT_TRUE(isClientReleased(currentBufferState, b1ClientMask)); |
| 288 | EXPECT_TRUE(isClientPosted(currentBufferState, b2ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 289 | } |
| 290 | |
| 291 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 292 | ASSERT_EQ(b1->gain(), 0); |
| 293 | ASSERT_EQ(b1->post(), 0); |
| 294 | ASSERT_TRUE(isAnyClientPosted(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 295 | |
| 296 | // Post from posted state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 297 | EXPECT_EQ(b1->post(), -EBUSY); |
| 298 | EXPECT_EQ(b2->post(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 299 | } |
| 300 | |
| 301 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromAcquiredState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 302 | ASSERT_EQ(b1->gain(), 0); |
| 303 | ASSERT_EQ(b1->post(), 0); |
| 304 | ASSERT_EQ(b2->acquire(), 0); |
| 305 | ASSERT_TRUE(isAnyClientAcquired(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 306 | |
| 307 | // Posting from acquired state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 308 | EXPECT_EQ(b1->post(), -EBUSY); |
| 309 | EXPECT_EQ(b2->post(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 310 | } |
| 311 | |
| 312 | TEST_F(BufferHubBufferStateTransitionTest, PostBuffer_fromReleasedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 313 | ASSERT_TRUE(b1->isReleased()); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 314 | |
| 315 | // Posting from released state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 316 | EXPECT_EQ(b1->post(), -EBUSY); |
| 317 | EXPECT_EQ(b2->post(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 318 | } |
| 319 | |
| 320 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromSelfInPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 321 | ASSERT_EQ(b1->gain(), 0); |
| 322 | ASSERT_EQ(b1->post(), 0); |
| 323 | ASSERT_TRUE(isClientPosted(b1->bufferState(), b2ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 324 | |
| 325 | // Acquire from posted state should pass. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 326 | EXPECT_EQ(b2->acquire(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 327 | } |
| 328 | |
| 329 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromOtherInPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 330 | ASSERT_EQ(b1->gain(), 0); |
| 331 | ASSERT_EQ(b1->post(), 0); |
| 332 | ASSERT_TRUE(isClientPosted(b1->bufferState(), b2ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 333 | |
| 334 | // Acquire from released state should fail, although there are other clients |
| 335 | // in posted state. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 336 | EXPECT_EQ(b1->acquire(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 337 | } |
| 338 | |
| 339 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromSelfInAcquiredState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 340 | ASSERT_EQ(b1->gain(), 0); |
| 341 | ASSERT_EQ(b1->post(), 0); |
| 342 | ASSERT_EQ(b2->acquire(), 0); |
| 343 | auto currentBufferState = b1->bufferState(); |
| 344 | ASSERT_TRUE(isClientAcquired(currentBufferState, b2ClientMask)); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 345 | |
| 346 | // Acquiring from acquired state by the same client should not error out. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 347 | EXPECT_EQ(b2->acquire(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 348 | } |
| 349 | |
| 350 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromReleasedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 351 | ASSERT_TRUE(b1->isReleased()); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 352 | |
| 353 | // Acquiring form released state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 354 | EXPECT_EQ(b1->acquire(), -EBUSY); |
| 355 | EXPECT_EQ(b2->acquire(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 356 | } |
| 357 | |
| 358 | TEST_F(BufferHubBufferStateTransitionTest, AcquireBuffer_fromGainedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 359 | ASSERT_EQ(b1->gain(), 0); |
| 360 | ASSERT_TRUE(isAnyClientGained(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 361 | |
| 362 | // Acquiring from gained state should fail. |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 363 | EXPECT_EQ(b1->acquire(), -EBUSY); |
| 364 | EXPECT_EQ(b2->acquire(), -EBUSY); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 365 | } |
| 366 | |
| 367 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInReleasedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 368 | ASSERT_TRUE(b1->isReleased()); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 369 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 370 | EXPECT_EQ(b1->release(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 371 | } |
| 372 | |
| 373 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInGainedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 374 | ASSERT_TRUE(b1->isReleased()); |
| 375 | ASSERT_EQ(b1->gain(), 0); |
| 376 | ASSERT_TRUE(isAnyClientGained(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 377 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 378 | EXPECT_EQ(b1->release(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 379 | } |
| 380 | |
| 381 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInPostedState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 382 | ASSERT_EQ(b1->gain(), 0); |
| 383 | ASSERT_EQ(b1->post(), 0); |
| 384 | ASSERT_TRUE(isAnyClientPosted(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 385 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 386 | EXPECT_EQ(b2->release(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 387 | } |
| 388 | |
| 389 | TEST_F(BufferHubBufferStateTransitionTest, ReleaseBuffer_fromSelfInAcquiredState) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 390 | ASSERT_EQ(b1->gain(), 0); |
| 391 | ASSERT_EQ(b1->post(), 0); |
| 392 | ASSERT_EQ(b2->acquire(), 0); |
| 393 | ASSERT_TRUE(isAnyClientAcquired(b1->bufferState())); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 394 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 395 | EXPECT_EQ(b2->release(), 0); |
Tianyu Jiang | b08b722 | 2018-11-16 17:55:26 -0800 | [diff] [blame] | 396 | } |
| 397 | |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 398 | TEST_F(BufferHubBufferStateTransitionTest, BasicUsage) { |
| 399 | // 1 producer buffer and 1 consumer buffer initialised in testcase setup. |
| 400 | // Test if this set of basic operation succeed: |
| 401 | // Producer post three times to the consumer, and released by consumer. |
| 402 | for (int i = 0; i < 3; ++i) { |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 403 | ASSERT_EQ(b1->gain(), 0); |
| 404 | ASSERT_EQ(b1->post(), 0); |
| 405 | ASSERT_EQ(b2->acquire(), 0); |
| 406 | ASSERT_EQ(b2->release(), 0); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 407 | } |
| 408 | } |
| 409 | |
| 410 | TEST_F(BufferHubBufferTest, createNewConsumerAfterGain) { |
| 411 | // Create a poducer buffer and gain. |
| 412 | std::unique_ptr<BufferHubBuffer> b1 = |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 413 | BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 414 | kUserMetadataSize); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 415 | ASSERT_THAT(b1, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 416 | ASSERT_EQ(b1->gain(), 0); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 417 | |
| 418 | // Create a consumer of the buffer and test if the consumer can acquire the |
| 419 | // buffer if producer posts. |
Tianyu Jiang | 8a7e765 | 2019-02-11 13:41:20 -0800 | [diff] [blame] | 420 | sp<NativeHandle> token = b1->duplicate(); |
| 421 | ASSERT_THAT(token, NotNull()); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 422 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 423 | std::unique_ptr<BufferHubBuffer> b2 = BufferHubBuffer::import(token); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 424 | |
| 425 | ASSERT_THAT(b2, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 426 | ASSERT_NE(b1->clientStateMask(), b2->clientStateMask()); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 427 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 428 | ASSERT_EQ(b1->post(), 0); |
| 429 | EXPECT_EQ(b2->acquire(), 0); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 430 | } |
| 431 | |
| 432 | TEST_F(BufferHubBufferTest, createNewConsumerAfterPost) { |
| 433 | // Create a poducer buffer and post. |
| 434 | std::unique_ptr<BufferHubBuffer> b1 = |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 435 | BufferHubBuffer::create(kWidth, kHeight, kLayerCount, kFormat, kUsage, |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 436 | kUserMetadataSize); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 437 | ASSERT_EQ(b1->gain(), 0); |
| 438 | ASSERT_EQ(b1->post(), 0); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 439 | |
| 440 | // Create a consumer of the buffer and test if the consumer can acquire the |
| 441 | // buffer if producer posts. |
Tianyu Jiang | 8a7e765 | 2019-02-11 13:41:20 -0800 | [diff] [blame] | 442 | sp<NativeHandle> token = b1->duplicate(); |
| 443 | ASSERT_THAT(token, NotNull()); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 444 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 445 | std::unique_ptr<BufferHubBuffer> b2 = BufferHubBuffer::import(token); |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 446 | |
| 447 | ASSERT_THAT(b2, NotNull()); |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 448 | ASSERT_NE(b1->clientStateMask(), b2->clientStateMask()); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 449 | |
Tianyu Jiang | 727ede4 | 2019-02-01 11:44:51 -0800 | [diff] [blame] | 450 | EXPECT_EQ(b2->acquire(), 0); |
Tianyu Jiang | ec97b76 | 2019-01-07 17:14:02 -0800 | [diff] [blame] | 451 | } |
| 452 | |
Fan Xu | ca70b7b | 2018-10-31 13:20:12 -0700 | [diff] [blame] | 453 | } // namespace |
Fan Xu | 021776e | 2018-12-05 13:34:48 -0800 | [diff] [blame] | 454 | |
Jiwen 'Steve' Cai | 0e7ee22 | 2018-10-12 17:20:40 -0700 | [diff] [blame] | 455 | } // namespace android |