Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c9ac46bb authored by Eric Laurent's avatar Eric Laurent
Browse files

audioflinger: fix mmap rerouting on track invalidation 

When mmap tracks are invalidated by audio policy manager to request
a reconnection of the client to another output, do not send an onTearDown()
callback to AAudioService but an onRoutingChanged() callback to force
the stream disconnection.

Bug: 246803857
Test: repro steps in bug
Change-Id: I92b759a90d13c36052c9b39de17731b0214bb673
Merged-In: I92b759a90d13c36052c9b39de17731b0214bb673
parent b9cefaf4

services/audioflinger/Threads.cpp

+10 −7
Original line number Diff line number Diff line
@@ -10222,19 +10222,22 @@ status_t AudioFlinger::MmapThread::checkEffectCompatibility_l( 


void AudioFlinger::MmapThread::checkInvalidTracks_l()

{

    sp<MmapStreamCallback> callback;

    for (const sp<MmapTrack> &track : mActiveTracks) {

        if (track->isInvalid()) {

            sp<MmapStreamCallback> callback = mCallback.promote();

            if (callback != 0) {

                mLock.unlock();

                callback->onTearDown(track->portId());

                mLock.lock();

            } else if (mNoCallbackWarningCount < kMaxNoCallbackWarnings) {

                ALOGW("Could not notify MMAP stream tear down: no onTearDown callback!");

            callback = mCallback.promote();

            if (callback == nullptr &&  mNoCallbackWarningCount < kMaxNoCallbackWarnings) {

                ALOGW("Could not notify MMAP stream tear down: no onRoutingChanged callback!");

                mNoCallbackWarningCount++;

            }

            break;

        }

    }

    if (callback != 0) {

        mLock.unlock();

        callback->onRoutingChanged(AUDIO_PORT_HANDLE_NONE);

        mLock.lock();

    }

}



void AudioFlinger::MmapThread::dumpInternals_l(int fd, const Vector<String16>& args __unused)