Add session mutex lock
[platform/core/api/audio-io.git] / src / cpp / CAudioOutput.cpp
index 80f17b9..c2ece71 100644 (file)
@@ -98,28 +98,33 @@ bool CAudioOutput::__IsReady() {
 }
 
 void CAudioOutput::initialize() {
-    if (__IsInit() == true) {
+    if (__IsInit() == true)
         return;
-    }
 
     try {
         CAudioIO::initialize();
+    } catch (CAudioError& e) {
+        finalize();
+        throw;
+    }
 
-        // Create ASM Handler
+    try {
+        internalSessionLock();
         mpAudioSessionHandler = new CAudioSessionHandler(CAudioSessionHandler::EAudioSessionType::AUDIO_SESSION_TYPE_PLAYBACK, mAudioInfo, this);
-
-        // Initialize ASM Handler
         mpAudioSessionHandler->initialize();
-
-        __setInit(true);
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
+        internalSessionUnlock();
     } catch (CAudioError& e) {
+        internalSessionUnlock();
         finalize();
         throw;
     } catch (const std::bad_alloc&) {
+        internalSessionUnlock();
         finalize();
         THROW_ERROR_MSG(CAudioError::EError::ERROR_OUT_OF_MEMORY, "Failed to allocate CAudioSessionHandler object");
     }
+
+    __setInit(true);
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
 }
 
 void CAudioOutput::finalize() {
@@ -128,8 +133,10 @@ void CAudioOutput::finalize() {
         return;
     }
 
+    internalSessionLock();
     SAFE_FINALIZE(mpAudioSessionHandler);
     SAFE_DELETE(mpAudioSessionHandler);
+    internalSessionUnlock();
 
     CAudioIO::finalize();
 
@@ -137,9 +144,8 @@ void CAudioOutput::finalize() {
 }
 
 void CAudioOutput::prepare() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED, "Did not initialize CAudioOutput");
-    }
 
     if (__IsReady() == true) {
         AUDIO_IO_LOGD("Already prepared CAudioOutput");
@@ -147,27 +153,43 @@ void CAudioOutput::prepare() {
         return;
     }
 
-    try {
-        internalLock();
-
-        // Check to invalid AudioType
-        CAudioInfo::EAudioType audioType = mAudioInfo.getAudioType();
-        if (audioType < CAudioInfo::EAudioType::AUDIO_OUT_TYPE_MEDIA || audioType >= CAudioInfo::EAudioType::AUDIO_TYPE_MAX) {
-            THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
-                                   "The audioType is invalid [type:%d]", static_cast<int>(audioType));
-        }
+    /* Check invalid AudioType */
+    CAudioInfo::EAudioType audioType = mAudioInfo.getAudioType();
+    if (audioType < CAudioInfo::EAudioType::AUDIO_OUT_TYPE_MEDIA ||
+        audioType >= CAudioInfo::EAudioType::AUDIO_TYPE_MAX)
+        THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
+                               "The audioType is invalid [type:%d]", static_cast<int>(audioType));
 
-        if (mpAudioSessionHandler->getId() < 0) {  // Did not registerSound()
-            if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false) {
-                // Register ASM Listener
+    try {
+        internalSessionLock();
+        if (mpAudioSessionHandler &&
+            mpAudioSessionHandler->getId() < 0) {  // Did not registerSound()
+            if (isForceIgnore() == false &&
+                mpAudioSessionHandler->isSkipSession() == false) {
                 AUDIO_IO_LOGD("Register ASM Listener");
                 mpAudioSessionHandler->registerSound();
             }
         }
+        internalSessionUnlock();
+    } catch (CAudioError& e) {
+        internalSessionUnlock();
+        throw;
+    }
 
-        CAudioIO::setInternalStreamInfo();
+    CAudioIO::setInternalStreamInfo();
 
-        // Init StreamSpec
+    try {
+        internalSessionLock();
+        if (isSessionEnabled() && isForceIgnore() == false)
+            mpAudioSessionHandler->updatePlaying();
+        internalSessionUnlock();
+    } catch (CAudioError& e) {
+        internalSessionUnlock();
+        throw;
+    }
+
+    try {
+        /* Init StreamSpec */
         CPulseStreamSpec::EStreamLatency streamSpec = CPulseStreamSpec::EStreamLatency::STREAM_LATENCY_OUTPUT_DEFAULT;
 #ifndef DISABLE_MOBILE_BACK_COMP
         if (!mStreamCallback.onStream) {
@@ -180,16 +202,11 @@ void CAudioOutput::prepare() {
 #endif
         CPulseStreamSpec spec(streamSpec, mAudioInfo);
 
-        // Create PulseAudio Handler
+        internalLock();
         mpPulseAudioClient = new CPulseAudioClient(CPulseAudioClient::EStreamDirection::STREAM_DIRECTION_PLAYBACK, spec, this);
-
-        if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false)
-            mpAudioSessionHandler->updatePlaying();
-
-        // Initialize PulseAudio Handler
         mpPulseAudioClient->initialize();
 #ifndef DISABLE_MOBILE_BACK_COMP
-        // Uncork stream which is created with CORKED flag */
+        /* Uncork stream which is created with CORKED flag */
         mpPulseAudioClient->cork(false);
 #endif
         internalUnlock();
@@ -207,10 +224,9 @@ void CAudioOutput::prepare() {
 }
 
 void CAudioOutput::unprepare() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize CAudioOutput");
-    }
 
     if (__IsReady() == false) {
         AUDIO_IO_LOGD("Already unprepared");
@@ -218,161 +234,134 @@ void CAudioOutput::unprepare() {
     }
 
     try {
-        if (mpAudioSessionHandler->getId() >= 0 && !mIsInterrupted) {
-            if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false) {
-                CPulseStreamSpec::EStreamLatency streamSpec;
-                streamSpec = mpPulseAudioClient->getStreamSpec().getStreamLatency();
-                if (CPulseStreamSpec::EStreamLatency::STREAM_LATENCY_OUTPUT_DEFAULT_ASYNC != streamSpec)
-                    CAudioIO::drain();
-            }
+        internalSessionLock();
+        if (isSessionEnabled() &&
+            !mIsInterrupted &&
+            !isForceIgnore() &&
+            mStreamCallback.onStream == NULL) {
+            CAudioIO::drain();
         }
-        CAudioIO::unprepare();
+        internalSessionUnlock();
+    } catch (CAudioError& e) {
+        internalSessionUnlock();
+        throw;
+    }
 
-        internalLock();
+    CAudioIO::unprepare();
 
+    try {
+        internalLock();
         SAFE_FINALIZE(mpPulseAudioClient);
         SAFE_DELETE(mpPulseAudioClient);
-
         internalUnlock();
+    } catch (CAudioError& e) {
+        internalUnlock();
+        throw;
+    }
 
-        if (mpAudioSessionHandler->getId() >= 0) {
-            if (isForceIgnore() == false && mpAudioSessionHandler->isSkipSession() == false)
-                mpAudioSessionHandler->updateStop();
-        }
+    try {
+        internalSessionLock();
+        if (isSessionEnabled() && isForceIgnore() == false)
+            mpAudioSessionHandler->updateStop();
+        internalSessionUnlock();
 
         CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_IDLE);
     } catch (CAudioError& e) {
-        internalUnlock();
+        internalSessionUnlock();
         throw;
     }
 }
 
 void CAudioOutput::pause() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING) {
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't pause if not in Running state");
-    }
 
-    if (mpPulseAudioClient->isInThread() == true) {
+    if (mpPulseAudioClient->isInThread() == true)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't pause in thread");
-    }
-
-    try {
-        CAudioIO::pause();
 
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::pause();
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED);
 }
 
 void CAudioOutput::resume() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED) {
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_PAUSED)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_STATE,
                         "Can't resume if not in Paused state");
-    }
 
-    if (mpPulseAudioClient->isInThread() == true) {
+    if (mpPulseAudioClient->isInThread() == true)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_OPERATION, "Can't resume in thread");
-    }
-
-    try {
-        CAudioIO::resume();
 
-        CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::resume();
+    CAudioIO::onStateChanged(CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING);
 }
 
 void CAudioOutput::drain() {
-    CPulseStreamSpec::EStreamLatency streamSpec;
-
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
-    streamSpec = mpPulseAudioClient->getStreamSpec().getStreamLatency();
-    if (CPulseStreamSpec::EStreamLatency::STREAM_LATENCY_OUTPUT_DEFAULT_ASYNC == streamSpec)
+    if (mStreamCallback.onStream)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION, "async type don't support drain");
 
-    try {
-        CAudioIO::drain();
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::drain();
 }
 
 void CAudioOutput::flush() {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
-    try {
-        CAudioIO::flush();
-    } catch (CAudioError& e) {
-        throw;
-    }
+    CAudioIO::flush();
 }
 
 int CAudioOutput::getBufferSize() {
-    if (__IsInit() == false) {
+    if (__IsInit() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
     /* FIXME : return calculated size here to satisfy backward compatibility */
     return (mAudioInfo.getSampleRate() * DEFAULT_PERIOD_SIZE) / 1000 * mAudioInfo.getSampleSize();
 }
 
 size_t CAudioOutput::write(const void* buffer, size_t length) {
-    if (__IsInit() == false || __IsReady() == false) {
+    if (__IsInit() == false || __IsReady() == false)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_NOT_INITIALIZED,
                         "Did not initialize or prepare CAudioOutput");
-    }
 
-    if (buffer == NULL) {
+    if (buffer == NULL)
         THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INVALID_ARGUMENT,
                                "Parameters are invalid - buffer:%p, length:%zu", buffer, length);
-    }
-    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING) {
+
+    if (CAudioIO::getState() != CAudioInfo::EAudioIOState::AUDIO_IO_STATE_RUNNING)
         THROW_ERROR_MSG(CAudioError::EError::ERROR_INVALID_OPERATION,
                         "Can't write if not in Running state");
-    }
 
     /* When write() is called in PulseAudio callback, bypass a pcm data to CPulseAudioClient (For Asynchronous) */
     if (mpPulseAudioClient && mpPulseAudioClient->isInThread() == true) {
         int ret = mpPulseAudioClient->write(buffer, length);
-        if (ret < 0) {
+        if (ret < 0)
             THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INTERNAL_OPERATION,
                                    "The written result is invalid ret:%d", ret);
-        }
-
 #ifdef _AUDIO_IO_DEBUG_TIMING_
         AUDIO_IO_LOGD("CPulseAudioClient->write(buffer:%p, length:%d)", buffer, length);
 #endif
-
         return length;
     }
 
     try {
         /* For synchronization */
         internalLock();
-        if (mIsInterrupted) {
+        if (mIsInterrupted)
             THROW_ERROR_MSG(CAudioError::EError::ERROR_POLICY_BLOCKED, "audio io is interrupted");
-        }
 
         // If another thread did call unprepare, do not write
         if (mpPulseAudioClient == NULL)
@@ -394,19 +383,17 @@ size_t CAudioOutput::write(const void* buffer, size_t length) {
                 internalWait();
             }
 
-            if (l > lengthIter) {
+            if (l > lengthIter)
                 l = lengthIter;
-            }
 
 #ifdef _AUDIO_IO_DEBUG_TIMING_
             AUDIO_IO_LOGD("CPulseAudioClient->write(buffer:%p, length:%d)", buffer, l);
 #endif
 
             int ret = mpPulseAudioClient->write(buffer, l);
-            if (ret < 0) {
+            if (ret < 0)
                 THROW_ERROR_MSG_FORMAT(CAudioError::EError::ERROR_INTERNAL_OPERATION,
                                        "The written result is invalid ret:%d", ret);
-            }
 
             buffer = static_cast<const uint8_t*>(buffer) + l;
             lengthIter -= l;
@@ -414,6 +401,7 @@ size_t CAudioOutput::write(const void* buffer, size_t length) {
 
         __mIsUsedSyncWrite = false;
         internalUnlock();
+
         sched_yield();
     } catch (CAudioError& e) {
         __mIsUsedSyncWrite = false;