commit 89c224e298fd280a3f2a69da8cc930ff6d036e18
author SathishKumar Mani <smani@codeaurora.org> Tue Oct 09 16:29:07 2012 -0700
committer Iliyan Malchev <malchev@google.com> Tue Oct 09 16:56:11 2012 -0700
tree bf0f47848a330918eece8f756ab9cf51eb687f4c
parent 2c79891211108cd8157124c1b6f9d393729602ff

alsa_sound: prevent lowlatency errors triggering WD reset

- HAL tries to recover from pcm write or read failure
  by closing and reopening pcm driver.
- There is no check when pcm is reopened. If the
  reopen fails it keeps trying multiple times in loop
- Fix the issue by checking error condition and
  return on failure.

Bug: 7253359
Change-Id: Ia15153f5b43bb0f255a8b34b70025a6215484cee
Signed-off-by: Iliyan Malchev <malchev@google.com>

alsa_sound/AudioStreamInALSA.cpp

diff --git a/alsa_sound/AudioStreamInALSA.cpp b/alsa_sound/AudioStreamInALSA.cpp
index 031d477..3b66c24 100644
--- a/alsa_sound/AudioStreamInALSA.cpp
+++ b/alsa_sound/AudioStreamInALSA.cpp
@@ -435,6 +435,13 @@
                 }
                 else
                     mHandle->module->open(mHandle);
+
+                if(mHandle->handle == NULL) {
+                   ALOGE("read:: PCM device re-open failed");
+                   mParent->mLock.unlock();
+                   return 0;
+                }
+
                 mParent->mLock.unlock();
                 continue;
             }