commit fc62917d0adf02637011960ecd2ac0c3d20317a7
author Andy Hung <hunga@google.com> Mon Jun 22 10:06:23 2020 -0700
committer Andy Hung <hunga@google.com> Mon Jun 22 10:41:48 2020 -0700
tree 7e9350c96601dca8ed5391324009e0d3b1fbb819
parent f3f98cc259b9ab73a1069e0a9bb63b3de8288dab

AudioFlinger: Fix spurious underrun logging

Removes false positive errors in metrics and debugging.

Test: Play Music, check underruns
Bug: 159620406
Change-Id: I008381bbf3aae13c0fe8289a0eeec47b7780e587

services/audioflinger/Tracks.cpp

diff --git a/services/audioflinger/Tracks.cpp b/services/audioflinger/Tracks.cpp
index be2f687..126015f 100644
--- a/services/audioflinger/Tracks.cpp
+++ b/services/audioflinger/Tracks.cpp
@@ -805,7 +805,7 @@
     status_t status = mServerProxy->obtainBuffer(&buf);
     buffer->frameCount = buf.mFrameCount;
     buffer->raw = buf.mRaw;
-    if (buf.mFrameCount == 0 && !isStopping() && !isStopped() && !isPaused()) {
+    if (buf.mFrameCount == 0 && !isStopping() && !isStopped() && !isPaused() && !isOffloaded()) {
         ALOGV("%s(%d): underrun,  framesReady(%zu) < framesDesired(%zd), state: %d",
                 __func__, mId, buf.mFrameCount, desiredFrames, mState);
         mAudioTrackServerProxy->tallyUnderrunFrames(desiredFrames);