Merge "Finalize O-MR1 compat matrix." into oc-mr1-dev

Merge changes I931dafbb,I845672b2 into oc-mr1-dev

* changes:
  Thermal 1.1 test statically links to hal def libs.
  Update thermal hal Android.bp.

Merge "Fix VtsHalGraphicsComposerV2_1TargetTest" into oc-mr1-dev

Merge "bug fix: restore support for broken flag" into oc-dev am: 40aad4e988
am: 062192a1be

Change-Id: I8349103fed48e5eafeffa9ea4dfc78cbe82e1d1b

Merge "bug fix: restore support for broken flag" into oc-dev
am: 40aad4e988

Change-Id: Ied23d04f5c38f81ad732424d3577edf4d3cc085d

Merge "bug fix: restore support for broken flag" into oc-dev

Merge "NNAPI Concurrent Query Management -- HAL and VTS update" into oc-mr1-dev

bug fix: restore support for broken flag

timestampDevTest flag can be used to disable timestamp deviation
tests. The flag is not working as intended. This is corrected.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64743270
Bug: 65166112

Merged-In: I611e96498a4677b6ccbff33c291d56eed7b48f6f
Change-Id: I611e96498a4677b6ccbff33c291d56eed7b48f6f

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

Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev am: febad058bd
am: a87079c3d4

Change-Id: Ied9396646a58ca9b3f123c7db25df418b040a478

Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev
am: febad058bd

Change-Id: I09128311ab40316b17d99daeae375ff7535d0e63

Merge "bug fix: disable timestamp deviation test for audio decoder" into oc-dev

NNAPI Concurrent Query Management -- HAL and VTS update

The NNAPI requires requests on a model to be asynchronously
processed. This CL implements a basic Event that can later
be used to block the runtime thread until the asynchronous
request has completed.

Bug: 63905942
Test: VtsHalNeuralnetworksV1_0TargetTest (32-bit, 64-bit) with sample driver enabled by cherry-pick
      frameworks/ml/nn/runtime/test with and without sample driver enabled
Change-Id: Ie27a574aaaac312e7cbb731750f9c06278357a1c

Modify SetDefaultPortParams test

1. For output port in video encoder and input port in video decoder,
it is allowed to configure the size of the buffer. Make adjustments
in the test application to validate this scenario.

2. useBuffer api call is allowed only after sending loaded to idle
state transition command. Ensure this requirement is met.

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 63796949

Change-Id: I1aba7703e1344b87f556d3388a4cde8199898499

bug fix: disable timestamp deviation test for audio decoder

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Test -l INFO

Bug: 64128634
Bug: 65166112

Change-Id: I8a08a8da7dd203cd16cbfbfd3f15e27966db711f

Cleanup the constants.

- Alphabetize the operations.
- Remove N_GRAM operation.
- Add TENSOR_INT32.

Test: Compiled and ran the tests.
Bug: 63905942
Change-Id: Ie7d9dec671a409256b686ef4665171492d1ac16a

Finalize O-MR1 compat matrix.

Note, only new HALs for various shipping API levels
are added. No required HALs are added.

Test: (sanity) boot device w/o 'internal problem' error
Bug: 63603463
Change-Id: Ia400bdaf5d167ce8637ecdf940c5ccd629c8a09a

Thermal 1.1 test statically links to hal def libs.

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

Update thermal hal Android.bp.

Bug: 32920003
Bug: 64040096
Test: compiles
Change-Id: I845672b2b07ddba9b73f702555231674b4f45ad1

Fix VtsHalGraphicsComposerV2_1TargetTest

Bug: 65252969
Test: make vts
Change-Id: Id4543b385ca9dce532bbf08ef206897924d66acb

Add OEM_OPERATION to NN HAL.

Bug: 63905942
Test: mm
Test: All unit tests pass on Sailfish
Change-Id: I8c8068b248d682fbf1db37de78666c9cbca376b4

Merge "Replace LOOKUP with EMBEDDING and HASHTABLE_LOOKUP" into oc-mr1-dev

Merge "Adding owners for gnss hardware interfaces" into oc-mr1-dev

Merge "cas: fix missing error code conversion" into oc-mr1-dev

Merge "Update vts test to use the updated test template." into oc-mr1-dev

Merge "Fix HALs to not have transitive depends." into oc-mr1-dev

Adding owners for gnss hardware interfaces

Bug # is example where owners file would have
appropriate expedited the fix.

Bug: 64482271
Test: Treehugger (if relevant) - OWNERS file only
Change-Id: If82a2794ae40d48542006674c3c57a1e21c5124b

cas: fix missing error code conversion

bug: 22804304
Change-Id: Ia950e12f7feef3eeb16596c0939aaeb61bfdf5df

Merge "Setting clock flags for GPS clock" into oc-mr1-dev

Update vts test to use the updated test template.

* For registerTestService/getServiceName, no need to pass
  any hard coded string of HAL service FQName.
* Affect test: VtsHalCameraProviderV2_4TargetTest,
               VtsHalGraphicsComposerV2_1TargetTest,
               VtsHalNeuralnetworksV1_0TargetTest

Bug: 62946472
Bug: 64203181

Test: make vts
      vts-tradefed run vts -m VtsHalCameraProviderV2_4Target

Merged-In: If365ab2ed9a91eb4013d71769804b9d4bf089d66
Change-Id: Id0bddbc2949337147557f45cc60dbfaa114ce25e
(cherry picked from commit d71b654d6d86245dec330aa5b338b39dcb728a22)

Merge "Move Broadcast Radio HAL to a separate binary." into oc-mr1-dev

Merge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev am: 4f3c28ad7a
am: b06f544d22

Change-Id: Id525d075289896bb5cb708aa104eef0ee622d61c

Merge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev
am: 4f3c28ad7a

Change-Id: I43a48595607a1c38475aec81232008ac76cdd5b1

Merge "Audio VTS: Look for Audio policy config in all supported folders" into oc-dev

Fix HALs to not have transitive depends.

Bug in hidl-gen allowed this to work.

Test: -Lcheck
Bug: 65055216
Change-Id: Ia0cbaac4c8bfd93be169f4fab5c9ce016ee6c473

Move Broadcast Radio HAL to a separate binary.

Bug: 63600413
Test: VTS, instrumentation
Change-Id: I34a779a6608f52bcadaca1dc159bcee02186b49f

Merge changes from topic "o-mr1-hal-freeze" into oc-mr1-dev

* changes:
  O-MR1: HIDL freeze-commit
  Move auto hash to right location in current.txt.

Merge "Configstore HAL is down-revisioned to 1.0" into oc-mr1-dev

Merge "Fix test that broke when number of ADD inputs increased from 2 to 3." into oc-mr1-dev

Fix test that broke when number of ADD inputs increased from 2 to 3.

Test: vts: nn32, nn64

Bug: 63905942

Change-Id: Ie4b237d0c16247d7774a13384a59453da94c5d91

Merge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev am: 751953859c
am: b8197386f9

Change-Id: Id6470f4a31469da02a8b3d4b4b258e863a9bb234

Merge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev
am: 751953859c

Change-Id: I8800d11d6b137488674f3159941135bce374d3a6

Merge "Sensor VTS testBatchingOperation flakiness fix" into oc-dev

O-MR1: HIDL freeze-commit

Freezing libraries which are being released in O-MR1.

Note the following remaining items:
- configstore@1.1 will be removed from O-MR1 (since it doesn't
  add anything)
- neuralnetworks@1.0 will be frozen later (as it has some
  remaining changes in progress)

Bug: 63603463
Test: -Lcheck
Change-Id: I888896a66e742613d0ce64c4ca6206972ac6aca8

Merge "Remove hardcoded <kernel> version from fwk comp mat" into oc-mr1-dev

Remove hardcoded <kernel> version from fwk comp mat

They are injected via assemble_vintf by the build system
at build time. See build/make/target/board/Android.mk.

Test: m system_compatibility_matrix.xml, and manually
inspect $OUT/system/compatibility_matrix.xml; it contains
<kernel> versions.

Bug: 64124223

Change-Id: Ibf82efa0805233dedb6badce7e5a4ff62a9ff769
Merged-In: Ibf82efa0805233dedb6badce7e5a4ff62a9ff769

Move auto hash to right location in current.txt.

This doesn't actually matter in practice, but this
will help keep the file sane and make sure the wrong
hashes aren't modified in the future.

Test: pass
Bug: 63603463
Change-Id: I89e320d36c41b08ebbcee320ac37565bb1aa708c

Merge "Bluetooth: Return false when not shut down" into oc-mr1-dev

Configstore HAL is down-revisioned to 1.0

Configstore was up-revisioned from 1.0 to 1.1. However no new
APIs were added to 1.1. Therefore configstore in mr1 will be down
revisioned to 1.0.

Bug: 64999074
Test: lshal | grep configsotre
Test: "cat proc/<configstore pid>/status | grep Seccomp " return:
    Seccomp: 2
Change-Id: Ia2614439cd620fbe5fbd427571ab55a15089cc47

Bluetooth: Return false when not shut down

Do not crash in the vendor layer.  The stack will crash
when it fails to initialize.

Bug: 64535243
Test: Bluetooth starts/stops
Change-Id: I69fdd2879c80fa2f006c50f0a42a85e9289b14fe

Bluetooth: Check H4 HCI packet types

An invalid type byte can cause an access outside
the bounds of the array.

Bug: 64565737
Test: Bluetooth On/Off
Change-Id: Ie11dd62942f66b9fc60ebce2e4339c37d597fdf0

Setting clock flags for GPS clock

Full bias nanosecond flag is now set correctly for
legacy GpsData (vs. modern GnssData) implementations
of gps.h measurements, going through default GNSS
HAL implementation.

Change-Id: Iec8f03994b6d9e509221e1323b54dc997319fccc
Fixes: 64482271
Test: Before/after on device with GnssLogger

Merge "Merge Vehicle HAL 2.1 iface to 2.0" into oc-mr1-dev

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 Vehicle HAL 2.1 iface to 2.0

Vehicle HAL is property based and all properties are considered
optional. It is quite a bit of pain to create new versions of VHAL only
to extend properties.

Bug: 64303350

Test: build and flash, verefied system works stable.
Change-Id: I8adb858f53a5d9d31f7444bf97ae09baf2fffafa

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

Audio VTS: Look for Audio policy config in all supported folders

audio_policy_configuration.xml can be loaded from the following places:
 - /system/etc/
 - /vendor/etc
 - /odm/etc
Nevertheless the config validation test was expecting it to be in
/vendor/etc exclusively.

This patch changes the test logic to:
 - look for the config files in all 3 folders
 - make sure the config is unique

Test: run the following script to check for regressions and test that
      invalid config make the test fail.
  (
  set -xe
  runVTS() { vts-tradefed run commandAndExit vts \
        --skip-all-system-status-check --primary-abi-only \
        --skip-preconditions --module VtsHalAudioV2_0Target \
        -t CheckConfig.audioPolicyConfigurationValidation; }

  echo "# Test valid config"
  runVTS

  echo "# Test multiple invalid match"
  adb shell touch /system/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell rm /system/etc/audio_policy_configuration.xml

  echo "# Test multiple valid match"
  adb shell cp /{vendor,system}/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell rm /system/etc/audio_policy_configuration.xml

  echo "# Test invalid config"
  adb shell sed -i /defaultOutputDevice/p /vendor/etc/audio_policy_configuration.xml
  ! runVTS
  adb shell sed -i '"/defaultOutputDevice/{p;N;d}"' /vendor/etc/audio_policy_configuration.xml

  echo "# Test that the test did not break the config"
  runVTS
  )

Bug: 64881365
Change-Id: I9db5e6f727d19fd654a3cc543a2aaab196682001
Signed-off-by: Kevin Rocard <krocard@google.com>

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

Replace LOOKUP with EMBEDDING and HASHTABLE_LOOKUP

Bug: 63905942

Test: adb shell /data/nativetest64/embedding_lookup_test/embedding_lookup_test
Change-Id: Ic9ac31f60ac8a6e2c59700a34909004f4b5dc9ff

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

Sensor VTS testBatchingOperation flakiness fix

Bug: 64732324
Bug: 63529148
Test: testBatchingOperation passes consistently
Change-Id: Id4e8075e249f3658f5683d53d6dc8403ee32bc4a

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