commit e9b2010fb1a8dcd0f23cac1731f510b0bfa4301f
author Treehugger Robot <treehugger-gerrit@google.com> Mon Sep 13 21:26:42 2021 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Sep 13 21:26:42 2021 +0000
tree 1302f830208bf2874248037510ac47806d25d658
parent 21fcad29cd4e8fbcff6d33c3616244af9ab3f9e8
parent 29936fa194516a906585eb3938d62d1944a21172

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

libs/gui/BufferQueueProducer.cpp

diff --git a/libs/gui/BufferQueueProducer.cpp b/libs/gui/BufferQueueProducer.cpp
index df308d8..5fe5e71 100644
--- a/libs/gui/BufferQueueProducer.cpp
+++ b/libs/gui/BufferQueueProducer.cpp
@@ -645,7 +645,10 @@
                     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) {