Merge RQ2A.210405.006 to stage-aosp-master - DO NOT MERGE

Merged-In: I269fdcdf301bb442f124a0813db066441ae2e29f
Change-Id: I8625ed264939f8ce9ff9795fa38d081291e95d01

vc: Initial field in constructor am: c435f425cb

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1671925

Change-Id: I7522a70f85a0c2a7086805d0e44f9f0528df4e51

vc: Initial field in constructor

The 'flags' field was our only field not initialized in our
constructor.  To assure consistent behavior, we initialize it
here.

Test: TreeHugger
Change-Id: I92c94eb5a995b5980d635f2c6d1cb12bf28ef4ca

btm_iso: Allow to start ISO by many clients am: 1c51c0afce

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1620999

Change-Id: Ibdd5c2a03e579a16df77c50a53d693a37271202a

btm_iso: Allow to start ISO by many clients

It turns out we might need btm_iso to be stated from more than one place
e.g. broadcast and unicast scenario.

Therefore, allow for that.

Bug: 150670922
Tag: #feature
Test: atest --host net_test_btm_iso
Sponsor: jpawlowski@

Change-Id: I8111a55707a00b5622a260f938e0da08fc37ae65

Merge "HCI: Accept erroneous ConnectionComplete" am: 4af4ff7566

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1667585

Change-Id: I0de79dd1131a5fb4f63dd8f90c30646025f3b91a

Merge "HCI: Accept erroneous ConnectionComplete"

audio: Introduce LE Audio support for client interface and HAL am: 944543dadb

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1589742

Change-Id: If100a9ae30f2b344e08d8b73b1c076dc992e67c4

audio: Introduce LE Audio support for client interface and HAL

Patch introduces support for client interface and HAL.

Tag: #feature
Test: atest bluetooth-test-audio-hal-interface
Sponsor: jpawlowski@
Bug: 150670922
Change-Id: Idddddc403aacc671d8010f8c6004038ffe53da8f

Re-log btif/src/stack_manager am: 7923f613fe

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1670988

Change-Id: I8462f1585e1ef5f2b008c00375a1a20863f58f08

Add layer of indirection to set hal_callbacks am: 6869395b5f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1670987

Change-Id: Idb81558fb6a18ffc5ca6b967fcba208fafaa176b

Cononicalize include pathnames am: 1dcbf80e0a

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1670986

Change-Id: I625540239f09f3ef04092795fb74b7ffd0548146

Remove execute bits btif/:: am: 6938800037

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1670985

Change-Id: I84a3cd6931b6aeeb0450d0d8a9fdb08b35473448

Print stopwatch log when the bluetooth HAL died. am: 1a7586dd08

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1669526

Change-Id: I3b1392d8102c893b86d41d482f34338789dd4054

Re-log btif/src/stack_manager

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I3403836b9e358ecd2967de25a72cba421641e24b

Add layer of indirection to set hal_callbacks

Used for test injection

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I1cfd4103bfe6b20aabff2885e5ace106e11b04e8

Cononicalize include pathnames

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I5a8f78346ae26fd4cdccc5730160cb76b3a01a9e

Remove execute bits btif/::

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I35654b3c03d365900c06e12a09934a27739b75b9

Print stopwatch log when the bluetooth HAL died.

Bug: 175754404
Test: atest BluetoothInstrumentationTests
Tag: #stability
Change-Id: I775066475cc3b14de3b613b8a3b60c34214c3703

Merge "Update to use CXX 1.0" am: 73f77c9369

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1669167

Change-Id: I7d28ba797c3340d6648e88a4a6fcb42ee9209853

Merge "Update to use CXX 1.0"

Merge "Add a hint for cert tests" am: 51f099ca2c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1668867

Change-Id: I12671d4a4459848b1a03ddffcbf0345ac9e4c561

Merge changes I3ccf85d5,I731adcff,I6f5f9cec,I9f773287,Iabf796dc, ... am: 88d3421141

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1667066

Change-Id: Id7317f98f5f402f462252ca28cb328f2340d288d

Merge "Add a hint for cert tests"

Merge changes I3ccf85d5,I731adcff,I6f5f9cec,I9f773287,Iabf796dc, ...

* changes:
  Update gitignore for Cargo.lock and tags
  Use local rules for flex/bison
  Specify bt_shim_ffi dependency to bluetoothtbd
  Fix rust build in build.py
  Avoid const value_type of vector
  Guard printing PKG_CONFIG_PATH in build.py
  Add missing #include <string.h> for memcpy
  Fix packetgen outputs

Merge "gd hci: Check credit before sending next fragment" am: 7476efe66c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1669525

Change-Id: I6f73f3f5b9c7b7150b5b2d314d8370a450357e76

HCI: Accept erroneous ConnectionComplete

Some controllers send ConnectionComplete with status code
Unknown Connection when cancelling a RemoteNameRequest.

Fixes: 184239841
Tag: #gd-refactor
Test: cert/run
Change-Id: Ibcaf68a13e18a629ab16be3bb4bfe9c1ff208095

Merge "gd hci: Check credit before sending next fragment"

gd hci: Check credit before sending next fragment

Tag: #gd-refactor
Bug: 180870443
Test: gd/cert/run
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines
Change-Id: Ifd2af34886a0cf40d5ca3f4fdd4869089276d919

Add a hint for cert tests

Bug: 184786243
Test: cert/run
Tag: #gd-refactor
Change-Id: Ie81ce626c23016dd66c8418aeff44687d50111d5

Add StopWatch for Hwbinder callback function am: 4311da6596

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1666379

Change-Id: I981dce97c04d24d8ff3871fe2c5ebd8444c71551

Update to use CXX 1.0

Test: m
Bug: 184799689
Tag: #refactor
Change-Id: I93c7c2e8f754501771fe13c9f30408940dfe66a6

Add StopWatch for Hwbinder callback function

Add stopWatch feature to monitor HCI callbacks and audio client
callbacks.

Bug: 175754854
Test: atest BluetoothInstrumentationTests
Tag: #stability
Change-Id: I87b9aac85166f345629ea110653cc873ca24c079

Update owners am: 63cecbe338

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1668433

Change-Id: I72b5486c69e23208691879cb3ab9892891548cc5

Update owners

Test: N/A
Change-Id: Ia98b958ea7a97b7aa5a47ce3b118148adcaa5d08

Merge "SCO: Use const reference instead of const value" am: 6d09b3f034

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1667119

Change-Id: I34f75e72130639a11fe5e5c9f79a0a5102925b54

Merge "SCO: Use const reference instead of const value"

Add Volume Control Profile API am: 06f084b01d

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1564640

Change-Id: I3de8beebd36dc467fdc4fd60bd9516abb9aee135

vc: Implement Volume Control Profile am: 3578a9e806

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1564641

Change-Id: I04457c3e0dd9e5658da0385b9f53ad460a1c7df0

Add Volume Control Profile API

Bug: 150670922
Test: compilation
Tag: #feature
Sponsor: jpawlowski@
Change-Id: I4a12847f9be7cb77d9094fd5a1a8f9c68053ad21

SCO: Use const reference instead of const value

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I538c46eeb9cf40400f4f4ec424c4da6ae804213c

 vc: Implement Volume Control Profile

 This implements basic version of VCP which allows to control Volume
 Control Service on ther remote side.

 This profile for expose only connect/disconnect/set volume

 Bug: 150670922
 Tag: #feature
 Test: atest --host bluetooth_test_vc
 Sponsor: jpawlowski@

Change-Id: Ie68c333ffece3958d68580fbfd3a86aec1186711

legacy: Plumb gd sco conn req to legacy stack am: c39bd0957d

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1666090

Change-Id: I7bc8b21f606c6c56e32f4b356dca7a65587be7e5

Const-ify btm_sco_conn_req::DEV_CLASS am: 85fb557c11

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1667068

Change-Id: I3a3079e175bb767f5090049cd42290fa3cfce97d

Merge "RFCOMM: Handle failed connection request" am: a2f6bff875

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1664098

Change-Id: I269d56ccfe71b5304c26a92f468c9352e8cafa8b

legacy: Plumb gd sco conn req to legacy stack

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I79fcfcf8dde071aff0a07f5440cfdca6e0e8134e

Const-ify btm_sco_conn_req::DEV_CLASS

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Iec78ab6a7b46837f9ab512d57b320b565d78f7cd

Merge "RFCOMM: Handle failed connection request"

Update gitignore for Cargo.lock and tags

Bug: 176847216
Tag: #floss
Test: atest --host bluetooth_test_gd

Change-Id: I3ccf85d54493630257957a1c5182e54470bcb16f

Use local rules for flex/bison

Use local flex and bison rules for the packet parser instead of
depending on the rules in common-mk. Right now, the packet parser is
using a re-entrant parser which can't be used with C++ codegen for flex.
Common-mk maintainers would like the flex code to use C++ codegen since
it is compiling the file as C++.

Bug: 176847216
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: I731adcff7c966543300d2cdf1f87ef2a69b8a211

Specify bt_shim_ffi dependency to bluetoothtbd

Specifying lib bt_shim_ffi explicitly before ssl so that ssl references
can be linked from bt_shim_ffi.

Bug: 176846220
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: I6f5f9cec960157559a57fe6721cd21ea0647c312

Fix rust build in build.py

This copies the rust build result to the gn output directory to be
accessible when linking from C++. Also adds an option to use crates.io
for rust crates.

Bug: 176846220
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: I9f7732878a2aced6f80f363c25aff3243d34f0d5

Avoid const value_type of vector

This change is required to build fluoride against libstdc++.
libstdc++ asserts that: std::vector must have a non-const, non-volatile
value_type.

Bug: 176846220
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: Iabf796dc356826782b480fd01887c04a86e9708a

Guard printing PKG_CONFIG_PATH in build.py

Bug: 176846220
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: Iba0c739a07bcfa7256ea624e2391d5cee016d2de

Add missing #include <string.h> for memcpy

Bug: 176846220
Tag: #floss
Test: atest --host bluetooth_test_gd
Change-Id: I62b5a032c1a346a006668145407cd00f3c6bddd5

Fix packetgen outputs

This changes action_foreach() to action() to be able to control the
outputs field correctly.

Tag: #floss
Bug: 176846220
Test: atest --host bluetooth_test_gd
Change-Id: I3cc9613e5c33aa3610a542107e97c7b4667e06c4

shim: Plumb e/sco through shim into legacy am: ace8f38d4a

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1666419

Change-Id: I70dd4894b74a2418bfbcb3e5e6d9d3e936487243

shim: Plumb e/sco connect request am: 7c4c80e3b6

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1666083

Change-Id: Id02f73dfc866996382539ea4e80a5d10eb81b144

Merge "gd: Plumb remote esco/sco connection requests through GD" am: 05f72ffbff

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1664730

Change-Id: I4b30e4d738146e125a401c65f187f023b0ea6132

shim: Plumb e/sco through shim into legacy

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Id3b9e0931e17a2e376dee207af86241836e96cbb

shim: Plumb e/sco connect request

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: If14d3bf2f8c632e9007fa846d514a9c3e7bbe18a

Merge "gd: Plumb remote esco/sco connection requests through GD"

Merge changes from topic "bt-gd-vse" am: 814f49d73c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1664345

Change-Id: I8d8253305ddaccbc5621f7c23a41e7f08b0dde41

Merge changes from topic "bt-gd-vse"

* changes:
  Hci: Include 0xFF in event loops
  GD: Crash on Root Inflammation Event
  BQR: Print common BQR events
  Revert "GD HCI: Pipe VSEs up in shim layer"
  Revert "GD HCI: Support vendor specific event in GD"

gd: Plumb remote esco/sco connection requests through GD

Bug: 183476377
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Ib36d3fded252a51be550510f11cc102df2277512

Hci: Include 0xFF in event loops

Bug: 183057550
Test: Enable VSEs
Tag: #gd-refactor
Change-Id: Ib06340af7965863343610c1b53d37d1f92c6c8da

RFCOMM: Handle failed connection request

Tag: #refactor
Bug: 183672787
Test: manual
Change-Id: Iae2682b9c6b8da5a3dead2c333b8c2054be4ac30

Merge "A2DP: Log BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED" am: 07cdb42b56

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1652248

Change-Id: Iace2f7495dbfce74fee06bc91b95ca1a133ce82e

Merge "A2DP: Log BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED"

GD: Crash on Root Inflammation Event

Use the crash timer to allow enough time for debugging logs.

Bug: 183057550
Test: Enable VSEs
Tag: #gd-refactor
Change-Id: I11d19c7fd012cd11e237bb13b84b9d77c83c1c1f

BQR: Print common BQR events

Bug: 183057550
Test: Enable BQR events
Tag: #logging
Change-Id: Ibda9f7041c7f68b0bb9fff203fea556a47c12079

Revert "GD HCI: Pipe VSEs up in shim layer"

This reverts commit e53576837aa0691199cba916b692711a41b499ba.

We prefer to keep the shim small and implement VSE
handling in btif, since it's mostly for logging.

Bug: 183057550
Test: Enable BQR events
Tag: #gd-refactor
Change-Id: I8f57c479eb57439f2ede486b645edc9b3975c25a

Revert "GD HCI: Support vendor specific event in GD"

This reverts commit 9462335b7fedd1579e5961a1dc8ddbfd1a4b35ed.

Bug: 183057550
Test: Enable BQR events
Tag: #gd-refactor
Change-Id: Ibb9302ae6bb9495880174a1ed8c6457a1e781027

Merge "Do not close the fd on drop." am: aeb13ee45c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1662765

Change-Id: I94233e774bc4174cc1be46ad37e7ab7bd2cbc0b8

Merge "Do not close the fd on drop."

Do not close the fd on drop.

The nix crate now closes the fd itself, so we cannot close it ourselves.

Test: libbt_common_inline_tests
Change-Id: I270c84ff198416ad472be86b7d9fb506e3a74232

Merge "Correct log stack/rfcomm/rfc_mx_fsm::rfc_mx_sm_execute" am: bd09e1666f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1662060

Change-Id: I2a29f24421ac101d0cb769c30c0e8f93e006a552

Merge "Correct log stack/rfcomm/rfc_mx_fsm::rfc_mx_sm_execute"

Correct log stack/rfcomm/rfc_mx_fsm::rfc_mx_sm_execute

Tag: #logging
Bug: 183672787
Test: manual
Change-Id: Id52a0d1c2c8ed18c79a4f48886e2b54487cc5bda

Merge "rusty-gd: Remove CancelableTaskTracker" am: 6a00c5a536

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1660763

Change-Id: I06e5e6c1da4c7c7f5162f4e0a5723085ef68f9ea

Merge "rusty-gd: Remove CancelableTaskTracker"

Merge "gd hci: Check acl connection when start round robin" am: ff86e8a812

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1659940

Change-Id: Ie76d92b3095e3d66a78674e96fdb69b06faeb3f9

Merge "gd hci: Check acl connection when start round robin"

Merge "[GD Controller] Return version info to legacy stack" am: 83a68cf92c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1659431

Change-Id: I0275139b5bdf7da4009820769e86d8649d56cef8

Merge "BluetoothAudioHAL: Don't fetch the provider without HAL version" am: 233433e76e

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1659939

Change-Id: I4f5f30d711e6dcb1df0693d9469836b62693a38c

Merge "[GD Controller] Return version info to legacy stack"

Merge "BluetoothAudioHAL: Don't fetch the provider without HAL version"

BluetoothAudioHAL: Don't fetch the provider without HAL version

This fixes a native crash while enabling BT with no bluetooth.audio HAL.

Fixes: 184147199
Bug: 184147199
Tag: #stability
Test: start Bluetooth on device without bluetooth.audio HAL
Change-Id: I96974b33316a73e1c0c291b9de05400d2e842e92

gd hci: Check acl connection when start round robin

Tag: #gd-refactor
Bug: 182541100
Test: gd/cert/run
Change-Id: I341fb4358e74b75bc1c6165af961c85cfc2ae739

Merge "gd hci: Handle overflow of acl credits" am: adb139eb52

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1652247

Change-Id: I8011397f11dea5baa5e19cdf6da10d98afa28899

Merge "gd hci: Handle overflow of acl credits"

Merge "A2DP: don't apply codec preference if service disabled" am: ede3dc1a48

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1655348

Change-Id: Ia8942d16e21525c2bdd847ea1ffe407c74f31845

rusty-gd: Remove CancelableTaskTracker

Pairing, connection and A2DP streaming works with rusty-gd.

Also, tested with the default flags to make sure I am not breaking
anything.

Bug: 171749953
Tag: #gd-refactor
Test: gd/cert/run --rhost DirectHciTest

Change-Id: I325e6aba146d9351aa7a06ab3c847cd410a502f3

Merge "A2DP: don't apply codec preference if service disabled"

gd hci: Handle overflow of acl credits

Tag: #gd-refactor
Bug: 178668450
Test: gd/cert/run
Change-Id: I3efc7a73cfbc90b876fa055b29250587c689df0c

[GD Controller] Return version info to legacy stack

* Copy and return version info to legacy stack
* Support parsing BT 5.2 version info

Tag: #gd-refactor
Bug: 183349385
Test: make, A2DP streaming
Change-Id: Ic8921570000599175b9b219052f95f6c7b4035a8

A2DP: Log BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED

Enable BLUETOOTH_A2DP_PLAYBACK_STATE_CHANGED Result metrics logged in stack

Tag: #feature
Bug: 183473902
Test: atest BluetoothInstrumentationTests
Change-Id: I9e9b7d1d0e6fd094978ccaad9c16a0e2306e7882

Merge "Use hci_root_inflamed_abort if received root inflamed packet" am: 3c6751a128

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1650169

Change-Id: I1d3c042e38f6263e6a377a96de4724d9bbc49322

Merge "Use hci_root_inflamed_abort if received root inflamed packet"

Merge "audio: Introduce client interface support for HAL 2.1" am: 052c371919

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1589741

Change-Id: If793a3ad5a59ddea4f2adb48dc346c9c4aed2186