Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2010 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 "ConsumerBase" |
| 18 | #define ATRACE_TAG ATRACE_TAG_GRAPHICS |
| 19 | //#define LOG_NDEBUG 0 |
| 20 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 21 | #define EGL_EGLEXT_PROTOTYPES |
| 22 | |
| 23 | #include <EGL/egl.h> |
| 24 | #include <EGL/eglext.h> |
| 25 | |
| 26 | #include <hardware/hardware.h> |
| 27 | |
Mathias Agopian | e9e9fe4 | 2017-02-28 16:25:16 -0800 | [diff] [blame] | 28 | #include <cutils/atomic.h> |
| 29 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 30 | #include <com_android_graphics_libgui_flags.h> |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 31 | #include <gui/BufferItem.h> |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 32 | #include <gui/BufferQueue.h> |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 33 | #include <gui/ConsumerBase.h> |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 34 | #include <gui/ISurfaceComposer.h> |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 35 | #include <gui/Surface.h> |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 36 | #include <gui/SurfaceComposerClient.h> |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 37 | |
| 38 | #include <private/gui/ComposerService.h> |
| 39 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 40 | #include <log/log.h> |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 41 | #include <utils/Log.h> |
| 42 | #include <utils/String8.h> |
| 43 | #include <utils/Trace.h> |
| 44 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 45 | #include <inttypes.h> |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 46 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 47 | // Macros for including the ConsumerBase name in log messages |
Tomasz Wasilczyk | b19fe0c | 2023-08-11 00:06:51 +0000 | [diff] [blame] | 48 | #define CB_LOGV(x, ...) ALOGV("[%s] " x, mName.c_str(), ##__VA_ARGS__) |
| 49 | // #define CB_LOGD(x, ...) ALOGD("[%s] " x, mName.c_str(), ##__VA_ARGS__) |
| 50 | // #define CB_LOGI(x, ...) ALOGI("[%s] " x, mName.c_str(), ##__VA_ARGS__) |
| 51 | // #define CB_LOGW(x, ...) ALOGW("[%s] " x, mName.c_str(), ##__VA_ARGS__) |
| 52 | #define CB_LOGE(x, ...) ALOGE("[%s] " x, mName.c_str(), ##__VA_ARGS__) |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 53 | |
| 54 | namespace android { |
| 55 | |
| 56 | // Get an ID that's unique within this process. |
| 57 | static int32_t createProcessUniqueId() { |
| 58 | static volatile int32_t globalCounter = 0; |
| 59 | return android_atomic_inc(&globalCounter); |
| 60 | } |
| 61 | |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 62 | ConsumerBase::ConsumerBase(const sp<IGraphicBufferConsumer>& bufferQueue, bool controlledByApp) : |
Jamie Gennis | 9fea342 | 2012-08-07 18:03:04 -0700 | [diff] [blame] | 63 | mAbandoned(false), |
Brian Anderson | 3546a3f | 2016-07-14 11:51:14 -0700 | [diff] [blame] | 64 | mConsumer(bufferQueue), |
| 65 | mPrevFinalReleaseFence(Fence::NO_FENCE) { |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 66 | initialize(controlledByApp); |
| 67 | } |
| 68 | |
| 69 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 70 | ConsumerBase::ConsumerBase(bool controlledByApp, bool consumerIsSurfaceFlinger) |
| 71 | : mAbandoned(false), mPrevFinalReleaseFence(Fence::NO_FENCE) { |
| 72 | sp<IGraphicBufferProducer> producer; |
| 73 | BufferQueue::createBufferQueue(&producer, &mConsumer, consumerIsSurfaceFlinger); |
| 74 | mSurface = sp<Surface>::make(producer, controlledByApp); |
| 75 | initialize(controlledByApp); |
| 76 | } |
| 77 | |
| 78 | ConsumerBase::ConsumerBase(const sp<IGraphicBufferProducer>& producer, |
| 79 | const sp<IGraphicBufferConsumer>& consumer, bool controlledByApp) |
| 80 | : mAbandoned(false), |
| 81 | mConsumer(consumer), |
| 82 | mSurface(sp<Surface>::make(producer, controlledByApp)), |
| 83 | mPrevFinalReleaseFence(Fence::NO_FENCE) { |
| 84 | initialize(controlledByApp); |
| 85 | } |
| 86 | |
| 87 | #endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 88 | |
| 89 | void ConsumerBase::initialize(bool controlledByApp) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 90 | // Choose a name using the PID and a process-unique ID. |
| 91 | mName = String8::format("unnamed-%d-%d", getpid(), createProcessUniqueId()); |
| 92 | |
| 93 | // Note that we can't create an sp<...>(this) in a ctor that will not keep a |
| 94 | // reference once the ctor ends, as that would cause the refcount of 'this' |
| 95 | // dropping to 0 at the end of the ctor. Since all we need is a wp<...> |
| 96 | // that's what we create. |
Mathias Agopian | a4e1952 | 2013-07-31 20:09:53 -0700 | [diff] [blame] | 97 | wp<ConsumerListener> listener = static_cast<ConsumerListener*>(this); |
| 98 | sp<IConsumerListener> proxy = new BufferQueue::ProxyConsumerListener(listener); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 99 | |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 100 | status_t err = mConsumer->consumerConnect(proxy, controlledByApp); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 101 | if (err != NO_ERROR) { |
Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 102 | CB_LOGE("ConsumerBase: error connecting to BufferQueue: %s (%d)", |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 103 | strerror(-err), err); |
| 104 | } else { |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 105 | mConsumer->setConsumerName(mName); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 106 | } |
| 107 | } |
| 108 | |
| 109 | ConsumerBase::~ConsumerBase() { |
Jamie Gennis | ad669b0 | 2013-04-05 16:41:27 -0700 | [diff] [blame] | 110 | CB_LOGV("~ConsumerBase"); |
Pablo Ceballos | 22b5702 | 2016-02-19 17:41:54 -0800 | [diff] [blame] | 111 | Mutex::Autolock lock(mMutex); |
Pablo Ceballos | e07e3e5 | 2016-03-15 15:07:54 -0700 | [diff] [blame] | 112 | |
Jamie Gennis | ad669b0 | 2013-04-05 16:41:27 -0700 | [diff] [blame] | 113 | // Verify that abandon() has been called before we get here. This should |
| 114 | // be done by ConsumerBase::onLastStrongRef(), but it's possible for a |
| 115 | // derived class to override that method and not call |
| 116 | // ConsumerBase::onLastStrongRef(). |
Tomasz Wasilczyk | b19fe0c | 2023-08-11 00:06:51 +0000 | [diff] [blame] | 117 | LOG_ALWAYS_FATAL_IF(!mAbandoned, |
| 118 | "[%s] ~ConsumerBase was called, but the " |
| 119 | "consumer is not abandoned!", |
| 120 | mName.c_str()); |
Jamie Gennis | ad669b0 | 2013-04-05 16:41:27 -0700 | [diff] [blame] | 121 | } |
| 122 | |
Igor Murashkin | 7d2d160 | 2013-11-12 18:02:20 -0800 | [diff] [blame] | 123 | void ConsumerBase::onLastStrongRef(const void* id __attribute__((unused))) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 124 | abandon(); |
| 125 | } |
| 126 | |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 127 | int ConsumerBase::getSlotForBufferLocked(const sp<GraphicBuffer>& buffer) { |
| 128 | if (!buffer) { |
| 129 | return BufferQueue::INVALID_BUFFER_SLOT; |
| 130 | } |
| 131 | |
| 132 | uint64_t id = buffer->getId(); |
| 133 | for (int i = 0; i < BufferQueueDefs::NUM_BUFFER_SLOTS; i++) { |
| 134 | auto& slot = mSlots[i]; |
| 135 | if (slot.mGraphicBuffer && slot.mGraphicBuffer->getId() == id) { |
| 136 | return i; |
| 137 | } |
| 138 | } |
| 139 | |
| 140 | return BufferQueue::INVALID_BUFFER_SLOT; |
| 141 | } |
| 142 | |
| 143 | status_t ConsumerBase::detachBufferLocked(int slotIndex) { |
| 144 | status_t result = mConsumer->detachBuffer(slotIndex); |
| 145 | |
| 146 | if (result != NO_ERROR) { |
| 147 | CB_LOGE("Failed to detach buffer: %d", result); |
| 148 | return result; |
| 149 | } |
| 150 | |
| 151 | freeBufferLocked(slotIndex); |
| 152 | |
| 153 | return result; |
| 154 | } |
| 155 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 156 | void ConsumerBase::freeBufferLocked(int slotIndex) { |
| 157 | CB_LOGV("freeBufferLocked: slotIndex=%d", slotIndex); |
Yi Kong | 48a619f | 2018-06-05 16:34:59 -0700 | [diff] [blame] | 158 | mSlots[slotIndex].mGraphicBuffer = nullptr; |
Jamie Gennis | 1df8c34 | 2012-12-20 14:05:45 -0800 | [diff] [blame] | 159 | mSlots[slotIndex].mFence = Fence::NO_FENCE; |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 160 | mSlots[slotIndex].mFrameNumber = 0; |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 161 | } |
| 162 | |
Adithya Srinivasan | 2e43438 | 2019-10-09 11:43:01 -0700 | [diff] [blame] | 163 | void ConsumerBase::onFrameDequeued(const uint64_t bufferId) { |
| 164 | CB_LOGV("onFrameDequeued"); |
| 165 | |
| 166 | sp<FrameAvailableListener> listener; |
| 167 | { |
| 168 | Mutex::Autolock lock(mFrameAvailableMutex); |
| 169 | listener = mFrameAvailableListener.promote(); |
| 170 | } |
| 171 | |
| 172 | if (listener != nullptr) { |
| 173 | listener->onFrameDequeued(bufferId); |
| 174 | } |
| 175 | } |
| 176 | |
| 177 | void ConsumerBase::onFrameCancelled(const uint64_t bufferId) { |
| 178 | CB_LOGV("onFrameCancelled"); |
| 179 | |
| 180 | sp<FrameAvailableListener> listener; |
| 181 | { |
| 182 | Mutex::Autolock lock(mFrameAvailableMutex); |
| 183 | listener = mFrameAvailableListener.promote(); |
| 184 | } |
| 185 | |
| 186 | if (listener != nullptr) { |
| 187 | listener->onFrameCancelled(bufferId); |
| 188 | } |
| 189 | } |
| 190 | |
| 191 | void ConsumerBase::onFrameDetached(const uint64_t bufferId) { |
| 192 | CB_LOGV("onFrameDetached"); |
| 193 | |
| 194 | sp<FrameAvailableListener> listener; |
| 195 | { |
| 196 | Mutex::Autolock lock(mFrameAvailableMutex); |
| 197 | listener = mFrameAvailableListener.promote(); |
| 198 | } |
| 199 | |
| 200 | if (listener != nullptr) { |
| 201 | listener->onFrameDetached(bufferId); |
| 202 | } |
| 203 | } |
| 204 | |
Dan Stoza | 8dc5539 | 2014-11-04 11:37:46 -0800 | [diff] [blame] | 205 | void ConsumerBase::onFrameAvailable(const BufferItem& item) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 206 | CB_LOGV("onFrameAvailable"); |
| 207 | |
| 208 | sp<FrameAvailableListener> listener; |
| 209 | { // scope for the lock |
Dan Stoza | 95971c8 | 2017-06-26 14:27:18 -0700 | [diff] [blame] | 210 | Mutex::Autolock lock(mFrameAvailableMutex); |
Igor Murashkin | a4a3149 | 2012-10-29 13:36:11 -0700 | [diff] [blame] | 211 | listener = mFrameAvailableListener.promote(); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 212 | } |
| 213 | |
Yi Kong | 48a619f | 2018-06-05 16:34:59 -0700 | [diff] [blame] | 214 | if (listener != nullptr) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 215 | CB_LOGV("actually calling onFrameAvailable"); |
Dan Stoza | 8dc5539 | 2014-11-04 11:37:46 -0800 | [diff] [blame] | 216 | listener->onFrameAvailable(item); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 217 | } |
| 218 | } |
| 219 | |
Dan Stoza | dc13c5b | 2015-05-11 15:33:01 -0700 | [diff] [blame] | 220 | void ConsumerBase::onFrameReplaced(const BufferItem &item) { |
| 221 | CB_LOGV("onFrameReplaced"); |
| 222 | |
| 223 | sp<FrameAvailableListener> listener; |
| 224 | { |
Dan Stoza | 95971c8 | 2017-06-26 14:27:18 -0700 | [diff] [blame] | 225 | Mutex::Autolock lock(mFrameAvailableMutex); |
Dan Stoza | dc13c5b | 2015-05-11 15:33:01 -0700 | [diff] [blame] | 226 | listener = mFrameAvailableListener.promote(); |
| 227 | } |
| 228 | |
Yi Kong | 48a619f | 2018-06-05 16:34:59 -0700 | [diff] [blame] | 229 | if (listener != nullptr) { |
Dan Stoza | dc13c5b | 2015-05-11 15:33:01 -0700 | [diff] [blame] | 230 | CB_LOGV("actually calling onFrameReplaced"); |
| 231 | listener->onFrameReplaced(item); |
| 232 | } |
| 233 | } |
| 234 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 235 | void ConsumerBase::onBuffersReleased() { |
Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 236 | Mutex::Autolock lock(mMutex); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 237 | |
Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 238 | CB_LOGV("onBuffersReleased"); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 239 | |
Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 240 | if (mAbandoned) { |
| 241 | // Nothing to do if we're already abandoned. |
| 242 | return; |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 243 | } |
| 244 | |
Dan Stoza | febd4f4 | 2014-04-09 16:14:51 -0700 | [diff] [blame] | 245 | uint64_t mask = 0; |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 246 | mConsumer->getReleasedBuffers(&mask); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 247 | for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { |
Dan Stoza | febd4f4 | 2014-04-09 16:14:51 -0700 | [diff] [blame] | 248 | if (mask & (1ULL << i)) { |
Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 249 | freeBufferLocked(i); |
| 250 | } |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 251 | } |
| 252 | } |
| 253 | |
Jesse Hall | 399184a | 2014-03-03 15:42:54 -0800 | [diff] [blame] | 254 | void ConsumerBase::onSidebandStreamChanged() { |
| 255 | } |
| 256 | |
Jim Shargo | 2e614a4 | 2024-10-02 19:31:53 +0000 | [diff] [blame^] | 257 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_UNLIMITED_SLOTS) |
| 258 | void ConsumerBase::onSlotCountChanged(int slotCount) { |
| 259 | CB_LOGV("onSlotCountChanged: %d", slotCount); |
| 260 | Mutex::Autolock lock(mMutex); |
| 261 | |
| 262 | if (slotCount > (int)mSlots.size()) { |
| 263 | mSlots.resize(slotCount); |
| 264 | } |
| 265 | } |
| 266 | #endif |
| 267 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 268 | void ConsumerBase::abandon() { |
| 269 | CB_LOGV("abandon"); |
| 270 | Mutex::Autolock lock(mMutex); |
| 271 | |
| 272 | if (!mAbandoned) { |
| 273 | abandonLocked(); |
| 274 | mAbandoned = true; |
| 275 | } |
| 276 | } |
| 277 | |
| 278 | void ConsumerBase::abandonLocked() { |
Brian Carlstrom | 83b1e68 | 2016-03-12 16:07:59 -0800 | [diff] [blame] | 279 | CB_LOGV("abandonLocked"); |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 280 | if (mAbandoned) { |
| 281 | CB_LOGE("abandonLocked: ConsumerBase is abandoned!"); |
| 282 | return; |
| 283 | } |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 284 | for (int i =0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { |
| 285 | freeBufferLocked(i); |
| 286 | } |
| 287 | // disconnect from the BufferQueue |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 288 | mConsumer->consumerDisconnect(); |
| 289 | mConsumer.clear(); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 290 | } |
| 291 | |
John Reck | e478305 | 2015-05-14 15:55:11 -0700 | [diff] [blame] | 292 | bool ConsumerBase::isAbandoned() { |
| 293 | Mutex::Autolock _l(mMutex); |
| 294 | return mAbandoned; |
| 295 | } |
| 296 | |
Chia-I Wu | a81bc49 | 2017-11-27 10:16:00 -0800 | [diff] [blame] | 297 | void ConsumerBase::setName(const String8& name) { |
| 298 | Mutex::Autolock _l(mMutex); |
| 299 | if (mAbandoned) { |
| 300 | CB_LOGE("setName: ConsumerBase is abandoned!"); |
| 301 | return; |
| 302 | } |
| 303 | mName = name; |
| 304 | mConsumer->setConsumerName(name); |
| 305 | } |
| 306 | |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 307 | void ConsumerBase::setFrameAvailableListener( |
Igor Murashkin | a4a3149 | 2012-10-29 13:36:11 -0700 | [diff] [blame] | 308 | const wp<FrameAvailableListener>& listener) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 309 | CB_LOGV("setFrameAvailableListener"); |
Dan Stoza | 95971c8 | 2017-06-26 14:27:18 -0700 | [diff] [blame] | 310 | Mutex::Autolock lock(mFrameAvailableMutex); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 311 | mFrameAvailableListener = listener; |
| 312 | } |
| 313 | |
Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 314 | status_t ConsumerBase::detachBuffer(int slot) { |
| 315 | CB_LOGV("detachBuffer"); |
| 316 | Mutex::Autolock lock(mMutex); |
| 317 | |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 318 | if (mAbandoned) { |
| 319 | CB_LOGE("detachBuffer: ConsumerBase is abandoned!"); |
| 320 | return NO_INIT; |
| 321 | } |
| 322 | |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 323 | return detachBufferLocked(slot); |
| 324 | } |
| 325 | |
| 326 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_PLATFORM_API_IMPROVEMENTS) |
| 327 | status_t ConsumerBase::detachBuffer(const sp<GraphicBuffer>& buffer) { |
| 328 | CB_LOGV("detachBuffer"); |
| 329 | Mutex::Autolock lock(mMutex); |
| 330 | |
| 331 | if (mAbandoned) { |
| 332 | CB_LOGE("detachBuffer: ConsumerBase is abandoned!"); |
| 333 | return NO_INIT; |
| 334 | } |
| 335 | if (buffer == nullptr) { |
| 336 | return BAD_VALUE; |
Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 337 | } |
| 338 | |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 339 | int slotIndex = getSlotForBufferLocked(buffer); |
| 340 | if (slotIndex == BufferQueue::INVALID_BUFFER_SLOT) { |
| 341 | return BAD_VALUE; |
| 342 | } |
Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 343 | |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 344 | return detachBufferLocked(slotIndex); |
Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 345 | } |
Jim Shargo | 28e9d65 | 2024-07-10 23:50:00 +0000 | [diff] [blame] | 346 | #endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_PLATFORM_API_IMPROVEMENTS) |
Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 347 | |
Michael Lentine | 847f11e | 2015-05-18 13:41:23 -0700 | [diff] [blame] | 348 | status_t ConsumerBase::setDefaultBufferSize(uint32_t width, uint32_t height) { |
| 349 | Mutex::Autolock _l(mMutex); |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 350 | if (mAbandoned) { |
| 351 | CB_LOGE("setDefaultBufferSize: ConsumerBase is abandoned!"); |
| 352 | return NO_INIT; |
| 353 | } |
Michael Lentine | 847f11e | 2015-05-18 13:41:23 -0700 | [diff] [blame] | 354 | return mConsumer->setDefaultBufferSize(width, height); |
| 355 | } |
| 356 | |
| 357 | status_t ConsumerBase::setDefaultBufferFormat(PixelFormat defaultFormat) { |
| 358 | Mutex::Autolock _l(mMutex); |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 359 | if (mAbandoned) { |
| 360 | CB_LOGE("setDefaultBufferFormat: ConsumerBase is abandoned!"); |
| 361 | return NO_INIT; |
| 362 | } |
Michael Lentine | 847f11e | 2015-05-18 13:41:23 -0700 | [diff] [blame] | 363 | return mConsumer->setDefaultBufferFormat(defaultFormat); |
| 364 | } |
| 365 | |
| 366 | status_t ConsumerBase::setDefaultBufferDataSpace( |
| 367 | android_dataspace defaultDataSpace) { |
| 368 | Mutex::Autolock _l(mMutex); |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 369 | if (mAbandoned) { |
| 370 | CB_LOGE("setDefaultBufferDataSpace: ConsumerBase is abandoned!"); |
| 371 | return NO_INIT; |
| 372 | } |
Michael Lentine | 847f11e | 2015-05-18 13:41:23 -0700 | [diff] [blame] | 373 | return mConsumer->setDefaultBufferDataSpace(defaultDataSpace); |
| 374 | } |
| 375 | |
Chia-I Wu | a81bc49 | 2017-11-27 10:16:00 -0800 | [diff] [blame] | 376 | status_t ConsumerBase::setConsumerUsageBits(uint64_t usage) { |
| 377 | Mutex::Autolock lock(mMutex); |
| 378 | if (mAbandoned) { |
| 379 | CB_LOGE("setConsumerUsageBits: ConsumerBase is abandoned!"); |
| 380 | return NO_INIT; |
| 381 | } |
| 382 | return mConsumer->setConsumerUsageBits(usage); |
| 383 | } |
| 384 | |
| 385 | status_t ConsumerBase::setTransformHint(uint32_t hint) { |
| 386 | Mutex::Autolock lock(mMutex); |
| 387 | if (mAbandoned) { |
| 388 | CB_LOGE("setTransformHint: ConsumerBase is abandoned!"); |
| 389 | return NO_INIT; |
| 390 | } |
| 391 | return mConsumer->setTransformHint(hint); |
| 392 | } |
| 393 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 394 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 395 | status_t ConsumerBase::setMaxBufferCount(int bufferCount) { |
| 396 | Mutex::Autolock lock(mMutex); |
| 397 | if (mAbandoned) { |
| 398 | CB_LOGE("setMaxBufferCount: ConsumerBase is abandoned!"); |
| 399 | return NO_INIT; |
| 400 | } |
| 401 | return mConsumer->setMaxBufferCount(bufferCount); |
| 402 | } |
| 403 | #endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 404 | |
Chia-I Wu | a81bc49 | 2017-11-27 10:16:00 -0800 | [diff] [blame] | 405 | status_t ConsumerBase::setMaxAcquiredBufferCount(int maxAcquiredBuffers) { |
| 406 | Mutex::Autolock lock(mMutex); |
| 407 | if (mAbandoned) { |
| 408 | CB_LOGE("setMaxAcquiredBufferCount: ConsumerBase is abandoned!"); |
| 409 | return NO_INIT; |
| 410 | } |
| 411 | return mConsumer->setMaxAcquiredBufferCount(maxAcquiredBuffers); |
| 412 | } |
| 413 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 414 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 415 | status_t ConsumerBase::setConsumerIsProtected(bool isProtected) { |
| 416 | Mutex::Autolock lock(mMutex); |
| 417 | if (mAbandoned) { |
| 418 | CB_LOGE("setConsumerIsProtected: ConsumerBase is abandoned!"); |
| 419 | return NO_INIT; |
| 420 | } |
| 421 | return mConsumer->setConsumerIsProtected(isProtected); |
| 422 | } |
| 423 | #endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 424 | |
Chia-I Wu | a81bc49 | 2017-11-27 10:16:00 -0800 | [diff] [blame] | 425 | sp<NativeHandle> ConsumerBase::getSidebandStream() const { |
| 426 | Mutex::Autolock _l(mMutex); |
| 427 | if (mAbandoned) { |
| 428 | CB_LOGE("getSidebandStream: ConsumerBase is abandoned!"); |
| 429 | return nullptr; |
| 430 | } |
| 431 | |
| 432 | sp<NativeHandle> stream; |
| 433 | status_t err = mConsumer->getSidebandStream(&stream); |
| 434 | if (err != NO_ERROR) { |
| 435 | CB_LOGE("failed to get sideband stream: %d", err); |
| 436 | return nullptr; |
| 437 | } |
| 438 | |
| 439 | return stream; |
| 440 | } |
| 441 | |
Dan Stoza | e77c766 | 2016-05-13 11:37:28 -0700 | [diff] [blame] | 442 | status_t ConsumerBase::getOccupancyHistory(bool forceFlush, |
| 443 | std::vector<OccupancyTracker::Segment>* outHistory) { |
| 444 | Mutex::Autolock _l(mMutex); |
| 445 | if (mAbandoned) { |
| 446 | CB_LOGE("getOccupancyHistory: ConsumerBase is abandoned!"); |
| 447 | return NO_INIT; |
| 448 | } |
| 449 | return mConsumer->getOccupancyHistory(forceFlush, outHistory); |
| 450 | } |
| 451 | |
Eino-Ville Talvala | bc2df65 | 2016-07-21 17:06:58 -0700 | [diff] [blame] | 452 | status_t ConsumerBase::discardFreeBuffers() { |
| 453 | Mutex::Autolock _l(mMutex); |
| 454 | if (mAbandoned) { |
| 455 | CB_LOGE("discardFreeBuffers: ConsumerBase is abandoned!"); |
| 456 | return NO_INIT; |
| 457 | } |
Eino-Ville Talvala | c6ff798 | 2017-06-13 17:09:11 -0700 | [diff] [blame] | 458 | status_t err = mConsumer->discardFreeBuffers(); |
| 459 | if (err != OK) { |
| 460 | return err; |
| 461 | } |
| 462 | uint64_t mask; |
| 463 | mConsumer->getReleasedBuffers(&mask); |
| 464 | for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { |
| 465 | if (mask & (1ULL << i)) { |
| 466 | freeBufferLocked(i); |
| 467 | } |
| 468 | } |
| 469 | return OK; |
Eino-Ville Talvala | bc2df65 | 2016-07-21 17:06:58 -0700 | [diff] [blame] | 470 | } |
| 471 | |
Colin Cross | dc78251 | 2016-09-26 18:10:16 -0700 | [diff] [blame] | 472 | void ConsumerBase::dumpState(String8& result) const { |
| 473 | dumpState(result, ""); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 474 | } |
| 475 | |
Colin Cross | dc78251 | 2016-09-26 18:10:16 -0700 | [diff] [blame] | 476 | void ConsumerBase::dumpState(String8& result, const char* prefix) const { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 477 | Mutex::Autolock _l(mMutex); |
Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 478 | dumpLocked(result, prefix); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 479 | } |
| 480 | |
Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 481 | void ConsumerBase::dumpLocked(String8& result, const char* prefix) const { |
| 482 | result.appendFormat("%smAbandoned=%d\n", prefix, int(mAbandoned)); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 483 | |
| 484 | if (!mAbandoned) { |
Dan Stoza | 0c9a1ed | 2017-04-06 15:10:21 -0700 | [diff] [blame] | 485 | String8 consumerState; |
| 486 | mConsumer->dumpState(String8(prefix), &consumerState); |
| 487 | result.append(consumerState); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 488 | } |
| 489 | } |
| 490 | |
Jim Shargo | d30823a | 2024-07-27 02:49:39 +0000 | [diff] [blame] | 491 | #if COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 492 | sp<Surface> ConsumerBase::getSurface() const { |
| 493 | LOG_ALWAYS_FATAL_IF(mSurface == nullptr, |
| 494 | "It's illegal to get the surface of a Consumer that does not own it. This " |
| 495 | "should be impossible once the old CTOR is removed."); |
| 496 | return mSurface; |
| 497 | } |
| 498 | |
| 499 | sp<IGraphicBufferConsumer> ConsumerBase::getIGraphicBufferConsumer() const { |
| 500 | return mConsumer; |
| 501 | } |
| 502 | #endif // COM_ANDROID_GRAPHICS_LIBGUI_FLAGS(WB_CONSUMER_BASE_OWNS_BQ) |
| 503 | |
Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 504 | status_t ConsumerBase::acquireBufferLocked(BufferItem *item, |
Dan Stoza | a4650a5 | 2015-05-12 12:56:16 -0700 | [diff] [blame] | 505 | nsecs_t presentWhen, uint64_t maxFrameNumber) { |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 506 | if (mAbandoned) { |
| 507 | CB_LOGE("acquireBufferLocked: ConsumerBase is abandoned!"); |
| 508 | return NO_INIT; |
| 509 | } |
| 510 | |
Dan Stoza | a4650a5 | 2015-05-12 12:56:16 -0700 | [diff] [blame] | 511 | status_t err = mConsumer->acquireBuffer(item, presentWhen, maxFrameNumber); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 512 | if (err != NO_ERROR) { |
| 513 | return err; |
| 514 | } |
| 515 | |
Yi Kong | 48a619f | 2018-06-05 16:34:59 -0700 | [diff] [blame] | 516 | if (item->mGraphicBuffer != nullptr) { |
| 517 | if (mSlots[item->mSlot].mGraphicBuffer != nullptr) { |
Yin-Chia Yeh | 723c489 | 2017-03-30 13:13:36 -0700 | [diff] [blame] | 518 | freeBufferLocked(item->mSlot); |
| 519 | } |
Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 520 | mSlots[item->mSlot].mGraphicBuffer = item->mGraphicBuffer; |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 521 | } |
| 522 | |
Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 523 | mSlots[item->mSlot].mFrameNumber = item->mFrameNumber; |
| 524 | mSlots[item->mSlot].mFence = item->mFence; |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 525 | |
Mark Salyzyn | 9110045 | 2014-06-09 14:27:45 -0700 | [diff] [blame] | 526 | CB_LOGV("acquireBufferLocked: -> slot=%d/%" PRIu64, |
Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 527 | item->mSlot, item->mFrameNumber); |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 528 | |
| 529 | return OK; |
| 530 | } |
| 531 | |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 532 | status_t ConsumerBase::addReleaseFence(int slot, |
| 533 | const sp<GraphicBuffer> graphicBuffer, const sp<Fence>& fence) { |
Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 534 | Mutex::Autolock lock(mMutex); |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 535 | return addReleaseFenceLocked(slot, graphicBuffer, fence); |
Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 536 | } |
| 537 | |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 538 | status_t ConsumerBase::addReleaseFenceLocked(int slot, |
| 539 | const sp<GraphicBuffer> graphicBuffer, const sp<Fence>& fence) { |
Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 540 | CB_LOGV("addReleaseFenceLocked: slot=%d", slot); |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 541 | |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 542 | // If consumer no longer tracks this graphicBuffer, we can safely |
| 543 | // drop this fence, as it will never be received by the producer. |
| 544 | if (!stillTracking(slot, graphicBuffer)) { |
| 545 | return OK; |
| 546 | } |
| 547 | |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 548 | if (!mSlots[slot].mFence.get()) { |
| 549 | mSlots[slot].mFence = fence; |
Matthew Bouyack | 377c203 | 2016-10-07 15:06:15 -0700 | [diff] [blame] | 550 | return OK; |
| 551 | } |
| 552 | |
Brian Anderson | 7b097e2 | 2017-08-08 16:31:37 -0700 | [diff] [blame] | 553 | // Check status of fences first because merging is expensive. |
| 554 | // Merging an invalid fence with any other fence results in an |
| 555 | // invalid fence. |
| 556 | auto currentStatus = mSlots[slot].mFence->getStatus(); |
| 557 | if (currentStatus == Fence::Status::Invalid) { |
| 558 | CB_LOGE("Existing fence has invalid state"); |
Matthew Bouyack | 377c203 | 2016-10-07 15:06:15 -0700 | [diff] [blame] | 559 | return BAD_VALUE; |
| 560 | } |
| 561 | |
Brian Anderson | 7b097e2 | 2017-08-08 16:31:37 -0700 | [diff] [blame] | 562 | auto incomingStatus = fence->getStatus(); |
| 563 | if (incomingStatus == Fence::Status::Invalid) { |
| 564 | CB_LOGE("New fence has invalid state"); |
Matthew Bouyack | 377c203 | 2016-10-07 15:06:15 -0700 | [diff] [blame] | 565 | mSlots[slot].mFence = fence; |
Brian Anderson | 7b097e2 | 2017-08-08 16:31:37 -0700 | [diff] [blame] | 566 | return BAD_VALUE; |
| 567 | } |
| 568 | |
| 569 | // If both fences are signaled or both are unsignaled, we need to merge |
| 570 | // them to get an accurate timestamp. |
| 571 | if (currentStatus == incomingStatus) { |
Matthew Bouyack | fd4c8c3 | 2016-10-07 14:26:47 -0700 | [diff] [blame] | 572 | char fenceName[32] = {}; |
Tomasz Wasilczyk | b19fe0c | 2023-08-11 00:06:51 +0000 | [diff] [blame] | 573 | snprintf(fenceName, 32, "%.28s:%d", mName.c_str(), slot); |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 574 | sp<Fence> mergedFence = Fence::merge( |
Matthew Bouyack | fd4c8c3 | 2016-10-07 14:26:47 -0700 | [diff] [blame] | 575 | fenceName, mSlots[slot].mFence, fence); |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 576 | if (!mergedFence.get()) { |
| 577 | CB_LOGE("failed to merge release fences"); |
| 578 | // synchronization is broken, the best we can do is hope fences |
| 579 | // signal in order so the new fence will act like a union |
| 580 | mSlots[slot].mFence = fence; |
| 581 | return BAD_VALUE; |
| 582 | } |
| 583 | mSlots[slot].mFence = mergedFence; |
Brian Anderson | 7b097e2 | 2017-08-08 16:31:37 -0700 | [diff] [blame] | 584 | } else if (incomingStatus == Fence::Status::Unsignaled) { |
| 585 | // If one fence has signaled and the other hasn't, the unsignaled |
| 586 | // fence will approximately correspond with the correct timestamp. |
| 587 | // There's a small race if both fences signal at about the same time |
| 588 | // and their statuses are retrieved with unfortunate timing. However, |
| 589 | // by this point, they will have both signaled and only the timestamp |
| 590 | // will be slightly off; any dependencies after this point will |
| 591 | // already have been met. |
| 592 | mSlots[slot].mFence = fence; |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 593 | } |
Brian Anderson | 7b097e2 | 2017-08-08 16:31:37 -0700 | [diff] [blame] | 594 | // else if (currentStatus == Fence::Status::Unsignaled) is a no-op. |
Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 595 | |
| 596 | return OK; |
| 597 | } |
| 598 | |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 599 | status_t ConsumerBase::releaseBufferLocked( |
| 600 | int slot, const sp<GraphicBuffer> graphicBuffer, |
| 601 | EGLDisplay display, EGLSyncKHR eglFence) { |
Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 602 | if (mAbandoned) { |
| 603 | CB_LOGE("releaseBufferLocked: ConsumerBase is abandoned!"); |
| 604 | return NO_INIT; |
| 605 | } |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 606 | // If consumer no longer tracks this graphicBuffer (we received a new |
| 607 | // buffer on the same slot), the buffer producer is definitely no longer |
| 608 | // tracking it. |
| 609 | if (!stillTracking(slot, graphicBuffer)) { |
| 610 | return OK; |
| 611 | } |
| 612 | |
Mark Salyzyn | 9110045 | 2014-06-09 14:27:45 -0700 | [diff] [blame] | 613 | CB_LOGV("releaseBufferLocked: slot=%d/%" PRIu64, |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 614 | slot, mSlots[slot].mFrameNumber); |
Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 615 | status_t err = mConsumer->releaseBuffer(slot, mSlots[slot].mFrameNumber, |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 616 | display, eglFence, mSlots[slot].mFence); |
Igor Murashkin | 7d2d160 | 2013-11-12 18:02:20 -0800 | [diff] [blame] | 617 | if (err == IGraphicBufferConsumer::STALE_BUFFER_SLOT) { |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 618 | freeBufferLocked(slot); |
| 619 | } |
| 620 | |
Brian Anderson | 3546a3f | 2016-07-14 11:51:14 -0700 | [diff] [blame] | 621 | mPrevFinalReleaseFence = mSlots[slot].mFence; |
Jamie Gennis | 1df8c34 | 2012-12-20 14:05:45 -0800 | [diff] [blame] | 622 | mSlots[slot].mFence = Fence::NO_FENCE; |
Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 623 | |
| 624 | return err; |
| 625 | } |
| 626 | |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 627 | bool ConsumerBase::stillTracking(int slot, |
| 628 | const sp<GraphicBuffer> graphicBuffer) { |
| 629 | if (slot < 0 || slot >= BufferQueue::NUM_BUFFER_SLOTS) { |
| 630 | return false; |
| 631 | } |
Yi Kong | 48a619f | 2018-06-05 16:34:59 -0700 | [diff] [blame] | 632 | return (mSlots[slot].mGraphicBuffer != nullptr && |
Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 633 | mSlots[slot].mGraphicBuffer->handle == graphicBuffer->handle); |
| 634 | } |
| 635 | |
Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 636 | } // namespace android |