Commit Graph

2713 Commits

Author SHA1 Message Date
Treehugger Robot
2f50682b45 Merge "audio: Add tracing to audio I/O and effect processing" into main am: c02d38d7b7 am: 8eb4b053f7
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2940231

Change-Id: I00dbfa0d32ff20502c233cb8e1dcbd57855d9aae
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-01 00:28:27 +00:00
Shunkai Yao
bd9dc52add Merge "BassBoost Process: Add tests to validate Bass Boost Process." into main am: 2d1e1bd332 am: bb110f463d
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2923712

Change-Id: Ida2e895df495d08d3fb262161993da044dd0fac3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-01 00:28:08 +00:00
Treehugger Robot
8eb4b053f7 Merge "audio: Add tracing to audio I/O and effect processing" into main am: c02d38d7b7
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2940231

Change-Id: I8628b1e11def48298f8f4adde20d49ebd9aaa05c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 23:49:06 +00:00
Shunkai Yao
bb110f463d Merge "BassBoost Process: Add tests to validate Bass Boost Process." into main am: 2d1e1bd332
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2923712

Change-Id: I063b9c8f0de2c67436a6093881026037f71eb118
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 23:48:11 +00:00
Treehugger Robot
c02d38d7b7 Merge "audio: Add tracing to audio I/O and effect processing" into main 2024-01-31 23:26:42 +00:00
Shunkai Yao
2d1e1bd332 Merge "BassBoost Process: Add tests to validate Bass Boost Process." into main 2024-01-31 23:25:28 +00:00
Mikhail Naganov
89a6bb7199 audio: Add tracing to audio I/O and effect processing
Emit trace events for audio reads and writes and effect
processing functions. This is to match the HIDL implementation.

Bug: 321233946
Test: `record_android_trace` with `audio` category enabled
Change-Id: I26907b09243fd3e5aaa470a0fb930b34addd3093
2024-01-31 14:19:08 -08:00
Aditya Choudhary
39ed8dc1ad [DON'T BLOCK] Test ownership migration rules am: c5c6c62995 am: 85d604d0db
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2939799

Change-Id: I68af10e082403fcfdbd89e7d63bb07f207bc06e2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 17:46:24 +00:00
Aditya Choudhary
85d604d0db [DON'T BLOCK] Test ownership migration rules am: c5c6c62995
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2939799

Change-Id: I1046113ffdcd09649fa65e023aae0969262772e0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 17:06:10 +00:00
Aditya Choudhary
c5c6c62995 [DON'T BLOCK] Test ownership migration rules
This CL is created as a best effort to migrate test targets
to the new android ownership model. If you find incorrect or unnecessary
attribution in this CL, please create a separate CL to fix that.

For more details please refer to the link below,
<add g3 doc link>

Bug: 304529413
Test: N/A
Change-Id: I3322344f595f974f730dc824af0110388076d838
2024-01-31 11:06:17 +00:00
Sneha Patil
e584923d98 BassBoost Process: Add tests to validate Bass Boost Process.
Added methods to set and validate parameters.
Added test to verify increasing strengths.

Bug:305866207
Test: atest VtsHalBassBoostTargetTest
Change-Id: I64e06deca199b3acc53eb15cf2007e6db27430f3
2024-01-31 14:13:02 +05:30
Shunkai Yao
c0c3ce95a4 vts: update SetCommonParameterAndReopen to DataPathTest am: d685aed029 am: e491392e99
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2937539

Change-Id: If18eee093eb5ad85a9bababb5467f28a1e415091
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 02:56:50 +00:00
Shunkai Yao
e491392e99 vts: update SetCommonParameterAndReopen to DataPathTest am: d685aed029
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2937539

Change-Id: Iad2853dd3854db6a81cb991a9704941ec5191235
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-31 01:09:00 +00:00
Xin Li
763c473c3d Merge Android 24Q1 Release (ab/11220357)
Bug: 319669529
Merged-In: I976f80006aeb88bde2ae34ca4f9be72cea727d9e
Change-Id: Id94b25af52bd2e2847b6858697ab21cc6ce27aa3
2024-01-30 10:36:50 -08:00
Shunkai Yao
d685aed029 vts: update SetCommonParameterAndReopen to DataPathTest
Bug: 323010068
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I05519481b69bb752f515bd54cb1eab0996ad5433
2024-01-30 18:35:26 +00:00
Treehugger Robot
2c1e429d34 Merge "audio: Reduce the interval between BT proxy registration retries" into main am: 320ef27af2 am: 121dcebc4e
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2933574

Change-Id: Iaa7c2b48609fe9a65b3fd35fa6a4f12f382886bc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-30 01:12:46 +00:00
Treehugger Robot
320ef27af2 Merge "audio: Reduce the interval between BT proxy registration retries" into main 2024-01-29 23:00:57 +00:00
Mikhail Naganov
19de94764d audio: Reduce the interval between BT proxy registration retries
A follow-up for aosp/2929441. The interval between retries
of connecting to BT proxy must be reduced significantly.
Otherwise, in situations when the BT is not connected
(that happens during tests), the resulting long delay causes
failures at the frameowork level due to resulting long
timeout.

Bug: 322820830
Test: atest audiosystem_tests --iterations 100
Change-Id: Idd84cb22b28d109151f98bb5883dca18a9ae928d
2024-01-29 13:40:27 -08:00
Shunkai Yao
56fe00d748 Merge "Update work buffer size with input/output frame size change" into main am: db9b8f08da am: 4ce527e8a9
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2925168

Change-Id: I9352ddeadbf19a7cb52bbbaad47c4de1b3cbc091
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-29 20:07:06 +00:00
Shunkai Yao
db9b8f08da Merge "Update work buffer size with input/output frame size change" into main 2024-01-29 18:51:47 +00:00
Mikhail Naganov
9db9570eec Merge "audio: Add retries for BT proxy port registration" into main am: 3552515d70 am: 2a3bc07b4e
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2929441

Change-Id: I616fce4c978a0effecbcee05adadffde05f8d6f2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-29 18:37:33 +00:00
Mikhail Naganov
3552515d70 Merge "audio: Add retries for BT proxy port registration" into main 2024-01-29 17:17:56 +00:00
Mikhail Naganov
a5c0437327 audio: Add retries for BT proxy port registration
When ModuleBluetooth::createProxy is invoked on BT device
connection, the BT stack may not be fully ready yet,
and port registration can fail. This is an intermittent
state and registration should succeed after retrying.

Bug: 320838889
Test: atest pts-bot:A2DP/SRC/REL/BV-02-I
Change-Id: I0c7cf7c1c6a8ee03ef55b004f89139e3b56ee9cd
2024-01-26 16:19:45 -08:00
Shunkai Yao
08c05dd017 Update work buffer size with input/output frame size change
call process with the minimal of input/output fmq data capacity

Bug: 321543210
Test: atest VtsHalAudioEffectTargetTest
Test: flash to pixel phone and test audio
Change-Id: I0c5087f541dc8526ac733a0c35f0d06f878d78a1
2024-01-26 18:59:13 +00:00
Bart Van Assche
51cde196c5 Remove superfluous include directives am: 5e7fcb4bbd am: 06101e215e
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2925152

Change-Id: I6c06a7ab6205dfce06e9b4802b53762fdc87918f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-24 01:05:35 +00:00
Bart Van Assche
5e7fcb4bbd Remove superfluous include directives
Remove #include "AudioHalBinderServiceUtil.h" from source files that do
not use the AudioHalBinderServiceUtil class directly.

Change-Id: Iff085856df91c2818dbd06bf790233cbb82a3379
Signed-off-by: Bart Van Assche <bvanassche@google.com>
2024-01-23 14:19:27 -08:00
Deyao Ren
e1e6ead010 Merge "Create vendor apex for cuttlefish audio and audio effect" into main am: fe55cba9bb am: 66f0387a35
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2863478

Change-Id: Ib31fbc62d5f600d37f5a46fb46af83a1c5efb9e9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-17 22:06:18 +00:00
Deyao Ren
0874626d77 Create vendor apex for cuttlefish audio and audio effect
Test: launch cuttlefish, CtsMediaAudioTestCases
Bug: 295803971
Change-Id: I056c3e81662f90207702b47664c23b7ddd1db378
2024-01-17 07:12:07 +00:00
Shunkai Yao
78265377d9 Merge "Effect VTS: update VtsHalDownmixTargetTest for data validation" into main 2024-01-13 03:50:02 +00:00
Treehugger Robot
dabe4a7902 Merge "audio: Fix handling of a thread exit command with a bad cookie" into main am: 759fb98667 am: 0ad71fd5fc am: 0e1a0a9328
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2908743

Change-Id: I73571a0d39b488d5aedd8162a4ffaa2cb673aef6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-13 03:30:37 +00:00
Treehugger Robot
759fb98667 Merge "audio: Fix handling of a thread exit command with a bad cookie" into main 2024-01-13 00:40:04 +00:00
Treehugger Robot
11646feb65 Merge "audio: Skip AudioModuleRemoteSubmixTest on Android U" into main am: 7f25c70740 am: 2681bdedf0 am: 2a0716f4cb
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2908298

Change-Id: I4f96c7848f0b5d40688dbe5806074f9ddd2603ac
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-12 23:20:45 +00:00
Mikhail Naganov
1850779bc4 audio: Fix handling of a thread exit command with a bad cookie
In case when the command was sent by the HAL itself (from another
thread), the worker thread must not post a reply. The only case
when a reply needs to be posted is in the case when the command
was sent from a VTS test. This case is identified by the fact that
the cookie has value '0'.

Bug: 300181540
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Ifeb0722b5cf7346a694c5a938f6b324f5fa825f1
2024-01-12 13:59:24 -08:00
Shunkai Yao
1319e986ab Effect VTS: update VtsHalDownmixTargetTest for data validation
For downmix, the output buffer size can be diff with input

Bug: 318926783
Bug: 317946442
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I0dc0009e1779b842a4f3cdcc047d225310a304f9
Merged-In: I0dc0009e1779b842a4f3cdcc047d225310a304f9
2024-01-12 21:54:17 +00:00
Treehugger Robot
3dd5a20ef5 Merge "Add android.virtualdevice.cts.VirtualAudioTest to postsubmit" into main am: 1fd5aef410 am: be7b1f22f0 am: 6be17c5eb2
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2907295

Change-Id: I341f2f3dac6321769d29eaa6b5584e8a77cfa001
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-12 21:36:08 +00:00
Treehugger Robot
7f25c70740 Merge "audio: Skip AudioModuleRemoteSubmixTest on Android U" into main 2024-01-12 21:10:04 +00:00
Shunkai Yao
3eb2a2c14f Merge changes from topic "effect_reopen" into main
* changes:
  Effect AIDL: implement IEffect.reopen
  Effect AIDL: add IEffect.reopen to update the effect instances data FMQ
2024-01-12 21:06:08 +00:00
Shunkai Yao
b34f133544 Merge "Effect VTS: update VtsHalDownmixTargetTest for data validation" into main 2024-01-12 21:01:51 +00:00
Mikhail Naganov
b2dab17b1e audio: Skip AudioModuleRemoteSubmixTest on Android U
This test was developed after Android U was cut, and it assumes
the new way of encoding of the remote submix device type.

Bug: 300181540
Test: run `atest VtsHalAudioCoreTargetTest` on UDC GSI
Change-Id: If53f92a55734e18a2acce6c790f9e5e259246684
2024-01-12 10:38:03 -08:00
Shunkai Yao
65c7c7051d Effect AIDL: implement IEffect.reopen
- add IEffect.reopen implementation
- now data MQs can update at runtime, sync
  EffectContext access
- add clang thread annotation

Bug: 302036943
Test: atest VtsHalAudioEffectTargetTest
Test: build and test audio effect on Pixel
Change-Id: I3e9fdc2d5eb50b8c1377e0da75573f0eba7ea3f1
Merged-In: I3e9fdc2d5eb50b8c1377e0da75573f0eba7ea3f1
2024-01-12 17:23:28 +00:00
Shunkai Yao
5b15e005a9 Effect AIDL: add IEffect.reopen to update the effect instances data FMQ
The effect instance may choose to reallocate the input data message
queue under specific conditions. For example, when the input format
changes, requiring an update to the data message queue allocated during
the open time.
In such cases, the effect instance can destroy the existing data message
queue, when the audio framework see a valid status MQ and invalid data MQ,
it call reopen to get the new data message queue.

Bug: 302036943
Test: m android.hardware.audio.effect-update-api, m
Change-Id: Ia245b154176f64bc3cc6e6049bca4f9c68ad482d
Merged-In: Ia245b154176f64bc3cc6e6049bca4f9c68ad482d
2024-01-12 17:23:14 +00:00
Shunkai Yao
0061ab1ae7 Merge "Effect AIDL: implement IEffect.reopen" into main 2024-01-12 16:26:19 +00:00
Mikhail Naganov
18fcb688bc Add android.virtualdevice.cts.VirtualAudioTest to postsubmit
These tests exercise audio playback and capture on CF and are
useful in spotting issues in the audio pipeline end-to-end.

Bug: 316017930
Test: atest CtsVirtualDevicesTestCases
Change-Id: If625cec5bc2d281f1e0d99b820f604f1bac7b621
2024-01-11 09:54:50 -08:00
Treehugger Robot
ee411a9b3a Merge "audio: Add libaudioclient tests to postsubmit for the AIDL HAL" into main am: 7fe11fd93d am: 4b454b6e9e am: e188448e49
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2904415

Change-Id: Iba604161761aae383fb5f5b99fd80e4583dff369
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-11 01:08:38 +00:00
Shunkai Yao
7d96d9e0ff Effect VTS: update VtsHalDownmixTargetTest for data validation
For downmix, the output buffer size can be diff with input

Bug: 318926783
Bug: 317946442
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I0dc0009e1779b842a4f3cdcc047d225310a304f9
2024-01-11 00:44:16 +00:00
Shunkai Yao
81bfcda77a Effect AIDL: implement IEffect.reopen
- add IEffect.reopen implementation
- now data MQs can update at runtime, sync
  EffectContext access
- add clang thread annotation

Bug: 302036943
Test: atest VtsHalAudioEffectTargetTest
Test: build and test audio effect on Pixel
Change-Id: I3e9fdc2d5eb50b8c1377e0da75573f0eba7ea3f1
2024-01-10 23:05:43 +00:00
Treehugger Robot
7fe11fd93d Merge "audio: Add libaudioclient tests to postsubmit for the AIDL HAL" into main 2024-01-10 23:02:04 +00:00
Mikhail Naganov
ff52e4fcba audio: Add libaudioclient tests to postsubmit for the AIDL HAL
These framework-side tests are helpful in detecting breakages
caused by changes in the HAL code.

Add to postsubmit first because the presubmit check does not
allow adding to presubmit without accumulating some history first.

Bug: 311830316
Test: atest
Change-Id: I1f946c71a89117d25f39fde34877988ea7e1d466
2024-01-10 13:47:57 -08:00
Mikhail Naganov
3daa193aa1 Merge "audio: Improve logging in remote submix module" into main am: 198e8f747c am: 13993da38a am: b3d65cec6c
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2901102

Change-Id: Iff8c545935a838b9ca593f2c0344042b115b2d90
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-10 21:12:38 +00:00
Mikhail Naganov
198e8f747c Merge "audio: Improve logging in remote submix module" into main 2024-01-10 17:53:05 +00:00