ConsumerBase.cpp revision 74d211ae26a0257c6075a823812e40b55aa1e653
11a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis/* 21a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * Copyright (C) 2010 The Android Open Source Project 31a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * 41a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * Licensed under the Apache License, Version 2.0 (the "License"); 51a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * you may not use this file except in compliance with the License. 61a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * You may obtain a copy of the License at 71a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * 81a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * http://www.apache.org/licenses/LICENSE-2.0 91a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * 101a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * Unless required by applicable law or agreed to in writing, software 111a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * distributed under the License is distributed on an "AS IS" BASIS, 121a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 131a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * See the License for the specific language governing permissions and 141a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis * limitations under the License. 151a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis */ 161a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 171a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define LOG_TAG "ConsumerBase" 181a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define ATRACE_TAG ATRACE_TAG_GRAPHICS 191a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis//#define LOG_NDEBUG 0 201a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 211a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define EGL_EGLEXT_PROTOTYPES 221a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 231a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <EGL/egl.h> 241a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <EGL/eglext.h> 251a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 261a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <hardware/hardware.h> 271a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 281a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <gui/IGraphicBufferAlloc.h> 291a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <gui/ISurfaceComposer.h> 301a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <gui/SurfaceComposerClient.h> 311a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <gui/ConsumerBase.h> 321a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 331a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <private/gui/ComposerService.h> 341a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 351a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <utils/Log.h> 361a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <utils/String8.h> 371a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#include <utils/Trace.h> 381a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 391a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis// Macros for including the ConsumerBase name in log messages 401a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define CB_LOGV(x, ...) ALOGV("[%s] "x, mName.string(), ##__VA_ARGS__) 411a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define CB_LOGD(x, ...) ALOGD("[%s] "x, mName.string(), ##__VA_ARGS__) 421a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define CB_LOGI(x, ...) ALOGI("[%s] "x, mName.string(), ##__VA_ARGS__) 431a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define CB_LOGW(x, ...) ALOGW("[%s] "x, mName.string(), ##__VA_ARGS__) 441a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis#define CB_LOGE(x, ...) ALOGE("[%s] "x, mName.string(), ##__VA_ARGS__) 451a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 461a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisnamespace android { 471a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 481a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis// Get an ID that's unique within this process. 491a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisstatic int32_t createProcessUniqueId() { 501a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis static volatile int32_t globalCounter = 0; 511a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis return android_atomic_inc(&globalCounter); 521a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 531a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 541a4d883dcc1725892bfb5c28dec255a233186524Jamie GennisConsumerBase::ConsumerBase(const sp<BufferQueue>& bufferQueue) : 559fea3421ffddf6480f57f55a25936a886043d909Jamie Gennis mAbandoned(false), 569fea3421ffddf6480f57f55a25936a886043d909Jamie Gennis mBufferQueue(bufferQueue) { 571a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // Choose a name using the PID and a process-unique ID. 581a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mName = String8::format("unnamed-%d-%d", getpid(), createProcessUniqueId()); 591a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 601a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // Note that we can't create an sp<...>(this) in a ctor that will not keep a 611a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // reference once the ctor ends, as that would cause the refcount of 'this' 621a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // dropping to 0 at the end of the ctor. Since all we need is a wp<...> 631a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // that's what we create. 641a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis wp<BufferQueue::ConsumerListener> listener; 651a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis sp<BufferQueue::ConsumerListener> proxy; 661a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis listener = static_cast<BufferQueue::ConsumerListener*>(this); 671a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis proxy = new BufferQueue::ProxyConsumerListener(listener); 681a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 691a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis status_t err = mBufferQueue->consumerConnect(proxy); 701a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (err != NO_ERROR) { 712adaf04fab35cf47c824d74d901b54094e01ccd3Andy McFadden CB_LOGE("ConsumerBase: error connecting to BufferQueue: %s (%d)", 721a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis strerror(-err), err); 731a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } else { 741a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mBufferQueue->setConsumerName(mName); 751a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 761a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 771a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 781a4d883dcc1725892bfb5c28dec255a233186524Jamie GennisConsumerBase::~ConsumerBase() { 79ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis CB_LOGV("~ConsumerBase"); 80ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis Mutex::Autolock lock(mMutex); 81ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis 82ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis // Verify that abandon() has been called before we get here. This should 83ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis // be done by ConsumerBase::onLastStrongRef(), but it's possible for a 84ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis // derived class to override that method and not call 85ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis // ConsumerBase::onLastStrongRef(). 86ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis LOG_ALWAYS_FATAL_IF(!mAbandoned, "[%s] ~ConsumerBase was called, but the " 87ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis "consumer is not abandoned!", mName.string()); 88ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis} 89ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennis 90ad669b04f4633957eea55b8ad2d8253adcefe39bJamie Gennisvoid ConsumerBase::onLastStrongRef(const void* id) { 911a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis abandon(); 921a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 931a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 941a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::freeBufferLocked(int slotIndex) { 951a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("freeBufferLocked: slotIndex=%d", slotIndex); 961a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mSlots[slotIndex].mGraphicBuffer = 0; 971df8c345854155cbbcb9f80de9d12d66ea70ac08Jamie Gennis mSlots[slotIndex].mFence = Fence::NO_FENCE; 981a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 991a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1001a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis// Used for refactoring, should not be in final interface 1011a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennissp<BufferQueue> ConsumerBase::getBufferQueue() const { 1021a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis Mutex::Autolock lock(mMutex); 1031a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis return mBufferQueue; 1041a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1051a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1061a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::onFrameAvailable() { 1071a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("onFrameAvailable"); 1081a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1091a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis sp<FrameAvailableListener> listener; 1101a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis { // scope for the lock 1111a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis Mutex::Autolock lock(mMutex); 112a4a3149a36bc69a06e4824aeae909ab910661070Igor Murashkin listener = mFrameAvailableListener.promote(); 1131a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1141a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1151a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (listener != NULL) { 1161a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("actually calling onFrameAvailable"); 1171a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis listener->onFrameAvailable(); 1181a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1191a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1201a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1211a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::onBuffersReleased() { 12272c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis Mutex::Autolock lock(mMutex); 123b21a4e3b5f7f07ed160ca6e1809313e2a8e2a6a4Jamie Gennis 12472c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis CB_LOGV("onBuffersReleased"); 1251a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 12672c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis if (mAbandoned) { 12772c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis // Nothing to do if we're already abandoned. 12872c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis return; 1291a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1301a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 13172c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis uint32_t mask = 0; 13272c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis mBufferQueue->getReleasedBuffers(&mask); 1331a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { 13472c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis if (mask & (1 << i)) { 13572c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis freeBufferLocked(i); 13672c3f7d88160b7c279f90f0efe3c1cb12cd140aeJamie Gennis } 1371a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1381a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1391a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1401a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::abandon() { 1411a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("abandon"); 1421a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis Mutex::Autolock lock(mMutex); 1431a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1441a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (!mAbandoned) { 1451a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis abandonLocked(); 1461a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mAbandoned = true; 1471a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1481a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1491a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1501a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::abandonLocked() { 1511a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("abandonLocked"); 1521a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis for (int i =0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { 1531a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis freeBufferLocked(i); 1541a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1551a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis // disconnect from the BufferQueue 1561a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mBufferQueue->consumerDisconnect(); 1571a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mBufferQueue.clear(); 1581a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1591a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1601a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::setFrameAvailableListener( 161a4a3149a36bc69a06e4824aeae909ab910661070Igor Murashkin const wp<FrameAvailableListener>& listener) { 1621a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("setFrameAvailableListener"); 1631a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis Mutex::Autolock lock(mMutex); 1641a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mFrameAvailableListener = listener; 1651a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1661a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1671a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisvoid ConsumerBase::dump(String8& result) const { 16874d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopian dump(result, ""); 1691a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1701a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 17174d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopianvoid ConsumerBase::dump(String8& result, const char* prefix) const { 1721a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis Mutex::Autolock _l(mMutex); 17374d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopian dumpLocked(result, prefix); 1741a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1751a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 17674d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopianvoid ConsumerBase::dumpLocked(String8& result, const char* prefix) const { 17774d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopian result.appendFormat("%smAbandoned=%d\n", prefix, int(mAbandoned)); 1781a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1791a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (!mAbandoned) { 18074d211ae26a0257c6075a823812e40b55aa1e653Mathias Agopian mBufferQueue->dump(result, prefix); 1811a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1821a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 1831a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1841a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisstatus_t ConsumerBase::acquireBufferLocked(BufferQueue::BufferItem *item) { 1851a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis status_t err = mBufferQueue->acquireBuffer(item); 1861a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (err != NO_ERROR) { 1871a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis return err; 1881a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1891a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1901a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (item->mGraphicBuffer != NULL) { 1911a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis mSlots[item->mBuf].mGraphicBuffer = item->mGraphicBuffer; 1921a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 1931a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 194b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[item->mBuf].mFence = item->mFence; 195b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis 1961a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("acquireBufferLocked: -> slot=%d", item->mBuf); 1971a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 1981a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis return OK; 1991a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 2001a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 201b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennisstatus_t ConsumerBase::addReleaseFence(int slot, const sp<Fence>& fence) { 2029504eb915c9628e130f45019bdefda0168089886Jesse Hall Mutex::Autolock lock(mMutex); 2039504eb915c9628e130f45019bdefda0168089886Jesse Hall return addReleaseFenceLocked(slot, fence); 2049504eb915c9628e130f45019bdefda0168089886Jesse Hall} 2059504eb915c9628e130f45019bdefda0168089886Jesse Hall 2069504eb915c9628e130f45019bdefda0168089886Jesse Hallstatus_t ConsumerBase::addReleaseFenceLocked(int slot, const sp<Fence>& fence) { 2079504eb915c9628e130f45019bdefda0168089886Jesse Hall CB_LOGV("addReleaseFenceLocked: slot=%d", slot); 208b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis 209b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis if (!mSlots[slot].mFence.get()) { 210b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[slot].mFence = fence; 211b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis } else { 212b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis sp<Fence> mergedFence = Fence::merge( 2137aff4a5de47bf32b0934f5744cd1df4ce666d2d2Jamie Gennis String8::format("%.28s:%d", mName.string(), slot), 214b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[slot].mFence, fence); 215b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis if (!mergedFence.get()) { 216b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis CB_LOGE("failed to merge release fences"); 217b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis // synchronization is broken, the best we can do is hope fences 218b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis // signal in order so the new fence will act like a union 219b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[slot].mFence = fence; 220b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis return BAD_VALUE; 221b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis } 222b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[slot].mFence = mergedFence; 223b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis } 224b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis 225b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis return OK; 226b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis} 227b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis 2281a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennisstatus_t ConsumerBase::releaseBufferLocked(int slot, EGLDisplay display, 229b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis EGLSyncKHR eglFence) { 2301a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis CB_LOGV("releaseBufferLocked: slot=%d", slot); 231b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis status_t err = mBufferQueue->releaseBuffer(slot, display, eglFence, 232b27254154642575dfb4bbfa79fbedde7d7ee23ddJamie Gennis mSlots[slot].mFence); 2331a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis if (err == BufferQueue::STALE_BUFFER_SLOT) { 2341a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis freeBufferLocked(slot); 2351a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis } 2361a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 2371df8c345854155cbbcb9f80de9d12d66ea70ac08Jamie Gennis mSlots[slot].mFence = Fence::NO_FENCE; 2381a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 2391a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis return err; 2401a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis} 2411a4d883dcc1725892bfb5c28dec255a233186524Jamie Gennis 2422adaf04fab35cf47c824d74d901b54094e01ccd3Andy McFadden} // namespace android 243