Commit graph

2931 commits

Author SHA1 Message Date
Treehugger Robot
1fdefd8d92 Merge "audio: Use more bursts in audio I/O VTS tests" into main am: 53eb12245b am: 35fdd00c34
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047956

Change-Id: I73ae68a26c70ecde5ca4a8e7088f664ecdc37e98
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-20 00:02:39 +00:00
Treehugger Robot
53eb12245b Merge "audio: Use more bursts in audio I/O VTS tests" into main 2024-04-19 23:25:33 +00:00
Mikhail Naganov
a2a9fa5003 audio: Use more bursts in audio I/O VTS tests
Some audio outputs use A/V sync and requre mode bursts
in order to start reporting the presentation position.

Bug: 300735639
Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icad0942f2ba1dcd6f030a7dc4f37e22fdbd6dd71
2024-04-19 21:56:00 +00:00
Treehugger Robot
b5ff39fb6a Merge changes Icd33f3cb,I737479d8 into main am: 0dccb6e265 am: 7f01390b34
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047703

Change-Id: I7192de767108802468a267a5b4e25f2fdc656bf3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-19 02:27:44 +00:00
Treehugger Robot
0dccb6e265 Merge changes Icd33f3cb,I737479d8 into main
* changes:
  audio: Fix AudioPatchTest/AudioModulePatch#UpdateInvalidPatchId VTS test
  audio: Skip stream I/O test for "echo reference" input device
2024-04-19 01:44:52 +00:00
Mikhail Naganov
ab64305031 Merge "audio: Update 'PauseSync' scenario in AudioStreamIoOutTest" into main am: 5e6d99b424 am: f453736f0d
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047078

Change-Id: I0ae1ee7ce46ab75e5ec46471ef2500bf3cd99133
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 23:57:42 +00:00
Mikhail Naganov
22e17d43bd audio: Update 'PauseSync' scenario in AudioStreamIoOutTest
According to the definition of the 'PAUSED' state
in StreamDescriptor.aidl, s/w (the client) stops
writing once the buffer is filled up. That means, it is
allowed for an output stream not to consume data
from the MQ while in the paused state, so allow that
in the test.

Also, update the state transition sequence in the test
to flush any data after making a burst while in the 'PAUSED'
state.

Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icb5fd02ca4ede63d7ae33613ab66cb96f3e6df29
2024-04-18 14:36:50 -07:00
Mikhail Naganov
8dd96d4c41 audio: Fix AudioPatchTest/AudioModulePatch#UpdateInvalidPatchId VTS test
The test was using '0' as an "invalid" patch ID value, however
this value is valid in the context of 'IModule.setAudioPatch'
method and means "create a new patch and allocate and ID for it".

Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icd33f3cbd1602ec5aa162fa72fc3ddd59ccffbef
2024-04-18 13:23:05 -07:00
Mikhail Naganov
a62c5df181 audio: Skip stream I/O test for "echo reference" input device
This is aligned with the HIDL implementation VTS. The echo reference
device can't provide any input until certain preconditions are met,
and modeling these preconditions in the test is not trivial.

Also, add the information into the mix port into the trace scope
for easier identification on test failure.

Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I737479d8ef1961791ac3bd82aeb779453d2e49f4
2024-04-18 13:22:32 -07:00
Mikhail Naganov
d5db4fd48e Merge "audio: Fix IStreamIn.getActiveMicrophones test" into main am: 403040b952 am: 7887f9706a
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047772

Change-Id: Ieaa3884d6adfa44a5439a852c56ec26fdb4ada50
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 23:48:15 +00:00
Mikhail Naganov
403040b952 Merge "audio: Fix IStreamIn.getActiveMicrophones test" into main 2024-04-17 23:15:45 +00:00
Mikhail Naganov
eee5499ba8 audio: Fix IStreamIn.getActiveMicrophones test
Since "active" means "used by the stream for acquiring data,"
it was unreasonable to expect the list of active microphones
to be non-empty prior to actually starting data acquisition.
This change adds running of 'burst' commands before calling
'getActiveMicrophones'.

To reuse existing code some refactorings have been made.
Added 'AudioInputFlags::HOTWORD_TAP' to the list of port
config flags for which I/O testing is not performed.

Bug: 328010709
Bug: 328362233
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I876c0b6d7365e104ec9ed8cf5033a83f822006b6
2024-04-17 14:47:16 -07:00
David Li
66fcc18bd9 Merge "add kDumpFromAudioServerArgument" into 24D1-dev am: c2b68f2bc1
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26983895

Change-Id: Id772372010b0e88bfad53d96bf90819d999e03b5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 18:22:27 +00:00
David Li
c2b68f2bc1 Merge "add kDumpFromAudioServerArgument" into 24D1-dev 2024-04-17 18:07:58 +00:00
Treehugger Robot
77b1c1082d Merge "audio: skip data path for offload Virtualizer" into main am: 7291b19db1 am: d62c8d7d92
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3042592

Change-Id: I7bd1f464765f2086fd87e3369cae80161cfd245b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 04:24:29 +00:00
Treehugger Robot
7291b19db1 Merge "audio: skip data path for offload Virtualizer" into main 2024-04-17 03:52:17 +00:00
David Li
adfb7d8c73 add kDumpFromAudioServerArgument
Bug: 275135031
Test: mm
Change-Id: I147fbe5eecae8cf0f318b1d77bb9a5faffb934be
2024-04-17 01:27:02 +08:00
Mikhail Naganov
66351e5a9f audio: Align bluetooth start behavior
When staring data transfer, if resuming of BT session fails,
the legacy implementation does not indicate an error, thus
causing an automatic retry on next transfer. The AIDL
implementation was switching the stream into an error state
instead. This could cause audio mute when the BT stack is
slow on resuming. This CL aligns AIDL with legacy behavior.

Bug: 333307789
Test: repro steps in the bug
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:3f4c7fca29db4b82d597bb56478f40a75cef618c)
Merged-In: I5080a7275671287f188930f041c5ff2215ed2e1e
Change-Id: I5080a7275671287f188930f041c5ff2215ed2e1e
2024-04-16 16:27:10 +00:00
Eric Laurent
711437527c Merge "audio: Align bluetooth start behavior" into 24D1-dev am: ef4519eeaa
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26946076

Change-Id: Ie3c889130f9328f4e4fc4d8f137d5e2c01d61320
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 14:18:52 +00:00
Eric Laurent
ef4519eeaa Merge "audio: Align bluetooth start behavior" into 24D1-dev 2024-04-16 13:47:01 +00:00
Shunkai Yao
8c47facb19 DynamicsProcessing VTS: remove stageInUse testing am: f137ba4d92
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26924921

Change-Id: I7372c7686b7480ad799165c234ce5dad9598dc26
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 08:08:52 +00:00
Jaideep Sharma
9f7c9102d3 audio: skip data path for offload Virtualizer
Skip the data path tests for offloading Virtualizer, as offloading
effects don't publish fmqs and don't actually process the data.

Bug: 334851716
Test: run vts -m VtsHalVirtualizerTargetTest
Change-Id: I0d96fe4b036bdbd6c7b65af906edaaaac39f7a10
2024-04-15 16:42:36 +00:00
Mikhail Naganov
3f4c7fca29 audio: Align bluetooth start behavior
When staring data transfer, if resuming of BT session fails,
the legacy implementation does not indicate an error, thus
causing an automatic retry on next transfer. The AIDL
implementation was switching the stream into an error state
instead. This could cause audio mute when the BT stack is
slow on resuming. This CL aligns AIDL with legacy behavior.

Bug: 333307789
Test: repro steps in the bug
Change-Id: I5080a7275671287f188930f041c5ff2215ed2e1e
2024-04-12 12:37:39 -07:00
Treehugger Robot
429baea512 Merge "PresetReverb: Add test to validate PresetReverb Effect" into main am: 0f141d1967 am: bdb87cf0ce
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3029145

Change-Id: I253596a95ee4a6dbac2a37299d73b797d82ce70f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 17:52:14 +00:00
Treehugger Robot
0f141d1967 Merge "PresetReverb: Add test to validate PresetReverb Effect" into main 2024-04-11 17:25:34 +00:00
Shunkai Yao
f137ba4d92 DynamicsProcessing VTS: remove stageInUse testing
As we relaxed the requirement of stageInUse to setParam

Bug: 328012516
Test: atest VtsHalDynamicsProcessingTargetTest
Change-Id: I62e7d07c54133a0e3aed1952f80010faba783a20
2024-04-11 17:16:34 +00:00
Mikhail Naganov
1676108800 audio: Use allow list for device connection types in TryConnectMissingDevice am: 7b9b9e03e5 am: 30cf023827
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3038652

Change-Id: I293c710b7bab3f5ab5d09022abe2f821210eba23
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 07:53:03 +00:00
Sneha Patil
94abce0998 PresetReverb: Add test to validate PresetReverb Effect
Added tests to verify increase in energy with decrease in room and hall size.
Remove preset reverb default implementation from the config file.

Bug: 305866207
Test: atest VtsHalPresetReverbTargetTest
Change-Id: I565612d90b7c7e52cb23bd5dbfaf02235f65cc67
2024-04-11 09:25:56 +05:30
Mikhail Naganov
7b9b9e03e5 audio: Use allow list for device connection types in TryConnectMissingDevice
Limit the connection types to test to the following:
 - HDMI*
 - IP_V4
 - USB

Only these connection types can be easily checked by the HAL
for presence of an external device.

Bug: 326888643
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I659e14a150b3043ead8d844cd89a2c4700d57efd
2024-04-10 23:50:37 +00:00
Kiyoung Kim
160c3d610c Merge "Remove VNDK definition(s)" into main 2024-04-09 02:13:37 +00:00
Kiyoung Kim
9e9d41d162 Remove VNDK definition(s)
As of VNDK deprecation, any libraries that defines vndk is no longer
valid anymore. This change removes all VNDK definition(s) from modules
which was VNDK.

Bug: 328994089
Test: AOSP CF build succeeded
Change-Id: Ifdade9c052db29c989ce7d804ba725361c4c0f8e
Merged-In: Ic4f19113c1a25dc1fb60cdfbf9c0f2d8d7d09726
2024-04-05 02:29:07 +00:00
Shunkai Yao
2475cd33e9 Merge "Effect AIDL VTS: relax dynamics processing effect parameter validations" into 24D1-dev am: ad6464d43b
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26767676

Change-Id: Ia9115becb833d3021f499708cf431e484d87bf0e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-04 18:44:54 +00:00
Shunkai Yao
ad6464d43b Merge "Effect AIDL VTS: relax dynamics processing effect parameter validations" into 24D1-dev 2024-04-04 18:18:58 +00:00
Shunkai Yao
1a84397f13 Effect VTS: skip VolumeDataTest for U am: 40d9d9ee7b
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26767393

Change-Id: I7dc684435f10e10a50ba7a4dd17a821b551bc80f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-02 20:46:28 +00:00
Shunkai Yao
76ea415c28 Merge changes I561d6864,I7e909cda into 24D1-dev
* changes:
  Use customized toString for UUID in VTS
  Effect VTS: skip VolumeDataTest for U
2024-04-02 20:30:03 +00:00
Shunkai Yao
653c996424 Merge "Use customized toString for UUID in VTS" into main am: e237f5bfaa am: ffac5b2200
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3020095

Change-Id: Ibbc6d195102bfb025e50241dd44ac36b08c53ba6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-02 17:34:30 +00:00
Shunkai Yao
6f8224538e Use customized toString for UUID in VTS
Bug: 331955595
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I561d68645f3f159b029135b0dfa66511a2800d72
Merged-In: I561d68645f3f159b029135b0dfa66511a2800d72
2024-04-02 17:03:38 +00:00
Shunkai Yao
e237f5bfaa Merge "Use customized toString for UUID in VTS" into main 2024-04-02 16:45:52 +00:00
Shunkai Yao
40d9d9ee7b Effect VTS: skip VolumeDataTest for U
Volume effect implementation was not ready on early U releases

Bug: 327497904
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I7e909cdabbd167bd2e012dbe378ccb3402d33f53
2024-03-30 00:13:23 +00:00
Shunkai Yao
53238b1afd Effect AIDL VTS: relax dynamics processing effect parameter validations
relaxing several parameter checking to align with HIDL
 - no need to have stage in use to set bands/channels
 - band settings no need to be sorted by frequency

Bug: 328012516
Test: atest VtsHalDynamicsProcessingTargetTest
Change-Id: If12d978ee69ee7f087a7e8758513a9c6bacf817f
2024-03-29 23:17:05 +00:00
Shunkai Yao
fb8f39452a Use customized toString for UUID in VTS
Bug: 331955595
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I561d68645f3f159b029135b0dfa66511a2800d72
2024-03-29 18:49:15 +00:00
Mikhail Naganov
efb21cb338 audio: Fix some VTS issues on real devices am: 69d60aa02c am: 677c969ad7
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3018422

Change-Id: Ia84d34b5233648d0b3540f083c9b5b506cd57241
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-29 02:51:52 +00:00
Mikhail Naganov
69d60aa02c audio: Fix some VTS issues on real devices
1. Skip testing of stream I/O on certain types of mix ports.

2. Skip testing of connection of BT SCO device.

Bug: 300735639
Bug: 326888356
Bug: 328010709
Bug: 331516432
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I9b8bbf2014e223375c8f8400ff2af32268803706
2024-03-28 17:02:27 -07:00
Mikhail Naganov
2677d603bc Merge "audio: Allow to inherit Binder RT priority in AIDL" into 24D1-dev am: 9a9a5d2a88
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26723206

Change-Id: Icdff933310168a01f1e2d6b56df3c7bec1fa1394
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-27 16:31:16 +00:00
Mikhail Naganov
efb45bc792 audio: Allow to inherit Binder RT priority in AIDL
This option is on by default for HIDL interfaces, however
in AIDL it needs to be enabled explicitly.

Bug: 331469541
Test: m
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:d975a9e6ec0b6f0eeac99673b9b06e11146874ba)
Merged-In: I971185a46da4222160fb04b8b77f805ef0396562
Change-Id: I971185a46da4222160fb04b8b77f805ef0396562
2024-03-27 16:20:33 +00:00
Mikhail Naganov
d975a9e6ec audio: Allow to inherit Binder RT priority in AIDL
This option is on by default for HIDL interfaces, however
in AIDL it needs to be enabled explicitly.

Bug: 331469541
Test: m
Change-Id: I971185a46da4222160fb04b8b77f805ef0396562
2024-03-26 17:22:47 -07:00
Mikhail Naganov
41e1e9e50c audio: Implement blocking in remote submix when there is no sink am: 6507376fb9
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26684914

Change-Id: I6a4a1b6c0eab6808db990e3bc341e9945812a1b9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-25 16:14:25 +00:00
Mikhail Naganov
a41ff5134d audio: Implement blocking in remote submix when there is no sink
The HIDL implementation blocked for the audio buffer duration
when the output side of the remote submix pipe does not have
the sink connected. This behavior was accidentally removed
when fixing b/327220024.

Also, limit the amount of the debug messages displayed when
the sink is shutdown to avoid spamming the syslog.

Bug: 328347445
Test: repro steps from the bug
Test: atest VtsHalAudioCoreTargetTest
Test: atest --test-filter=".*AudioPlaybackCaptureTest.*" CtsMediaAudioTestCases
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:6507376fb940198e8c39ad6d5345d132b1dd7219)
Merged-In: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
Change-Id: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
2024-03-25 16:04:27 +00:00
Mikhail Naganov
6507376fb9 audio: Implement blocking in remote submix when there is no sink
The HIDL implementation blocked for the audio buffer duration
when the output side of the remote submix pipe does not have
the sink connected. This behavior was accidentally removed
when fixing b/327220024.

Also, limit the amount of the debug messages displayed when
the sink is shutdown to avoid spamming the syslog.

Bug: 328347445
Test: repro steps from the bug
Test: atest VtsHalAudioCoreTargetTest
Test: atest --test-filter=".*AudioPlaybackCaptureTest.*" CtsMediaAudioTestCases
Change-Id: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
2024-03-22 13:05:48 -07:00
Mikhail Naganov
7f50205333 Merge "audio: Adjust stream buffer size calculation" into main am: 222d034461 am: a5855a1592
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3008508

Change-Id: I8484832349e3af69cec9ca2a56f315f225b83427
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-22 04:08:11 +00:00