commit fa546d3397bcd3bf94f26e37b4d62ddb46227f45
author Chih-Yu Huang <akahuang@google.com> Thu Jul 02 16:26:52 2020 +0900
committer Chih-Yu Huang <akahuang@google.com> Fri Jul 03 10:14:37 2020 +0900
tree 27caf6df6dc37fe51dd8ae36c8fbdf9f83cfa1ae
parent dc952450309a0a16365bf4b6b62d6152b2d4df59

components: Polish some logs.

This CL adds or fixes some information at logging, not changing the
code logic.

Bug: 160110407
Test: m -j32

Change-Id: I97d2cd43a526e56000abcc16886b1f0f6e0f6b71

components/V4L2Decoder.cpp

diff --git a/components/V4L2Decoder.cpp b/components/V4L2Decoder.cpp
index 59e3b0c..6a60dda 100644
--- a/components/V4L2Decoder.cpp
+++ b/components/V4L2Decoder.cpp
@@ -329,10 +329,13 @@
 }
 
 void V4L2Decoder::serviceDeviceTask(bool event) {
-    ALOGV("%s(event=%d) state=%s InputQueue:%zu+%zu/%zu, OutputQueue:%zu+%zu/%zu", __func__, event,
-          StateToString(mState), mInputQueue->FreeBuffersCount(), mInputQueue->QueuedBuffersCount(),
-          mInputQueue->AllocatedBuffersCount(), mOutputQueue->FreeBuffersCount(),
-          mOutputQueue->QueuedBuffersCount(), mOutputQueue->AllocatedBuffersCount());
+    ALOGV("%s(event=%d) state=%s InputQueue(%s):%zu+%zu/%zu, OutputQueue(%s):%zu+%zu/%zu", __func__,
+          event, StateToString(mState), (mInputQueue->IsStreaming() ? "streamon" : "streamoff"),
+          mInputQueue->FreeBuffersCount(), mInputQueue->QueuedBuffersCount(),
+          mInputQueue->AllocatedBuffersCount(),
+          (mOutputQueue->IsStreaming() ? "streamon" : "streamoff"),
+          mOutputQueue->FreeBuffersCount(), mOutputQueue->QueuedBuffersCount(),
+          mOutputQueue->AllocatedBuffersCount());
     ALOG_ASSERT(mTaskRunner->RunsTasksInCurrentSequence());
 
     if (mState == State::Error) return;
@@ -344,7 +347,7 @@
         media::V4L2ReadableBufferRef dequeuedBuffer;
         std::tie(success, dequeuedBuffer) = mInputQueue->DequeueBuffer();
         if (!success) {
-            ALOGE("Failed to dequeue buffer from output queue.");
+            ALOGE("Failed to dequeue buffer from input queue.");
             onError();
             return;
         }
@@ -519,7 +522,7 @@
     }
 
     size_t bufferId = outputBuffer.BufferId();
-    ALOGE("QBUF to output queue, bufferId=%zu", bufferId);
+    ALOGV("QBUF to output queue, bufferId=%zu", bufferId);
     std::move(outputBuffer).QueueDMABuf(frame->getFDs());
     mFrameAtDevice.insert(std::make_pair(bufferId, std::move(frame)));