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

Commit 900e05ed authored by Mikhail Naganov's avatar Mikhail Naganov
Browse files

audioflinger: Temporary disable post-close dumping of input streams 

There is a suspicion that dumping HAL state for a recently closed
input stream can hang forever, blocking audioflinger and stucking
system boot.

In order to verify, we disable this dumping to see if it would
help with presubmit bots.

Bug: 116211629
Test: see steps in b/116211629#9
Change-Id: I189461187a2310c213278cc2b3af2469d1896965
parent 3237d6b5

services/audioflinger/AudioFlinger.cpp

+4 −0
Original line number Diff line number Diff line
@@ -2500,7 +2500,9 @@ status_t AudioFlinger::closeInput_nonvirtual(audio_io_handle_t input) 
        if (recordThread != 0) {

            ALOGV("closeInput() %d", input);



#if 0

            dumpToThreadLog_l(recordThread);

#endif



            // If we still have effect chains, it means that a client still holds a handle

            // on at least one effect. We must either move the chain to an existing thread with the
@@ -2544,7 +2546,9 @@ status_t AudioFlinger::closeInput_nonvirtual(audio_io_handle_t input) 
            if (mmapThread == 0) {

                return BAD_VALUE;

            }

#if 0

            dumpToThreadLog_l(mmapThread);

#endif

            mMmapThreads.removeItem(input);

        }

        const sp<AudioIoDescriptor> ioDesc = new AudioIoDescriptor();