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