From b687a2814ca9db576eb1ea33dea90ac35cd61bc1 Mon Sep 17 00:00:00 2001 From: Pablo Ceballos Date: Tue, 6 Oct 2015 11:14:51 -0700 Subject: BQ: fix Volantis test failures The Volantis driver will dequeue a buffer in eglCreateWindowSurface. - Remove the requirement that no buffers be dequeued when calling setAsyncMode() on a BufferQueueProducer, since this gets called from eglSwapInterval. - Modify the tests to call setMaxDequeuedBufferCount before calling eglCreateWindowSurface. Change-Id: Icc64e9933f151771bbd57035549cd5928c0b7216 --- libs/gui/BufferQueueProducer.cpp | 8 -------- 1 file changed, 8 deletions(-) (limited to 'libs/gui/BufferQueueProducer.cpp') diff --git a/libs/gui/BufferQueueProducer.cpp b/libs/gui/BufferQueueProducer.cpp index deec3305b8..0cb018ccae 100644 --- a/libs/gui/BufferQueueProducer.cpp +++ b/libs/gui/BufferQueueProducer.cpp @@ -159,14 +159,6 @@ status_t BufferQueueProducer::setAsyncMode(bool async) { return NO_INIT; } - // There must be no dequeued buffers when changing the async mode. - for (int s = 0; s < BufferQueueDefs::NUM_BUFFER_SLOTS; ++s) { - if (mSlots[s].mBufferState == BufferSlot::DEQUEUED) { - BQ_LOGE("setAsyncMode: buffer owned by producer"); - return BAD_VALUE; - } - } - if ((mCore->mMaxAcquiredBufferCount + mCore->mMaxDequeuedBufferCount + (async || mCore->mDequeueBufferCannotBlock ? 1 : 0)) > mCore->mMaxBufferCount) { -- cgit v1.2.3-59-g8ed1b