From 4c5a3675ed6b429dfaa2707acf2b3faf2808baf2 Mon Sep 17 00:00:00 2001 From: Phil Burk Date: Thu, 30 Apr 2015 16:18:53 -0700 Subject: AudioTrack: reduce retrograde motion spew Change-Id: I96aced52b136ceea924aa3d9ef56374dd4c49784 Signed-off-by: Phil Burk --- media/libmedia/AudioTrack.cpp | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) (limited to 'media/libmedia/AudioTrack.cpp') diff --git a/media/libmedia/AudioTrack.cpp b/media/libmedia/AudioTrack.cpp index 76d9169..bb47d3e 100644 --- a/media/libmedia/AudioTrack.cpp +++ b/media/libmedia/AudioTrack.cpp @@ -2227,11 +2227,18 @@ status_t AudioTrack::getTimestamp(AudioTimestamp& timestamp) - mPreviousTimestamp.mPosition); // position can bobble slightly as an artifact; this hides the bobble static const int32_t MINIMUM_POSITION_DELTA = 8; - ALOGW_IF(deltaPosition < 0, - "retrograde timestamp position corrected, %d = %u - %u", - deltaPosition, - timestamp.mPosition, - mPreviousTimestamp.mPosition); + if (deltaPosition < 0) { + // Only report once per position instead of spamming the log. + if (!mRetrogradeMotionReported) { + ALOGW("retrograde timestamp position corrected, %d = %u - %u", + deltaPosition, + timestamp.mPosition, + mPreviousTimestamp.mPosition); + mRetrogradeMotionReported = true; + } + } else { + mRetrogradeMotionReported = false; + } if (deltaPosition < MINIMUM_POSITION_DELTA) { timestamp = mPreviousTimestamp; // Use last valid timestamp. } -- cgit v1.1