Commit Graph

2696 Commits

Author SHA1 Message Date
Treehugger Robot
74dc4cc5d7 Merge "[DON'T BLOCK] Test ownership migration rules" into main am: 27e97308c4
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2950589

Change-Id: Ie4f36b193820490e606e7f5100ca29200f832fa2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-07 01:45:21 +00:00
Treehugger Robot
27e97308c4 Merge "[DON'T BLOCK] Test ownership migration rules" into main 2024-02-07 00:55:34 +00:00
Shunkai Yao
35f4bb6089 Merge "Add backward compatibility in effect hal default implementation" into main am: 3ed9b6bbce
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2951489

Change-Id: If5d00b8810e95e0a870fe455951572d687da33cd
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-06 18:58:18 +00:00
Jooyung Han
6b6d0ef23c Merge "Revert "Install bluetooth audio VINTF fragment in apex"" into main am: c5c0b5eb6a
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2951227

Change-Id: Ia075e0a1eecf75781f243947a53536eb2a688e19
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-06 06:40:20 +00:00
Ke-Yu Lu
e9f8c8c4f5 Revert "Install bluetooth audio VINTF fragment in apex"
Revert submission 2949647-bt-audio-vintf

Reason for revert: b/323989115

Reverted changes: /q/submissionid:2949647-bt-audio-vintf

Change-Id: Ie335aa9a0ad2bb847cabe6f85ef37352e9065e3a
2024-02-06 04:04:01 +00:00
Jooyung Han
e5596af7e1 Merge "Install bluetooth audio VINTF fragment in apex" into main am: 0cacd8c7f5
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2949685

Change-Id: I043fbc9d39e21537b05e66d31d962addaaf8eb02
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-06 01:20:20 +00:00
Shunkai Yao
7e2c3719ff Add backward compatibility in effect hal default implementation
Skipping the reopen sequence for unsupported version
Add member initialize for EffectContext

Bug: 322780092
Test: latest libAudioHal with V1 effect HAL
Test: atest VtsHalAudioEffectTargetTest

Change-Id: I597194e5ebf25566d5adda533e151da2e99781f4
2024-02-05 22:58:54 +00:00
Aditya Choudhary
ad2ba1b851 [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,
go/new-android-ownership-model

Bug: 304529413
Test: N/A
Change-Id: Ic8c4bd6ac246e7efa98bbdaf822d12ae8e1230a5
2024-02-05 15:45:22 +00:00
Jooyung Han
92ca5545ca Install bluetooth audio VINTF fragment in apex
Previously, android.hardware.bluetooth.audio-impl was installed with no
use and the attached vintf was fulfilled by the
com.android.hardware.audio apex.

For cleanup, we no longer install android.hardware.bluetooth.audio-impl
separately (for cuttlefish) and install the VINTF inside the apex.

Bug: 312265159
Test: atest VtsHalBluetoothAudioTargetTest
Change-Id: I31e0ccd6a8c3c00565159f2be7fe3bf4d70e9ddf
2024-02-05 11:04:02 +09:00
Shunkai Yao
4221f8783d Merge "Effect: verify all AIDL IEffect versions are same as IFactory" into main am: 047484de60
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2945531

Change-Id: I4ca7e560dbe9a60dfb8b9420493edf5628e8560d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-02 20:39:08 +00:00
Mikhail Naganov
64edc33110 Merge "audio: Move stable libaudioclient tests to presubmit for the AIDL HAL" into main am: cb49cde0ad
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2893423

Change-Id: I2f827ef54fed5f2ed80cf64176bb7dd199f46de8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-02 20:36:44 +00:00
Shunkai Yao
047484de60 Merge "Effect: verify all AIDL IEffect versions are same as IFactory" into main 2024-02-02 20:23:29 +00:00
Mikhail Naganov
cb49cde0ad Merge "audio: Move stable libaudioclient tests to presubmit for the AIDL HAL" into main 2024-02-02 19:53:44 +00:00
Jooyung Han
3f23aa7523 Merge "Use LLNDK guard instead of builtin_available" into main am: 63b64e2995
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2943125

Change-Id: I85ae07b6cd65ac885f18dff099444573369dd528
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-02 02:19:53 +00:00
Jooyung Han
63b64e2995 Merge "Use LLNDK guard instead of builtin_available" into main 2024-02-02 01:58:22 +00:00
Shunkai Yao
910a8153b2 Effect: verify all AIDL IEffect versions are same as IFactory
Bug: 322780092
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: If47f70b9d8f72a45f3ecf927294fb93fda28851b
2024-02-02 01:46:09 +00:00
Mikhail Naganov
03761c7e8e audio: Move stable libaudioclient tests to presubmit for the AIDL HAL
These framework-side tests are helpful in detecting breakages
caused by changes in the HAL code.

Data from postsubmit runs:

Test module audioeffect_tests: Latency (p90): 3s, Flakiness: 0.00%
Test module trackplayerbase_tests: Latency (p90): 13s, Flakiness: 0.00%

Bug: 311830316
Test: presubmit
Test-Mapping-Slo-Bypass-Bug: b/318425155
Change-Id: I5c329239cd3890e98edc74283bc9edb83e7e7ba7
2024-02-01 10:47:21 -08:00
Jooyung Han
23e5bf2873 Use LLNDK guard instead of builtin_available
APEXSUPPORT_API was a workaround when we had no support for
__ANDROID_VENDOR_API__. Since we have it now, use it instead of
builtin_available(android APEXSUPPORT_API).

Bug: 302088370
Test: CtsMediaAudioTestCases
Change-Id: I28b0e0ebab5f357a2f5cc29f244ae5494e379fbe
2024-02-01 12:49:35 +09:00
Treehugger Robot
178592e696 Merge "AHAL: add default values for dynamic port config of hardcoded config." into main am: a64a264f2d
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2941766

Change-Id: I06ea348c29c0d4ed3055884f0151f2974b508f4e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-01 02:16:04 +00:00
Treehugger Robot
a64a264f2d Merge "AHAL: add default values for dynamic port config of hardcoded config." into main 2024-02-01 01:42:36 +00:00
jiabin
129b145133 AHAL: add default values for dynamic port config of hardcoded config.
If the values(format, sample rate, channel mask) are left as unset, they
will be null. In that case, when using them will require checking
nullablitiy to avoid crash.

Bug: 323255506
Test: make and boot
Change-Id: Ic66bf0b6f53750ff8700e531cca13f03fbb72936
2024-01-31 23:56:26 +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
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
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
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
db9b8f08da Merge "Update work buffer size with input/output frame size change" into main 2024-01-29 18:51:47 +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
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
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
759fb98667 Merge "audio: Fix handling of a thread exit command with a bad cookie" into main 2024-01-13 00:40:04 +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
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
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
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