Merge changes Id207ae31,Ib18c6d2e,I4af19080,I03686e55,Iea1d7d22, ...

* changes:
  Audio HAL VTS: Add tests for IStreamIn
  Audio HAL VTS: Add at least negative tests for IStream
  Audio HAL VTS: Test IStream::debugDump
  Audio HAL VTS: Split big test for better traceability
  Audio hal VTS: pretty print AudioDevice and ChannelMask
  Audio Hal VTS: Refactor ASSERT_RESULT helper
  Audio hal VTS: generate names for parametrized tests
  Audio hal VTS: factorize master volume test
  Fix potentiel deadlock in audio VTS

Audio HAL VTS: Add tests for IStreamIn

For now most tests are negative.

Test: run test on board

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

Audio HAL VTS: Add at least negative tests for IStream

Add negative test for the remaining methods of IStream
and positive one for most.

Test: run test on board

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

Audio HAL VTS: Test IStream::debugDump

Test: run it on target
Bug: 34170075
Change-Id: I4af190806f79e0c03b59e1cd60b6f58a08b05322
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio HAL VTS: Split big test for better traceability

As some part of this big test were failing, it was difficult to actually
track on the dashboard what was wrong.

Split it in small independent equivalent tests.

Test: mm & run on target

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

Audio hal VTS: pretty print AudioDevice and ChannelMask

This eases the debuging of failing tests.
Also print hex value of the enums as well of their literal values.

Test: run the test on target
Bug: 34170075
Change-Id: Iea1d7d2269e0e5b413240aaa905daae755846e74
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio Hal VTS: Refactor ASSERT_RESULT helper

ASSERT_INVALID_ARGUMENTS was a macro that asserted that a given Result
or Return contained INVALID_ARGUMENT. The problem was that a result can
have lots of other values like INVALID_STATE or NOT_SUPPORTED.
Additionally not all test expect only one possible result.

Introduce two overload of ASSERT_RESULT()
The first one takes an expected Result value and compare it to the
obtained one.
The second take a list and expect the obtained one to be in this list.

Test: run the test on target
Bug: 34170075
Change-Id: I798729f27f723c98292610bfb43dbdb2724ec2ca
Signed-off-by: Kevin Rocard <krocard@google.com>

Merge changes from topic 'configstore' am: e3a0c607cd am: fb64badb3e
am: e0ca61a6be  -s ours

Change-Id: I12c920d8b2e1a1734c822d3badc4195dee82bcfc

configstore: add new configuration item am: cb71410f16 am: 35ad8cf50d
am: 4b7759c251  -s ours

Change-Id: Ieb5e5416a3eecd1fde02cc3caaabeb95c1d8a7c8

configstore: add utility library functions for configstore am: ad54c30863 am: 71e1808de8
am: 20a8fde005  -s ours

Change-Id: I072c59bd3c543ea6fb3cb9cc26d6c24e6ed47914

configstore: add configstore@1.0 HAL am: c5d7c8a27a am: 06be01518b
am: c5a9bd19f6  -s ours

Change-Id: I41c0b04abe4ffbf84ea7cd5581104543f3896190

Merge changes from topic 'configstore' am: e3a0c607cd
am: fb64badb3e

Change-Id: Ia4da72dc31ec235dd3aa0710846fc1504a0fd884

configstore: add new configuration item am: cb71410f16
am: 35ad8cf50d

Change-Id: I66849f126efe18b86cd6010027d59909e67799c5

configstore: add utility library functions for configstore am: ad54c30863
am: 71e1808de8

Change-Id: I9acd3b567d8daa137f60eeb456e31be05e20a396

configstore: add configstore@1.0 HAL am: c5d7c8a27a
am: 06be01518b

Change-Id: Ifa830fa1d3128c9319e52152e20451b0886f6db8

Merge changes from topic 'configstore'
am: e3a0c607cd

Change-Id: I3b50189813bde7b84e9e545e37717a4e8bda5801

configstore: add new configuration item
am: cb71410f16

Change-Id: I9dcf29ba10593d3bca6d0db4269c072687d7ab51

configstore: add utility library functions for configstore
am: ad54c30863

Change-Id: I4d2fed4d6c46b4ab22c955f020f590f1f66f5927

configstore: add configstore@1.0 HAL
am: c5d7c8a27a

Change-Id: Id8da8ce348021dd6d8976071370c5d75118c2fa2

Merge changes from topic 'configstore'

* changes:
  configstore: build configstore-utils as header library
  configstore: add new configuration item
  configstore: add utility library functions for configstore
  configstore: add configstore@1.0 HAL

Merge "Bluetooth: No power off before power on" am: 2e429d6b64 am: 3805d65284
am: 0dba862f10

Change-Id: I2277fc9cae7055f3b915d9556e2a17ed657fecb3

Merge "Bluetooth: No power off before power on" am: 2e429d6b64
am: 3805d65284

Change-Id: I03d5b9fc13750baa4a771b645d032ce2331b1523

Merge "Bluetooth: No power off before power on"
am: 2e429d6b64

Change-Id: If65b972e390b50add2412c1d91c259210405aa5c

Merge "Bluetooth: No power off before power on"

wifi: Link the new libcld80211 library am: 48df739a4d am: 02354cb6b6
am: 50ad0c5171

Change-Id: I6d7ccaa1c960b3e650ee0cbea8cddd4bbb145ce5

wifi: Link the new libcld80211 library am: 48df739a4d
am: 02354cb6b6

Change-Id: Ie1a5036050c5edc42597a1b70467b9da08c15183

wifi: Link the new libcld80211 library
am: 48df739a4d

Change-Id: Ia0676ab9802360aaa67dcb0f67f478fa820e06d4

wifi: Link the new libcld80211 library

Bug: 32775496
Change-Id: I700dec21bfc3a36b8a3b09251b3004449fd49de7

Bluetooth: No power off before power on

Test: Switch users (toggles Bluetooth under load)
Bug: 36026072
Change-Id: I7eef35e19fbb4cf825c2583ec0af573733b24f9d

Audio hal VTS: generate names for parametrized tests

The gtest dashboard only shows test names, not their parameters.
This mean that parametrized tests are names 0,1,2...
Thus when one test fails, the dashboard info is not very useful.

This patch appends the audio config to the test name.

Test: run the test on target
Bug: 34170075
Change-Id: I77090a3a17e4a3e97ecb72a8a89ad2d88626d96f
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio hal VTS: factorize master volume test

Master volume was the first pair of getter/setter tested
and did not use the generic test*Accessor function.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Fix potentiel deadlock in audio VTS

The IDevice::debugDump method dumps debug info in a file descriptor.
Such file descriptor was previously the writing end of a pipe.

As the test is not multithreaded, if the dump was bigger than the pipe
buffer, a deadlock would occur.

With this patch, the test uses a file instead of a pipe. Thus write
will never block infinitely.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

configstore: build configstore-utils as header library

This change makes android.hardware.configstore-utils library built as
a header-only library instead of a static library.

Design doc: go/design-confighal
Bug: 35397068
Test: build, run

Change-Id: I59c7c8802aeb1fb687af24cfdb2bbb38723e6221
Merged-In: Idee48816a1a271bdb9ba8cf907d98c3823925da4

Merge "[WIFI][HIDL] Add device-based feature flag support" am: 48b4ce2e99 am: d15376f05c
am: 733015b56e

Change-Id: Ia12e5aef8c2260a2a3b35da1318ee1a73f6a6728

Merge "[WIFI][HIDL] Add device-based feature flag support" am: 48b4ce2e99
am: d15376f05c

Change-Id: If13d2a7e8993d93f23082fcd4ccf2732aaacba6f

Merge "[WIFI][HIDL] Add device-based feature flag support"
am: 48b4ce2e99

Change-Id: I74d285740511b9e8604a9bac5bcc8ab54688c0d8

Merge "[WIFI][HIDL] Add device-based feature flag support"

Merge "Revert "Revert "Add return status to IGraphicBufferSource methods"""

Merge "EvsCamera: %zd -> %zu for size_t print"

wifi(implementation): Add null checks for RTT result am: 78cfef9f0c am: 841ba1b367
am: 53c158606b

Change-Id: Ia36f323770af2f72bf416f686ac6e521adcf5ff4

wifi(implementation): Add null checks for RTT result am: 78cfef9f0c
am: 841ba1b367

Change-Id: Id2ce5421a1d4ae08fbedb6983477c80542e90001

wifi(implementation): Add null checks for RTT result
am: 78cfef9f0c

Change-Id: I9a58ca156d3e2f61fda8346d39b8de355302690f

wifi(implementation): Add null checks for RTT result

According to the legacy HAL header file, the LCI & LCR fields may not
always be populated. So, add null checks for them.

Bug: 36141536
Test: Compiles
Change-Id: Id2a5f5fd0eaf21b3a4d5cfcd26124c9fb6c90869

EvsCamera: %zd -> %zu for size_t print

%zd doesn't throw a warning, but %zu is technically correct.

Change-Id: I3762d01cc424ecf84e73c21afe5a5a3d295fe782

Revert "Revert "Add return status to IGraphicBufferSource methods""

This reverts commit b729637648a319616654160785cb088d10ddf95b.

Bug: 36110748
Change-Id: Ia12c00d57b9e8d4b66265018c3970416c265091a

Merge "memtrack: eliminate dependency on surfaceflinger in VTS test"

Merge "Revert "Add return status to IGraphicBufferSource methods""

Merge "audio: use vts target test base class"

Merge "Do not crash fatally when recovering from system server restart."

audio: use vts target test base class

Test: make vts succeed
Change-Id: I8008392e016e0604f8aa9a292f1ec6c6e7c3b384

Revert "Add return status to IGraphicBufferSource methods"

Bug: 36110748

This reverts commit ed32ff5150ce5bf9897c4dd00cf3b8cad385fcab.

Change-Id: Ibb5e72fc11f705a571491103fc5c68005cf54a30

Change vts target base test class name

Test: make vts succeed, vts kernel tests and nfc hal tests passed
Change-Id: I95f10ad7a66f261c9030357edd411fe6d94150e8

Merge "Change vts target base test class name" am: 4f4fdaf664 am: a8239094f4
am: 649a2c13d7

Change-Id: I376e83c09515ba50a7e033e3cb75e0c5f8a24ccf

Merge "Change vts target base test class name" am: 4f4fdaf664
am: a8239094f4

Change-Id: If6245264c41f92dc6f89a0bfd6001db07e211de4

Merge "Change vts target base test class name"
am: 4f4fdaf664

Change-Id: Id82295a163c4abbd0b9fc1ac8fde12ebd9a43030

Merge "Change vts target base test class name"

Do not crash fatally when recovering from system server restart.

Log(FATAL) is creating lots of nuisance bugs.

Test: Sensor service continues to recover from a system server
      restart and sensors are available.
      No crash log is created.
Bug: 36094522
Bug: 36115683

Change-Id: Idc2d086709dc2132c98aab806aa44728fbdaa189

Merge "graphics: fix a potential use after free"

Merge "graphics: really reset HWC2 on SF crashes"

Merge "Add SurfaceFlinger phase offset to configStore"

Bluetooth: Power off after closing the UART am: 66a4ca3c92 am: a62b176c5c
am: 0eb1c4a753

Change-Id: I09afc5f2c4b086749a093fe4114ece747e498c2d

Bluetooth: Power off after closing the UART am: 66a4ca3c92
am: a62b176c5c

Change-Id: I18c58eb31b35b8f589134541babff80194232bbe

Bluetooth: Power off after closing the UART
am: 66a4ca3c92

Change-Id: Iccea3216281ddb5e6ccd608d9743c49dca21293a

graphics: fix a potential use after free

We cannot lookup _and_ update buffer cache entry in lookupBuffer.
The old buffer is still in use by hwcomposer2.  Add updateBuffer to
do the update after the new buffer has replaced the old buffer in
hwcomposer2.

While at it, s/BufferClone/BufferCacheEntry/g.

Bug: 36064845
Test: manual
Change-Id: I59b61c0198ad528c40020fdebbe27a6cc359226f

Merge changes Ic451761c,I0652f1bf,I928cae71

* changes:
  Add VTS for the audio primary device
  Fix segfault in Primary audio hal hidl wrapper getDevice
  Do not skip failing Audio vts tests

[WIFI][HIDL] Add device-based feature flag support

Add mechanism to define feature support flags to
configure set of Wi-Fi features supported on a particular
device.

Bug: 32997844
Test: integration tests
Change-Id: Iffbf3ba237013f1030b24c0173641234a9c27e5c

Bluetooth: Power off after closing the UART

Bug: 36084266
Test: angler & marlin switch users (Toggle Bluetooth under load)
Change-Id: I1e0c34e334d11b9dcf9df62cbde3d5556c537bd7

Merge "Add generic flags to ProgramInfo struct."

Change vts target base test class name

Test: make vts succeed, vts kernel tests and nfc hal tests passed
Change-Id: I38ae6cc940d0f17bb3352f360efdf9c70697c76b

configstore: add new configuration item

This change adds to configstore@1.0 HAL a new configuration item
for surfaceflinger; useTripleFramebuffer()

In the default implementation, useTripleFramebuffer() will return true
only if the build flag NUM_FRAMEBUFFER_SURFACE_BUFFERS is defined as 3.

Bug: 34723491
Test: build & run. log message "SurfaceFlinger FrameBuffer: ?"
      should be seen.

Change-Id: I7ab9f17f43d68ee7b37b5db8c24b542db4d1c216
Merged-In: Iac97e25cd69717c73a135f6b2a44538124544987

configstore: add utility library functions for configstore

this change adds a set of library functions to facilitate
accessing(retrieving) configuration items.

Specifically, added template functions for accessing configuration
items for following types:

OptionalBool, OptionalInt32, OptionalUInt32, OptionalInt64,
OptionalUInt64, OptionalString

Bug: 34724435
Test: build, run

Change-Id: I28d565dcfc04bb71bf891d1b48ce2ec6192eb21f
Merged-in: Icca56d0d9e086b9d94c9b1168df041bf7d418698

configstore: add configstore@1.0 HAL

This change adds configstore@1.0 HAL which provides static configuration values
to the framework. As a first attempt, a configuration item in the
surfaceflinger (VSYNC_EVENT_PHASE_OFFSET_NS) is choosen and converted to
a HAL function named vsyncEventPhaseoffsetNs().

Bug: 34314793
Test: build & run. make sure that configstore is up and running.
$ ps -A | grep configstore

Change-Id: I0692190180cadc8e3f64ee840c737e05b519a135
Merged-in: Ibc70b14176d2e0144a5745d71ceba49ebc3ca706

Add VTS for the audio primary device

Its base class was tested but not the primary
device getters and setters.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Fix segfault in Primary audio hal hidl wrapper getDevice

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Do not skip failing Audio vts tests

Previously failing tests were commented because I thought
our implementation had to pass all tests.

As a mater of facts, it is not the case and tests are allowed to fail.
Bugs will be raised for those failure.
This is much better way to track the fixes that FIXMEs.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Merge changes If83c1cbe,I12ad2665 am: cdca05557d am: 3b815eee2a
am: b005b3ae2d

Change-Id: Ib9f77fbe6c43c4706cce89a6bcd99d9b840915e1

Merge changes If83c1cbe,I12ad2665 am: cdca05557d
am: 3b815eee2a

Change-Id: I3fe19d41a610b80fcb28e0db4f1264770b1659f3

Merge changes If83c1cbe,I12ad2665
am: cdca05557d

Change-Id: Ifc3dbe11f88ceb83d77f0334894f739bbc64e1c9

Merge "Fix static properties to comply with VTS"

Merge changes If83c1cbe,I12ad2665

* changes:
  wifi(implementation): Move out clearing on stop complete
  wifi(vts): Some IWifiChip/IWifiStaIface tests

Merge "Treat warnings as errors in VHAL unit test"

Merge "[AWARE] Relocate ranging field in legacy HAL - match in HIDL impl" am: 255cdb29a0 am: 556b385fa4
am: 17ea7d9060

Change-Id: I8ba8ebc238214879d6e4e53be32c2c7fb818b58b

wifi(implementation): Move out clearing on stop complete

Currently, we invoke WifiLegacyHal.invalidate() inside the
|on_stop_complete_internal_callback| std::function. |invalidate|
nulls out |on_stop_complete_internal_callback| inside it which might be
considered a violation since we're deleting the std::function object
as we're executing it.

Bug: 36095153
Test: Compiles, start/stop wifi multiple times.
Test: Verified that a crash is not seen in asan build.
Change-Id: If83c1cbee78a1b6eca12686fcc1ec1de67134f13

Merge "[AWARE] Relocate ranging field in legacy HAL - match in HIDL impl" am: 255cdb29a0
am: 556b385fa4

Change-Id: I2aed12f7984f14a01fef071c23282153931680aa

Merge "[AWARE] Relocate ranging field in legacy HAL - match in HIDL impl"
am: 255cdb29a0

Change-Id: I2221282979fdf9b93495650fb82a0b8bfc9d4146

Merge "[AWARE] Relocate ranging field in legacy HAL - match in HIDL impl"

Merge "camera.device@1.0: Fix minor doc bug"

Merge "vts: fingerprint: properly set timeouts for tests"

Merge "Add VTS for voice telephony apis." am: 6b8077be1b am: 89f6259553
am: 452a9dff53

Change-Id: Ie2f973da848eae1b8adcef3efbe7fef2be6d3417

Merge "Add VTS for voice telephony apis." am: 6b8077be1b
am: 89f6259553

Change-Id: I5f7611d361abe225a5141639c7527da1a55e9bf4

Merge "Add VTS for voice telephony apis."
am: 6b8077be1b

Change-Id: I40bcd56f02fe1c0a7641aec82065a518665473b1

Merge "Add VTS for voice telephony apis."

vts: fingerprint: properly set timeouts for tests

Test:
  make vts;
  adb push
    out/host/linux-x86/vts/android-vts/testcases/DATA/nativetest64/VtsHalBiometricsFingerprintV2_1TargetTest/VtsHalBiometricsFingerprintV2_1TargetTest
    /data/local/tmp/.;
  adb shell /data/local/tmp/VtsHalBiometricsFingerprintV2_1TargetTest

Bug: 32022721
Bug: 34179855
Change-Id: I92ba6157d378fe04b5319b951fa0ef06f6cbaa42

Bluetooth: Protect callbacks during shutdown am: 9ef1f71982 am: d222c44873
am: a6a12995e5

Change-Id: I7548d7550b7448f3ed9e1b19bc5a8b8cc6b6a553

Bluetooth: Protect callbacks during shutdown am: 9ef1f71982
am: d222c44873

Change-Id: I1516a786e6c9a561f419f9dceadb5a21f75f840f

Bluetooth: Protect callbacks during shutdown
am: 9ef1f71982

Change-Id: I12d75f2262ed199e42fa014d5c6c5f0270de2e18

Bluetooth: Protect callbacks during shutdown

- Hold the mutex during callbacks
- Disable power mode before deleting hci_

Bug: 36084266
Test: Switch users repeatedly (toggles Bluetooth)
Change-Id: I7e4d405e9028c3363c112a8b09c03b8aea652b7a

Add VTS for voice telephony apis.

Bug: 32022332
Test: vts
Change-Id: I94f86f48d0c227f00c84051d14e6116d0bf2d1b3

Add SurfaceFlinger phase offset to configStore

Test: Manual
Bug: None
Change-Id: I64e07154903fedc40c153f4f53fb9a9aeae24dea

wifi(vts): Some IWifiChip/IWifiStaIface tests

These tests basically just ensure that we can call the corresponding
API's successfully. More tests will be added in further CL's.

Bug: 33459306
Test: mmm -j32 hardware/interfaces/wifi/1.0/vts/functional/
&& adb push out/target/product/angler/data/nativetest64/
VtsHalWifiV1_0TargetTest/VtsHalWifiV1_0TargetTest /data/ &&
adb shell /data/VtsHalWifiV1_0TargetTest

Change-Id: I12ad26653993ce7167d007a887e7aff07fd0a5ce