commit 643825e4943995bbefd13d55c58bffd5e6904e90
author Andy Hung <hunga@google.com> Sat Feb 24 14:15:05 2018 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Sat Feb 24 14:15:05 2018 +0000
tree 065eac1e1623e5340edd77f25b98dcd1ad4377ec
parent 8231ac5a67006a8c0e9755fd143f788059930c45
parent 7840c30d04a749da9817682339f97b4ab128dffd

Merge "Revert "Ignore config error if non-offloadable effect is on offload thread""

services/audioflinger/Effects.cpp

diff --git a/services/audioflinger/Effects.cpp b/services/audioflinger/Effects.cpp
index 0ce5350..979290f 100644
--- a/services/audioflinger/Effects.cpp
+++ b/services/audioflinger/Effects.cpp
@@ -589,13 +589,6 @@
         status = cmdStatus;
     }
 
-    // Ignore error if non-offloadable effect is created on an offload thread.
-    // Will be switched to non-offload thread when the effect is enabled.
-    if (status != NO_ERROR && thread->type() == ThreadBase::OFFLOAD && !isOffloaded()) {
-        ALOGV("Ignore error %d on non-offloadable effect on offload thread", status);
-        status = NO_ERROR;
-    }
-
 #ifdef MULTICHANNEL_EFFECT_CHAIN
     if (status != NO_ERROR &&
             (mConfig.inputCfg.channels != AUDIO_CHANNEL_OUT_STEREO