commit 7e75c7da3452ba3ddf509536969c6a1431d53317
author Treehugger Robot <treehugger-gerrit@google.com> Mon Sep 13 22:03:01 2021 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Mon Sep 13 22:03:01 2021 +0000
tree 010898dcc696c95126c7c8f7eb7b6df0bfb82a8c
parent 8b245c803049691f5703bed05946b67135e8dfac
parent 2824e91bef3c03ee0e1bcca680d24b1776ce45f1

Merge "BufferQueueProducer: Log verbose log to warning instead of error." am: e9b2010fb1 am: bf8edbe041 am: 2824e91bef

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

Change-Id: I69b8a1a7aa23d5ebc5f6125ef92fd70cb1c3bfad

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) {