Audio VTS: Workaround async HIDL destructor by sleeping
am: d87a288e82

Change-Id: I6af4b725b9917d99c450287ba4bfce864371a104

Merge "health.filesystem::IFileSystem -> health.storage::IStorage"

Merge "Add command to pass oem commands to HAL"

health.filesystem::IFileSystem -> health.storage::IStorage

Rename the HAL because it is regarding 'storage health',
not 'filesystem health'.

Bug: 111655771
Test: compiles
Change-Id: I07072c876fb07c88be8bd74cf6b84aec6020a697
Merged-In: I07072c876fb07c88be8bd74cf6b84aec6020a697

Audio VTS: Workaround async HIDL destructor by sleeping

The test creates an instance of the Audio HAL then destroys it and create it again.
The test assumed that the destruction was synchronous when in fact it is async with
no way of knowing when the object has been destroyed.

As a result, until a better solution is found, sleep for 100ms to hopefully let enouth
time for the HAL destructor to return.

Bug: 112566489
Test: adb shell /data/nativetest64/VtsHalAudioV4_0TargetTest/VtsHalAudioV4_0TargetTest --gtest_filter=*OpenPrimaryDeviceUsingGetDevice
      atest VtsHalAudioV4_0TargetTest
Change-Id: I0ec75c12007d39060232632708722df5bf0f99d7
Signed-off-by: Kevin Rocard <krocard@google.com>

Merge "Audio VTS: Always check for Result::isOk()" into pie-vts-dev
am: 2070f8b0b8

Change-Id: Icf4a294d1fef7626b8248500dffc6bcd0189571e

Merge "Audio VTS: Always check for Result::isOk()" into pie-vts-dev

Add command to pass oem commands to HAL

Bug: 79480454
Bug: 78793464
Test: fastboot oem command

Change-Id: Ifa20badace91550812742702f3b16f5abe56be72

Merge "ISafeUnion: add case w/ enum"

ISafeUnion: add case w/ enum

Bug: 115638948
Test: hidl_test
Test: hidl_test_java
Change-Id: I26b5cc25bd29c420160686b5572a005a7629784c

Merge "Fix failing drm 1.0 vts tests"

Merge "health filesystem: update garbageCollect() function."

Fix failing drm 1.0 vts tests

The shared library path was incorrectly set based on
the drm.64bit.enabled flag. It should be set based
on whether the drm service is running as 32-bit or
64-bit.

Test: vts-tradefed run commandAndExit vts -m VtsHalDrmV1_0Target

bug:111289939
Change-Id: I388dc87bd4566211dc2901feccf1e6e1c90bfe67
Merged-In: I388dc87bd4566211dc2901feccf1e6e1c90bfe67

health filesystem: update garbageCollect() function.

Bug: 111655771
Test: run dev gc on vold

Change-Id: I2cef404f9e2720c1be7822895187a590837763c8

Fix logics for floating-point comparision in VTS test.
am: 39c865b3aa

Change-Id: Iccb497fc3aa87e05a361427e350a3f51dd6f273c

keymaster 3.0: make service use nobody as user and remove system group

Only the drmrpc group is necessary

Test: VtsHalKeymasterV3_0TargetTest
Change-Id: I2be255215df827c9f17ecaffcb9d0ba402dd3405

Merge "Note confusion-causing breaks in HIDL convention."

Note confusion-causing breaks in HIDL convention.

These were either made at a time when the convention was not fully
formed or missed during review.

It is somewhat misleading since method overloading isn't supported
and method names need this kind of prefix, but nothing else does.

The reason for this is that everything is namespaced, but methods are
all in the same namespace. The reason method overloading is not supported
in HIDL is because the HIDL types may map to types that collide in the
target languages, and this would cause any sort of overloading rules
to require complicated machinery.

Bug: N/A
Test: hidl-gen -Lcheck android.hardware.usb@1.1 android.hardware.vibrator@1.1 && echo ":)"
:)

Change-Id: Iac23c9311925ed140ff1e15d1366829b078c8866

Merge "Get partition file system type from Fastboot HAL"

Merge "[Graphics] Only validate display in PresentOrValidateDisplay."

Skip NoUserConfirmation VTS test for Strongbox

Since Confirmation UI is optional for Strongbox
implementation, skipping the test.

Bug: 112189538
Test: This is an update to the vts test
Change-Id: Ie3485a1de92444b0c49670b198de30ea25e0673e
Signed-off-by: Max Bires <jbires@google.com>

Get partition file system type from Fastboot HAL

Partition type is required for fastbootd to
format partitions.

Test: make
Bug: 79480454
Change-Id: I49f21fe6f1626cf3da566b33cc84131b3e3c85ff

Fix logics for floating-point comparision in VTS test.

Set the acceptable error range based on both absolute tolerance and
relative tolerance.

Currently, absolute tolerance is set to 1e-5 for FP32 and 5 epsilon
(~5e-3) for FP16 relaxed computation. The relative tolerance is set to
5ULP of the corresponding precision. Add a TODO mark for potential
future adjustment on error limit based on testing.

Bug: 111768023

Test: none
Change-Id: Idedcec3e09fd7de9696811b93c81d0f180e896ef

Fix logics for floating-point comparision in VTS test.

Set the acceptable error range based on both absolute tolerance and
relative tolerance.

Currently, absolute tolerance is set to 1e-5 for FP32 and 5 epsilon
(~5e-3) for FP16 relaxed computation. The relative tolerance is set to
5ULP of the corresponding precision. Add a TODO mark for potential
future adjustment on error limit based on testing.

Bug: 111768023

Test: none
Change-Id: Idedcec3e09fd7de9696811b93c81d0f180e896ef

Audio VTS: Always check for Result::isOk()

A test was not checking for the Result::isOk() resulting
in an assert being triggered in HIDL code if the HAL crashed.

Test: (sleep 0.3; adb shell pkill audio) &
      adb shell /data/nativetest64/VtsHalAudioV4_0TargetTest/VtsHalAudioV4_0TargetTest
      # Test finishes in failure but does not crash
Signed-off-by: Kevin Rocard <krocard@google.com>
Change-Id: I97a6213fe82a5e9a6eb333dc1dc2476102610b58

Merge "Add IRadioResponse/Indication 1.3 and Emergency Function HAL"

Merge "Correct the comments for radio hal types"

Add IRadioResponse/Indication 1.3 and Emergency Function HAL

- Add IRadioResponse/Indication in 1.3
- Add Emergency Functions:
1) getCurrentEmergencyNumberList/getCurrentEmergencyNumberListResponse
2) emergencyDial/emergencyDialResponse
3) currentEmergencyNumberList
4) EmergencyNumber/EmergencyServiceCategory

Test: compile (will add VTS later)
Bug: 112657134
Change-Id: Iafe12d5fd0e4c28fcf6e4d8e61cfb954db7fa68a

Correct the comments for radio hal types

The type of mcc and mnc is String instead of Int now. They should be an
empty string if unknown. Also added a test case for their values.

Bug: 111703979
Test: Vts
Change-Id: Ie0426453dc426ccc6cf203b315806e78511ce14d

Merge "Fix KM VTS tests for Strongbox implementations"

Merge "Camera: support torch control on external camera"

Merge "Camera: Allow 3% ISO tolerance during ISO burst VTS" into pie-vts-dev
am: 391eb588cf

Change-Id: Id897eee2251eee92267b4e32f7d245aca3eb1e88

Merge "Allow getCapturePosition to return 0 frames for unprepared streams" into pie-vts-dev
am: 137194ef5e

Change-Id: Ibef9acb447f71a94c27ad62ffd93b87126637143

Merge "Camera: Allow 3% ISO tolerance during ISO burst VTS" into pie-vts-dev

Merge "Allow getCapturePosition to return 0 frames for unprepared streams" into pie-vts-dev

Merge "Set the floor value of time estimate as 2017 Jan 01 00:00:00 GMT" into pie-vts-dev
am: 4e0c7c1760

Change-Id: Ia8b0d2e61772da26e399d5a6d029159eb70b88d7

Merge "Set the floor value of time estimate as 2017 Jan 01 00:00:00 GMT" into pie-vts-dev

Merge "[ConfigStore] Clarify hasHDRDisplay spec."

[Graphics] Only validate display in PresentOrValidateDisplay.

Previously we require display to be validated in onRefresh, however, onRefresh
can be called while validateDisplay is executing, results in next
presentDisplay being skipped. This patch makes sure we don't check validation
state when presentDisplay is called.

BUG: 80063800
Test: build, flash, boot and play Youtube videos.
Change-Id: I3d8686db3274436afb6605812641768296f1af0e
Merged-In: I3d8686db3274436afb6605812641768296f1af0e

[automerger skipped] Merge "Fix setAllowedCarrier" into pie-vts-dev
am: 052f207b42  -s ours

Change-Id: Ic6f3763761f13361d21d14594042011e797099f1

Merge "Fix setAllowedCarrier" into pie-vts-dev

Camera: Allow 3% ISO tolerance during ISO burst VTS

The advertised ISO ranges may not be entirely supported on
some devices. Expect a 3% tolerance when comparing the applied
results versus the expected ISO value.

Bug: 80025874
Test: run vts --skip-all-system-status-check --skip-preconditions
--primary-abi-only --module VtsHalCameraProviderV2_4Target -l INFO

Merged-In: Ibd2758e48a0924e81877e35dd27433669274fdb3
Change-Id: Ibd2758e48a0924e81877e35dd27433669274fdb3
(cherry picked from commit 0fdd7cdc0eab249e0f0615fad1034231c09987dd)

Set the floor value of time estimate as 2017 Jan 01 00:00:00 GMT

- The VTS test checks if the time estimate is a sane value. Thus, any
arbitrary sane value is acceptable. Here we make it consistent to
IGnssDebug.hal, where a comment mentions 2017 Jan 1 as an example of
the value.

Bug: 110094003
Test: make vts, and tested on device.
Merged-In: Ic4f6b597a718ea3e98a67c7939f9e6930d44d224
Change-Id: Ic4f6b597a718ea3e98a67c7939f9e6930d44d224
(cherry picked from commit 5fa43c83e90ccd0f852c89feb0e071aad195d7b2)
(cherry picked from commit 4f0d19ebff1f4a3eb565d8432f44b561ebd4abe2)

Allow getCapturePosition to return 0 frames for unprepared streams

For device supporting getCapturePosition,
VTS only allowed unprepared stream to return INVALID_STATE.
Now also allow for the stream to return 0 frames similarly to the other
non started states.

Test: vts-tradefed run commandAndExit vts --module VtsHalAudioV2_0Target
Bug: 110367728
Cherry-piked from: d01dc3edac9e92981e7ae419d0e7622f753acf94
Merged-In: Ibdf8df8cb8809e98c40a50035371df6893fe4da4
Change-Id: Ibdf8df8cb8809e98c40a50035371df6893fe4da4
Signed-off-by: Kevin Rocard <krocard@google.com>
(cherry picked from commit 6c9c6e62c193461ed0de4666d0995631ba39e009)

[automerger skipped] Merge "Change ImportWrappedKeyTest back to SHA1" into pie-vts-dev
am: e7300721a0  -s ours

Change-Id: Ie8af40cfea7a0893f9f90d4c42a7cf8513e38818

[automerger skipped] Merge "Require keymaster4 attestations to contain the right version." into pie-vts-dev
am: 516718071b  -s ours

Change-Id: I15537d95bc28ff14aa3942d932378e99d5793f42

[automerger skipped] Merge "keymaster: skip SHA2 digest tests for strongbox" into pie-vts-dev
am: 86104dcc18  -s ours

Change-Id: Ib420f757159aaf8bbf462f98d88be3808d84c7d1

[automerger skipped] Merge changes Ic41afbd0,Ibe264d08 into pie-vts-dev
am: e101aa87a3  -s ours

Change-Id: Ifff697b20cb2e0c9f89594fe965dde3545447175

Merge "Fixing Keymaster documentation." into pie-vts-dev
am: 33f7970672

Change-Id: I0bd196af03f96817cbd23620b109e7665456f586

Merge "Change ImportWrappedKeyTest back to SHA1" into pie-vts-dev

Merge "Require keymaster4 attestations to contain the right version." into pie-vts-dev

Merge "keymaster: skip SHA2 digest tests for strongbox" into pie-vts-dev

Merge changes Ic41afbd0,Ibe264d08 into pie-vts-dev

* changes:
  keymaster: spec does not require that update produce output
  Fix attestation test.

Merge "Fixing Keymaster documentation." into pie-vts-dev

[automerger skipped] Allow general errors for getImsiForApp().
am: 1f3d0a68a4  -s ours

Change-Id: I2b7df276429ce16a63e00ed5f3e2654bbb9c63ed

Fix setAllowedCarrier

If function is not supported or executed successfully, do not
expect its effect on cardStatus.

Bug: 112008372
Test: run vts
Change-Id: I4532a39af2cfcf0e44eafe29c3c7f6779ae101f5
Merged-In: I4532a39af2cfcf0e44eafe29c3c7f6779ae101f5
(cherry picked from commit 44b129f7288016a4113f5e6ff059d63987d63385)

Allow general errors for getImsiForApp().

This is to match it with other 1.0 tests where general errors are
allowed. For newer tests we have decided to not allow these errors,
but a failure for this old test is reported when run with SIM
present.

Test: run vts -m VtsHalRadioV1_0Target
Bug: 109889468
Change-Id: If36083b7832706a50805932e8ba08e4eb397f3fe
Merged-In: If36083b7832706a50805932e8ba08e4eb397f3fe
(cherry picked from commit 817848e59e18334a30f623ef143cf182716fd3fa)

[automerger skipped] Merge "Loose startNetworkScan cases" into pie-vts-dev
am: 3c233d344c  -s ours

Change-Id: Ie83195654dc60810651b9cd883d1a5f1f360f5fa

[automerger skipped] Merge "Fix missed refactor of serial" into pie-vts-dev
am: d4f9ad2c59  -s ours

Change-Id: I95f73d92932ffa352b6299c7bb8d5fe4ec71e023

[automerger skipped] Merge "Allow NOT_SUPPORTED for setLinkCapacityReportingCriteria on GERAN." into pie-vts-dev
am: 429da8656e  -s ours

Change-Id: Ie29dbb55092bcfecc0f01a4ba3405ea8dd8db518

Merge "Loose startNetworkScan cases" into pie-vts-dev

[automerger skipped] Merge "Allow REQUEST_NOT_SUPPORTED as an error for startNetworkScan" into pie-vts-dev
am: 00cdcb77f8  -s ours

Change-Id: Ie8c2baaacc0369048448d608ab002c1369b9b68f

Merge "Fix missed refactor of serial" into pie-vts-dev

Merge "Allow NOT_SUPPORTED for setLinkCapacityReportingCriteria on GERAN." into pie-vts-dev

Merge "Allow REQUEST_NOT_SUPPORTED as an error for startNetworkScan" into pie-vts-dev

[automerger skipped] Secure_element: vts: Modify openBasicChannel as per OMAPI standard
am: ecdec18c73  -s ours

Change-Id: I84c5b1003063990e4d70941344a0b920b28a2867

Loose startNetworkScan cases

startNetworkScan:
Some vendor may not support the required manual GSM search
functionality.

startNetworkScan_GoodRequest1 and startNetworkScan_GoodRequest2:
Some vendor may not support max search time of 360s.

Test: sanity
Bug: 109765420
Merged-In: I456847815057d76561bfb3e840016619ac753476
Change-Id: I456847815057d76561bfb3e840016619ac753476
(cherry picked from commit 12f7d6127b301b140ee81696fed19f81a7ea50c6)

Fix missed refactor of serial

Bug: 109822806
Test: run vts
Merged-In: If88384699dbb4c2ddd1c580d2b35be38afd0f8e8
Change-Id: If88384699dbb4c2ddd1c580d2b35be38afd0f8e8
(cherry picked from commit c0e2e2b071df5cdff10193556163e2975f0b682c)

Allow NOT_SUPPORTED for setLinkCapacityReportingCriteria on GERAN.

Test: run vts -m VtsHalRadioV1_2Target
Bug: 110716988
Merged-In: I9a5f014d498db00d818870cbd73e119f66562ef1
Change-Id: I9a5f014d498db00d818870cbd73e119f66562ef1
(cherry picked from commit c7896501db413c9a0fa44d2697da4079610075d8)

Allow REQUEST_NOT_SUPPORTED as an error for startNetworkScan

We are trying to tighten the APIs. However for this case since the
documentation was not updated, we are allowing NOT_SUPPORTED for
now and will be cleaned up in a later release.

Test: run vts -m VtsHalRadioV1_2Target
Bug: 110118713
Merged-In: Id9dd3d7bac99bed36ceb9c906189f1fea78d5a2c
Change-Id: Id9dd3d7bac99bed36ceb9c906189f1fea78d5a2c
(cherry picked from commit a7587b5a7fdace0d193ad333e6f5cda8b995570b)

Secure_element: vts: Modify openBasicChannel as per OMAPI standard

CHANNEL_NOT_AVAILABLE should be returned when opening basic channel
is not permitted on the secure element

Test: Test with SIM1 HAL
Bug: 110945797
Merged-In: I2dd4eafaf4c534b49a37690a9baadcc5ac93190c
Change-Id: I2dd4eafaf4c534b49a37690a9baadcc5ac93190c
(cherry picked from commit ec5aa6af4d8b330af6c8599830942e29294aa688)

Change ImportWrappedKeyTest back to SHA1

Change I5f877b2a1ac66026a876e145416ba078d486e4b5 inadvertently changed
the digest used for ImportWrappedKey, breaking the test.  This CL
reverts that portion of the change.

Test: VtsHalKeymasterV4_0TargetTest
Bug: 112279922
Merged-In: Ib8e2e7793ba46ae0d29d8407bb730a35bdb5ea98
Change-Id: Ib8e2e7793ba46ae0d29d8407bb730a35bdb5ea98
(cherry picked from commit 0dba88861267b2b37355b41504b9d1e0d89bbe0f)

Require keymaster4 attestations to contain the right version.

Note that devices with KM4 will fail to pass VTS after this
lands, until the fix from Qualcomm arrives.

Test: VtsHalKeymasterV4_0TargetTest
Bug: 112040197
Merged-In: Ie2cd917af704b9f19de3537297b3a7e4f0c861e9
Change-Id: Ie2cd917af704b9f19de3537297b3a7e4f0c861e9
(cherry picked from commit 4e006c2b92be6ac71927b975b6725ddadf0b3342)

keymaster: skip SHA2 digest tests for strongbox

Strongbox is not required to support SHA-2 digests,
so skip the related tests.

Bug: 109771020
Merged-In: I5f877b2a1ac66026a876e145416ba078d486e4b5
Change-Id: I5f877b2a1ac66026a876e145416ba078d486e4b5
(cherry picked from commit 8cec80be1fc3d2d30fe095fa009dfa83cb7651ce)

keymaster: spec does not require that update produce output

Remove out of spec enforcement on the amount of data returned
by update, as this is not specified in the HAL.

Bug: 109771020
Test: yes it is
Merged-In: Ic41afbd01d51faf48d3c0fe090409ebcd257cc1e
Change-Id: Ic41afbd01d51faf48d3c0fe090409ebcd257cc1e
(cherry picked from commit 7b75f015a7a7ebc40722fd4f36c9ca70e7e58e59)

Fix attestation test.

Bug: 77588764
Test: VtsHalKeymasterV4_0TargetTest
Merged-In: Ibe264d08ae7b3333a6949761a92759f5305b3fcb
Change-Id: Ibe264d08ae7b3333a6949761a92759f5305b3fcb
(cherry picked from commit d898d0a4223df75225cfdf23e6f184166933cdb3)

[automerger skipped] Merge changes Ia3b96e83,I80d924cc into pie-vts-dev
am: 0b1461687f  -s ours

Change-Id: I8a2912ea0ea9fc396f3d6513db7c466f5cdec0c5

Fixing Keymaster documentation.

Keymaster HAL documentation documents the bootPatchLevel as having
tag 718, while types.hal indicates the tag value for it is actually
719.

Test: N/A
Bug: 78104779
Merged-In: I0dde0b3c863081f2594e20466d8e82866a5f2d2e
Change-Id: I0dde0b3c863081f2594e20466d8e82866a5f2d2e
(cherry picked from commit ae8da1b70ad563edb79f4f7429b704b54c1a9e19)

Merge changes Ia3b96e83,I80d924cc into pie-vts-dev

* changes:
  Add REQUEST_NOT_SUPPORTED for pin and pin2 Icc VTS
  Add SIM_PUK2 for Icc VTS tests

Improve VTS GNSS 1.1 reliability.
am: 2d86da7b28

Change-Id: I401190490a49bdf529d5fd697216cdf5b051729d

Delete time/position in InjectDelete instead of Delete_all
am: 66fb7a1b88

Change-Id: I6651f1f2e181ad2022ae2cdc6b9f6c1d4a4d0d06

Inject last location in InjectBestLocation VTS test
am: 75b0c4bba2

Change-Id: I934dd9ac9a49d1813f10f6d2f31739c3a6a6babf

Improve VTS GNSS 1.1 reliability.

Allow VTS test for low power mode to handle
certain vendor issues that supply one extra location
at the start of low power mode.

Allow more time for GNSS first fix, and warmup as
needed given limited VTS access to AGPS.

Bug: 110626730
Bug: 110987651
Test: (TODO) Passes on device.
Merged-In: Ieeefd7fcd45890b03dffbbee965e1d9b17805c4c
Change-Id: Ieeefd7fcd45890b03dffbbee965e1d9b17805c4c
(cherry picked from commit 5ea5dda2c33f9d2f1db77d6b4493f0eb356c2937)

Delete time/position in InjectDelete instead of Delete_all

- DELETE_ALL will delete XTRA.

Bug: 110626730
Test: run VTS test on device.
Merged-In: I4f01430f24f92238193d3de6b6fbb31e81e9f006
Change-Id: I4f01430f24f92238193d3de6b6fbb31e81e9f006
(cherry picked from commit 84cff99b2fcae8757704b0384552aebe290165ed)

Inject last location in InjectBestLocation VTS test

Bug: 110626730
Test: run VTS on device.
Merged-In: I6f57dd7f9164dacf011f25412e55fcd22821aca0
Change-Id: I6f57dd7f9164dacf011f25412e55fcd22821aca0
(cherry picked from commit bb372e5984a612cb6972e837662d64a90785fbd8)

Merge "Tolerate 1 sv status in GNSS blacklist VTS test" into pie-vts-dev
am: a9ad41d1ad

Change-Id: I0bce1b24e7472bebd96614bc502a863bca020e8f

Merge "VTS: Do not require XML for Audio effect V2" into pie-vts-dev
am: 5e6dd1288e

Change-Id: If389674923030e041dfdb95cd99cd0316603c917

Merge "Camera: Don't hold 'mLock' when closing the camera session" into pie-vts-dev
am: 75ceb36b27

Change-Id: I3b3d8c2051dd89b0993bd17af77a6a19cd7792b6

Merge "Tolerate 1 sv status in GNSS blacklist VTS test" into pie-vts-dev

Merge "VTS: Do not require XML for Audio effect V2" into pie-vts-dev

Merge "Camera: Don't hold 'mLock' when closing the camera session" into pie-vts-dev

Merge "Audio VTS was testing wrong uninitialized variable due to typo" into pie-vts-dev
am: b3e86d489a

Change-Id: I13b546ec0e8785040e4096a7658474f2eff6c90e

Add VTS test to verify scheduling capability
am: 2910e64d24

Change-Id: Ife2f3b299e9c3762d5aad18007b28e62da4fe20f

[automerger skipped] Add setIndicationFilter response notification in 1.2 VTS
am: 388883a3ec  -s ours

Change-Id: Iddbbd11134cc9c05320cec945d513126e6662641

Merge "Audio VTS was testing wrong uninitialized variable due to typo" into pie-vts-dev

Camera: support torch control on external camera

Claim torch API support though there is no device
actually has a flash unit. This is to workaround
an issue in camera service where it falls back
to legacy camera HAL1 control if the device does
not report torch API support (and external camera
HAL does not implement HAL1)

Test: CtsVerifier FlashLight test
Bug: 113336515
Change-Id: I945b35fecfed3b2698b1e9bb1d8faad8699258d2

Tolerate 1 sv status in GNSS blacklist VTS test

Bug: 110969497
Test: run VTS test on device.
Merged-In: I26345bb62513da8530cb48672e5f59a915817cb4
Change-Id: I26345bb62513da8530cb48672e5f59a915817cb4
(cherry picked from commit 91cc8f4832882e8fa9baa04d717184636a57314d)

VTS: Do not require XML for Audio effect V2

In O, the Audio effect V2 HAL allows not to have an audio effect XML
configuration but to use a .conf format.

In P, before this patch, the Audio effect V2 HAL _requires_ to have an
audio effect XML configuration and forbids the use of a .conf format.

This is discouraged by Treble as it means that an unchanged HAL V2
implementation that was compatible with O VTS will not be with P VTS.

As a result, revert to the Oreo behavior for Audio Effect V2 VTS.

Note that the audio effect V4 VTS is not changed by this patch and
still requires an audio effect XML configuration.
Aka device _updating_ to P will not have to upgrade to XML but new
devices lunching on P are still required to have an audio effect
XML configuration.

This commit only changes VTS code.

Test: cd hardware/interfaces/audio/effect/
      mmm 2.0/vts/functional 4.0/vts/functional && adb sync data
      adb push *.0/xml/audio_effects_conf_V*.xsd /data/local/tmp/
      adb shell
      cd /data/nativetest64
      OPS=--gtest_filter=CheckConfig.audioEffectsConfigurationValidation
      VtsHalAudioEffectV2_0TargetTest/VtsHalAudioEffectV2_0TargetTest $OPS
      VtsHalAudioEffectV4_0TargetTest/VtsHalAudioEffectV4_0TargetTest $OPS
      rm /{odm,vendor,system}/etc/audio_effects.xml || true
      VtsHalAudioEffectV2_0TargetTest/VtsHalAudioEffectV2_0TargetTest $OPS
      ! VtsHalAudioEffectV4_0TargetTest/VtsHalAudioEffectV4_0TargetTest $OPS
Bug: 111421676
Change-Id: Ifb15d65ae4aa65759c8ebaa91cbc8c02234030dc
Merged-In: Ifb15d65ae4aa65759c8ebaa91cbc8c02234030dc
Signed-off-by: Kevin Rocard <krocard@google.com>
(cherry picked from commit b41b575475f3c93fe30216b9cdb2db441b92854b)

Camera: Don't hold 'mLock' when closing the camera session

After flushing the camera requests don't try to close the
camera session while still holding 'mLock'. Different Hals
could still try and return additional results which then will
get blocked on 'mLock'. Closing the session could potentially
introduce a deadlock if the Hal implementation waits for any
of the result calls to return.

Bug: 111836165
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I0f309f435cc76db3f9b6ba500da65a232b014717
Merged-In: I0f309f435cc76db3f9b6ba500da65a232b014717
(cherry picked from commit 8e920568371136b011c79e6c1d15bcb4c047d0a3)

Fix KM VTS tests for Strongbox implementations

Modify RSA keysize used in various tests
to ensure both TEE and Strongbox implementations
can be validated.
Skip invalid keysizes that Strongbox does not
support.

Test: Patches the strongbox tests
Bug: 112189538
Change-Id: I46ab01ce9b8224403e2a334a894967761d6799c9
Signed-off-by: Max Bires <jbires@google.com>