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

Commit 4361749d authored by Glenn Kasten's avatar Glenn Kasten Committed by Gerrit Code Review
Browse files

Merge "AudioTrack: Never try to use the fast path if resampling is required"

parents 7142d10a 398f2134

media/libmedia/AudioTrack.cpp

+6 −0
Original line number Diff line number Diff line
@@ -878,6 +878,12 @@ status_t AudioTrack::createTrack_l( 
    }

    ALOGV("createTrack_l() output %d afLatency %d", output, afLatency);



    if ((flags & AUDIO_OUTPUT_FLAG_FAST) && sampleRate != afSampleRate) {

        ALOGW("AUDIO_OUTPUT_FLAG_FAST denied by client due to mismatching sample rate (%d vs %d)",

              sampleRate, afSampleRate);

        flags = (audio_output_flags_t) (flags & ~AUDIO_OUTPUT_FLAG_FAST);

    }



    // The client's AudioTrack buffer is divided into n parts for purpose of wakeup by server, where

    //  n = 1   fast track with single buffering; nBuffering is ignored

    //  n = 2   fast track with double buffering

services/audioflinger/Configuration.h

+0 −3
Original line number Diff line number Diff line
@@ -32,9 +32,6 @@ 
// uncomment to enable fast mixer to take performance samples for later statistical analysis

#define FAST_MIXER_STATISTICS



// uncomment to allow fast tracks at non-native sample rate

//#define FAST_TRACKS_AT_NON_NATIVE_SAMPLE_RATE



// uncomment for debugging timing problems related to StateQueue::push()

//#define STATE_QUEUE_DUMP

services/audioflinger/Threads.cpp

+0 −2
Original line number Diff line number Diff line
@@ -1218,10 +1218,8 @@ sp<AudioFlinger::PlaybackThread::Track> AudioFlinger::PlaybackThread::createTrac 
            // mono or stereo

            ( (channelMask == AUDIO_CHANNEL_OUT_MONO) ||

              (channelMask == AUDIO_CHANNEL_OUT_STEREO) ) &&

#ifndef FAST_TRACKS_AT_NON_NATIVE_SAMPLE_RATE

            // hardware sample rate

            (sampleRate == mSampleRate) &&

#endif

            // normal mixer has an associated fast mixer

            hasFastMixer() &&

            // there are sufficient fast track slots available