commit b7e30fa0947dd71231845637e5475a052bd87a91
author Robert Wu <robertwu@google.com> Thu Dec 09 01:00:16 2021 +0000
committer Robert Wu <robertwu@google.com> Wed Dec 15 00:04:04 2021 +0000
tree c945c70ab8d0f0d4caa5dcdc7c0827aaa8fb181c
parent af0dd0d1643878711893c8bf0f03597cbc25d5f6

Don't apply extra AAudio volume ramps

For shared Audio streams, both the exclusive version and the shared version per client will be applying volume changes to the volume ramp.

The fix is to not apply volume changes to shared streams since the exclusive stream will do the ramp.

Bug: 209877538
Test: OboeTester MMAP exclusive/shared
Test: atest AAudioTests
Change-Id: I6172e7ace0b36f84f2a2005e30ad7667e210fe49

media/libaaudio/src/client/AudioStreamInternalPlay.cpp

diff --git a/media/libaaudio/src/client/AudioStreamInternalPlay.cpp b/media/libaaudio/src/client/AudioStreamInternalPlay.cpp
index 8292573..450d390 100644
--- a/media/libaaudio/src/client/AudioStreamInternalPlay.cpp
+++ b/media/libaaudio/src/client/AudioStreamInternalPlay.cpp
@@ -54,7 +54,8 @@
                              getDeviceFormat(),
                              getDeviceChannelCount(),
                              getRequireMonoBlend(),
-                             getAudioBalance());
+                             getAudioBalance(),
+                             (getSharingMode() == AAUDIO_SHARING_MODE_EXCLUSIVE));
 
         if (result != AAUDIO_OK) {
             safeReleaseClose();