Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2012 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 "BufferQueue_test" |
| 18 | //#define LOG_NDEBUG 0 |
| 19 | |
Dan Stoza | c6f30bd | 2015-06-08 09:32:50 -0700 | [diff] [blame] | 20 | #include "DummyConsumer.h" |
| 21 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 22 | #include <gui/BufferItem.h> |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 23 | #include <gui/BufferQueue.h> |
| 24 | #include <gui/IProducerListener.h> |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 25 | |
| 26 | #include <ui/GraphicBuffer.h> |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 27 | |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 28 | #include <binder/IPCThreadState.h> |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 29 | #include <binder/IServiceManager.h> |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 30 | #include <binder/ProcessState.h> |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 31 | |
| 32 | #include <utils/String8.h> |
| 33 | #include <utils/threads.h> |
| 34 | |
| 35 | #include <gtest/gtest.h> |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 36 | |
| 37 | namespace android { |
| 38 | |
| 39 | class BufferQueueTest : public ::testing::Test { |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 40 | |
| 41 | public: |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 42 | protected: |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 43 | BufferQueueTest() { |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 44 | const ::testing::TestInfo* const testInfo = |
| 45 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 46 | ALOGV("Begin test: %s.%s", testInfo->test_case_name(), |
| 47 | testInfo->name()); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 48 | } |
| 49 | |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 50 | ~BufferQueueTest() { |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 51 | const ::testing::TestInfo* const testInfo = |
| 52 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 53 | ALOGV("End test: %s.%s", testInfo->test_case_name(), |
| 54 | testInfo->name()); |
| 55 | } |
| 56 | |
Igor Murashkin | 7ea777f | 2013-11-18 16:58:36 -0800 | [diff] [blame] | 57 | void GetMinUndequeuedBufferCount(int* bufferCount) { |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 58 | ASSERT_TRUE(bufferCount != NULL); |
| 59 | ASSERT_EQ(OK, mProducer->query(NATIVE_WINDOW_MIN_UNDEQUEUED_BUFFERS, |
| 60 | bufferCount)); |
| 61 | ASSERT_GE(*bufferCount, 0); |
Igor Murashkin | 7ea777f | 2013-11-18 16:58:36 -0800 | [diff] [blame] | 62 | } |
| 63 | |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 64 | void createBufferQueue() { |
| 65 | BufferQueue::createBufferQueue(&mProducer, &mConsumer); |
| 66 | } |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 67 | |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 68 | sp<IGraphicBufferProducer> mProducer; |
| 69 | sp<IGraphicBufferConsumer> mConsumer; |
| 70 | }; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 71 | |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 72 | static const uint32_t TEST_DATA = 0x12345678u; |
| 73 | |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 74 | // XXX: Tests that fork a process to hold the BufferQueue must run before tests |
| 75 | // that use a local BufferQueue, or else Binder will get unhappy |
| 76 | TEST_F(BufferQueueTest, BufferQueueInAnotherProcess) { |
| 77 | const String16 PRODUCER_NAME = String16("BQTestProducer"); |
| 78 | const String16 CONSUMER_NAME = String16("BQTestConsumer"); |
| 79 | |
| 80 | pid_t forkPid = fork(); |
| 81 | ASSERT_NE(forkPid, -1); |
| 82 | |
| 83 | if (forkPid == 0) { |
| 84 | // Child process |
Dan Stoza | b3d0bdf | 2014-04-07 16:33:59 -0700 | [diff] [blame] | 85 | sp<IGraphicBufferProducer> producer; |
| 86 | sp<IGraphicBufferConsumer> consumer; |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 87 | BufferQueue::createBufferQueue(&producer, &consumer); |
| 88 | sp<IServiceManager> serviceManager = defaultServiceManager(); |
Marco Nelissen | 097ca27 | 2014-11-14 08:01:01 -0800 | [diff] [blame] | 89 | serviceManager->addService(PRODUCER_NAME, IInterface::asBinder(producer)); |
| 90 | serviceManager->addService(CONSUMER_NAME, IInterface::asBinder(consumer)); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 91 | ProcessState::self()->startThreadPool(); |
| 92 | IPCThreadState::self()->joinThreadPool(); |
| 93 | LOG_ALWAYS_FATAL("Shouldn't be here"); |
| 94 | } |
| 95 | |
| 96 | sp<IServiceManager> serviceManager = defaultServiceManager(); |
| 97 | sp<IBinder> binderProducer = |
| 98 | serviceManager->getService(PRODUCER_NAME); |
| 99 | mProducer = interface_cast<IGraphicBufferProducer>(binderProducer); |
| 100 | EXPECT_TRUE(mProducer != NULL); |
| 101 | sp<IBinder> binderConsumer = |
| 102 | serviceManager->getService(CONSUMER_NAME); |
| 103 | mConsumer = interface_cast<IGraphicBufferConsumer>(binderConsumer); |
| 104 | EXPECT_TRUE(mConsumer != NULL); |
| 105 | |
| 106 | sp<DummyConsumer> dc(new DummyConsumer); |
| 107 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, false)); |
| 108 | IGraphicBufferProducer::QueueBufferOutput output; |
| 109 | ASSERT_EQ(OK, |
| 110 | mProducer->connect(NULL, NATIVE_WINDOW_API_CPU, false, &output)); |
| 111 | |
| 112 | int slot; |
| 113 | sp<Fence> fence; |
| 114 | sp<GraphicBuffer> buffer; |
| 115 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 116 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 117 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 118 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
| 119 | |
| 120 | uint32_t* dataIn; |
| 121 | ASSERT_EQ(OK, buffer->lock(GraphicBuffer::USAGE_SW_WRITE_OFTEN, |
| 122 | reinterpret_cast<void**>(&dataIn))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 123 | *dataIn = TEST_DATA; |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 124 | ASSERT_EQ(OK, buffer->unlock()); |
| 125 | |
Eino-Ville Talvala | 82c6bcc | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 126 | IGraphicBufferProducer::QueueBufferInput input(0, false, |
| 127 | HAL_DATASPACE_UNKNOWN, Rect(0, 0, 1, 1), |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 128 | NATIVE_WINDOW_SCALING_MODE_FREEZE, 0, Fence::NO_FENCE); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 129 | ASSERT_EQ(OK, mProducer->queueBuffer(slot, input, &output)); |
| 130 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 131 | BufferItem item; |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 132 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, 0)); |
| 133 | |
| 134 | uint32_t* dataOut; |
| 135 | ASSERT_EQ(OK, item.mGraphicBuffer->lock(GraphicBuffer::USAGE_SW_READ_OFTEN, |
| 136 | reinterpret_cast<void**>(&dataOut))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 137 | ASSERT_EQ(*dataOut, TEST_DATA); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 138 | ASSERT_EQ(OK, item.mGraphicBuffer->unlock()); |
| 139 | } |
| 140 | |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 141 | TEST_F(BufferQueueTest, AcquireBuffer_ExceedsMaxAcquireCount_Fails) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 142 | createBufferQueue(); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 143 | sp<DummyConsumer> dc(new DummyConsumer); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 144 | mConsumer->consumerConnect(dc, false); |
Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 145 | IGraphicBufferProducer::QueueBufferOutput qbo; |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 146 | mProducer->connect(new DummyProducerListener, NATIVE_WINDOW_API_CPU, false, |
| 147 | &qbo); |
Pablo Ceballos | fa45535 | 2015-08-12 17:47:47 -0700 | [diff] [blame] | 148 | mProducer->setMaxDequeuedBufferCount(3); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 149 | |
| 150 | int slot; |
| 151 | sp<Fence> fence; |
| 152 | sp<GraphicBuffer> buf; |
Eino-Ville Talvala | 82c6bcc | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 153 | IGraphicBufferProducer::QueueBufferInput qbi(0, false, |
| 154 | HAL_DATASPACE_UNKNOWN, Rect(0, 0, 1, 1), |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 155 | NATIVE_WINDOW_SCALING_MODE_FREEZE, 0, Fence::NO_FENCE); |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 156 | BufferItem item; |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 157 | |
| 158 | for (int i = 0; i < 2; i++) { |
Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 159 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 160 | mProducer->dequeueBuffer(&slot, &fence, 1, 1, 0, |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 161 | GRALLOC_USAGE_SW_READ_OFTEN)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 162 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buf)); |
| 163 | ASSERT_EQ(OK, mProducer->queueBuffer(slot, qbi, &qbo)); |
| 164 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, 0)); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 165 | } |
| 166 | |
Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 167 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 168 | mProducer->dequeueBuffer(&slot, &fence, 1, 1, 0, |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 169 | GRALLOC_USAGE_SW_READ_OFTEN)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 170 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buf)); |
| 171 | ASSERT_EQ(OK, mProducer->queueBuffer(slot, qbi, &qbo)); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 172 | |
| 173 | // Acquire the third buffer, which should fail. |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 174 | ASSERT_EQ(INVALID_OPERATION, mConsumer->acquireBuffer(&item, 0)); |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 175 | } |
| 176 | |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 177 | TEST_F(BufferQueueTest, SetMaxAcquiredBufferCountWithIllegalValues_ReturnsError) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 178 | createBufferQueue(); |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 179 | sp<DummyConsumer> dc(new DummyConsumer); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 180 | mConsumer->consumerConnect(dc, false); |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 181 | |
Igor Murashkin | 7ea777f | 2013-11-18 16:58:36 -0800 | [diff] [blame] | 182 | int minBufferCount; |
| 183 | ASSERT_NO_FATAL_FAILURE(GetMinUndequeuedBufferCount(&minBufferCount)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 184 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount( |
| 185 | minBufferCount - 1)); |
Igor Murashkin | 7ea777f | 2013-11-18 16:58:36 -0800 | [diff] [blame] | 186 | |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 187 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount(0)); |
| 188 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount(-3)); |
| 189 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount( |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 190 | BufferQueue::MAX_MAX_ACQUIRED_BUFFERS+1)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 191 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount(100)); |
Pablo Ceballos | 19e3e06 | 2015-08-19 16:16:06 -0700 | [diff] [blame] | 192 | |
| 193 | EXPECT_EQ(OK, mConsumer->setMaxBufferCount(5)); |
| 194 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxAcquiredBufferCount(5)); |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 195 | } |
| 196 | |
| 197 | TEST_F(BufferQueueTest, SetMaxAcquiredBufferCountWithLegalValues_Succeeds) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 198 | createBufferQueue(); |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 199 | sp<DummyConsumer> dc(new DummyConsumer); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 200 | mConsumer->consumerConnect(dc, false); |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 201 | |
Igor Murashkin | 7ea777f | 2013-11-18 16:58:36 -0800 | [diff] [blame] | 202 | int minBufferCount; |
| 203 | ASSERT_NO_FATAL_FAILURE(GetMinUndequeuedBufferCount(&minBufferCount)); |
| 204 | |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 205 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount(1)); |
| 206 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount(2)); |
| 207 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount(minBufferCount)); |
| 208 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount( |
Jamie Gennis | c68f2ec | 2012-08-30 18:36:22 -0700 | [diff] [blame] | 209 | BufferQueue::MAX_MAX_ACQUIRED_BUFFERS)); |
| 210 | } |
| 211 | |
Pablo Ceballos | 19e3e06 | 2015-08-19 16:16:06 -0700 | [diff] [blame] | 212 | TEST_F(BufferQueueTest, SetMaxBufferCountWithLegalValues_Succeeds) { |
| 213 | createBufferQueue(); |
| 214 | sp<DummyConsumer> dc(new DummyConsumer); |
| 215 | mConsumer->consumerConnect(dc, false); |
| 216 | |
| 217 | // Test single buffer mode |
| 218 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount(1)); |
| 219 | } |
| 220 | |
| 221 | TEST_F(BufferQueueTest, SetMaxBufferCountWithIllegalValues_ReturnsError) { |
| 222 | createBufferQueue(); |
| 223 | sp<DummyConsumer> dc(new DummyConsumer); |
| 224 | mConsumer->consumerConnect(dc, false); |
| 225 | |
| 226 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxBufferCount(0)); |
| 227 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxBufferCount( |
| 228 | BufferQueue::NUM_BUFFER_SLOTS + 1)); |
| 229 | |
| 230 | EXPECT_EQ(OK, mConsumer->setMaxAcquiredBufferCount(5)); |
| 231 | EXPECT_EQ(BAD_VALUE, mConsumer->setMaxBufferCount(3)); |
| 232 | } |
| 233 | |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 234 | TEST_F(BufferQueueTest, DetachAndReattachOnProducerSide) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 235 | createBufferQueue(); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 236 | sp<DummyConsumer> dc(new DummyConsumer); |
| 237 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, false)); |
| 238 | IGraphicBufferProducer::QueueBufferOutput output; |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 239 | ASSERT_EQ(OK, mProducer->connect(new DummyProducerListener, |
| 240 | NATIVE_WINDOW_API_CPU, false, &output)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 241 | |
| 242 | ASSERT_EQ(BAD_VALUE, mProducer->detachBuffer(-1)); // Index too low |
| 243 | ASSERT_EQ(BAD_VALUE, mProducer->detachBuffer( |
| 244 | BufferQueueDefs::NUM_BUFFER_SLOTS)); // Index too high |
| 245 | ASSERT_EQ(BAD_VALUE, mProducer->detachBuffer(0)); // Not dequeued |
| 246 | |
| 247 | int slot; |
| 248 | sp<Fence> fence; |
| 249 | sp<GraphicBuffer> buffer; |
| 250 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 251 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 252 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 253 | ASSERT_EQ(BAD_VALUE, mProducer->detachBuffer(slot)); // Not requested |
| 254 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
| 255 | ASSERT_EQ(OK, mProducer->detachBuffer(slot)); |
| 256 | ASSERT_EQ(BAD_VALUE, mProducer->detachBuffer(slot)); // Not dequeued |
| 257 | |
| 258 | sp<GraphicBuffer> safeToClobberBuffer; |
| 259 | // Can no longer request buffer from this slot |
| 260 | ASSERT_EQ(BAD_VALUE, mProducer->requestBuffer(slot, &safeToClobberBuffer)); |
| 261 | |
| 262 | uint32_t* dataIn; |
| 263 | ASSERT_EQ(OK, buffer->lock(GraphicBuffer::USAGE_SW_WRITE_OFTEN, |
| 264 | reinterpret_cast<void**>(&dataIn))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 265 | *dataIn = TEST_DATA; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 266 | ASSERT_EQ(OK, buffer->unlock()); |
| 267 | |
| 268 | int newSlot; |
| 269 | ASSERT_EQ(BAD_VALUE, mProducer->attachBuffer(NULL, safeToClobberBuffer)); |
| 270 | ASSERT_EQ(BAD_VALUE, mProducer->attachBuffer(&newSlot, NULL)); |
| 271 | |
| 272 | ASSERT_EQ(OK, mProducer->attachBuffer(&newSlot, buffer)); |
Eino-Ville Talvala | 82c6bcc | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 273 | IGraphicBufferProducer::QueueBufferInput input(0, false, |
| 274 | HAL_DATASPACE_UNKNOWN, Rect(0, 0, 1, 1), |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 275 | NATIVE_WINDOW_SCALING_MODE_FREEZE, 0, Fence::NO_FENCE); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 276 | ASSERT_EQ(OK, mProducer->queueBuffer(newSlot, input, &output)); |
| 277 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 278 | BufferItem item; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 279 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, static_cast<nsecs_t>(0))); |
| 280 | |
| 281 | uint32_t* dataOut; |
| 282 | ASSERT_EQ(OK, item.mGraphicBuffer->lock(GraphicBuffer::USAGE_SW_READ_OFTEN, |
| 283 | reinterpret_cast<void**>(&dataOut))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 284 | ASSERT_EQ(*dataOut, TEST_DATA); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 285 | ASSERT_EQ(OK, item.mGraphicBuffer->unlock()); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 286 | } |
| 287 | |
| 288 | TEST_F(BufferQueueTest, DetachAndReattachOnConsumerSide) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 289 | createBufferQueue(); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 290 | sp<DummyConsumer> dc(new DummyConsumer); |
| 291 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, false)); |
| 292 | IGraphicBufferProducer::QueueBufferOutput output; |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 293 | ASSERT_EQ(OK, mProducer->connect(new DummyProducerListener, |
| 294 | NATIVE_WINDOW_API_CPU, false, &output)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 295 | |
| 296 | int slot; |
| 297 | sp<Fence> fence; |
| 298 | sp<GraphicBuffer> buffer; |
| 299 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 300 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 301 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 302 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
Eino-Ville Talvala | 82c6bcc | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 303 | IGraphicBufferProducer::QueueBufferInput input(0, false, |
| 304 | HAL_DATASPACE_UNKNOWN, Rect(0, 0, 1, 1), |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 305 | NATIVE_WINDOW_SCALING_MODE_FREEZE, 0, Fence::NO_FENCE); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 306 | ASSERT_EQ(OK, mProducer->queueBuffer(slot, input, &output)); |
| 307 | |
| 308 | ASSERT_EQ(BAD_VALUE, mConsumer->detachBuffer(-1)); // Index too low |
| 309 | ASSERT_EQ(BAD_VALUE, mConsumer->detachBuffer( |
| 310 | BufferQueueDefs::NUM_BUFFER_SLOTS)); // Index too high |
| 311 | ASSERT_EQ(BAD_VALUE, mConsumer->detachBuffer(0)); // Not acquired |
| 312 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 313 | BufferItem item; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 314 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, static_cast<nsecs_t>(0))); |
| 315 | |
Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 316 | ASSERT_EQ(OK, mConsumer->detachBuffer(item.mSlot)); |
| 317 | ASSERT_EQ(BAD_VALUE, mConsumer->detachBuffer(item.mSlot)); // Not acquired |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 318 | |
| 319 | uint32_t* dataIn; |
| 320 | ASSERT_EQ(OK, item.mGraphicBuffer->lock( |
| 321 | GraphicBuffer::USAGE_SW_WRITE_OFTEN, |
| 322 | reinterpret_cast<void**>(&dataIn))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 323 | *dataIn = TEST_DATA; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 324 | ASSERT_EQ(OK, item.mGraphicBuffer->unlock()); |
| 325 | |
| 326 | int newSlot; |
| 327 | sp<GraphicBuffer> safeToClobberBuffer; |
| 328 | ASSERT_EQ(BAD_VALUE, mConsumer->attachBuffer(NULL, safeToClobberBuffer)); |
| 329 | ASSERT_EQ(BAD_VALUE, mConsumer->attachBuffer(&newSlot, NULL)); |
| 330 | ASSERT_EQ(OK, mConsumer->attachBuffer(&newSlot, item.mGraphicBuffer)); |
| 331 | |
Dan Stoza | 99b18b4 | 2014-03-28 15:34:33 -0700 | [diff] [blame] | 332 | ASSERT_EQ(OK, mConsumer->releaseBuffer(newSlot, 0, EGL_NO_DISPLAY, |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 333 | EGL_NO_SYNC_KHR, Fence::NO_FENCE)); |
| 334 | |
| 335 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 336 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 337 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 338 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
| 339 | |
| 340 | uint32_t* dataOut; |
| 341 | ASSERT_EQ(OK, buffer->lock(GraphicBuffer::USAGE_SW_READ_OFTEN, |
| 342 | reinterpret_cast<void**>(&dataOut))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 343 | ASSERT_EQ(*dataOut, TEST_DATA); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 344 | ASSERT_EQ(OK, buffer->unlock()); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 345 | } |
| 346 | |
| 347 | TEST_F(BufferQueueTest, MoveFromConsumerToProducer) { |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 348 | createBufferQueue(); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 349 | sp<DummyConsumer> dc(new DummyConsumer); |
| 350 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, false)); |
| 351 | IGraphicBufferProducer::QueueBufferOutput output; |
Dan Stoza | f0eaf25 | 2014-03-21 13:05:51 -0700 | [diff] [blame] | 352 | ASSERT_EQ(OK, mProducer->connect(new DummyProducerListener, |
| 353 | NATIVE_WINDOW_API_CPU, false, &output)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 354 | |
| 355 | int slot; |
| 356 | sp<Fence> fence; |
| 357 | sp<GraphicBuffer> buffer; |
| 358 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 359 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 360 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 361 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
| 362 | |
| 363 | uint32_t* dataIn; |
| 364 | ASSERT_EQ(OK, buffer->lock(GraphicBuffer::USAGE_SW_WRITE_OFTEN, |
| 365 | reinterpret_cast<void**>(&dataIn))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 366 | *dataIn = TEST_DATA; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 367 | ASSERT_EQ(OK, buffer->unlock()); |
| 368 | |
Eino-Ville Talvala | 82c6bcc | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 369 | IGraphicBufferProducer::QueueBufferInput input(0, false, |
| 370 | HAL_DATASPACE_UNKNOWN, Rect(0, 0, 1, 1), |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 371 | NATIVE_WINDOW_SCALING_MODE_FREEZE, 0, Fence::NO_FENCE); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 372 | ASSERT_EQ(OK, mProducer->queueBuffer(slot, input, &output)); |
| 373 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 374 | BufferItem item; |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 375 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, static_cast<nsecs_t>(0))); |
Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 376 | ASSERT_EQ(OK, mConsumer->detachBuffer(item.mSlot)); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 377 | |
| 378 | int newSlot; |
| 379 | ASSERT_EQ(OK, mProducer->attachBuffer(&newSlot, item.mGraphicBuffer)); |
| 380 | ASSERT_EQ(OK, mProducer->queueBuffer(newSlot, input, &output)); |
| 381 | ASSERT_EQ(OK, mConsumer->acquireBuffer(&item, static_cast<nsecs_t>(0))); |
| 382 | |
| 383 | uint32_t* dataOut; |
| 384 | ASSERT_EQ(OK, item.mGraphicBuffer->lock(GraphicBuffer::USAGE_SW_READ_OFTEN, |
| 385 | reinterpret_cast<void**>(&dataOut))); |
Dan Stoza | f8cebe5 | 2015-04-20 12:09:38 -0700 | [diff] [blame] | 386 | ASSERT_EQ(*dataOut, TEST_DATA); |
Dan Stoza | 1a0b861 | 2014-03-20 15:36:31 -0700 | [diff] [blame] | 387 | ASSERT_EQ(OK, item.mGraphicBuffer->unlock()); |
Dan Stoza | 9f3053d | 2014-03-06 15:14:33 -0800 | [diff] [blame] | 388 | } |
| 389 | |
Dan Stoza | 9de7293 | 2015-04-16 17:28:43 -0700 | [diff] [blame] | 390 | TEST_F(BufferQueueTest, TestDisallowingAllocation) { |
| 391 | createBufferQueue(); |
| 392 | sp<DummyConsumer> dc(new DummyConsumer); |
| 393 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, true)); |
| 394 | IGraphicBufferProducer::QueueBufferOutput output; |
| 395 | ASSERT_EQ(OK, mProducer->connect(new DummyProducerListener, |
| 396 | NATIVE_WINDOW_API_CPU, true, &output)); |
| 397 | |
| 398 | static const uint32_t WIDTH = 320; |
| 399 | static const uint32_t HEIGHT = 240; |
| 400 | |
| 401 | ASSERT_EQ(OK, mConsumer->setDefaultBufferSize(WIDTH, HEIGHT)); |
| 402 | |
| 403 | int slot; |
| 404 | sp<Fence> fence; |
| 405 | sp<GraphicBuffer> buffer; |
| 406 | // This should return an error since it would require an allocation |
| 407 | ASSERT_EQ(OK, mProducer->allowAllocation(false)); |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 408 | ASSERT_EQ(WOULD_BLOCK, mProducer->dequeueBuffer(&slot, &fence, 0, 0, |
Dan Stoza | 9de7293 | 2015-04-16 17:28:43 -0700 | [diff] [blame] | 409 | 0, GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 410 | |
| 411 | // This should succeed, now that we've lifted the prohibition |
| 412 | ASSERT_EQ(OK, mProducer->allowAllocation(true)); |
| 413 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 414 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, |
Dan Stoza | 9de7293 | 2015-04-16 17:28:43 -0700 | [diff] [blame] | 415 | GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 416 | |
| 417 | // Release the previous buffer back to the BufferQueue |
| 418 | mProducer->cancelBuffer(slot, fence); |
| 419 | |
| 420 | // This should fail since we're requesting a different size |
| 421 | ASSERT_EQ(OK, mProducer->allowAllocation(false)); |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 422 | ASSERT_EQ(WOULD_BLOCK, mProducer->dequeueBuffer(&slot, &fence, |
Dan Stoza | 9de7293 | 2015-04-16 17:28:43 -0700 | [diff] [blame] | 423 | WIDTH * 2, HEIGHT * 2, 0, GRALLOC_USAGE_SW_WRITE_OFTEN)); |
| 424 | } |
| 425 | |
Dan Stoza | 812ed06 | 2015-06-02 15:45:22 -0700 | [diff] [blame] | 426 | TEST_F(BufferQueueTest, TestGenerationNumbers) { |
| 427 | createBufferQueue(); |
| 428 | sp<DummyConsumer> dc(new DummyConsumer); |
| 429 | ASSERT_EQ(OK, mConsumer->consumerConnect(dc, true)); |
| 430 | IGraphicBufferProducer::QueueBufferOutput output; |
| 431 | ASSERT_EQ(OK, mProducer->connect(new DummyProducerListener, |
| 432 | NATIVE_WINDOW_API_CPU, true, &output)); |
| 433 | |
| 434 | ASSERT_EQ(OK, mProducer->setGenerationNumber(1)); |
| 435 | |
| 436 | // Get one buffer to play with |
| 437 | int slot; |
| 438 | sp<Fence> fence; |
| 439 | ASSERT_EQ(IGraphicBufferProducer::BUFFER_NEEDS_REALLOCATION, |
Pablo Ceballos | 567dbbb | 2015-08-26 18:59:08 -0700 | [diff] [blame^] | 440 | mProducer->dequeueBuffer(&slot, &fence, 0, 0, 0, 0)); |
Dan Stoza | 812ed06 | 2015-06-02 15:45:22 -0700 | [diff] [blame] | 441 | |
| 442 | sp<GraphicBuffer> buffer; |
| 443 | ASSERT_EQ(OK, mProducer->requestBuffer(slot, &buffer)); |
| 444 | |
| 445 | // Ensure that the generation number we set propagates to allocated buffers |
| 446 | ASSERT_EQ(1U, buffer->getGenerationNumber()); |
| 447 | |
| 448 | ASSERT_EQ(OK, mProducer->detachBuffer(slot)); |
| 449 | |
| 450 | ASSERT_EQ(OK, mProducer->setGenerationNumber(2)); |
| 451 | |
| 452 | // These should fail, since we've changed the generation number on the queue |
| 453 | int outSlot; |
| 454 | ASSERT_EQ(BAD_VALUE, mProducer->attachBuffer(&outSlot, buffer)); |
| 455 | ASSERT_EQ(BAD_VALUE, mConsumer->attachBuffer(&outSlot, buffer)); |
| 456 | |
| 457 | buffer->setGenerationNumber(2); |
| 458 | |
| 459 | // This should succeed now that we've changed the buffer's generation number |
| 460 | ASSERT_EQ(OK, mProducer->attachBuffer(&outSlot, buffer)); |
| 461 | |
| 462 | ASSERT_EQ(OK, mProducer->detachBuffer(outSlot)); |
| 463 | |
| 464 | // This should also succeed with the new generation number |
| 465 | ASSERT_EQ(OK, mConsumer->attachBuffer(&outSlot, buffer)); |
| 466 | } |
| 467 | |
Jamie Gennis | 9e75ddd | 2012-08-31 15:32:45 -0700 | [diff] [blame] | 468 | } // namespace android |