| 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 |  | 
| Mark Salyzyn | 9110045 | 2014-06-09 14:27:45 -0700 | [diff] [blame] | 17 | #include <inttypes.h> | 
|  | 18 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 19 | #define LOG_TAG "ConsumerBase" | 
|  | 20 | #define ATRACE_TAG ATRACE_TAG_GRAPHICS | 
|  | 21 | //#define LOG_NDEBUG 0 | 
|  | 22 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 23 | #define EGL_EGLEXT_PROTOTYPES | 
|  | 24 |  | 
|  | 25 | #include <EGL/egl.h> | 
|  | 26 | #include <EGL/eglext.h> | 
|  | 27 |  | 
|  | 28 | #include <hardware/hardware.h> | 
|  | 29 |  | 
| Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 30 | #include <gui/BufferItem.h> | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 31 | #include <gui/IGraphicBufferAlloc.h> | 
|  | 32 | #include <gui/ISurfaceComposer.h> | 
|  | 33 | #include <gui/SurfaceComposerClient.h> | 
|  | 34 | #include <gui/ConsumerBase.h> | 
|  | 35 |  | 
|  | 36 | #include <private/gui/ComposerService.h> | 
|  | 37 |  | 
|  | 38 | #include <utils/Log.h> | 
|  | 39 | #include <utils/String8.h> | 
|  | 40 | #include <utils/Trace.h> | 
|  | 41 |  | 
|  | 42 | // Macros for including the ConsumerBase name in log messages | 
| Dan Albert | 8b49125 | 2014-09-08 18:53:39 -0700 | [diff] [blame] | 43 | #define CB_LOGV(x, ...) ALOGV("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 44 | //#define CB_LOGD(x, ...) ALOGD("[%s] " x, mName.string(), ##__VA_ARGS__) | 
|  | 45 | //#define CB_LOGI(x, ...) ALOGI("[%s] " x, mName.string(), ##__VA_ARGS__) | 
|  | 46 | //#define CB_LOGW(x, ...) ALOGW("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Dan Albert | 8b49125 | 2014-09-08 18:53:39 -0700 | [diff] [blame] | 47 | #define CB_LOGE(x, ...) ALOGE("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 48 |  | 
|  | 49 | namespace android { | 
|  | 50 |  | 
|  | 51 | // Get an ID that's unique within this process. | 
|  | 52 | static int32_t createProcessUniqueId() { | 
|  | 53 | static volatile int32_t globalCounter = 0; | 
|  | 54 | return android_atomic_inc(&globalCounter); | 
|  | 55 | } | 
|  | 56 |  | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 57 | ConsumerBase::ConsumerBase(const sp<IGraphicBufferConsumer>& bufferQueue, bool controlledByApp) : | 
| Jamie Gennis | 9fea342 | 2012-08-07 18:03:04 -0700 | [diff] [blame] | 58 | mAbandoned(false), | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 59 | mConsumer(bufferQueue) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 60 | // Choose a name using the PID and a process-unique ID. | 
|  | 61 | mName = String8::format("unnamed-%d-%d", getpid(), createProcessUniqueId()); | 
|  | 62 |  | 
|  | 63 | // Note that we can't create an sp<...>(this) in a ctor that will not keep a | 
|  | 64 | // reference once the ctor ends, as that would cause the refcount of 'this' | 
|  | 65 | // dropping to 0 at the end of the ctor.  Since all we need is a wp<...> | 
|  | 66 | // that's what we create. | 
| Mathias Agopian | a4e1952 | 2013-07-31 20:09:53 -0700 | [diff] [blame] | 67 | wp<ConsumerListener> listener = static_cast<ConsumerListener*>(this); | 
|  | 68 | sp<IConsumerListener> proxy = new BufferQueue::ProxyConsumerListener(listener); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 69 |  | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 70 | status_t err = mConsumer->consumerConnect(proxy, controlledByApp); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 71 | if (err != NO_ERROR) { | 
| Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 72 | CB_LOGE("ConsumerBase: error connecting to BufferQueue: %s (%d)", | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 73 | strerror(-err), err); | 
|  | 74 | } else { | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 75 | mConsumer->setConsumerName(mName); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 76 | } | 
|  | 77 | } | 
|  | 78 |  | 
|  | 79 | ConsumerBase::~ConsumerBase() { | 
| Jamie Gennis | ad669b0 | 2013-04-05 16:41:27 -0700 | [diff] [blame] | 80 | CB_LOGV("~ConsumerBase"); | 
| Pablo Ceballos | 22b5702 | 2016-02-19 17:41:54 -0800 | [diff] [blame] | 81 | Mutex::Autolock lock(mMutex); | 
| Pablo Ceballos | e07e3e5 | 2016-03-15 15:07:54 -0700 | [diff] [blame] | 82 |  | 
| Jamie Gennis | ad669b0 | 2013-04-05 16:41:27 -0700 | [diff] [blame] | 83 | // Verify that abandon() has been called before we get here.  This should | 
|  | 84 | // be done by ConsumerBase::onLastStrongRef(), but it's possible for a | 
|  | 85 | // derived class to override that method and not call | 
|  | 86 | // ConsumerBase::onLastStrongRef(). | 
|  | 87 | LOG_ALWAYS_FATAL_IF(!mAbandoned, "[%s] ~ConsumerBase was called, but the " | 
|  | 88 | "consumer is not abandoned!", mName.string()); | 
|  | 89 | } | 
|  | 90 |  | 
| Igor Murashkin | 7d2d160 | 2013-11-12 18:02:20 -0800 | [diff] [blame] | 91 | void ConsumerBase::onLastStrongRef(const void* id __attribute__((unused))) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 92 | abandon(); | 
|  | 93 | } | 
|  | 94 |  | 
|  | 95 | void ConsumerBase::freeBufferLocked(int slotIndex) { | 
|  | 96 | CB_LOGV("freeBufferLocked: slotIndex=%d", slotIndex); | 
|  | 97 | mSlots[slotIndex].mGraphicBuffer = 0; | 
| Jamie Gennis | 1df8c34 | 2012-12-20 14:05:45 -0800 | [diff] [blame] | 98 | mSlots[slotIndex].mFence = Fence::NO_FENCE; | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 99 | mSlots[slotIndex].mFrameNumber = 0; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 100 | } | 
|  | 101 |  | 
| Dan Stoza | 8dc5539 | 2014-11-04 11:37:46 -0800 | [diff] [blame] | 102 | void ConsumerBase::onFrameAvailable(const BufferItem& item) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 103 | CB_LOGV("onFrameAvailable"); | 
|  | 104 |  | 
|  | 105 | sp<FrameAvailableListener> listener; | 
|  | 106 | { // scope for the lock | 
|  | 107 | Mutex::Autolock lock(mMutex); | 
| Igor Murashkin | a4a3149 | 2012-10-29 13:36:11 -0700 | [diff] [blame] | 108 | listener = mFrameAvailableListener.promote(); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 109 | } | 
|  | 110 |  | 
|  | 111 | if (listener != NULL) { | 
|  | 112 | CB_LOGV("actually calling onFrameAvailable"); | 
| Dan Stoza | 8dc5539 | 2014-11-04 11:37:46 -0800 | [diff] [blame] | 113 | listener->onFrameAvailable(item); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 114 | } | 
|  | 115 | } | 
|  | 116 |  | 
| Dan Stoza | dc13c5b | 2015-05-11 15:33:01 -0700 | [diff] [blame] | 117 | void ConsumerBase::onFrameReplaced(const BufferItem &item) { | 
|  | 118 | CB_LOGV("onFrameReplaced"); | 
|  | 119 |  | 
|  | 120 | sp<FrameAvailableListener> listener; | 
|  | 121 | { | 
|  | 122 | Mutex::Autolock lock(mMutex); | 
|  | 123 | listener = mFrameAvailableListener.promote(); | 
|  | 124 | } | 
|  | 125 |  | 
|  | 126 | if (listener != NULL) { | 
|  | 127 | CB_LOGV("actually calling onFrameReplaced"); | 
|  | 128 | listener->onFrameReplaced(item); | 
|  | 129 | } | 
|  | 130 | } | 
|  | 131 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 132 | void ConsumerBase::onBuffersReleased() { | 
| Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 133 | Mutex::Autolock lock(mMutex); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 134 |  | 
| Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 135 | CB_LOGV("onBuffersReleased"); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 136 |  | 
| Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 137 | if (mAbandoned) { | 
|  | 138 | // Nothing to do if we're already abandoned. | 
|  | 139 | return; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 140 | } | 
|  | 141 |  | 
| Dan Stoza | febd4f4 | 2014-04-09 16:14:51 -0700 | [diff] [blame] | 142 | uint64_t mask = 0; | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 143 | mConsumer->getReleasedBuffers(&mask); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 144 | for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { | 
| Dan Stoza | febd4f4 | 2014-04-09 16:14:51 -0700 | [diff] [blame] | 145 | if (mask & (1ULL << i)) { | 
| Jamie Gennis | 72c3f7d | 2012-12-07 00:41:56 -0800 | [diff] [blame] | 146 | freeBufferLocked(i); | 
|  | 147 | } | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 148 | } | 
|  | 149 | } | 
|  | 150 |  | 
| Jesse Hall | 399184a | 2014-03-03 15:42:54 -0800 | [diff] [blame] | 151 | void ConsumerBase::onSidebandStreamChanged() { | 
|  | 152 | } | 
|  | 153 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 154 | void ConsumerBase::abandon() { | 
|  | 155 | CB_LOGV("abandon"); | 
|  | 156 | Mutex::Autolock lock(mMutex); | 
|  | 157 |  | 
|  | 158 | if (!mAbandoned) { | 
|  | 159 | abandonLocked(); | 
|  | 160 | mAbandoned = true; | 
|  | 161 | } | 
|  | 162 | } | 
|  | 163 |  | 
|  | 164 | void ConsumerBase::abandonLocked() { | 
| Brian Carlstrom | 83b1e68 | 2016-03-12 16:07:59 -0800 | [diff] [blame] | 165 | CB_LOGV("abandonLocked"); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 166 | for (int i =0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { | 
|  | 167 | freeBufferLocked(i); | 
|  | 168 | } | 
|  | 169 | // disconnect from the BufferQueue | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 170 | mConsumer->consumerDisconnect(); | 
|  | 171 | mConsumer.clear(); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 172 | } | 
|  | 173 |  | 
| John Reck | e478305 | 2015-05-14 15:55:11 -0700 | [diff] [blame] | 174 | bool ConsumerBase::isAbandoned() { | 
|  | 175 | Mutex::Autolock _l(mMutex); | 
|  | 176 | return mAbandoned; | 
|  | 177 | } | 
|  | 178 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 179 | void ConsumerBase::setFrameAvailableListener( | 
| Igor Murashkin | a4a3149 | 2012-10-29 13:36:11 -0700 | [diff] [blame] | 180 | const wp<FrameAvailableListener>& listener) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 181 | CB_LOGV("setFrameAvailableListener"); | 
|  | 182 | Mutex::Autolock lock(mMutex); | 
|  | 183 | mFrameAvailableListener = listener; | 
|  | 184 | } | 
|  | 185 |  | 
| Dan Stoza | 634f5ee | 2015-04-03 14:22:05 -0700 | [diff] [blame] | 186 | status_t ConsumerBase::detachBuffer(int slot) { | 
|  | 187 | CB_LOGV("detachBuffer"); | 
|  | 188 | Mutex::Autolock lock(mMutex); | 
|  | 189 |  | 
|  | 190 | status_t result = mConsumer->detachBuffer(slot); | 
|  | 191 | if (result != NO_ERROR) { | 
|  | 192 | CB_LOGE("Failed to detach buffer: %d", result); | 
|  | 193 | return result; | 
|  | 194 | } | 
|  | 195 |  | 
|  | 196 | freeBufferLocked(slot); | 
|  | 197 |  | 
|  | 198 | return result; | 
|  | 199 | } | 
|  | 200 |  | 
| Michael Lentine | 847f11e | 2015-05-18 13:41:23 -0700 | [diff] [blame] | 201 | status_t ConsumerBase::setDefaultBufferSize(uint32_t width, uint32_t height) { | 
|  | 202 | Mutex::Autolock _l(mMutex); | 
|  | 203 | return mConsumer->setDefaultBufferSize(width, height); | 
|  | 204 | } | 
|  | 205 |  | 
|  | 206 | status_t ConsumerBase::setDefaultBufferFormat(PixelFormat defaultFormat) { | 
|  | 207 | Mutex::Autolock _l(mMutex); | 
|  | 208 | return mConsumer->setDefaultBufferFormat(defaultFormat); | 
|  | 209 | } | 
|  | 210 |  | 
|  | 211 | status_t ConsumerBase::setDefaultBufferDataSpace( | 
|  | 212 | android_dataspace defaultDataSpace) { | 
|  | 213 | Mutex::Autolock _l(mMutex); | 
|  | 214 | return mConsumer->setDefaultBufferDataSpace(defaultDataSpace); | 
|  | 215 | } | 
|  | 216 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 217 | void ConsumerBase::dump(String8& result) const { | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 218 | dump(result, ""); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 219 | } | 
|  | 220 |  | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 221 | void ConsumerBase::dump(String8& result, const char* prefix) const { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 222 | Mutex::Autolock _l(mMutex); | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 223 | dumpLocked(result, prefix); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 224 | } | 
|  | 225 |  | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 226 | void ConsumerBase::dumpLocked(String8& result, const char* prefix) const { | 
|  | 227 | result.appendFormat("%smAbandoned=%d\n", prefix, int(mAbandoned)); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 228 |  | 
|  | 229 | if (!mAbandoned) { | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 230 | mConsumer->dump(result, prefix); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 231 | } | 
|  | 232 | } | 
|  | 233 |  | 
| Dan Stoza | cf3834d | 2015-03-11 14:04:22 -0700 | [diff] [blame] | 234 | status_t ConsumerBase::acquireBufferLocked(BufferItem *item, | 
| Dan Stoza | a4650a5 | 2015-05-12 12:56:16 -0700 | [diff] [blame] | 235 | nsecs_t presentWhen, uint64_t maxFrameNumber) { | 
|  | 236 | status_t err = mConsumer->acquireBuffer(item, presentWhen, maxFrameNumber); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 237 | if (err != NO_ERROR) { | 
|  | 238 | return err; | 
|  | 239 | } | 
|  | 240 |  | 
|  | 241 | if (item->mGraphicBuffer != NULL) { | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 242 | mSlots[item->mSlot].mGraphicBuffer = item->mGraphicBuffer; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 243 | } | 
|  | 244 |  | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 245 | mSlots[item->mSlot].mFrameNumber = item->mFrameNumber; | 
|  | 246 | mSlots[item->mSlot].mFence = item->mFence; | 
| Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 247 |  | 
| Mark Salyzyn | 9110045 | 2014-06-09 14:27:45 -0700 | [diff] [blame] | 248 | CB_LOGV("acquireBufferLocked: -> slot=%d/%" PRIu64, | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 249 | item->mSlot, item->mFrameNumber); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 250 |  | 
|  | 251 | return OK; | 
|  | 252 | } | 
|  | 253 |  | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 254 | status_t ConsumerBase::addReleaseFence(int slot, | 
|  | 255 | const sp<GraphicBuffer> graphicBuffer, const sp<Fence>& fence) { | 
| Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 256 | Mutex::Autolock lock(mMutex); | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 257 | return addReleaseFenceLocked(slot, graphicBuffer, fence); | 
| Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 258 | } | 
|  | 259 |  | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 260 | status_t ConsumerBase::addReleaseFenceLocked(int slot, | 
|  | 261 | const sp<GraphicBuffer> graphicBuffer, const sp<Fence>& fence) { | 
| Jesse Hall | 9504eb9 | 2012-10-05 14:34:21 -0700 | [diff] [blame] | 262 | CB_LOGV("addReleaseFenceLocked: slot=%d", slot); | 
| Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 263 |  | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 264 | // If consumer no longer tracks this graphicBuffer, we can safely | 
|  | 265 | // drop this fence, as it will never be received by the producer. | 
|  | 266 | if (!stillTracking(slot, graphicBuffer)) { | 
|  | 267 | return OK; | 
|  | 268 | } | 
|  | 269 |  | 
| Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 270 | if (!mSlots[slot].mFence.get()) { | 
|  | 271 | mSlots[slot].mFence = fence; | 
|  | 272 | } else { | 
|  | 273 | sp<Fence> mergedFence = Fence::merge( | 
| Jamie Gennis | 7aff4a5 | 2012-09-24 12:25:15 -0700 | [diff] [blame] | 274 | String8::format("%.28s:%d", mName.string(), slot), | 
| Jamie Gennis | b272541 | 2012-09-05 20:09:05 -0700 | [diff] [blame] | 275 | mSlots[slot].mFence, fence); | 
|  | 276 | if (!mergedFence.get()) { | 
|  | 277 | CB_LOGE("failed to merge release fences"); | 
|  | 278 | // synchronization is broken, the best we can do is hope fences | 
|  | 279 | // signal in order so the new fence will act like a union | 
|  | 280 | mSlots[slot].mFence = fence; | 
|  | 281 | return BAD_VALUE; | 
|  | 282 | } | 
|  | 283 | mSlots[slot].mFence = mergedFence; | 
|  | 284 | } | 
|  | 285 |  | 
|  | 286 | return OK; | 
|  | 287 | } | 
|  | 288 |  | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 289 | status_t ConsumerBase::releaseBufferLocked( | 
|  | 290 | int slot, const sp<GraphicBuffer> graphicBuffer, | 
|  | 291 | EGLDisplay display, EGLSyncKHR eglFence) { | 
|  | 292 | // If consumer no longer tracks this graphicBuffer (we received a new | 
|  | 293 | // buffer on the same slot), the buffer producer is definitely no longer | 
|  | 294 | // tracking it. | 
|  | 295 | if (!stillTracking(slot, graphicBuffer)) { | 
|  | 296 | return OK; | 
|  | 297 | } | 
|  | 298 |  | 
| Mark Salyzyn | 9110045 | 2014-06-09 14:27:45 -0700 | [diff] [blame] | 299 | CB_LOGV("releaseBufferLocked: slot=%d/%" PRIu64, | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 300 | slot, mSlots[slot].mFrameNumber); | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 301 | status_t err = mConsumer->releaseBuffer(slot, mSlots[slot].mFrameNumber, | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 302 | display, eglFence, mSlots[slot].mFence); | 
| Igor Murashkin | 7d2d160 | 2013-11-12 18:02:20 -0800 | [diff] [blame] | 303 | if (err == IGraphicBufferConsumer::STALE_BUFFER_SLOT) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 304 | freeBufferLocked(slot); | 
|  | 305 | } | 
|  | 306 |  | 
| Jamie Gennis | 1df8c34 | 2012-12-20 14:05:45 -0800 | [diff] [blame] | 307 | mSlots[slot].mFence = Fence::NO_FENCE; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 308 |  | 
|  | 309 | return err; | 
|  | 310 | } | 
|  | 311 |  | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 312 | bool ConsumerBase::stillTracking(int slot, | 
|  | 313 | const sp<GraphicBuffer> graphicBuffer) { | 
|  | 314 | if (slot < 0 || slot >= BufferQueue::NUM_BUFFER_SLOTS) { | 
|  | 315 | return false; | 
|  | 316 | } | 
|  | 317 | return (mSlots[slot].mGraphicBuffer != NULL && | 
|  | 318 | mSlots[slot].mGraphicBuffer->handle == graphicBuffer->handle); | 
|  | 319 | } | 
|  | 320 |  | 
| Andy McFadden | 2adaf04 | 2012-12-18 09:49:45 -0800 | [diff] [blame] | 321 | } // namespace android |