Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2019 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 | |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 17 | #undef LOG_TAG |
| 18 | #define LOG_TAG "BLASTBufferQueue" |
| 19 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 20 | #define ATRACE_TAG ATRACE_TAG_GRAPHICS |
Vishnu Nair | e1a4232 | 2020-10-02 17:42:04 -0700 | [diff] [blame] | 21 | //#define LOG_NDEBUG 0 |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 22 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 23 | #include <gui/BLASTBufferQueue.h> |
| 24 | #include <gui/BufferItemConsumer.h> |
Vishnu Nair | 8949612 | 2020-12-14 17:14:53 -0800 | [diff] [blame] | 25 | #include <gui/BufferQueueConsumer.h> |
| 26 | #include <gui/BufferQueueCore.h> |
| 27 | #include <gui/BufferQueueProducer.h> |
Valerie Hau | 45e4b3b | 2019-12-03 10:49:17 -0800 | [diff] [blame] | 28 | #include <gui/GLConsumer.h> |
Vishnu Nair | 8949612 | 2020-12-14 17:14:53 -0800 | [diff] [blame] | 29 | #include <gui/IProducerListener.h> |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 30 | #include <gui/Surface.h> |
Vishnu Nair | 8949612 | 2020-12-14 17:14:53 -0800 | [diff] [blame] | 31 | #include <utils/Singleton.h> |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 32 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 33 | #include <utils/Trace.h> |
| 34 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 35 | #include <chrono> |
| 36 | |
| 37 | using namespace std::chrono_literals; |
| 38 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 39 | namespace { |
| 40 | inline const char* toString(bool b) { |
| 41 | return b ? "true" : "false"; |
| 42 | } |
| 43 | } // namespace |
| 44 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 45 | namespace android { |
| 46 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 47 | // Macros to include adapter info in log messages |
| 48 | #define BQA_LOGV(x, ...) \ |
| 49 | ALOGV("[%s](f:%u,a:%u) " x, mName.c_str(), mNumFrameAvailable, mNumAcquired, ##__VA_ARGS__) |
Vishnu Nair | c6f89ee | 2020-12-11 14:27:32 -0800 | [diff] [blame] | 50 | // enable logs for a single layer |
| 51 | //#define BQA_LOGV(x, ...) \ |
| 52 | // ALOGV_IF((strstr(mName.c_str(), "SurfaceView") != nullptr), "[%s](f:%u,a:%u) " x, \ |
| 53 | // mName.c_str(), mNumFrameAvailable, mNumAcquired, ##__VA_ARGS__) |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 54 | #define BQA_LOGE(x, ...) \ |
| 55 | ALOGE("[%s](f:%u,a:%u) " x, mName.c_str(), mNumFrameAvailable, mNumAcquired, ##__VA_ARGS__) |
| 56 | |
Valerie Hau | 871d635 | 2020-01-29 08:44:02 -0800 | [diff] [blame] | 57 | void BLASTBufferItemConsumer::onDisconnect() { |
| 58 | Mutex::Autolock lock(mFrameEventHistoryMutex); |
| 59 | mPreviouslyConnected = mCurrentlyConnected; |
| 60 | mCurrentlyConnected = false; |
| 61 | if (mPreviouslyConnected) { |
| 62 | mDisconnectEvents.push(mCurrentFrameNumber); |
| 63 | } |
| 64 | mFrameEventHistory.onDisconnect(); |
| 65 | } |
| 66 | |
| 67 | void BLASTBufferItemConsumer::addAndGetFrameTimestamps(const NewFrameEventsEntry* newTimestamps, |
| 68 | FrameEventHistoryDelta* outDelta) { |
| 69 | Mutex::Autolock lock(mFrameEventHistoryMutex); |
| 70 | if (newTimestamps) { |
| 71 | // BufferQueueProducer only adds a new timestamp on |
| 72 | // queueBuffer |
| 73 | mCurrentFrameNumber = newTimestamps->frameNumber; |
| 74 | mFrameEventHistory.addQueue(*newTimestamps); |
| 75 | } |
| 76 | if (outDelta) { |
| 77 | // frame event histories will be processed |
| 78 | // only after the producer connects and requests |
| 79 | // deltas for the first time. Forward this intent |
| 80 | // to SF-side to turn event processing back on |
| 81 | mPreviouslyConnected = mCurrentlyConnected; |
| 82 | mCurrentlyConnected = true; |
| 83 | mFrameEventHistory.getAndResetDelta(outDelta); |
| 84 | } |
| 85 | } |
| 86 | |
| 87 | void BLASTBufferItemConsumer::updateFrameTimestamps(uint64_t frameNumber, nsecs_t refreshStartTime, |
| 88 | const sp<Fence>& glDoneFence, |
| 89 | const sp<Fence>& presentFence, |
| 90 | const sp<Fence>& prevReleaseFence, |
| 91 | CompositorTiming compositorTiming, |
| 92 | nsecs_t latchTime, nsecs_t dequeueReadyTime) { |
| 93 | Mutex::Autolock lock(mFrameEventHistoryMutex); |
| 94 | |
| 95 | // if the producer is not connected, don't bother updating, |
| 96 | // the next producer that connects won't access this frame event |
| 97 | if (!mCurrentlyConnected) return; |
| 98 | std::shared_ptr<FenceTime> glDoneFenceTime = std::make_shared<FenceTime>(glDoneFence); |
| 99 | std::shared_ptr<FenceTime> presentFenceTime = std::make_shared<FenceTime>(presentFence); |
| 100 | std::shared_ptr<FenceTime> releaseFenceTime = std::make_shared<FenceTime>(prevReleaseFence); |
| 101 | |
| 102 | mFrameEventHistory.addLatch(frameNumber, latchTime); |
| 103 | mFrameEventHistory.addRelease(frameNumber, dequeueReadyTime, std::move(releaseFenceTime)); |
| 104 | mFrameEventHistory.addPreComposition(frameNumber, refreshStartTime); |
| 105 | mFrameEventHistory.addPostComposition(frameNumber, glDoneFenceTime, presentFenceTime, |
| 106 | compositorTiming); |
| 107 | } |
| 108 | |
| 109 | void BLASTBufferItemConsumer::getConnectionEvents(uint64_t frameNumber, bool* needsDisconnect) { |
| 110 | bool disconnect = false; |
| 111 | Mutex::Autolock lock(mFrameEventHistoryMutex); |
| 112 | while (!mDisconnectEvents.empty() && mDisconnectEvents.front() <= frameNumber) { |
| 113 | disconnect = true; |
| 114 | mDisconnectEvents.pop(); |
| 115 | } |
| 116 | if (needsDisconnect != nullptr) *needsDisconnect = disconnect; |
| 117 | } |
| 118 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 119 | BLASTBufferQueue::BLASTBufferQueue(const std::string& name, const sp<SurfaceControl>& surface, |
| 120 | int width, int height, bool enableTripleBuffering) |
| 121 | : mName(name), |
| 122 | mSurfaceControl(surface), |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 123 | mSize(width, height), |
| 124 | mRequestedSize(mSize), |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 125 | mNextTransaction(nullptr) { |
Vishnu Nair | 8949612 | 2020-12-14 17:14:53 -0800 | [diff] [blame] | 126 | createBufferQueue(&mProducer, &mConsumer); |
Valerie Hau | 0889c62 | 2020-02-19 15:04:47 -0800 | [diff] [blame] | 127 | // since the adapter is in the client process, set dequeue timeout |
| 128 | // explicitly so that dequeueBuffer will block |
| 129 | mProducer->setDequeueTimeout(std::numeric_limits<int64_t>::max()); |
Valerie Hau | 65b8e87 | 2020-02-13 09:45:14 -0800 | [diff] [blame] | 130 | |
Robert Carr | cedef05 | 2020-04-22 15:58:23 -0700 | [diff] [blame] | 131 | if (enableTripleBuffering) { |
Valerie Hau | 65b8e87 | 2020-02-13 09:45:14 -0800 | [diff] [blame] | 132 | mProducer->setMaxDequeuedBufferCount(2); |
| 133 | } |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 134 | mBufferItemConsumer = |
Robert Carr | 8304f7f | 2020-12-02 15:47:16 -0800 | [diff] [blame] | 135 | new BLASTBufferItemConsumer(mConsumer, GraphicBuffer::USAGE_HW_COMPOSER, 1, false); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 136 | static int32_t id = 0; |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 137 | auto consumerName = mName + "(BLAST Consumer)" + std::to_string(id); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 138 | id++; |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 139 | mBufferItemConsumer->setName(String8(consumerName.c_str())); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 140 | mBufferItemConsumer->setFrameAvailableListener(this); |
| 141 | mBufferItemConsumer->setBufferFreedListener(this); |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 142 | mBufferItemConsumer->setDefaultBufferSize(mSize.width, mSize.height); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 143 | mBufferItemConsumer->setDefaultBufferFormat(PIXEL_FORMAT_RGBA_8888); |
Robert Carr | 9f133d7 | 2020-04-01 15:51:46 -0700 | [diff] [blame] | 144 | |
Valerie Hau | 2882e98 | 2020-01-23 13:33:10 -0800 | [diff] [blame] | 145 | mTransformHint = mSurfaceControl->getTransformHint(); |
Robert Carr | 9f133d7 | 2020-04-01 15:51:46 -0700 | [diff] [blame] | 146 | mBufferItemConsumer->setTransformHint(mTransformHint); |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 147 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 148 | mNumAcquired = 0; |
| 149 | mNumFrameAvailable = 0; |
| 150 | mPendingReleaseItem.item = BufferItem(); |
| 151 | mPendingReleaseItem.releaseFence = nullptr; |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 152 | } |
| 153 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 154 | void BLASTBufferQueue::update(const sp<SurfaceControl>& surface, uint32_t width, uint32_t height) { |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 155 | std::unique_lock _lock{mMutex}; |
| 156 | mSurfaceControl = surface; |
Robert Carr | fc41651 | 2020-04-02 12:32:44 -0700 | [diff] [blame] | 157 | |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 158 | ui::Size newSize(width, height); |
| 159 | if (mRequestedSize != newSize) { |
| 160 | mRequestedSize.set(newSize); |
| 161 | mBufferItemConsumer->setDefaultBufferSize(mRequestedSize.width, mRequestedSize.height); |
Vishnu Nair | 53c936c | 2020-12-03 11:46:37 -0800 | [diff] [blame] | 162 | if (mLastBufferScalingMode != NATIVE_WINDOW_SCALING_MODE_FREEZE) { |
| 163 | // If the buffer supports scaling, update the frame immediately since the client may |
| 164 | // want to scale the existing buffer to the new size. |
| 165 | mSize = mRequestedSize; |
| 166 | SurfaceComposerClient::Transaction t; |
| 167 | t.setFrame(mSurfaceControl, |
| 168 | {0, 0, static_cast<int32_t>(mSize.width), |
| 169 | static_cast<int32_t>(mSize.height)}); |
| 170 | t.apply(); |
| 171 | } |
Robert Carr | fc41651 | 2020-04-02 12:32:44 -0700 | [diff] [blame] | 172 | } |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 173 | } |
| 174 | |
| 175 | static void transactionCallbackThunk(void* context, nsecs_t latchTime, |
| 176 | const sp<Fence>& presentFence, |
| 177 | const std::vector<SurfaceControlStats>& stats) { |
| 178 | if (context == nullptr) { |
| 179 | return; |
| 180 | } |
Robert Carr | fbcbb4c | 2020-11-02 14:14:34 -0800 | [diff] [blame] | 181 | sp<BLASTBufferQueue> bq = static_cast<BLASTBufferQueue*>(context); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 182 | bq->transactionCallback(latchTime, presentFence, stats); |
| 183 | } |
| 184 | |
| 185 | void BLASTBufferQueue::transactionCallback(nsecs_t /*latchTime*/, const sp<Fence>& /*presentFence*/, |
| 186 | const std::vector<SurfaceControlStats>& stats) { |
| 187 | std::unique_lock _lock{mMutex}; |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 188 | ATRACE_CALL(); |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 189 | BQA_LOGV("transactionCallback"); |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 190 | mInitialCallbackReceived = true; |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 191 | |
Valerie Hau | 871d635 | 2020-01-29 08:44:02 -0800 | [diff] [blame] | 192 | if (!stats.empty()) { |
| 193 | mTransformHint = stats[0].transformHint; |
| 194 | mBufferItemConsumer->setTransformHint(mTransformHint); |
| 195 | mBufferItemConsumer->updateFrameTimestamps(stats[0].frameEventStats.frameNumber, |
| 196 | stats[0].frameEventStats.refreshStartTime, |
| 197 | stats[0].frameEventStats.gpuCompositionDoneFence, |
| 198 | stats[0].presentFence, |
| 199 | stats[0].previousReleaseFence, |
| 200 | stats[0].frameEventStats.compositorTiming, |
| 201 | stats[0].latchTime, |
| 202 | stats[0].frameEventStats.dequeueReadyTime); |
| 203 | } |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 204 | if (mPendingReleaseItem.item.mGraphicBuffer != nullptr) { |
Valerie Hau | 871d635 | 2020-01-29 08:44:02 -0800 | [diff] [blame] | 205 | if (!stats.empty()) { |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 206 | mPendingReleaseItem.releaseFence = stats[0].previousReleaseFence; |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 207 | } else { |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 208 | BQA_LOGE("Warning: no SurfaceControlStats returned in BLASTBufferQueue callback"); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 209 | mPendingReleaseItem.releaseFence = nullptr; |
| 210 | } |
| 211 | mBufferItemConsumer->releaseBuffer(mPendingReleaseItem.item, |
| 212 | mPendingReleaseItem.releaseFence |
| 213 | ? mPendingReleaseItem.releaseFence |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 214 | : Fence::NO_FENCE); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 215 | mNumAcquired--; |
| 216 | mPendingReleaseItem.item = BufferItem(); |
| 217 | mPendingReleaseItem.releaseFence = nullptr; |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 218 | } |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 219 | |
| 220 | if (mSubmitted.empty()) { |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 221 | BQA_LOGE("ERROR: callback with no corresponding submitted buffer item"); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 222 | } |
| 223 | mPendingReleaseItem.item = std::move(mSubmitted.front()); |
| 224 | mSubmitted.pop(); |
Robert Carr | e4943eb | 2020-02-15 18:34:59 -0800 | [diff] [blame] | 225 | |
Robert Carr | 255acdc | 2020-04-17 14:08:55 -0700 | [diff] [blame] | 226 | processNextBufferLocked(false); |
Robert Carr | e4943eb | 2020-02-15 18:34:59 -0800 | [diff] [blame] | 227 | |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 228 | mCallbackCV.notify_all(); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 229 | decStrong((void*)transactionCallbackThunk); |
| 230 | } |
| 231 | |
Robert Carr | 255acdc | 2020-04-17 14:08:55 -0700 | [diff] [blame] | 232 | void BLASTBufferQueue::processNextBufferLocked(bool useNextTransaction) { |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 233 | ATRACE_CALL(); |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 234 | BQA_LOGV("processNextBufferLocked useNextTransaction=%s", toString(useNextTransaction)); |
| 235 | |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 236 | // Wait to acquire a buffer if there are no frames available or we have acquired the max |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 237 | // number of buffers. |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 238 | if (mNumFrameAvailable == 0 || maxBuffersAcquired()) { |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 239 | BQA_LOGV("processNextBufferLocked waiting for frame available or callback"); |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 240 | mCallbackCV.notify_all(); |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 241 | return; |
| 242 | } |
| 243 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 244 | if (mSurfaceControl == nullptr) { |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 245 | BQA_LOGE("ERROR : surface control is null"); |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 246 | return; |
| 247 | } |
| 248 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 249 | SurfaceComposerClient::Transaction localTransaction; |
| 250 | bool applyTransaction = true; |
| 251 | SurfaceComposerClient::Transaction* t = &localTransaction; |
Robert Carr | 255acdc | 2020-04-17 14:08:55 -0700 | [diff] [blame] | 252 | if (mNextTransaction != nullptr && useNextTransaction) { |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 253 | t = mNextTransaction; |
| 254 | mNextTransaction = nullptr; |
| 255 | applyTransaction = false; |
| 256 | } |
| 257 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 258 | BufferItem bufferItem; |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 259 | |
Vishnu Nair | c6f89ee | 2020-12-11 14:27:32 -0800 | [diff] [blame] | 260 | status_t status = |
| 261 | mBufferItemConsumer->acquireBuffer(&bufferItem, 0 /* expectedPresent */, false); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 262 | if (status != OK) { |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 263 | BQA_LOGE("Failed to acquire a buffer, err=%s", statusToString(status).c_str()); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 264 | return; |
| 265 | } |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 266 | auto buffer = bufferItem.mGraphicBuffer; |
| 267 | mNumFrameAvailable--; |
| 268 | |
| 269 | if (buffer == nullptr) { |
| 270 | mBufferItemConsumer->releaseBuffer(bufferItem, Fence::NO_FENCE); |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 271 | BQA_LOGE("Buffer was empty"); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 272 | return; |
| 273 | } |
| 274 | |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 275 | if (rejectBuffer(bufferItem)) { |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 276 | BQA_LOGE("rejecting buffer:active_size=%dx%d, requested_size=%dx%d" |
| 277 | "buffer{size=%dx%d transform=%d}", |
| 278 | mSize.width, mSize.height, mRequestedSize.width, mRequestedSize.height, |
| 279 | buffer->getWidth(), buffer->getHeight(), bufferItem.mTransform); |
| 280 | mBufferItemConsumer->releaseBuffer(bufferItem, Fence::NO_FENCE); |
| 281 | processNextBufferLocked(useNextTransaction); |
| 282 | return; |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 283 | } |
| 284 | |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 285 | mNumAcquired++; |
| 286 | mSubmitted.push(bufferItem); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 287 | |
Valerie Hau | 871d635 | 2020-01-29 08:44:02 -0800 | [diff] [blame] | 288 | bool needsDisconnect = false; |
| 289 | mBufferItemConsumer->getConnectionEvents(bufferItem.mFrameNumber, &needsDisconnect); |
| 290 | |
| 291 | // if producer disconnected before, notify SurfaceFlinger |
| 292 | if (needsDisconnect) { |
| 293 | t->notifyProducerDisconnect(mSurfaceControl); |
| 294 | } |
| 295 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 296 | // Ensure BLASTBufferQueue stays alive until we receive the transaction complete callback. |
| 297 | incStrong((void*)transactionCallbackThunk); |
| 298 | |
Vishnu Nair | 53c936c | 2020-12-03 11:46:37 -0800 | [diff] [blame] | 299 | mLastBufferScalingMode = bufferItem.mScalingMode; |
| 300 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 301 | t->setBuffer(mSurfaceControl, buffer); |
John Reck | 137069e | 2020-12-10 22:07:37 -0500 | [diff] [blame] | 302 | t->setDataspace(mSurfaceControl, static_cast<ui::Dataspace>(bufferItem.mDataSpace)); |
| 303 | t->setHdrMetadata(mSurfaceControl, bufferItem.mHdrMetadata); |
| 304 | t->setSurfaceDamageRegion(mSurfaceControl, bufferItem.mSurfaceDamage); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 305 | t->setAcquireFence(mSurfaceControl, |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 306 | bufferItem.mFence ? new Fence(bufferItem.mFence->dup()) : Fence::NO_FENCE); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 307 | t->addTransactionCompletedCallback(transactionCallbackThunk, static_cast<void*>(this)); |
| 308 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 309 | t->setFrame(mSurfaceControl, |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 310 | {0, 0, static_cast<int32_t>(mSize.width), static_cast<int32_t>(mSize.height)}); |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 311 | t->setCrop(mSurfaceControl, computeCrop(bufferItem)); |
| 312 | t->setTransform(mSurfaceControl, bufferItem.mTransform); |
Valerie Hau | 2882e98 | 2020-01-23 13:33:10 -0800 | [diff] [blame] | 313 | t->setTransformToDisplayInverse(mSurfaceControl, bufferItem.mTransformToDisplayInverse); |
Valerie Hau | 181abd3 | 2020-01-27 14:18:28 -0800 | [diff] [blame] | 314 | t->setDesiredPresentTime(bufferItem.mTimestamp); |
Vishnu Nair | 6b7c5c9 | 2020-09-29 17:27:05 -0700 | [diff] [blame] | 315 | t->setFrameNumber(mSurfaceControl, bufferItem.mFrameNumber); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 316 | |
Jorim Jaggi | a3fe67b | 2020-12-01 00:24:33 +0100 | [diff] [blame] | 317 | if (!mNextFrameTimelineVsyncIdQueue.empty()) { |
| 318 | t->setFrameTimelineVsync(mSurfaceControl, mNextFrameTimelineVsyncIdQueue.front()); |
| 319 | mNextFrameTimelineVsyncIdQueue.pop(); |
| 320 | } |
| 321 | |
Vishnu Nair | cf26a0a | 2020-11-13 12:56:20 -0800 | [diff] [blame] | 322 | if (mAutoRefresh != bufferItem.mAutoRefresh) { |
| 323 | t->setAutoRefresh(mSurfaceControl, bufferItem.mAutoRefresh); |
| 324 | mAutoRefresh = bufferItem.mAutoRefresh; |
| 325 | } |
| 326 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 327 | if (applyTransaction) { |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 328 | t->apply(); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 329 | } |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 330 | |
| 331 | BQA_LOGV("processNextBufferLocked size=%dx%d mFrameNumber=%" PRIu64 |
| 332 | " applyTransaction=%s mTimestamp=%" PRId64, |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 333 | mSize.width, mSize.height, bufferItem.mFrameNumber, toString(applyTransaction), |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 334 | bufferItem.mTimestamp); |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 335 | } |
| 336 | |
Valerie Hau | 45e4b3b | 2019-12-03 10:49:17 -0800 | [diff] [blame] | 337 | Rect BLASTBufferQueue::computeCrop(const BufferItem& item) { |
| 338 | if (item.mScalingMode == NATIVE_WINDOW_SCALING_MODE_SCALE_CROP) { |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 339 | return GLConsumer::scaleDownCrop(item.mCrop, mSize.width, mSize.height); |
Valerie Hau | 45e4b3b | 2019-12-03 10:49:17 -0800 | [diff] [blame] | 340 | } |
| 341 | return item.mCrop; |
| 342 | } |
| 343 | |
Vishnu Nair | aef1de9 | 2020-10-22 12:15:53 -0700 | [diff] [blame] | 344 | void BLASTBufferQueue::onFrameAvailable(const BufferItem& item) { |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 345 | ATRACE_CALL(); |
Valerie Hau | 0188adf | 2020-02-13 08:29:20 -0800 | [diff] [blame] | 346 | std::unique_lock _lock{mMutex}; |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 347 | |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 348 | const bool nextTransactionSet = mNextTransaction != nullptr; |
Vishnu Nair | aef1de9 | 2020-10-22 12:15:53 -0700 | [diff] [blame] | 349 | BQA_LOGV("onFrameAvailable framenumber=%" PRIu64 " nextTransactionSet=%s mFlushShadowQueue=%s", |
| 350 | item.mFrameNumber, toString(nextTransactionSet), toString(mFlushShadowQueue)); |
Vishnu Nair | dab9409 | 2020-09-29 16:09:04 -0700 | [diff] [blame] | 351 | |
Vishnu Nair | 7eb670a | 2020-10-15 12:16:10 -0700 | [diff] [blame] | 352 | if (nextTransactionSet || mFlushShadowQueue) { |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 353 | while (mNumFrameAvailable > 0 || maxBuffersAcquired()) { |
Vishnu Nair | 7eb670a | 2020-10-15 12:16:10 -0700 | [diff] [blame] | 354 | BQA_LOGV("waiting in onFrameAvailable..."); |
Valerie Hau | 0188adf | 2020-02-13 08:29:20 -0800 | [diff] [blame] | 355 | mCallbackCV.wait(_lock); |
| 356 | } |
| 357 | } |
Vishnu Nair | 7eb670a | 2020-10-15 12:16:10 -0700 | [diff] [blame] | 358 | mFlushShadowQueue = false; |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 359 | // add to shadow queue |
Valerie Hau | a32c552 | 2019-12-09 10:11:08 -0800 | [diff] [blame] | 360 | mNumFrameAvailable++; |
Robert Carr | 255acdc | 2020-04-17 14:08:55 -0700 | [diff] [blame] | 361 | processNextBufferLocked(true); |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 362 | } |
| 363 | |
Vishnu Nair | aef1de9 | 2020-10-22 12:15:53 -0700 | [diff] [blame] | 364 | void BLASTBufferQueue::onFrameReplaced(const BufferItem& item) { |
| 365 | BQA_LOGV("onFrameReplaced framenumber=%" PRIu64, item.mFrameNumber); |
| 366 | // Do nothing since we are not storing unacquired buffer items locally. |
| 367 | } |
| 368 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 369 | void BLASTBufferQueue::setNextTransaction(SurfaceComposerClient::Transaction* t) { |
Valerie Hau | d3b90d2 | 2019-11-06 09:37:31 -0800 | [diff] [blame] | 370 | std::lock_guard _lock{mMutex}; |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 371 | mNextTransaction = t; |
| 372 | } |
| 373 | |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 374 | bool BLASTBufferQueue::rejectBuffer(const BufferItem& item) { |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 375 | if (item.mScalingMode != NATIVE_WINDOW_SCALING_MODE_FREEZE) { |
| 376 | // Only reject buffers if scaling mode is freeze. |
| 377 | return false; |
| 378 | } |
| 379 | |
Vishnu Nair | e1a4232 | 2020-10-02 17:42:04 -0700 | [diff] [blame] | 380 | uint32_t bufWidth = item.mGraphicBuffer->getWidth(); |
| 381 | uint32_t bufHeight = item.mGraphicBuffer->getHeight(); |
| 382 | |
| 383 | // Take the buffer's orientation into account |
| 384 | if (item.mTransform & ui::Transform::ROT_90) { |
| 385 | std::swap(bufWidth, bufHeight); |
| 386 | } |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 387 | ui::Size bufferSize(bufWidth, bufHeight); |
| 388 | if (mRequestedSize != mSize && mRequestedSize == bufferSize) { |
| 389 | mSize = mRequestedSize; |
| 390 | return false; |
| 391 | } |
Vishnu Nair | e1a4232 | 2020-10-02 17:42:04 -0700 | [diff] [blame] | 392 | |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 393 | // reject buffers if the buffer size doesn't match. |
Vishnu Nair | ea0de00 | 2020-11-17 17:42:37 -0800 | [diff] [blame] | 394 | return mSize != bufferSize; |
Vishnu Nair | 670b3f7 | 2020-09-29 17:52:18 -0700 | [diff] [blame] | 395 | } |
Vishnu Nair | bf25577 | 2020-10-16 10:54:41 -0700 | [diff] [blame] | 396 | |
| 397 | // Check if we have acquired the maximum number of buffers. |
| 398 | // As a special case, we wait for the first callback before acquiring the second buffer so we |
| 399 | // can ensure the first buffer is presented if multiple buffers are queued in succession. |
| 400 | bool BLASTBufferQueue::maxBuffersAcquired() const { |
| 401 | return mNumAcquired == MAX_ACQUIRED_BUFFERS + 1 || |
| 402 | (!mInitialCallbackReceived && mNumAcquired == 1); |
| 403 | } |
| 404 | |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 405 | class BBQSurface : public Surface { |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 406 | private: |
| 407 | sp<BLASTBufferQueue> mBbq; |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 408 | public: |
Vishnu Nair | 992496b | 2020-10-22 17:27:21 -0700 | [diff] [blame] | 409 | BBQSurface(const sp<IGraphicBufferProducer>& igbp, bool controlledByApp, |
| 410 | const sp<IBinder>& scHandle, const sp<BLASTBufferQueue>& bbq) |
| 411 | : Surface(igbp, controlledByApp, scHandle), mBbq(bbq) {} |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 412 | |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 413 | void allocateBuffers() override { |
| 414 | uint32_t reqWidth = mReqWidth ? mReqWidth : mUserWidth; |
| 415 | uint32_t reqHeight = mReqHeight ? mReqHeight : mUserHeight; |
| 416 | auto gbp = getIGraphicBufferProducer(); |
| 417 | std::thread ([reqWidth, reqHeight, gbp=getIGraphicBufferProducer(), |
| 418 | reqFormat=mReqFormat, reqUsage=mReqUsage] () { |
| 419 | gbp->allocateBuffers(reqWidth, reqHeight, |
| 420 | reqFormat, reqUsage); |
| 421 | |
| 422 | }).detach(); |
| 423 | } |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 424 | |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 425 | status_t setFrameRate(float frameRate, int8_t compatibility, bool shouldBeSeamless) override { |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 426 | if (!ValidateFrameRate(frameRate, compatibility, "BBQSurface::setFrameRate")) { |
| 427 | return BAD_VALUE; |
| 428 | } |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 429 | return mBbq->setFrameRate(frameRate, compatibility, shouldBeSeamless); |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 430 | } |
Robert Carr | 9b611b7 | 2020-10-19 12:00:23 -0700 | [diff] [blame] | 431 | |
| 432 | status_t setFrameTimelineVsync(int64_t frameTimelineVsyncId) override { |
| 433 | return mBbq->setFrameTimelineVsync(frameTimelineVsyncId); |
| 434 | } |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 435 | }; |
| 436 | |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 437 | // TODO: Can we coalesce this with frame updates? Need to confirm |
| 438 | // no timing issues. |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 439 | status_t BLASTBufferQueue::setFrameRate(float frameRate, int8_t compatibility, |
| 440 | bool shouldBeSeamless) { |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 441 | std::unique_lock _lock{mMutex}; |
| 442 | SurfaceComposerClient::Transaction t; |
| 443 | |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 444 | return t.setFrameRate(mSurfaceControl, frameRate, compatibility, shouldBeSeamless).apply(); |
Robert Carr | 9c006e0 | 2020-10-14 13:41:57 -0700 | [diff] [blame] | 445 | } |
| 446 | |
Robert Carr | 9b611b7 | 2020-10-19 12:00:23 -0700 | [diff] [blame] | 447 | status_t BLASTBufferQueue::setFrameTimelineVsync(int64_t frameTimelineVsyncId) { |
| 448 | std::unique_lock _lock{mMutex}; |
Jorim Jaggi | a3fe67b | 2020-12-01 00:24:33 +0100 | [diff] [blame] | 449 | mNextFrameTimelineVsyncIdQueue.push(frameTimelineVsyncId); |
| 450 | return OK; |
Robert Carr | 9b611b7 | 2020-10-19 12:00:23 -0700 | [diff] [blame] | 451 | } |
| 452 | |
Vishnu Nair | 992496b | 2020-10-22 17:27:21 -0700 | [diff] [blame] | 453 | sp<Surface> BLASTBufferQueue::getSurface(bool includeSurfaceControlHandle) { |
| 454 | std::unique_lock _lock{mMutex}; |
| 455 | sp<IBinder> scHandle = nullptr; |
| 456 | if (includeSurfaceControlHandle && mSurfaceControl) { |
| 457 | scHandle = mSurfaceControl->getHandle(); |
| 458 | } |
| 459 | return new BBQSurface(mProducer, true, scHandle, this); |
Robert Carr | 05086b2 | 2020-10-13 18:22:51 -0700 | [diff] [blame] | 460 | } |
| 461 | |
Vishnu Nair | 8949612 | 2020-12-14 17:14:53 -0800 | [diff] [blame] | 462 | // Maintains a single worker thread per process that services a list of runnables. |
| 463 | class AsyncWorker : public Singleton<AsyncWorker> { |
| 464 | private: |
| 465 | std::thread mThread; |
| 466 | bool mDone = false; |
| 467 | std::deque<std::function<void()>> mRunnables; |
| 468 | std::mutex mMutex; |
| 469 | std::condition_variable mCv; |
| 470 | void run() { |
| 471 | std::unique_lock<std::mutex> lock(mMutex); |
| 472 | while (!mDone) { |
| 473 | mCv.wait(lock); |
| 474 | while (!mRunnables.empty()) { |
| 475 | std::function<void()> runnable = mRunnables.front(); |
| 476 | mRunnables.pop_front(); |
| 477 | runnable(); |
| 478 | } |
| 479 | } |
| 480 | } |
| 481 | |
| 482 | public: |
| 483 | AsyncWorker() : Singleton<AsyncWorker>() { mThread = std::thread(&AsyncWorker::run, this); } |
| 484 | |
| 485 | ~AsyncWorker() { |
| 486 | mDone = true; |
| 487 | mCv.notify_all(); |
| 488 | if (mThread.joinable()) { |
| 489 | mThread.join(); |
| 490 | } |
| 491 | } |
| 492 | |
| 493 | void post(std::function<void()> runnable) { |
| 494 | std::unique_lock<std::mutex> lock(mMutex); |
| 495 | mRunnables.emplace_back(std::move(runnable)); |
| 496 | mCv.notify_one(); |
| 497 | } |
| 498 | }; |
| 499 | ANDROID_SINGLETON_STATIC_INSTANCE(AsyncWorker); |
| 500 | |
| 501 | // Asynchronously calls ProducerListener functions so we can emulate one way binder calls. |
| 502 | class AsyncProducerListener : public BnProducerListener { |
| 503 | private: |
| 504 | const sp<IProducerListener> mListener; |
| 505 | |
| 506 | public: |
| 507 | AsyncProducerListener(const sp<IProducerListener>& listener) : mListener(listener) {} |
| 508 | |
| 509 | void onBufferReleased() override { |
| 510 | AsyncWorker::getInstance().post([listener = mListener]() { listener->onBufferReleased(); }); |
| 511 | } |
| 512 | |
| 513 | void onBuffersDiscarded(const std::vector<int32_t>& slots) override { |
| 514 | AsyncWorker::getInstance().post( |
| 515 | [listener = mListener, slots = slots]() { listener->onBuffersDiscarded(slots); }); |
| 516 | } |
| 517 | }; |
| 518 | |
| 519 | // Extends the BufferQueueProducer to create a wrapper around the listener so the listener calls |
| 520 | // can be non-blocking when the producer is in the client process. |
| 521 | class BBQBufferQueueProducer : public BufferQueueProducer { |
| 522 | public: |
| 523 | BBQBufferQueueProducer(const sp<BufferQueueCore>& core) |
| 524 | : BufferQueueProducer(core, false /* consumerIsSurfaceFlinger*/) {} |
| 525 | |
| 526 | status_t connect(const sp<IProducerListener>& listener, int api, bool producerControlledByApp, |
| 527 | QueueBufferOutput* output) override { |
| 528 | if (!listener) { |
| 529 | return BufferQueueProducer::connect(listener, api, producerControlledByApp, output); |
| 530 | } |
| 531 | |
| 532 | return BufferQueueProducer::connect(new AsyncProducerListener(listener), api, |
| 533 | producerControlledByApp, output); |
| 534 | } |
| 535 | }; |
| 536 | |
| 537 | // Similar to BufferQueue::createBufferQueue but creates an adapter specific bufferqueue producer. |
| 538 | // This BQP allows invoking client specified ProducerListeners and invoke them asynchronously, |
| 539 | // emulating one way binder call behavior. Without this, if the listener calls back into the queue, |
| 540 | // we can deadlock. |
| 541 | void BLASTBufferQueue::createBufferQueue(sp<IGraphicBufferProducer>* outProducer, |
| 542 | sp<IGraphicBufferConsumer>* outConsumer) { |
| 543 | LOG_ALWAYS_FATAL_IF(outProducer == nullptr, "BLASTBufferQueue: outProducer must not be NULL"); |
| 544 | LOG_ALWAYS_FATAL_IF(outConsumer == nullptr, "BLASTBufferQueue: outConsumer must not be NULL"); |
| 545 | |
| 546 | sp<BufferQueueCore> core(new BufferQueueCore()); |
| 547 | LOG_ALWAYS_FATAL_IF(core == nullptr, "BLASTBufferQueue: failed to create BufferQueueCore"); |
| 548 | |
| 549 | sp<IGraphicBufferProducer> producer(new BBQBufferQueueProducer(core)); |
| 550 | LOG_ALWAYS_FATAL_IF(producer == nullptr, |
| 551 | "BLASTBufferQueue: failed to create BBQBufferQueueProducer"); |
| 552 | |
| 553 | sp<IGraphicBufferConsumer> consumer(new BufferQueueConsumer(core)); |
| 554 | LOG_ALWAYS_FATAL_IF(consumer == nullptr, |
| 555 | "BLASTBufferQueue: failed to create BufferQueueConsumer"); |
| 556 | |
| 557 | *outProducer = producer; |
| 558 | *outConsumer = consumer; |
| 559 | } |
| 560 | |
Robert Carr | 78c25dd | 2019-08-15 14:10:33 -0700 | [diff] [blame] | 561 | } // namespace android |