Revert "libgui: Plumb attach/detach through CpuConsumer"

This reverts commit 43078e2827.

Change-Id: Ifef774d9922bb90450384e810e4767b56cc29de3
This commit is contained in:
Dan Stoza 2015-03-17 15:54:50 +00:00
parent 43078e2827
commit 724768f916
3 changed files with 2 additions and 75 deletions

View File

@ -114,20 +114,6 @@ class CpuConsumer : public ConsumerBase
// lockNextBuffer.
status_t unlockBuffer(const LockedBuffer &nativeBuffer);
// Gets the next buffer from the producer (if any), and transfers ownership
// of it from the queue to the caller.
//
// See IGraphicBufferConsumer::{acquireBuffer,detachBuffer} for possible
// error codes.
status_t detachNextBuffer(BufferItem* outItem);
// Transfers ownership of the buffer from the caller to the queue, and
// releases the buffer for use by the producer.
//
// See IGraphicBufferConsumer::{attachBuffer,releaseBuffer} for possible
// error codes.
status_t attachAndReleaseBuffer(const sp<GraphicBuffer>& buffer);
private:
// Maximum number of buffers that can be locked at a time
size_t mMaxLockedBuffers;

View File

@ -247,16 +247,12 @@ status_t BufferQueueConsumer::attachBuffer(int* outSlot,
ATRACE_BUFFER_INDEX(*outSlot);
BQ_LOGV("attachBuffer(C): returning slot %d", *outSlot);
// If these are modified, they also need to be modified in
// CpuConsumer::attachAndReleaseBuffer
mSlots[*outSlot].mGraphicBuffer = buffer;
mSlots[*outSlot].mFence = Fence::NO_FENCE;
mSlots[*outSlot].mFrameNumber = 0;
// Changes to these do not need to be propagated to CpuConsumer
mSlots[*outSlot].mBufferState = BufferSlot::ACQUIRED;
mSlots[*outSlot].mAttachedByConsumer = true;
mSlots[*outSlot].mNeedsCleanupOnRelease = false;
mSlots[*outSlot].mFence = Fence::NO_FENCE;
mSlots[*outSlot].mFrameNumber = 0;
// mAcquireCalled tells BufferQueue that it doesn't need to send a valid
// GraphicBuffer pointer on the next acquireBuffer call, which decreases

View File

@ -277,59 +277,4 @@ void CpuConsumer::freeBufferLocked(int slotIndex) {
ConsumerBase::freeBufferLocked(slotIndex);
}
status_t CpuConsumer::detachNextBuffer(BufferItem* outItem) {
if (outItem == NULL) {
return BAD_VALUE;
}
Mutex::Autolock lock(mMutex);
BufferItem item;
status_t result = acquireBufferLocked(&item, 0);
if (result != NO_ERROR) {
CC_LOGE("%s: Failed to acquire buffer (%d)", __FUNCTION__, result);
return result;
}
result = mConsumer->detachBuffer(item.mSlot);
if (result != NO_ERROR) {
CC_LOGE("%s: Failed to detach buffer (%d)", __FUNCTION__, result);
return result;
}
freeBufferLocked(item.mSlot);
*outItem = item;
return NO_ERROR;
}
status_t CpuConsumer::attachAndReleaseBuffer(const sp<GraphicBuffer>& buffer) {
if (buffer == NULL) {
return BAD_VALUE;
}
Mutex::Autolock lock(mMutex);
int slot = -1;
status_t result = mConsumer->attachBuffer(&slot, buffer);
if (result != NO_ERROR) {
CC_LOGE("%s: Failed to attach buffer (%d)", __FUNCTION__, result);
return result;
}
// These behaviors must be kept in sync with
// BufferQueueConsumer::attachBuffer
mSlots[slot].mGraphicBuffer = buffer;
mSlots[slot].mFence = Fence::NO_FENCE;
mSlots[slot].mFrameNumber = 0;
result = releaseBufferLocked(slot, buffer, EGL_NO_DISPLAY, EGL_NO_SYNC_KHR);
if (result != NO_ERROR) {
CC_LOGE("%s: Failed to release buffer (%d)", __FUNCTION__, result);
return result
}
return NO_ERROR;
}
} // namespace android