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

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

audio policy: fix offload when spatial audio is enabled. 

The fact that the spatializer effect is enabled on the output stage
session of the spatializer mixer should not prevent compressed audio
playback from being offloaded.
The logic verifying that non offloadable effects are enabled is
updated to ignore output stage (post processing) effects because those
are not "music effects" but instead apply to all use cases on a given mixer.

Bug: 225119469
Test: enable spatial audio and verify MP3 offload
Change-Id: If612ffb8f7687c345fb0c4671a87f9ab4ad11ea7
parent 2545ce7c

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

+4 −3
Original line number Diff line number Diff line
@@ -3012,9 +3012,10 @@ status_t AudioPolicyManager::registerEffect(const effect_descriptor_t *desc, 
            }

        }

    }

    return mEffects.registerEffect(desc, io, session, id,

                                   (strategy == streamToStrategy(AUDIO_STREAM_MUSIC) ||

                                   strategy == PRODUCT_STRATEGY_NONE));

    bool isMusicEffect = (session != AUDIO_SESSION_OUTPUT_STAGE)

                            && ((strategy == streamToStrategy(AUDIO_STREAM_MUSIC)

                                    || strategy == PRODUCT_STRATEGY_NONE));

    return mEffects.registerEffect(desc, io, session, id, isMusicEffect);

}



status_t AudioPolicyManager::unregisterEffect(int id)