Commit graph

47510 commits

Author SHA1 Message Date
Treehugger Robot
7f2619ddff Merge "audio: Fix remote submix module device ports handling" into main 2023-08-19 03:53:26 +00:00
Andrew Scull
2332583ad1 Reference definition of the Android Profile for DICE
Remove the inline definition of the Android Profile for DICE and instead
reference the definition that exists alongside the Open Profile for DICE
and is now the source of truth for the profile.

Test: n/a
Change-Id: Ia71a674234be13542ad0ce4db0b764e8ee0c7a62
2023-08-18 12:39:39 +00:00
Igor Tolkov
3b3f7f06fa Move StopServicesSetup later in VtsHalBluetoothV1.0TargetTest am: 8c32f711b5 am: 12320fd071 am: cd47557c62
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2709435

Change-Id: I478ec9b89d874d516b384c211f1ee9acaf022288
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-18 08:26:35 +00:00
Igor Tolkov
cd47557c62 Move StopServicesSetup later in VtsHalBluetoothV1.0TargetTest am: 8c32f711b5 am: 12320fd071
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2709435

Change-Id: Id19f7f18a9a374e92a5790240adf260aa1f92b4a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-18 07:57:21 +00:00
Igor Tolkov
12320fd071 Move StopServicesSetup later in VtsHalBluetoothV1.0TargetTest am: 8c32f711b5
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2709435

Change-Id: Ib6d8d85f2aafbaed23264a55d66c7e0aa3d36bf0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-18 07:26:57 +00:00
Tri Vo
f6d445c32f [automerger skipped] Merge changes I8b34a94c,I46e62ab9 into android13-tests-dev am: c8b9250b82 -s ours
am skip reason: Merged-In I8b34a94c11d7cc174821483f624fd083ca1763ac with SHA-1 bf75a4070d is already in history

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

Change-Id: Ie184dbf272230673b3b54b6aa98ef906b770acd9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-17 20:55:45 +00:00
Tri Vo
a5995c24f7 [automerger skipped] KeyMint: Add Root-of-Trust test cases am: e0fb4d0ce5 -s ours
am skip reason: Merged-In I8b34a94c11d7cc174821483f624fd083ca1763ac with SHA-1 bf75a4070d is already in history

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

Change-Id: I395f91173c417e431c8ba6cace93cff753f59ba4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-17 20:55:27 +00:00
Tri Vo
c8b9250b82 Merge changes I8b34a94c,I46e62ab9 into android13-tests-dev
* changes:
  KeyMint: Add Root-of-Trust test cases
  KM 4: Add Root-of-Trust test cases
2023-08-17 20:20:16 +00:00
Tomasz Wasilczyk
fed7d54dfc Don't use String8::empty
With the current definition, it's an alias of default constructor. This
has very different meaning from std::string empty, especially that
static method String8::empty can still be called from instance context.

One of such bugs are present in in ParametersUtil.

Bug: 295394788
Test: make checkbuild
Change-Id: I9116a0c42a6a9e94116eaaaa0fd60568db428838
2023-08-17 13:53:55 +00:00
Henri Chataing
77971b371b uwb: Rewrite implementation of sendUciMessage to use the opened File
The current implementation used fs::write() as workaround since
the write permission was missing from the fs::open() invocation

Test: m android.hardware.uwb-service
Change-Id: Ifdb460843fcb9525ad498f928b1dc2ed9beb5410
2023-08-16 17:10:20 +00:00
David Drysdale
ae8c281824 Merge "KeyMint: allow extra error code" into main 2023-08-16 10:20:21 +00:00
Igor Tolkov
8c32f711b5 Move StopServicesSetup later in VtsHalBluetoothV1.0TargetTest
Move StopServicesSetup after RunCommandTargetPreparer in
VtsHalBluetoothV1_0TargetTest. RunCommandTargetPreparer requires
settings service, which is not available after StopServicesSetup.

Bug: 293822620
Test: run vts -m VtsHalBluetoothV1_0TargetTest and inspect logs to make
sure no preparer steps fail.

Change-Id: I935d19c5e941bfdb122554d8c413d3095fef27ac
2023-08-16 15:24:25 +08:00
Treehugger Robot
a2441d9090 Merge "Add VSR annotation to KeyMint tests" into main 2023-08-15 14:17:24 +00:00
Treehugger Robot
1452142a46 Merge "Whenever generateKey fails updated AttestKeyTests to abort instead of continuing the execution of the test." into main 2023-08-15 11:30:45 +00:00
William Escande
046165292d Sync wait for the bluetooth to be disabled / enabled am: 29120123a8 am: 311d918dde am: f4924ed7a6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2682567

Change-Id: Ia55b92b16098b38a599e396276661cf4961197db
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-14 05:14:51 +00:00
William Escande
f4924ed7a6 Sync wait for the bluetooth to be disabled / enabled am: 29120123a8 am: 311d918dde
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2682567

Change-Id: I8615128eef8eb118f5b5ff055fadcad51e89118f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-14 04:53:10 +00:00
William Escande
311d918dde Sync wait for the bluetooth to be disabled / enabled am: 29120123a8
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2682567

Change-Id: If48483f0c01b96f0c6bb26515886406bbe42c641
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-14 04:11:47 +00:00
David Drysdale
84b685adf5 KeyMint: allow extra error code
Bug: 295055603
Test: VtsAidlKeyMintTargetTest
Change-Id: Ifbd4a899364c38bb6ad63bb5b5a683c69edfb5b7
2023-08-11 16:00:32 +01:00
Jooyung Han
49e6433617 Mark keymint HAL and its friends as updatable-via-apex
Cuttlefish implementation for these HALs are moved into an APEX along
with VINTF fragments. Hence we need to mark them so in FCM.

Bug: 197988234
Test: aosp_cf_x64_64_phone boots
  keymint service starts from apex
Change-Id: I669ef4e215abc203f51bb21b7251aff601531ac7
2023-08-11 16:34:14 +09:00
David Li
b089c0cfee audio: Fix remote submix module device ports handling
- remove the default address "0" for IN_SUBMIX and OUT_SUBMIX;
 - remove the profiles in the device port and assign profiles when
   connecting;
 - make remote submix input to use "virtual" connection type,
   same as the output;
 - fix ModuleConfig in VTS to avoid returning devices with virtual
   connections as "external devices" because they can actually
   be connected even when connection simulation is disabled;
 - fix TryConnectMissingDevice VTS test to disconnect the device
   if the operation has unexpectedly succeeded.

Bug: 286914845
Bug: 294976817
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Ife11c9c356d1b5dc587d08cef47294e3b29f65c5
2023-08-10 14:37:31 -07:00
Jaewan Kim
5b454f372e Disable verbose nlinterceptor log
Test: No verbose log on Cloud Android with Wi-Fi is turned on
Bug: 256518068
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:7df5c1c34538d1e051dc3df63fcba563e40713c8)
Merged-In: I062e698b64114f139997cd24b16f90dd52048247
Change-Id: I062e698b64114f139997cd24b16f90dd52048247
2023-08-10 18:03:56 +00:00
Henri Chataing
477100915e Merge "bluetooth/aidl: Enable HCI reset in default hal only when running on emulator devices" into main 2023-08-10 00:57:18 +00:00
Henri Chataing
1267ff2c46 bluetooth/aidl: Enable HCI reset in default hal only when running on emulator devices
Bug: 290329516
Test: m
Change-Id: I624f26824372fb708598e6be322adf06e40cb5a2
2023-08-09 22:57:26 +00:00
Yifan Hong
76d87305d5 Merge changes from topic "fcm_exclude_aidl_version" into main
* changes:
  fcm_exclude: mark bluetooth.audio@1 as deprecated.
  matrices: Add gnss.* to fcm_exclude.
  fcm_exclude: Aidl HALs contains versions.
2023-08-09 20:35:12 +00:00
Arthur Ishiguro
802c53da89 Add CPU_WRITE_RARELY flag
Without this flag, the test may fail with SEGV_ACCERR.

Bug: 293475360
Test: Run test
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:8c5976f06a9c9b0cb92735c3eeba7f7485f4e7ca)
Merged-In: I2d7cfba94145239afde196567400c966a2769419
Change-Id: I2d7cfba94145239afde196567400c966a2769419
2023-08-09 18:17:23 +00:00
Treehugger Robot
d99d7730b8 Merge "Update the TimeoutAuthenticationMultiSid test" into main 2023-08-09 14:25:05 +00:00
Eric Biggers
15d95e8d65 Merge "Test that the password isn't truncated (again)" into main 2023-08-08 23:21:36 +00:00
Henry Fang
15ec04889d Merge "CAS: Add SYS_NICE Capability" into main 2023-08-08 22:44:19 +00:00
Subrahmanya Manikanta Venkateswarlu Bhamidipati Kameswara Sri
07c7d28a84 Update the TimeoutAuthenticationMultiSid test
Update TimeoutAuthenticationMultiSid test to support
generateKey for Strongbox implementations without
factory attestation.

Bug: 293211157
Test: run vts -m VtsAidlKeyMintTarget
Change-Id: I27bf08d2fd2d9e0217a90ee8ccb789adfd9d5f7f
2023-08-08 22:33:37 +00:00
Yifan Hong
19d015a45e fcm_exclude: mark bluetooth.audio@1 as deprecated.
It never shows up in any matrices. Only bluetooth.audio@2
and beyond are accepted.

Test: TH
Bug: 255383566
Change-Id: I110b73b2ed610d86aa351216fe46d9f803f8c3cf
2023-08-08 13:32:19 -07:00
Yifan Hong
a3975a563f matrices: Add gnss.* to fcm_exclude.
They aren't top level interfaces.

Test: TH
Bug: 255383566
Change-Id: I8265f531880d1dcff1bf7edfd035a25f9e2a4d70
2023-08-08 13:32:12 -07:00
Yifan Hong
68ed69dd7c fcm_exclude: Aidl HALs contains versions.
ShouldCheckMissingAidlHalsInFcm now accepts arguments of
the format android.hardware.foo@1.

Test: TH
Bug: 255383566
Change-Id: I891f749190a8db8725ec1e05b190e73ab4468254
2023-08-08 13:32:11 -07:00
Shreshta Manu
595041e6fb [uwb-interfaces] Add support for prioritized channel list
Test: compiles
Bug: 293959095
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:cef396ca26136d812405e78009e7a1374ca89e5c)
Merged-In: I18da378128b96b73ae457045354d22a4add988d0
Change-Id: I18da378128b96b73ae457045354d22a4add988d0
2023-08-08 20:08:45 +00:00
David Drysdale
955fb9e5fc Merge "KeyMint: use a smaller invalid IMEI value" into main 2023-08-08 10:16:23 +00:00
Venkatarama Avadhani
90e664dfa6 CAS: Add SYS_NICE Capability
Add SYS_NICE capability to the CAS services. This will enable the
service to set its priority.

Bug: 286011513
Test: Build
Change-Id: I5cd58a3a9db53fcc004557a8e9f509ebd6291c2d
2023-08-08 15:35:08 +05:30
Treehugger Robot
ff1f1f3494 Merge "audio: Implement the major functionality of the primary CF HAL" into main 2023-08-08 07:56:05 +00:00
Treehugger Robot
41243dc213 Merge "audio: Ensure proper priority and scheduler for service threads" into main 2023-08-08 02:19:23 +00:00
Mikhail Naganov
4bf6899cc7 audio: Ensure proper priority and scheduler for service threads
Change audio worker threads priority to ..._URGENT_AUDIO to match
the coupled thread in the audio flinger.

Set SCHED_FIFO scheduler for FAST threads (also to match AF fast
mixer/capture threads). In order to enable that, grand SYS_NICE
capability to the HAL service process and provide "getTid"
function in the StreamWorker class. For testing, add a "FAST"
output to the "stub" module in the HAL configuration.

Bug: 286914845
Test: atest libaudioaidlcommon_test
Test: atest VtsHalAudioCoreTargetTest
Test: adb shell ps -A -T -o PID,TID,NI,PCY,PRI,RTPRIO,SCHED,CMD
      for the HAL service process during VTS test. For regular
      "reader"/"writer" threads should see 'NI 19, PRI 38',
      for FAST "writer" should see 'NI -19, PRI 43, RTPRIO 3, SCH 1'
Change-Id: Iab7e21ebc139ff11cf9b7f4a1645960db8dadd43
2023-08-07 18:15:19 -07:00
Treehugger Robot
203ca985ce Merge "Update version for audiocontrol" into main 2023-08-07 22:59:39 +00:00
Mikhail Naganov
cf824f65c8 audio: Implement the major functionality of the primary CF HAL
Core HAL changes:

1. Add StreamPrimary implemented via StreamAlsa.

2. Align the configuration with the HIDL HAL.

3. Fix position retrieval vs. standby call.

4. Fix sleeps in StreamAlsa.

VTS changes:

1. Use several bursts for stream I/O test scenarios that check
   observable position increase. This is because the position may
   not be available until a couple of transfers have been made.

2. Do not require position increase for the scenarios that do
   not make several bursts. As specified above, the position may
   not have been increased for the ALSA case. Whereas, using
   multiple bursts in all scenarios will increase test time, and
   make the state machine transitions graph more complicated.

3. Hook up the test config file to shut down audioserver during
   VTS tests, fix the test config file.

Bug: 286914845
Test: atest VtsHalAudioCoreTargetTest
Test: compare APM dumps for AIDL vs. HIDL
Change-Id: I85271564c664fa40008d60e82b32eaa66a99c68f
2023-08-07 13:38:50 -07:00
David Drysdale
0215cb3d3e KeyMint: use a smaller invalid IMEI value
The invalid value used for the second IMEI attestation test is
potentially wrong in two ways:
- It doesn't match the provisioned value.
- It's not a valid IMEI, not least because it is longer than 16 bytes.

Make the test value shorter so the second failure doesn't apply and
the test can reliably expect CANNOT_ATTEST_IDS.

Bug: 292959871
Test: VtsAidlKeyMintTargetTest
Change-Id: If8c6b9e08b48e6caf5c767578e1ac43964214619
2023-08-07 11:53:46 +01:00
kuanyuhuang
25b4819e08 BT: add LOW_LATENCY mode if low latency allowed
LatencyMode is not updatable after audio session started.
Currently, if low latency is allowed, audio session should have
LOW_LATENCY latency mode.
Create a workaround to add LOW_LATENCY when low_latency_allowed_ set to
ture at ReportLowLatencyModeAllowedChanged.

Bug: 291760415
Bug: 294498919
Test: play low latency audio with supported headset and check
LOW_LATENCY mode can be selected

Change-Id: Ia324f0a368e47ae8b8056c190d8d1f4a2e09ecec
2023-08-07 05:05:52 +00:00
Eric Biggers
8bf0780fd9 Test that the password isn't truncated (again)
Test that Gatekeeper doesn't truncate passwords, either due to them
containing NUL bytes or being long.

This is https://r.android.com/2151558 ported to the AIDL test.  Even
though the AIDL test wasn't added until after my change, it was forked
from an earlier version of the HIDL test that didn't have my change.

Bug: 238919794
Test: atest VtsHalGatekeeperTargetTest # on Cuttlefish
Change-Id: I6fec951e67a35d5275a67244fbef07d1435c9f4f
2023-08-05 02:47:59 +00:00
Terry Guan
b324444c8a Update version for audiocontrol
Bug: 255369346
Bug: 255383566
Test: atest vts_treble_vintf_vendor_test
Change-Id: I17000be75963e511d2a56583be0f619a3dee66f1
Merged-In: I17000be75963e511d2a56583be0f619a3dee66f1
Merged-In: Ic42e51b616782a2763eb80e1cad5ca776cc079d4
2023-08-04 16:50:58 -07:00
Yifan Hong
966a4f13a5 matrices: automotive.audiocontrol in 7.xml has max version
Updates version from 1 to 1-2 to reflect the fact
that version 2 is released in Android 13.

Test: TH
Bug: 255383566
Change-Id: I290aefb51fcc54d5e5a02f8c7da5876e86f61cd9
2023-08-04 16:47:29 -07:00
Yifan Hong
23847d4411 Merge "fcm_exclude: split HIDL and AIDL predicate." into main 2023-08-04 23:16:13 +00:00
Treehugger Robot
e8d695d9b2 Merge "audio: Make renamings suggested by reviewers" into main 2023-08-04 15:46:13 +00:00
Treehugger Robot
5a15a1c838 Merge "Remove authsecret HIDL entry in compat matrix." into main 2023-08-03 22:12:23 +00:00
Mikhail Naganov
459b7331e9 audio: Make renamings suggested by reviewers
Rename DriverInterface::getPosition -> refinePosition. This reflects
the fact that the driver may provide more precise position, however
this is an optional operation.

Rename StreamIn/Out::mContext -> mContextInstance. This is
to emphasize that StreamIn/Out actually own the context and they
hold the real context instance, not just a reference/pointer
to it, like other classes involved in the inheritance chain.

Bug: 286914845
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Id7159f9bf3ee020dc7cf594ec35f5d59d23d7e31
2023-08-03 10:31:12 -07:00
Mikhail Naganov
09d45c473e Merge "audio: Address comments from an internal review" into main 2023-08-03 15:58:56 +00:00