ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

am d4e70620: am 72c3f7d8: Revert "ConsumerBase: free buffers outside the lock"

Browse Source 
* commit 'd4e706208c99ff1716fb0009ec867ff29ff04a7c':
  Revert "ConsumerBase: free buffers outside the lock"
This commit is contained in:
Jamie Gennis 2012-12-07 11:07:03 -08:00 committed by Android Git Automerger
commit 50f8d2856b
1 changed files with 10 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
libs/gui/ConsumerBase.cpp
View File

@ -109,35 +109,21 @@ void ConsumerBase::onFrameAvailable() {
} }
void ConsumerBase::onBuffersReleased() { void ConsumerBase::onBuffersReleased() {
sp<GraphicBuffer> bufRefs[BufferQueue::NUM_BUFFER_SLOTS]; Mutex::Autolock lock(mMutex);
{ // Scope for the lock CB_LOGV("onBuffersReleased");
Mutex::Autolock lock(mMutex);
CB_LOGV("onBuffersReleased"); if (mAbandoned) {
// Nothing to do if we're already abandoned.
if (mAbandoned) { return;
// Nothing to do if we're already abandoned.
return;
}
uint32_t mask = 0;
mBufferQueue->getReleasedBuffers(&mask);
for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) {
if (mask & (1 << i)) {
// Grab a local reference to the buffers so that they don't
// get freed while the lock is held.
bufRefs[i] = mSlots[i].mGraphicBuffer;
freeBufferLocked(i);
}
}
} }
// Clear the local buffer references. This would happen automatically uint32_t mask = 0;
// when the array gets dtor'd, but I'm doing it explicitly for clarity. mBufferQueue->getReleasedBuffers(&mask);
for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) { for (int i = 0; i < BufferQueue::NUM_BUFFER_SLOTS; i++) {
bufRefs[i].clear(); if (mask & (1 << i)) {
freeBufferLocked(i);
}
} }
} }