Merge "bug fix: output colorformat configuration fixed" into oc-dev am: 1e15657ca1
am: 0e42ac3399  -s ours

Change-Id: Iee420fcb4df06d89f56966779cc4dd4d885321e7

Merge "bug fix: output colorformat configuration fixed" into oc-dev
am: 1e15657ca1

Change-Id: I1bf3c443058ab57baba7c51e3f9f0ae662e61882

Merge "bug fix: output colorformat configuration fixed" into oc-dev

Merge "Drm test static links to HAL def libs." into oc-mr1-dev

Merge "Camera: Add new minor revision camera.device@3.3" into oc-mr1-dev

Merge "configstore: Allow syscalls needed by crash_dump" into oc-mr1-dev

Merge "contexthub: Pass target API version in new fields" into oc-mr1-dev

Merge "Modify StateTransitions test" into oc-mr1-dev

configstore: Allow syscalls needed by crash_dump

Information such as tombstones and callstack are important when
debugging why a process crashed. configstore's seccomp filter
and selinux policy are currently overly strict and are blocking
the collection of crash data.

In order to keep configstore's sandbox as strict as possible, use
Crash_dump's fallback mechanism to collect crash data. This uses a
local socket to send crash data to tombstoned.

Bug: 64768925
Test: killall -ABRT android.hardware.configstore@1.1-service
    Verify that configstore callstack gets dumped to logcat.
    Verify that a crash tombstone gets recorded to /data/tombstones
Test: proto_fuzzer runs without crashing

Change-Id: I3c3e13d6aa1c1e2dda2a619dfa815375ee7ebff6
(cherry picked from commit 60ced291129198d7c7a954c32811dec9bd71d83a)

Camera: Add new minor revision camera.device@3.3

This adds a single new member to the HalStream structure, used by
camera.device@3.2::ICameraDeviceSession.configureStreams, for
HALs to override the requested stream dataSpace.

This override may be used when the requested format is
IMPLEMENTATION_DEFINED.

Test: Builds
Bug: 62358514
Change-Id: Ie43e6cfd2ac52657c9366ff5297daa22c0bfa46c

contexthub: Pass target API version in new fields

Update default implementation to use the new fields defined explicitly
for passing in target CHRE API version information, rather than adding
this data into the reserved area.

Bug: 64852297
Test: run CHQTS on Nanohub, rebuild test app targeting 1.2 and confirm
      it's interpreted correctly
Change-Id: Id11f9b510a2d239d285436e119b7b8dbece4c5df

Drm test static links to HAL def libs.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalDrmV1_0Target
Change-Id: If23f1fda06e9188b84fc94ada0e22ca7e5b45aa9

Merge "Camera test statically links to dependencies." into oc-mr1-dev

Merge "bug fix: close file pointer" into oc-dev am: 8735fb45a5
am: 51f04884ba  -s ours

Change-Id: Iaa1a97e0771874b65adf0eab0a48de7eed78be6d

Merge "bug fix: configure input port buffer size" into oc-dev am: bc769a7e5a
am: 2eedb46745  -s ours

Change-Id: Ic49186056e830ac1edbe299fc35ced4105f2690c

Merge "bug fix: close file pointer" into oc-dev
am: 8735fb45a5

Change-Id: I7ca0f4992804da30eaac7ca24b2dbe9004a5ac55

Merge "bug fix: configure input port buffer size" into oc-dev
am: bc769a7e5a

Change-Id: I3a9b89f4c64d1129ea59a0144937832d93ed81fd

Merge "bug fix: close file pointer" into oc-dev

Merge "bug fix: configure input port buffer size" into oc-dev

biometrics: fingerprint: add locking to default impl

There exists the following race condition:
a). thread A receives setNotify and sets the callback to some object
b). thread B of the wrapped implementation calls
BiometricsFingerprint::notify which it was given a handle to. Thread B
executes past the nullptr check:
c). thread A receives setNotify and sets the callback to some other
object (or nullptr)
d). thread B resumes in notify with unknown state

Add mutex to protect access to mClientCallback.

Change-Id: I9163204ff5802e9246056caeb2a7857e6138531c
Fixes: 64802340
Test: VtsHalBiometricsFingerprintV2_1IfaceFuzzer

Merge "Increase thread pool for VHAL 2.1" into oc-mr1-dev

Increase thread pool for VHAL 2.1

Fixes: 36510399

Test: runtest -x
packages/services/Car/tests/vehiclehal_test/src/com/android/car/vehiclehal/test/E2ePerformanceTest.java

Change-Id: Id5638f63c6ff23bb78675fa07a2f3922bb7e3d73

bug fix: output colorformat configuration fixed

The eColorFormat field in OMX_PARAM_PORTDEFINITIONTYPE structure
has no effect on the component. This has to be configured from
OMX_VIDEO_PARAM_PORTFORMATTYPE. This is corrected. This color format
information is passed to graphic buffer during its allocation

Bug: 63796949

Merged-In: I51b1e59631647d1291ecfac373781ac5e5a0ffd5
Change-Id: I51b1e59631647d1291ecfac373781ac5e5a0ffd5

Allow clearkey tests to run if no vendor modules am: 5db2e67293
am: 66812dc364

Change-Id: I6369dd486c35c842c1d0f1f151b3b468ef94ab4c

Allow clearkey tests to run if no vendor modules
am: 5db2e67293

Change-Id: I594b8b5e9857d4a6c19d0847acf7c13949e13614

Merge "Load libRS_internal.so from the rs namespace" into oc-mr1-dev

Allow clearkey tests to run if no vendor modules

DRM cannot skip all tests when vendor modules are
not present. Currently even the clearkey tests are
skipping when there is no vendor lib. This can be
replicated by just pushing the DRM VTS test binary
build from master (or DR or MR) and executing the
binary without VTS. All test cases skip (including
clearkey).

This change prevents exiting when no vendor
modules are found.

Test: Push the test binary and confirm that the
clearkey tests run.

bug:64459011
Change-Id: Ic5f554d9b223d6ca4bf26f92f072254cc18f4620

bug fix: close file pointer

Bug: 63796949

Merged-In: Ifd8a27d851cbc659ea8108e0879747cd9169680a
Change-Id: Ifd8a27d851cbc659ea8108e0879747cd9169680a

bug fix: configure input port buffer size

In video decoders, size of a single input frame (elementary-stream)
is dependent on width, height, color format, profile settings, ...
Most of this information is part of sps, pps and requires parsing.
But as the max size of the stream is known ahead, use it to configure
the input buffer size requirements

Bug: 63875287
Bug: 63796949

Merged-In: Ib760c4f55b094260a0abd120f852dcf1899df4e0
Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0

Merge "Remove the data type from the OperationType enum." into oc-mr1-dev

Merge "Add test for additional ClearKey UUID." into oc-mr1-dev

Merge "Audio test statically links to dependencies." into oc-mr1-dev

Load libRS_internal.so from the rs namespace

Bug: 64747884
Test: VtsHalRenderscriptV1_0TargetTest successful on the device built
with BOARD_VNDK_VERSION=current and [system] namespace config is applied
to /data/nativetest[64]/* processes.

Change-Id: I9e967c80ac2dba718cd47e1f378bcbf18abe9ad2

Merge "bug fix: close file pointer" into oc-mr1-dev

Merge "bug fix: configure input port buffer size" into oc-mr1-dev

Merge "[AWARE] Fix maxNdpSessions documentation" into oc-mr1-dev

Merge "wifi(implementation): Reduce logging severity" into oc-mr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev
am: 92d3bc1e1a  -s ours

Change-Id: I1a648056655a978d867b50a53d929c646fc6d035

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev am: e2dae8c258
am: 5d18c87e88  -s ours

Change-Id: Ib03c1512099ec88e1c65672dadbe29ef3b9d9086

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-mr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
am: e2dae8c258

Change-Id: I396a157f3522843c3d36870719cb3e5176754dcc

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

Audio test statically links to dependencies.

This test now statically links to libs not guaranteed to be on the
device.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalAudioEffectV2_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalAudioV2_0Target

Change-Id: I331d7e4abdf7e520df6d508a8059b6358ad4398f

wifi(implementation): Reduce logging severity

IE elements should not be present in the batched scan results, but it
being there doesn't cause any problems. The IE elements will be ignored
when we send it over the HIDL interface, so reduce the log level to
prevent log spamming.

Bug: 64448186
Test: None
Change-Id: Ia0d0312a113c907bb0992bb8f41e734fe9d05114

Modify StateTransitions test

During loaded <-> idle state transition, check for events
is done after allocating all the buffers on a port. This is now
done after allocation of every single buffer. This is done to
improve scope of testing

Bug: 63796949

Change-Id: Icf3405ac6716f9533d31189f6c4a4c4ac908da5f

bug fix: close file pointer

Bug: 63796949

Change-Id: Ifd8a27d851cbc659ea8108e0879747cd9169680a

bug fix: configure input port buffer size

In video decoders, size of a single input frame (elementary-stream)
is dependent on width, height, color format, profile settings, ...
Most of this information is part of sps, pps and requires parsing.
But as the max size of the stream is known ahead, use it to configure
the input buffer size requirements

Bug: 63875287
Bug: 63796949

Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0

Merge "Tetheroffload tests static link to HAL def libs." into oc-mr1-dev

[AWARE] Fix maxNdpSessions documentation

Clarify meaning of maxNdpSessions capability. Total, not per interface.

Bug: 64685830
Test: builds
Change-Id: I24fc0a97172a4ca161f432b6c5c1dc55a44dfb12

Remove the data type from the OperationType enum.

  - Now the driver needs to report a list of tuple
    {OperationType, OperandType} for capabilities.
  - Any time Operation information is passed across HIDL,
    it should be passed as the tuple {OperationType, OperandType}

Bug: 63905942
Test: mm
Change-Id: I909b5acf4936f65c242ee0925d3a1ac665f46131

Merge "Widen dummy band range to cover all regions." into oc-mr1-dev

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev am: 52af675d14
am: fc73b0fcbd  -s ours

Change-Id: I4f9fcf22f98f9c40290d321fabe05a082524da19

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
am: 52af675d14

Change-Id: Ic5080cb6b22cd3984d0eb39d87cabf7c8a30178d

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev

Tetheroffload tests static link to HAL def libs.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadControlV1_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadConfigV1_0Target
Change-Id: Ib771c07646f3e219c7b9ba4be5515552d7fd6348

Update broadcastradio HAL 1.0 VTS tests.

This is a cherry-pick of the following VTS changes from O MR1 to O:
 - ag/2462390: Fix Broadcast radio 1.0 VTS to not fail on
   other radio types than AM/FM.
 - ag/2477726: Make tuned field description more precise and
   fix VTS tests to obey it.
 - ag/2549733: Implement out-of-band metadata images (a test for in-band).

Additionally, it fixes config callback handling in OpenTunerTwice test.

Bug: b/64324703
Test: VTS
Change-Id: I590b9b4d927fb0cd41f92e919343b5b94ad0e246
Merged-In: I590b9b4d927fb0cd41f92e919343b5b94ad0e246

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-mr1-dev

Merge "configstore: sandbox with seccomp filter" into oc-mr1-dev

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

Update broadcastradio HAL 1.0 VTS tests.

This is a cherry-pick of the following VTS changes from O MR1 to O:
 - ag/2462390: Fix Broadcast radio 1.0 VTS to not fail on
   other radio types than AM/FM.
 - ag/2477726: Make tuned field description more precise and
   fix VTS tests to obey it.
 - ag/2549733: Implement out-of-band metadata images (a test for in-band).

Additionally, it fixes config callback handling in OpenTunerTwice test.

Bug: b/64324703
Test: VTS
Change-Id: I590b9b4d927fb0cd41f92e919343b5b94ad0e246

Merge "Mark diagnostic values as either being numerically equivalent to OBD2 values, or providing the equivalent OBD2 value" into oc-mr1-dev

Merge "Keymaster test statically links to dependencies." into oc-mr1-dev

Widen dummy band range to cover all regions.

Bug: b/64115131
Test: VTS, instrumentation
Change-Id: I34e9aac440a003226200243c835e6116ba7eae90

Merge "Neural networks test static links to HAL def libs." into oc-mr1-dev

Keymaster test statically links to dependencies.

This test now statically links to libs not guaranteed to be on the
device.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalKeymasterV3_0Target
Change-Id: I6a7b8c116153f18f61a71e5b5bef98343a4de43b

Merge "VTS tests now static link to HAL def libs. #2" into oc-mr1-dev

VTS tests now static link to HAL def libs. #2

And use VtsHalTargetTestDefaults.

Bug: 64040096
Test: crop out all non-affected tests from vts-hal-hidl.xml and run
vts-tradefed run commandAndExit vts-hal-hidl --skip-all-system-status-check
--skip-preconditions
2 failures, same as without this change:
GatekeeperHidlTest.DeleteAllUsersTest
GatekeeperHidlTest.DeleteUserTest

Change-Id: I8f6995e9536a9aefe283ee3effec9f5a7f03b620

configstore: sandbox with seccomp filter

Configstore HAL is accessible to third party apps and thus requires
a tight sandbox that reflects the limited system access this HAL
needs.

We use two primary mechanisms to sandbox configstore, selinux and
seccomp, with the goal of restricting its access to userspace and
the kernel. The addition of a seccomp filter is primarily aimed
at reducing the kernel's attack surface that is reachable by
configstore HAL.

Seccomp filters are architecture dependent, so filters need to be
added for each architecture. This change adds a seccomp filter for
arm64 and issues a non-fatal runtime warning for other architectures
which still require a seccomp filter.

Bug: 36453956
Test: boot Marlin and Angler. Verify that configstore is not aborting
    due to seccomp violations.
Test: "cat proc/<configstore pid>/status | grep seccomp " returns:
    seccomp: 2
    Which indicates that configstore is using seccomp-bpf.

Change-Id: Iab014ff357b7329085a5e18a92f51838d2c72371
(cherry picked from commit ed95043d6433c5900984f24dbb101bba78109775)

Merge "Graphics tests statically link to HAL definition libs." into oc-mr1-dev

Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev
am: 269d9ba09a

Change-Id: I1260d97229e5330481178c8f802e2c7f6b651582

Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev

Merge "Renderscript test static link to HAL def libs." into oc-mr1-dev

Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
am: 5397fa8175

Change-Id: I1587c5451950a89c6aac991a0d8b7dacea16fe21

NFC: Add OWNERS file for default/
am: a4d1a74700

Change-Id: I43572572901298d1e915c9753798361bdd5d38c2

Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev

Merge "Media omx tests static link to dependencies." into oc-mr1-dev

NFC: Add OWNERS file for default/

Test: build
Change-Id: I6e854923fdd66991bcc7ed3446106d148abea982
(cherry picked from commit 31439b1d24d49fcb431dfa48bdaa0b1d57deded4)

Bluetooth: Add OWNERS file for default/

Test: build
Change-Id: Ib4cbd5502ac0791d4bc700f28de68d0548256de6
(cherry picked from commit 7c127dc4cbf90713bcdbce89fa166228c912847d)

Merge changes I51b1e596,I3aece23a into oc-mr1-dev

* changes:
  bug fix: output colorformat configuration fixed
  bug fix: android list management in dequeueMsg()

Bluetooth: Check the number of file descriptors

Opening the serial port(s) returns -1 when it fails, check
that the number of file descriptors is greater than 0 and
less than the size of the array.

Test: VtsHalBluetoothV1_0TargetTest, Bluetooth start/stop
Bug: 64420262
Change-Id: Icbffa5bd8c02030e389bac03aa5e1e78cc3c6057
(cherry picked from commit 89316af23eb7aa300015a3d80bc97891cfe6031e)

Merge "Skip direct report test if sensor is not available" into oc-dev am: 63524f9fcc
am: 7bea962c9c

Change-Id: I99d4eb05d1348e24910c5ebe2e87b6b334138b31

Merge "Skip direct report test if sensor is not available" into oc-dev
am: 63524f9fcc

Change-Id: Iffe683356ed925142ef877ecb5289463d8d4ac1e

Merge "Skip direct report test if sensor is not available" into oc-dev

wifi(vts): Remove a tight loop of toggling wifi state
am: 0d1bb97447

Change-Id: Icec63b9ffd5d12003da44bdde73edde04e55f11a

wifi(vts): Remove a tight loop of toggling wifi state

The tight loop of HAL start/stop exposes a race condition within the HAL.
Adding a fix for preventing this race would need a fix which would be
pretty risky at this point of the release. The issue itself is unlikely
to happen in real use cases because wifi toggles (user initiated or test
scripts initiated) via framework goes through a series of processing
before it invokes the HAL start/stop.

Bug: 64195190
Test: `make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l
INFO`

Change-Id: I4e4d65f8b6e2e423a3a5f26e5a97a78b7e99c3e8

Merge "Fix typo in in offload control VTS test" into oc-dr1-dev
am: 5d77efddbd

Change-Id: I2be944da8800040540bf98aeb197a933d1da0e42

Merge "Fix typo in in offload control VTS test" into oc-dr1-dev

Fix typo in in offload control VTS test

Test: built, flashed, run
Bug: 38220415
Bug: 64479437
Change-Id: I3aa4afa3b52297dd92e37b4f39a5bd25b129ba04

Add test for additional ClearKey UUID.

Test for both ClearKey plugin UUIDs in DrmHalClearkeyFactoryTest.

Test: VTS - VtsHalDrmV1_0TargetTest

bug: 64386897
Change-Id: Id7f140518910a87efdf9eaa19ce16113e206264f

Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev
am: 3a57cca59c

Change-Id: I6a1aeea07e64b4748151f62ae4827132eba32781

Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev

Merge "Add OWNERS for broadcast radio HAL 1.1." into oc-mr1-dev

Add the error code "OPERATION_NOT_ALLOWED" and log the error.

Test: Telephony sanity tests, run vts -m VtsHalRadioV1_1Target
Bug: 64426938
Change-Id: Id4f06842186b10a52640372c23bc2b368cc1e9ab

wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE
am: 8c216524a3

Change-Id: I1bf9be99d4e673209edb893040483bf772ee3ca0

wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE

When IWifi.stop() is invoked back to back (happens in the ConfigureChip
vts test), the HAL would return ERROR_NOT_AVAILABLE if the previous stop
is still being processed. This is not an error that needs to fail the test,
but a legitimate status for stop. We have a retry mechanism to handle
this in both the VTS test and framework for the case where IWifi.start()
is invoked while the previous stop is being processed.

While there, corrected a few log messages emitted by the HAL to debug
such startup/stop issues better.

Bug: 63971806
Test: `vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l
INFO`

Change-Id: I5e3470ac97541a6ea10aceec9b737e5d03ed5206

Merge "Update IRadioResponse.hal to add new error codes." into oc-mr1-dev

Merge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev
am: 6d5cf73b75

Change-Id: I004e81a764313fdaf23f9580b144e1ee2ba79b27