commit 93bb77da5481ab75c2cd6e3aa681839273c6e43d
author Glenn Kasten <gkasten@google.com> Mon Jun 24 12:10:45 2013 -0700
committer Glenn Kasten <gkasten@google.com> Mon Jun 24 13:34:08 2013 -0700
tree 024bebbba1fd4692be502a1fc2c7297055a2a3cc
parent 0d09a9bec07b3bec78bd473ff0bfcf0a261f3f25

Workaround AudioRecord bug for large buffer sizes

Bug: 9556436
Change-Id: I92d1238b623d2cfd648e0a684d0e710fb0bd8b43

media/libmedia/AudioTrackShared.cpp

diff --git a/media/libmedia/AudioTrackShared.cpp b/media/libmedia/AudioTrackShared.cpp
index 4b7f368..5f8f292 100644
--- a/media/libmedia/AudioTrackShared.cpp
+++ b/media/libmedia/AudioTrackShared.cpp
@@ -482,7 +482,8 @@
     } else if (minimum > half) {
         minimum = half;
     }
-    if (!mDeferWake && mAvailToClient + stepCount >= minimum) {
+    // FIXME AudioRecord wakeup needs to be optimized; it currently wakes up client every time
+    if (!mIsOut || (!mDeferWake && mAvailToClient + stepCount >= minimum)) {
         ALOGV("mAvailToClient=%u stepCount=%u minimum=%u", mAvailToClient, stepCount, minimum);
         int32_t old = android_atomic_or(CBLK_FUTEX_WAKE, &cblk->mFutex);
         if (!(old & CBLK_FUTEX_WAKE)) {