Commit graph

64400 commits

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

Change-Id: I335575c6d8f2fa227f3900d9877583b12f91a178
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-19 23:43:37 +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
7f01390b34 Merge changes Icd33f3cb,I737479d8 into main am: 0dccb6e265
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047703

Change-Id: Ieeaa40bc1909142cfdac6ac193704f93cc024da7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-19 02:08:02 +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
f453736f0d Merge "audio: Update 'PauseSync' scenario in AudioStreamIoOutTest" into main am: 5e6d99b424
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047078

Change-Id: I22b6ec53a90a95e56a62fe7a9e1c7d4ddfa75cde
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 23:39:28 +00:00
Mikhail Naganov
5e6d99b424 Merge "audio: Update 'PauseSync' scenario in AudioStreamIoOutTest" into main 2024-04-18 23:19:40 +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
Bruno Carneiro
20b6a001ab [automerger skipped] Improve OpenAfterOpen VTS test am: 8d3cb52ead -s ours am: 0af575b2f1 -s ours
am skip reason: Merged-In Idd78c0dae944e3d39a6828b43db5a608adbaf1d9 with SHA-1 056aa140ca is already in history

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047257

Change-Id: Id128f94342dac5863309fa19ff801092937ec80b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 18:58:39 +00:00
Bruno Carneiro
0af575b2f1 [automerger skipped] Improve OpenAfterOpen VTS test am: 8d3cb52ead -s ours
am skip reason: Merged-In Idd78c0dae944e3d39a6828b43db5a608adbaf1d9 with SHA-1 056aa140ca is already in history

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047257

Change-Id: I04959cdfd8c40c3c289ffb3e675db9678286cdec
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-18 18:25:48 +00:00
Bruno Carneiro
8d3cb52ead Improve OpenAfterOpen VTS test
A callback should be executed every time the NFC interface is opened,
even in OpenAfterOpen scenarios. If a callback is not executed, a
crash may occur where the NfcService hangs while waiting for an
OPEN_CPLT event.

Bug: 327582461
Merged-In: Idd78c0dae944e3d39a6828b43db5a608adbaf1d9
Change-Id: Idd78c0dae944e3d39a6828b43db5a608adbaf1d9
Signed-off-by: Bruno Carneiro <bccunha@motorola.com>
(cherry picked from commit 056aa140ca)
2024-04-18 04:21:08 +00:00
Jiawei Li
77098341d1 [automerger skipped] Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US am: 59aeb3182b -s ours am: 5c93471127 -s ours
am skip reason: Merged-In Idad4127ce26188145c7b07d60d5e6ce40a80aa5b with SHA-1 73e55bdbcf is already in history

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3044892

Change-Id: Ib91aff7142b1aba8d916cc71991ff351c70f9585
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 23:33:45 +00:00
Mikhail Naganov
7887f9706a Merge "audio: Fix IStreamIn.getActiveMicrophones test" into main am: 403040b952
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047772

Change-Id: Id97ed2680ea273ee455020f796dfdfcf2ba80b74
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 23:33:25 +00:00
Jiawei Li
5c93471127 [automerger skipped] Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US am: 59aeb3182b -s ours
am skip reason: Merged-In Idad4127ce26188145c7b07d60d5e6ce40a80aa5b with SHA-1 73e55bdbcf is already in history

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3044892

Change-Id: I6b5eba0db4184caf5ffe4822d3dc6b9e4c6de974
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 23:27:00 +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
Hsin-Yi Chen
e835d02eb8 Merge "Move libimapper_stablec ABI dumps from prebuilts/abi-dumps" into main am: cdf6fc5424
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3043453

Change-Id: I011091853e14c292e734e3dbd1aa601c18859b4f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 07:47:09 +00:00
Hsin-Yi Chen
cdf6fc5424 Merge "Move libimapper_stablec ABI dumps from prebuilts/abi-dumps" into main 2024-04-17 07:31:27 +00:00
Treehugger Robot
d62c8d7d92 Merge "audio: skip data path for offload Virtualizer" into main am: 7291b19db1
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3042592

Change-Id: Id58e0288698d3b03d39b05f5ac671b67ce559982
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-17 04:10:45 +00:00
Treehugger Robot
7291b19db1 Merge "audio: skip data path for offload Virtualizer" into main 2024-04-17 03:52:17 +00:00
Mikhail Naganov
d6cd4c1993 audio: Align bluetooth start behavior am: 66351e5a9f
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3044732

Change-Id: Ia5267369d96824a1daf21b1d050b1764d6fa32c9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 17:34:16 +00:00
Jiawei Li
59aeb3182b Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US
Bug: 323694622
Bug: 334053791
Test: VTS SupplicantStaIfaceAidlTest
(cherry picked from https://android-review.googlesource.com/q/commit:73e55bdbcf03a990fafd06abc585e1d2521b9c18)
Merged-In: Idad4127ce26188145c7b07d60d5e6ce40a80aa5b
Change-Id: Idad4127ce26188145c7b07d60d5e6ce40a80aa5b
2024-04-16 16:59:12 +00: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
Hsin-Yi Chen
219876109a Move libimapper_stablec ABI dumps from prebuilts/abi-dumps
The update process for prebuilts/abi-dumps/platform will be adapted for
trunk stable. The ABI dumps for libimapper_stablec are moved to
"abi-dumps" in this project so that the trunk stable release process
will not affect them.

Test: m libimapper_stablec_abicheck
Test: development/vndk/tools/header-checker/utils/create_reference_dumps.py \
      -libs libimapper_stablec_abicheck --lib-variant PLATFORM \
      -ref-dump-dir hardware/interfaces/graphics/mapper/stable-c/abi-dumps \
      -products aosp_arm64
Bug: 333532038
Change-Id: Ia69daa721ef983c18118bd6d0a70dd5b7a0a47a2
2024-04-16 20:21:56 +08:00
Treehugger Robot
bcc352f43b Merge "Annotate Health HAL test for VSR." into main am: 613782fa78
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3020094

Change-Id: I69df1ed0f4451fea87efdb5b63d58ae1dff7be32
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-15 22:49:44 +00:00
Treehugger Robot
0cf9620136 Merge "Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US" into main am: 84c6d5e8c6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3039132

Change-Id: If5478897cb7b52624272201402f22b142a84d36a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-15 22:49:30 +00:00
Treehugger Robot
613782fa78 Merge "Annotate Health HAL test for VSR." into main 2024-04-15 22:48:18 +00:00
Treehugger Robot
84c6d5e8c6 Merge "Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US" into main 2024-04-15 22:26:58 +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
Cody Schuffelen
e6273eff1d Merge "C++ AIDL on-device GateKeeper / SharedSecret impl" into main am: e23e450487
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3023986

Change-Id: I6532238eb92f6000366df38c87b170e623d72a44
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-14 22:26:59 +00:00
Cody Schuffelen
e23e450487 Merge "C++ AIDL on-device GateKeeper / SharedSecret impl" into main 2024-04-14 22:08:37 +00:00
Jiawei Li
73e55bdbcf Update VTS SupplicantStaIfaceAidlTest#SetCountryCode to US
Bug: 323694622
Bug: 334053791
Test: VTS SupplicantStaIfaceAidlTest
Change-Id: Idad4127ce26188145c7b07d60d5e6ce40a80aa5b
2024-04-13 01:07:31 +00:00
Bruno Carneiro
bdf398486d Improve OpenAfterOpen VTS test am: 056aa140ca
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3040299

Change-Id: I1ba8e032c62f3e5b21844f776ea0fc918d2f2529
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-12 21:40:08 +00:00
Bruno Carneiro
056aa140ca Improve OpenAfterOpen VTS test
A callback should be executed every time the NFC interface is opened,
even in OpenAfterOpen scenarios. If a callback is not executed, a
crash may occur where the NfcService hangs while waiting for an
OPEN_CPLT event.

Change-Id: Idd78c0dae944e3d39a6828b43db5a608adbaf1d9
Signed-off-by: Bruno Carneiro <bccunha@motorola.com>
2024-04-12 14:43:22 -03:00
A. Cody Schuffelen
097e625335 C++ AIDL on-device GateKeeper / SharedSecret impl
Because Keystore2 always requires valid auth tokens for user keys, this
copies and extends the existing in-guest GateKeeper implementation to
negotiate a shared secret key with Keymint in order to generate a
session key for signing auth tokens.

This implementation also uses AIDL rather than HIDL to interact with the
framework. It's also packaged in an APEX.

The files are cobbled together from a few sources:

- SoftGateKeeper.h is based on hardware/interfaces/gatekeeper/1.0/software/SoftGateKeeper.h
- GateKeeper.{cpp,h} are based on device/google/cuttlefish/guest/hals/gatekeeper/remote/remote_gatekeeper.{cpp,h}
- SharedSecret.{cpp,h} are based on device/google/cuttlefish/guest/hals/keymint/remote/remote_shared_secret.{cpp,h}
- Apex files are based on device/google/cuttlefish/guest/hals/keymint/rust/

Keymint modifications to use BOOT_TIME are lifted from

https://android-review.git.corp.google.com/c/platform/hardware/interfaces/+/2856649/6..8/security/keymint/aidl/default/ta/clock.rs#38

Bug: 332376454
Change-Id: I81845d5e6370bdddb1a24d67437964e03a5fb243
Test: Run with rust-nonsecure keymint on Cuttlefish
2024-04-11 13:53:32 -07:00
Treehugger Robot
bdb87cf0ce Merge "PresetReverb: Add test to validate PresetReverb Effect" into main am: 0f141d1967
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3029145

Change-Id: Ib0fcf6e703c83f1c09e887dfd411a2f45a78d0ee
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 17:37:48 +00:00
Treehugger Robot
094dfcfacf Merge "Bluetooth vts hal: Restore to 2s startup" into main am: a6c9097dd0
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3037779

Change-Id: Icf2a60fd56912893557dd559aa243713cb799a2a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 17:35:31 +00:00
Treehugger Robot
0f141d1967 Merge "PresetReverb: Add test to validate PresetReverb Effect" into main 2024-04-11 17:25:34 +00:00
Treehugger Robot
a6c9097dd0 Merge "Bluetooth vts hal: Restore to 2s startup" into main 2024-04-11 17:21:34 +00:00
William Escande
ac0f34f6cd Bluetooth vts hal: Restore to 2s startup
1s should really be the max limit, but it requires time to investigate
the failures and optimize this

Bug: 327273567
Fix: 332815453
Test: atest VtsHalBluetoothTargetTest
Change-Id: I43767b5241d967cb643401711fd7b2e015e99455
2024-04-11 09:06:31 -07:00
Treehugger Robot
35c50c21e2 Merge "Allow vendor ISharedSecret HALs by any name" into main am: 776e39b725
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3038254

Change-Id: I9bd960e6fa72912bc32cb2bde26aff589dd017b7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 09:54:14 +00:00
Treehugger Robot
776e39b725 Merge "Allow vendor ISharedSecret HALs by any name" into main 2024-04-11 09:36:52 +00:00
Treehugger Robot
fad8a7b6d0 Merge "[Thread] make sure buffer size is enough" into main am: 82bb6fcfdc
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3039092

Change-Id: Ie1372d5e4f7152426fdb9a6619617ac2d67e5895
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 08:33:14 +00:00
Treehugger Robot
82bb6fcfdc Merge "[Thread] make sure buffer size is enough" into main 2024-04-11 08:18:51 +00:00
Mikhail Naganov
30cf023827 audio: Use allow list for device connection types in TryConnectMissingDevice am: 7b9b9e03e5
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3038652

Change-Id: I304b1d7bb2d42151aa24d9c16387e4f2bfc14711
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 07:36:49 +00:00
shihchienc
cb8a9d74a0 [Thread] make sure buffer size is enough
inotify event size is dynamic, we need to make sure read would not
return error when having inotify event.

Bug: 333829392
Test: manual check `Invalid argument` error won't happen
Change-Id: I3cac8d1519659c6b1b53c1d41013818adab876c7
2024-04-11 06:20:38 +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