| 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 | 3be1c6b | 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> | 
| Eino-Ville Talvala | 1c2001e | 2015-02-19 16:10:43 -0800 | [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 |  | 
| Dan Albert | 8b49125 | 2014-09-08 18:53:39 -0700 | [diff] [blame] | 26 | #define CC_LOGV(x, ...) ALOGV("[%s] " x, mName.string(), ##__VA_ARGS__) | 
| Dan Stoza | 3be1c6b | 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 | 3be1c6b | 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 | 3be1c6b | 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); | 
|  | 55 | mName = name; | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 56 | mConsumer->setConsumerName(name); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 57 | } | 
|  | 58 |  | 
| Zhijun He | b4b6370 | 2013-06-06 11:59:21 -0700 | [diff] [blame] | 59 | status_t CpuConsumer::setDefaultBufferSize(uint32_t width, uint32_t height) | 
|  | 60 | { | 
|  | 61 | Mutex::Autolock _l(mMutex); | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 62 | return mConsumer->setDefaultBufferSize(width, height); | 
| Zhijun He | b4b6370 | 2013-06-06 11:59:21 -0700 | [diff] [blame] | 63 | } | 
|  | 64 |  | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 65 | status_t CpuConsumer::setDefaultBufferFormat(PixelFormat defaultFormat) | 
| Zhijun He | b4b6370 | 2013-06-06 11:59:21 -0700 | [diff] [blame] | 66 | { | 
|  | 67 | Mutex::Autolock _l(mMutex); | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 68 | return mConsumer->setDefaultBufferFormat(defaultFormat); | 
| Zhijun He | b4b6370 | 2013-06-06 11:59:21 -0700 | [diff] [blame] | 69 | } | 
|  | 70 |  | 
| Eino-Ville Talvala | 1c2001e | 2015-02-19 16:10:43 -0800 | [diff] [blame^] | 71 | status_t CpuConsumer::setDefaultBufferDataSpace( | 
|  | 72 | android_dataspace defaultDataSpace) | 
|  | 73 | { | 
|  | 74 | Mutex::Autolock _l(mMutex); | 
|  | 75 | return mConsumer->setDefaultBufferDataSpace(defaultDataSpace); | 
|  | 76 | } | 
|  | 77 |  | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 78 | static bool isPossiblyYUV(PixelFormat format) { | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 79 | switch (static_cast<int>(format)) { | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 80 | case HAL_PIXEL_FORMAT_RGBA_8888: | 
|  | 81 | case HAL_PIXEL_FORMAT_RGBX_8888: | 
|  | 82 | case HAL_PIXEL_FORMAT_RGB_888: | 
|  | 83 | case HAL_PIXEL_FORMAT_RGB_565: | 
|  | 84 | case HAL_PIXEL_FORMAT_BGRA_8888: | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 85 | case HAL_PIXEL_FORMAT_Y8: | 
|  | 86 | case HAL_PIXEL_FORMAT_Y16: | 
| Eino-Ville Talvala | 1c2001e | 2015-02-19 16:10:43 -0800 | [diff] [blame^] | 87 | case HAL_PIXEL_FORMAT_RAW16: | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 88 | case HAL_PIXEL_FORMAT_RAW10: | 
|  | 89 | case HAL_PIXEL_FORMAT_RAW_OPAQUE: | 
|  | 90 | case HAL_PIXEL_FORMAT_BLOB: | 
|  | 91 | case HAL_PIXEL_FORMAT_IMPLEMENTATION_DEFINED: | 
|  | 92 | return false; | 
|  | 93 |  | 
|  | 94 | case HAL_PIXEL_FORMAT_YV12: | 
|  | 95 | case HAL_PIXEL_FORMAT_YCbCr_420_888: | 
|  | 96 | case HAL_PIXEL_FORMAT_YCbCr_422_SP: | 
|  | 97 | case HAL_PIXEL_FORMAT_YCrCb_420_SP: | 
|  | 98 | case HAL_PIXEL_FORMAT_YCbCr_422_I: | 
|  | 99 | default: | 
|  | 100 | return true; | 
|  | 101 | } | 
|  | 102 | } | 
|  | 103 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 104 | status_t CpuConsumer::lockNextBuffer(LockedBuffer *nativeBuffer) { | 
|  | 105 | status_t err; | 
|  | 106 |  | 
|  | 107 | if (!nativeBuffer) return BAD_VALUE; | 
|  | 108 | if (mCurrentLockedBuffers == mMaxLockedBuffers) { | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 109 | CC_LOGW("Max buffers have been locked (%zd), cannot lock anymore.", | 
| Igor Murashkin | a5b7513 | 2013-08-14 18:49:12 -0700 | [diff] [blame] | 110 | mMaxLockedBuffers); | 
|  | 111 | return NOT_ENOUGH_DATA; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 112 | } | 
|  | 113 |  | 
| Eino-Ville Talvala | 1c2001e | 2015-02-19 16:10:43 -0800 | [diff] [blame^] | 114 | BufferItem b; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 115 |  | 
|  | 116 | Mutex::Autolock _l(mMutex); | 
|  | 117 |  | 
| Andy McFadden | 1585c4d | 2013-06-28 13:52:40 -0700 | [diff] [blame] | 118 | err = acquireBufferLocked(&b, 0); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 119 | if (err != OK) { | 
|  | 120 | if (err == BufferQueue::NO_BUFFER_AVAILABLE) { | 
|  | 121 | return BAD_VALUE; | 
|  | 122 | } else { | 
|  | 123 | CC_LOGE("Error acquiring buffer: %s (%d)", strerror(err), err); | 
|  | 124 | return err; | 
|  | 125 | } | 
|  | 126 | } | 
|  | 127 |  | 
|  | 128 | int buf = b.mBuf; | 
|  | 129 |  | 
| Eino-Ville Talvala | 64d8b19 | 2013-02-28 14:08:34 -0800 | [diff] [blame] | 130 | void *bufferPointer = NULL; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 131 | android_ycbcr ycbcr = android_ycbcr(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 132 |  | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 133 | PixelFormat format = mSlots[buf].mGraphicBuffer->getPixelFormat(); | 
|  | 134 | PixelFormat flexFormat = format; | 
|  | 135 | if (isPossiblyYUV(format)) { | 
|  | 136 | if (b.mFence.get()) { | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 137 | err = mSlots[buf].mGraphicBuffer->lockAsyncYCbCr( | 
|  | 138 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 139 | b.mCrop, | 
|  | 140 | &ycbcr, | 
|  | 141 | b.mFence->dup()); | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 142 | } else { | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 143 | err = mSlots[buf].mGraphicBuffer->lockYCbCr( | 
|  | 144 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 145 | b.mCrop, | 
|  | 146 | &ycbcr); | 
|  | 147 | } | 
|  | 148 | if (err == OK) { | 
|  | 149 | bufferPointer = ycbcr.y; | 
|  | 150 | flexFormat = HAL_PIXEL_FORMAT_YCbCr_420_888; | 
|  | 151 | if (format != HAL_PIXEL_FORMAT_YCbCr_420_888) { | 
|  | 152 | CC_LOGV("locking buffer of format %#x as flex YUV", format); | 
|  | 153 | } | 
|  | 154 | } else if (format == HAL_PIXEL_FORMAT_YCbCr_420_888) { | 
|  | 155 | CC_LOGE("Unable to lock YCbCr buffer for CPU reading: %s (%d)", | 
|  | 156 | strerror(-err), err); | 
|  | 157 | return err; | 
|  | 158 | } | 
|  | 159 | } | 
|  | 160 |  | 
|  | 161 | if (bufferPointer == NULL) { // not flexible YUV | 
|  | 162 | if (b.mFence.get()) { | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 163 | err = mSlots[buf].mGraphicBuffer->lockAsync( | 
|  | 164 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 165 | b.mCrop, | 
|  | 166 | &bufferPointer, | 
|  | 167 | b.mFence->dup()); | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 168 | } else { | 
|  | 169 | err = mSlots[buf].mGraphicBuffer->lock( | 
|  | 170 | GraphicBuffer::USAGE_SW_READ_OFTEN, | 
|  | 171 | b.mCrop, | 
|  | 172 | &bufferPointer); | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 173 | } | 
|  | 174 | if (err != OK) { | 
|  | 175 | CC_LOGE("Unable to lock buffer for CPU reading: %s (%d)", | 
|  | 176 | strerror(-err), err); | 
|  | 177 | return err; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 178 | } | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 179 | } | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 180 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 181 | size_t lockedIdx = 0; | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 182 | for (; lockedIdx < static_cast<size_t>(mMaxLockedBuffers); lockedIdx++) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 183 | if (mAcquiredBuffers[lockedIdx].mSlot == | 
|  | 184 | BufferQueue::INVALID_BUFFER_SLOT) { | 
|  | 185 | break; | 
|  | 186 | } | 
|  | 187 | } | 
|  | 188 | assert(lockedIdx < mMaxLockedBuffers); | 
|  | 189 |  | 
|  | 190 | AcquiredBuffer &ab = mAcquiredBuffers.editItemAt(lockedIdx); | 
|  | 191 | ab.mSlot = buf; | 
|  | 192 | ab.mBufferPointer = bufferPointer; | 
|  | 193 | ab.mGraphicBuffer = mSlots[buf].mGraphicBuffer; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 194 |  | 
| Eino-Ville Talvala | 64d8b19 | 2013-02-28 14:08:34 -0800 | [diff] [blame] | 195 | nativeBuffer->data   = | 
|  | 196 | reinterpret_cast<uint8_t*>(bufferPointer); | 
| Eino-Ville Talvala | f57e754 | 2012-08-20 15:44:40 -0700 | [diff] [blame] | 197 | nativeBuffer->width  = mSlots[buf].mGraphicBuffer->getWidth(); | 
|  | 198 | nativeBuffer->height = mSlots[buf].mGraphicBuffer->getHeight(); | 
| Lajos Molnar | 6a26be7 | 2015-01-22 16:37:37 -0800 | [diff] [blame] | 199 | nativeBuffer->format = format; | 
|  | 200 | nativeBuffer->flexFormat = flexFormat; | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 201 | nativeBuffer->stride = (ycbcr.y != NULL) ? | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 202 | static_cast<uint32_t>(ycbcr.ystride) : | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 203 | mSlots[buf].mGraphicBuffer->getStride(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 204 |  | 
|  | 205 | nativeBuffer->crop        = b.mCrop; | 
|  | 206 | nativeBuffer->transform   = b.mTransform; | 
|  | 207 | nativeBuffer->scalingMode = b.mScalingMode; | 
|  | 208 | nativeBuffer->timestamp   = b.mTimestamp; | 
| Eino-Ville Talvala | 1c2001e | 2015-02-19 16:10:43 -0800 | [diff] [blame^] | 209 | nativeBuffer->dataSpace   = b.mDataSpace; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 210 | nativeBuffer->frameNumber = b.mFrameNumber; | 
|  | 211 |  | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 212 | nativeBuffer->dataCb       = reinterpret_cast<uint8_t*>(ycbcr.cb); | 
|  | 213 | nativeBuffer->dataCr       = reinterpret_cast<uint8_t*>(ycbcr.cr); | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 214 | nativeBuffer->chromaStride = static_cast<uint32_t>(ycbcr.cstride); | 
|  | 215 | nativeBuffer->chromaStep   = static_cast<uint32_t>(ycbcr.chroma_step); | 
| Eino-Ville Talvala | c43946b | 2013-05-04 18:07:43 -0700 | [diff] [blame] | 216 |  | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 217 | mCurrentLockedBuffers++; | 
|  | 218 |  | 
|  | 219 | return OK; | 
|  | 220 | } | 
|  | 221 |  | 
|  | 222 | status_t CpuConsumer::unlockBuffer(const LockedBuffer &nativeBuffer) { | 
|  | 223 | Mutex::Autolock _l(mMutex); | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 224 | size_t lockedIdx = 0; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 225 |  | 
|  | 226 | void *bufPtr = reinterpret_cast<void *>(nativeBuffer.data); | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 227 | for (; lockedIdx < static_cast<size_t>(mMaxLockedBuffers); lockedIdx++) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 228 | if (bufPtr == mAcquiredBuffers[lockedIdx].mBufferPointer) break; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 229 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 230 | if (lockedIdx == mMaxLockedBuffers) { | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 231 | CC_LOGE("%s: Can't find buffer to free", __FUNCTION__); | 
|  | 232 | return BAD_VALUE; | 
|  | 233 | } | 
|  | 234 |  | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 235 | return releaseAcquiredBufferLocked(lockedIdx); | 
|  | 236 | } | 
|  | 237 |  | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 238 | status_t CpuConsumer::releaseAcquiredBufferLocked(size_t lockedIdx) { | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 239 | status_t err; | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 240 | int fd = -1; | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 241 |  | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 242 | err = mAcquiredBuffers[lockedIdx].mGraphicBuffer->unlockAsync(&fd); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 243 | if (err != OK) { | 
| Dan Stoza | 3be1c6b | 2014-11-18 10:24:03 -0800 | [diff] [blame] | 244 | CC_LOGE("%s: Unable to unlock graphic buffer %zd", __FUNCTION__, | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 245 | lockedIdx); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 246 | return err; | 
|  | 247 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 248 | int buf = mAcquiredBuffers[lockedIdx].mSlot; | 
| Riley Andrews | d53e052 | 2014-08-18 16:57:11 -0700 | [diff] [blame] | 249 | if (CC_LIKELY(fd != -1)) { | 
|  | 250 | sp<Fence> fence(new Fence(fd)); | 
|  | 251 | addReleaseFenceLocked( | 
|  | 252 | mAcquiredBuffers[lockedIdx].mSlot, | 
|  | 253 | mSlots[buf].mGraphicBuffer, | 
|  | 254 | fence); | 
|  | 255 | } | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 256 |  | 
|  | 257 | // release the buffer if it hasn't already been freed by the BufferQueue. | 
|  | 258 | // This can happen, for example, when the producer of this buffer | 
|  | 259 | // disconnected after this buffer was acquired. | 
|  | 260 | if (CC_LIKELY(mAcquiredBuffers[lockedIdx].mGraphicBuffer == | 
|  | 261 | mSlots[buf].mGraphicBuffer)) { | 
| Lajos Molnar | c5d7b7d | 2013-05-03 14:50:50 -0700 | [diff] [blame] | 262 | releaseBufferLocked( | 
|  | 263 | buf, mAcquiredBuffers[lockedIdx].mGraphicBuffer, | 
|  | 264 | EGL_NO_DISPLAY, EGL_NO_SYNC_KHR); | 
| Eino-Ville Talvala | 042ecee | 2013-02-28 18:23:24 -0800 | [diff] [blame] | 265 | } | 
|  | 266 |  | 
|  | 267 | AcquiredBuffer &ab = mAcquiredBuffers.editItemAt(lockedIdx); | 
|  | 268 | ab.mSlot = BufferQueue::INVALID_BUFFER_SLOT; | 
|  | 269 | ab.mBufferPointer = NULL; | 
|  | 270 | ab.mGraphicBuffer.clear(); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 271 |  | 
|  | 272 | mCurrentLockedBuffers--; | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 273 | return OK; | 
|  | 274 | } | 
|  | 275 |  | 
| Eino-Ville Talvala | f57e754 | 2012-08-20 15:44:40 -0700 | [diff] [blame] | 276 | void CpuConsumer::freeBufferLocked(int slotIndex) { | 
| Eino-Ville Talvala | f57e754 | 2012-08-20 15:44:40 -0700 | [diff] [blame] | 277 | ConsumerBase::freeBufferLocked(slotIndex); | 
| Eino-Ville Talvala | e41b318 | 2012-04-16 17:54:33 -0700 | [diff] [blame] | 278 | } | 
|  | 279 |  | 
|  | 280 | } // namespace android |