commit 2fc14730e4697a6f456b4631549c9981f6b0b115
author Glenn Kasten <gkasten@google.com> Mon Aug 05 14:58:14 2013 -0700
committer Glenn Kasten <gkasten@google.com> Tue Aug 13 09:24:18 2013 -0700
tree 0f48a06513b295104f8c085a971fcbdf3feca1fe
parent c0029a8551d65929cbf51a55bd3bae3342a1f2fe

Comments

Change-Id: I56f82904cb3855bad6f33ad1ff4ddf66a61c5720

services/audioflinger/AudioFlinger.cpp

diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp
index 1bac09e..91cf6ce 100644
--- a/services/audioflinger/AudioFlinger.cpp
+++ b/services/audioflinger/AudioFlinger.cpp
@@ -500,10 +500,12 @@
 
         track = thread->createTrack_l(client, streamType, sampleRate, format,
                 channelMask, frameCount, sharedBuffer, lSessionId, flags, tid, &lStatus);
+        // we don't abort yet if lStatus != NO_ERROR; there is still work to be done regardless
 
         // move effect chain to this output thread if an effect on same session was waiting
         // for a track to be created
         if (lStatus == NO_ERROR && effectThread != NULL) {
+            // no risk of deadlock because AudioFlinger::mLock is held
             Mutex::Autolock _dl(thread->mLock);
             Mutex::Autolock _sl(effectThread->mLock);
             moveEffectChain_l(lSessionId, effectThread, thread, true);
@@ -1269,7 +1271,7 @@
         goto Exit;
     }
 
-    // return to handle to client
+    // return handle to client
     recordHandle = new RecordHandle(recordTrack);
 
 Exit: