Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 829c1976 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "BufferQueueProducer: Log verbose log to warning instead of error." am:... 

Merge "BufferQueueProducer: Log verbose log to warning instead of error." am: e9b2010f am: bf8edbe0 am: 2824e91b am: 7e75c7da am: 1e5d4e56

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1822354

Change-Id: I0d1bd13e3ca64e6f2fab3ee302358c6d99ca0aa9
parents 6fbb4991 1e5d4e56

libs/gui/BufferQueueProducer.cpp

+4 −1
Original line number Diff line number Diff line
@@ -645,7 +645,10 @@ status_t BufferQueueProducer::detachBuffer(int slot) { 
                    slot, BufferQueueDefs::NUM_BUFFER_SLOTS);

            return BAD_VALUE;

        } else if (!mSlots[slot].mBufferState.isDequeued()) {

            BQ_LOGE("detachBuffer: slot %d is not owned by the producer "

            // TODO(http://b/140581935): This message is BQ_LOGW because it

            // often logs when no actionable errors are present. Return to

            // using BQ_LOGE after ensuring this only logs during errors.

            BQ_LOGW("detachBuffer: slot %d is not owned by the producer "

                    "(state = %s)", slot, mSlots[slot].mBufferState.string());

            return BAD_VALUE;

        } else if (!mSlots[slot].mRequestBufferCalled) {