| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright (C) 2012 The Android Open Source Project | 
|  | 3 | * | 
|  | 4 | * Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 5 | * you may not use this file except in compliance with the License. | 
|  | 6 | * You may obtain a copy of the License at | 
|  | 7 | * | 
|  | 8 | *      http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 9 | * | 
|  | 10 | * Unless required by applicable law or agreed to in writing, software | 
|  | 11 | * distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 13 | * See the License for the specific language governing permissions and | 
|  | 14 | * limitations under the License. | 
|  | 15 | */ | 
|  | 16 |  | 
|  | 17 | //#define LOG_NDEBUG 0 | 
|  | 18 | #define LOG_TAG "CpuConsumer" | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 19 | //#define ATRACE_TAG ATRACE_TAG_GRAPHICS | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 20 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 21 | #include <cutils/compiler.h> | 
|  | 22 | #include <utils/Log.h> | 
| Dan Stoza | dd26416 | 2015-03-12 13:58:47 -0700 | [diff] [blame] | 23 | #include <gui/BufferItem.h> | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 24 | #include <gui/CpuConsumer.h> | 
|  | 25 |  | 
| Lajos Molnar | b3ca72c | 2015-01-29 10:55:21 -0800 | [diff] [blame] | 26 | #define CC_LOGV(x, ...) ALOGV("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 27 | //#define CC_LOGD(x, ...) ALOGD("[%s] " x, mName.string(), ##__VA_ARGS__) | 
|  | 28 | //#define CC_LOGI(x, ...) ALOGI("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Dan Albert | 8b49125 | 2014-09-08 18:53:39 -0700 | [diff] [blame] | 29 | #define CC_LOGW(x, ...) ALOGW("[%s] " x, mName.string(), ##__VA_ARGS__) | 
|  | 30 | #define CC_LOGE(x, ...) ALOGE("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 31 |  | 
|  | 32 | namespace android { | 
|  | 33 |  | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 34 | CpuConsumer::CpuConsumer(const sp<IGraphicBufferConsumer>& bq, | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 35 | size_t maxLockedBuffers, bool controlledByApp) : | 
| Mathias Agopian | 595264f | 2013-07-16 22:56:09 -0700 | [diff] [blame] | 36 | ConsumerBase(bq, controlledByApp), | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 37 | mMaxLockedBuffers(maxLockedBuffers), | 
|  | 38 | mCurrentLockedBuffers(0) | 
|  | 39 | { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 40 | // Create tracking entries for locked buffers | 
|  | 41 | mAcquiredBuffers.insertAt(0, maxLockedBuffers); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 42 |  | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 43 | mConsumer->setConsumerUsageBits(GRALLOC_USAGE_SW_READ_OFTEN); | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 44 | mConsumer->setMaxAcquiredBufferCount(static_cast<int32_t>(maxLockedBuffers)); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 45 | } | 
|  | 46 |  | 
| Eino-Ville Talvala | e232fdc | 2012-08-21 13:37:35 -0700 | [diff] [blame] | 47 | CpuConsumer::~CpuConsumer() { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 48 | // ConsumerBase destructor does all the work. | 
| Eino-Ville Talvala | e232fdc | 2012-08-21 13:37:35 -0700 | [diff] [blame] | 49 | } | 
|  | 50 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 51 |  | 
|  | 52 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 53 | void CpuConsumer::setName(const String8& name) { | 
|  | 54 | Mutex::Autolock _l(mMutex); | 
| Pablo Ceballos | 65d9f6d | 2016-05-04 13:59:35 -0700 | [diff] [blame] | 55 | if (mAbandoned) { | 
|  | 56 | CC_LOGE("setName: CpuConsumer is abandoned!"); | 
|  | 57 | return; | 
|  | 58 | } | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 59 | mName = name; | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 60 | mConsumer->setConsumerName(name); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 61 | } | 
|  | 62 |  | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 63 | static bool isPossiblyYUV(PixelFormat format) { | 
| Lajos Molnar | b3ca72c | 2015-01-29 10:55:21 -0800 | [diff] [blame] | 64 | switch (static_cast<int>(format)) { | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 65 | case HAL_PIXEL_FORMAT_RGBA_8888: | 
|  | 66 | case HAL_PIXEL_FORMAT_RGBX_8888: | 
| Romain Guy | ff41514 | 2016-12-13 16:51:25 -0800 | [diff] [blame] | 67 | case HAL_PIXEL_FORMAT_RGBA_FP16: | 
| Romain Guy | 541f226 | 2017-02-10 18:50:17 -0800 | [diff] [blame] | 68 | case HAL_PIXEL_FORMAT_RGBA_1010102: | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 69 | case HAL_PIXEL_FORMAT_RGB_888: | 
|  | 70 | case HAL_PIXEL_FORMAT_RGB_565: | 
|  | 71 | case HAL_PIXEL_FORMAT_BGRA_8888: | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 72 | case HAL_PIXEL_FORMAT_Y8: | 
|  | 73 | case HAL_PIXEL_FORMAT_Y16: | 
| Eino-Ville Talvala | 5b75a51 | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 74 | case HAL_PIXEL_FORMAT_RAW16: | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 75 | case HAL_PIXEL_FORMAT_RAW10: | 
|  | 76 | case HAL_PIXEL_FORMAT_RAW_OPAQUE: | 
|  | 77 | case HAL_PIXEL_FORMAT_BLOB: | 
|  | 78 | case HAL_PIXEL_FORMAT_IMPLEMENTATION_DEFINED: | 
|  | 79 | return false; | 
|  | 80 |  | 
|  | 81 | case HAL_PIXEL_FORMAT_YV12: | 
|  | 82 | case HAL_PIXEL_FORMAT_YCbCr_420_888: | 
|  | 83 | case HAL_PIXEL_FORMAT_YCbCr_422_SP: | 
|  | 84 | case HAL_PIXEL_FORMAT_YCrCb_420_SP: | 
|  | 85 | case HAL_PIXEL_FORMAT_YCbCr_422_I: | 
|  | 86 | default: | 
|  | 87 | return true; | 
|  | 88 | } | 
|  | 89 | } | 
|  | 90 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 91 | status_t CpuConsumer::lockNextBuffer(LockedBuffer *nativeBuffer) { | 
|  | 92 | status_t err; | 
|  | 93 |  | 
|  | 94 | if (!nativeBuffer) return BAD_VALUE; | 
|  | 95 | if (mCurrentLockedBuffers == mMaxLockedBuffers) { | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 96 | CC_LOGW("Max buffers have been locked (%zd), cannot lock anymore.", | 
| Igor Murashkin | a5b7513 | 2013-08-14 18:49:12 -0700 | [diff] [blame] | 97 | mMaxLockedBuffers); | 
|  | 98 | return NOT_ENOUGH_DATA; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 99 | } | 
|  | 100 |  | 
| Dan Stoza | dd26416 | 2015-03-12 13:58:47 -0700 | [diff] [blame] | 101 | BufferItem b; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 102 |  | 
|  | 103 | Mutex::Autolock _l(mMutex); | 
|  | 104 |  | 
| Andy McFadden | 1585c4d | 2013-06-28 13:52:40 -0700 | [diff] [blame] | 105 | err = acquireBufferLocked(&b, 0); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 106 | if (err != OK) { | 
|  | 107 | if (err == BufferQueue::NO_BUFFER_AVAILABLE) { | 
|  | 108 | return BAD_VALUE; | 
|  | 109 | } else { | 
|  | 110 | CC_LOGE("Error acquiring buffer: %s (%d)", strerror(err), err); | 
|  | 111 | return err; | 
|  | 112 | } | 
|  | 113 | } | 
|  | 114 |  | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 115 | int slot = b.mSlot; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 116 |  | 
| Eino-Ville Talvala | 64d8b19 | 2013-02-28 14:08:34 -0800 | [diff] [blame] | 117 | void *bufferPointer = NULL; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 118 | android_ycbcr ycbcr = android_ycbcr(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 119 |  | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 120 | PixelFormat format = mSlots[slot].mGraphicBuffer->getPixelFormat(); | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 121 | PixelFormat flexFormat = format; | 
|  | 122 | if (isPossiblyYUV(format)) { | 
|  | 123 | if (b.mFence.get()) { | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 124 | err = mSlots[slot].mGraphicBuffer->lockAsyncYCbCr( | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 125 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 126 | b.mCrop, | 
|  | 127 | &ycbcr, | 
|  | 128 | b.mFence->dup()); | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 129 | } else { | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 130 | err = mSlots[slot].mGraphicBuffer->lockYCbCr( | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 131 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 132 | b.mCrop, | 
|  | 133 | &ycbcr); | 
|  | 134 | } | 
|  | 135 | if (err == OK) { | 
|  | 136 | bufferPointer = ycbcr.y; | 
|  | 137 | flexFormat = HAL_PIXEL_FORMAT_YCbCr_420_888; | 
|  | 138 | if (format != HAL_PIXEL_FORMAT_YCbCr_420_888) { | 
|  | 139 | CC_LOGV("locking buffer of format %#x as flex YUV", format); | 
|  | 140 | } | 
|  | 141 | } else if (format == HAL_PIXEL_FORMAT_YCbCr_420_888) { | 
|  | 142 | CC_LOGE("Unable to lock YCbCr buffer for CPU reading: %s (%d)", | 
|  | 143 | strerror(-err), err); | 
|  | 144 | return err; | 
|  | 145 | } | 
|  | 146 | } | 
|  | 147 |  | 
|  | 148 | if (bufferPointer == NULL) { // not flexible YUV | 
|  | 149 | if (b.mFence.get()) { | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 150 | err = mSlots[slot].mGraphicBuffer->lockAsync( | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 151 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 152 | b.mCrop, | 
|  | 153 | &bufferPointer, | 
|  | 154 | b.mFence->dup()); | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 155 | } else { | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 156 | err = mSlots[slot].mGraphicBuffer->lock( | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 157 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 158 | b.mCrop, | 
|  | 159 | &bufferPointer); | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 160 | } | 
|  | 161 | if (err != OK) { | 
|  | 162 | CC_LOGE("Unable to lock buffer for CPU reading: %s (%d)", | 
|  | 163 | strerror(-err), err); | 
|  | 164 | return err; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 165 | } | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 166 | } | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 167 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 168 | size_t lockedIdx = 0; | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 169 | for (; lockedIdx < static_cast<size_t>(mMaxLockedBuffers); lockedIdx++) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 170 | if (mAcquiredBuffers[lockedIdx].mSlot == | 
|  | 171 | BufferQueue::INVALID_BUFFER_SLOT) { | 
|  | 172 | break; | 
|  | 173 | } | 
|  | 174 | } | 
|  | 175 | assert(lockedIdx < mMaxLockedBuffers); | 
|  | 176 |  | 
|  | 177 | AcquiredBuffer &ab = mAcquiredBuffers.editItemAt(lockedIdx); | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 178 | ab.mSlot = slot; | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 179 | ab.mBufferPointer = bufferPointer; | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 180 | ab.mGraphicBuffer = mSlots[slot].mGraphicBuffer; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 181 |  | 
| Eino-Ville Talvala | 64d8b19 | 2013-02-28 14:08:34 -0800 | [diff] [blame] | 182 | nativeBuffer->data   = | 
|  | 183 | reinterpret_cast<uint8_t*>(bufferPointer); | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 184 | nativeBuffer->width  = mSlots[slot].mGraphicBuffer->getWidth(); | 
|  | 185 | nativeBuffer->height = mSlots[slot].mGraphicBuffer->getHeight(); | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 186 | nativeBuffer->format = format; | 
|  | 187 | nativeBuffer->flexFormat = flexFormat; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 188 | nativeBuffer->stride = (ycbcr.y != NULL) ? | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 189 | static_cast<uint32_t>(ycbcr.ystride) : | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 190 | mSlots[slot].mGraphicBuffer->getStride(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 191 |  | 
|  | 192 | nativeBuffer->crop        = b.mCrop; | 
|  | 193 | nativeBuffer->transform   = b.mTransform; | 
|  | 194 | nativeBuffer->scalingMode = b.mScalingMode; | 
|  | 195 | nativeBuffer->timestamp   = b.mTimestamp; | 
| Eino-Ville Talvala | 5b75a51 | 2015-02-19 16:10:43 -0800 | [diff] [blame] | 196 | nativeBuffer->dataSpace   = b.mDataSpace; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 197 | nativeBuffer->frameNumber = b.mFrameNumber; | 
|  | 198 |  | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 199 | nativeBuffer->dataCb       = reinterpret_cast<uint8_t*>(ycbcr.cb); | 
|  | 200 | nativeBuffer->dataCr       = reinterpret_cast<uint8_t*>(ycbcr.cr); | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 201 | nativeBuffer->chromaStride = static_cast<uint32_t>(ycbcr.cstride); | 
|  | 202 | nativeBuffer->chromaStep   = static_cast<uint32_t>(ycbcr.chroma_step); | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 203 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 204 | mCurrentLockedBuffers++; | 
|  | 205 |  | 
|  | 206 | return OK; | 
|  | 207 | } | 
|  | 208 |  | 
|  | 209 | status_t CpuConsumer::unlockBuffer(const LockedBuffer &nativeBuffer) { | 
|  | 210 | Mutex::Autolock _l(mMutex); | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 211 | size_t lockedIdx = 0; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 212 |  | 
|  | 213 | void *bufPtr = reinterpret_cast<void *>(nativeBuffer.data); | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 214 | for (; lockedIdx < static_cast<size_t>(mMaxLockedBuffers); lockedIdx++) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 215 | if (bufPtr == mAcquiredBuffers[lockedIdx].mBufferPointer) break; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 216 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 217 | if (lockedIdx == mMaxLockedBuffers) { | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 218 | CC_LOGE("%s: Can't find buffer to free", __FUNCTION__); | 
|  | 219 | return BAD_VALUE; | 
|  | 220 | } | 
|  | 221 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 222 | return releaseAcquiredBufferLocked(lockedIdx); | 
|  | 223 | } | 
|  | 224 |  | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 225 | status_t CpuConsumer::releaseAcquiredBufferLocked(size_t lockedIdx) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 226 | status_t err; | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 227 | int fd = -1; | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 228 |  | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 229 | err = mAcquiredBuffers[lockedIdx].mGraphicBuffer->unlockAsync(&fd); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 230 | if (err != OK) { | 
| Dan Stoza | d723bd7 | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 231 | CC_LOGE("%s: Unable to unlock graphic buffer %zd", __FUNCTION__, | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 232 | lockedIdx); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 233 | return err; | 
|  | 234 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 235 | int buf = mAcquiredBuffers[lockedIdx].mSlot; | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 236 | if (CC_LIKELY(fd != -1)) { | 
|  | 237 | sp<Fence> fence(new Fence(fd)); | 
|  | 238 | addReleaseFenceLocked( | 
|  | 239 | mAcquiredBuffers[lockedIdx].mSlot, | 
|  | 240 | mSlots[buf].mGraphicBuffer, | 
|  | 241 | fence); | 
|  | 242 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 243 |  | 
|  | 244 | // release the buffer if it hasn't already been freed by the BufferQueue. | 
|  | 245 | // This can happen, for example, when the producer of this buffer | 
|  | 246 | // disconnected after this buffer was acquired. | 
|  | 247 | if (CC_LIKELY(mAcquiredBuffers[lockedIdx].mGraphicBuffer == | 
|  | 248 | mSlots[buf].mGraphicBuffer)) { | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 249 | releaseBufferLocked( | 
|  | 250 | buf, mAcquiredBuffers[lockedIdx].mGraphicBuffer, | 
|  | 251 | EGL_NO_DISPLAY, EGL_NO_SYNC_KHR); | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 252 | } | 
|  | 253 |  | 
|  | 254 | AcquiredBuffer &ab = mAcquiredBuffers.editItemAt(lockedIdx); | 
|  | 255 | ab.mSlot = BufferQueue::INVALID_BUFFER_SLOT; | 
|  | 256 | ab.mBufferPointer = NULL; | 
|  | 257 | ab.mGraphicBuffer.clear(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 258 |  | 
|  | 259 | mCurrentLockedBuffers--; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 260 | return OK; | 
|  | 261 | } | 
|  | 262 |  | 
| Eino-Ville Talvala | f57e754 | 2012-08-20 15:44:40 -0700 | [diff] [blame] | 263 | void CpuConsumer::freeBufferLocked(int slotIndex) { | 
| Eino-Ville Talvala | f57e754 | 2012-08-20 15:44:40 -0700 | [diff] [blame] | 264 | ConsumerBase::freeBufferLocked(slotIndex); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 265 | } | 
|  | 266 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 267 | } // namespace android |