Merge "GD metrics: Logging link layer events from GD"

gd: Enforce modules to present a unique name

A rogue module should be identified if it hangs

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

Change-Id: I0514ff36c7b1654e8de30a262b9a3a05a305a537

headless: Add docs and more information

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

Change-Id: I44c8a3a38914581a335446f975c9626a9587e73f

headless: Add initial/final sentinal log markers

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

Change-Id: Ic965827c03487d9d7984382178411e520c5a5057

Explict default init types module_t

Toward initialized code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor

Change-Id: I343a777b57138017f825823d260a3c956f2aead0

Enum-ify tSMP_EVT

Toward readable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor

Change-Id: I12e2d8d7d1f8ddb05a6b9b673c74013fda5a0d22

Enum-ify tBTA_STATUS

Toward readable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor

Change-Id: Id79e619d24484ace55409e32f99368d5d9bce06e

testing: Use proper types

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
Change-Id: I959e0522ca0e7394e7d1287859694d83d9e50e83

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

Merged-In: Ia5ca732e8464d02a3c5a7b1b75f4c6afbe441fb1
Merged-In: I4021d889879a11a7ad2a59a1b63c4371970cc823
Change-Id: I2db132a530a87a3664f7ea24c0fc5d4ac0a85822

GD metrics: Logging link layer events from GD

Events include:
* LinkLayerConnection,
* ReadRssi,
* ReadFailedContactCounter,
* ReadTxPower,
* RemoteVersionInfo,
* HciTimeout
Also move the gd metrics logging from common to os

Test: gd/cert/run
Manual test: build and run Bluetooth (connection, pairing, music and phone
call)
Bug: 181819141
Tag: #gd-refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Id84651a4cc0343ed0438083290b3aaab1ead6535

Merge "RFCOMM: Handle configuration for pending connection"

Merge "OOB: Implement generateLocalOobData API"

Merge "Add time unit to AVDTP delay reporting value displayed for debugging."

Merge "Add logging message to indicate collision with previous pending connection from the same remote device."

Add time unit to AVDTP delay reporting value displayed for debugging.

Tag: #refactor
Test: ADT3 with Pixel Buds
Bug: 182167357
Change-Id: Ie2527cc4c2e9cb871d15eb26417ae4814953de84

RFCOMM: Handle configuration for pending connection

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

Merge "SCO: change D1 max latency to 0xFFFF"

Merge "Initial btif/test/btif_stack_test"

Add logging message to indicate collision with previous pending connection from the same remote device.

Currently, When there's already a pending connecting procedure going on in the native layer (even if it were the same remote device), the native layer returns BT_STATUS_BUSY which is then interpreted by the JNI layer as a failure to connect, resulting in a disconnection request.

This adds logging message when this happens, until there's a clear
indication to prioritize a complete fix.

Tag: #stability
Bug: 174127136
Test: manual through repeated connections in ADT3 OOBE
Change-Id: Ifc84e883f8ae248ecfa99556465c22b32ce82f79

SCO: change D1 max latency to 0xFFFF

Although classic SCO cannot retransmit, the max latency should be
0xFFFF(Don't care) instead of 0(reserved, not 0ms).

Tag: #compatibility
Bug: 185093884
Test: atest BluetoothInstrumentationTests
Change-Id: Ia287c32e771eff1e5550fbb2098d70c520689250

Remove not Hwbinder thread stopwatch caller

Bug: 185083209
Test: atest BluetoothInstrumentationTests
Tag: #stability
Change-Id: I14aa41d741c30a9470929a19b633d0d936adcf04

Initial btif/test/btif_stack_test

Bug: 182544206
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines
No-Typo-Check: Legacy code mocks

Change-Id: I5cf4b24651b436a96bfbba43ad33db28707352e3

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

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"

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

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"

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

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

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

Test: N/A
Change-Id: Ia98b958ea7a97b7aa5a47ce3b118148adcaa5d08

OOB: Implement generateLocalOobData API

Ignore-AOSP-First: Conflict with internal
Bug: 178007935
Tag: #feature
Test: manual
Change-Id: I149907ef7ee3b6f9600ac57950afbc9616381dc0
Merged-In: I149907ef7ee3b6f9600ac57950afbc9616381dc0

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

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

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

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"

* 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"

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."

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"

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"

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

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"

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"

Merge "audio: Introduce client interface support for HAL 2.1"

Merge "eatt: Adjust to new L2CAP API behaviour"

eatt: Adjust to new L2CAP API behaviour

There were a changes in the L2CAP layer which needs to be taken into
account by EATT.

L2CAP calls L2CA_DisconnectInd_Cb only when channel
has been disconnected by peer and there is also no confirmation on
L2CA_DisconnectReq.

Having above in mind, EATT needs to clear its channels list just
after calling L2CA_DisconnectReq() othwerwise state of EATT will
be incorrect.

Tag: #feature
Bug: 159786353
Sponsor: jpawlowski@
Test: atest --host net_test_eatt

Change-Id: I49949ca071f40abb9ca38dcd337b4de63f493f09

audio: Introduce client interface support for HAL 2.1

Patch introduces HAL 2.1 client interface support with backward compatibility.

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

Merge "Use proper UUID for PBAP-PCE profile descriptor list"

A2DP: don't apply codec preference if service disabled

Because the codec preference is saved at upper-layer, it was timing that
JNI callbacks were reset by BT turning off while querying the
preference. In order to prevent NPE, this CL checks whether A2DP is
still enabled, and uses a JNI variable to store those callbacks.

Tag: #stability
Fixes: 182756107
Test: atest net_test_btif
Change-Id: Iacfb9fee0e69a47ca7c7ed7c5fc8b4782b647f84

Merge changes I60b334ef,I390a385c,I0a562e1b

* changes:
  Revert "Remove unused API BTM_IsPhy2mSupported"
  Revert "Remove unused API BTM_RequestPeerSCA"
  Revert "Remove unused API BTM_GetPeerSCA"

Use hci_root_inflamed_abort if received root inflamed packet

Sometimes, Since the hci_timeout_abort abort_timer triggered is faster
then the bluetooth HIDL service died, the bt_stack doesn't call
hci_root_inflamed_abort even if received root inflamed packet. Use
hci_root_inflamed_abort when HCI command timeout if it received the root
inflamed packet.

Bug: 182940721
Test: atest BluetoothInstrumentationTests
Tag: #feature
Change-Id: I877ac13e46ac12ef42fc149a7d4a16d932f014d2

Revert "Remove unused API BTM_IsPhy2mSupported"

Reason for revert: This is needed for LE Audio

This reverts commit 5f3ca2bb910c5f18927a15be017879a8387c78fe.

Change-Id: I60b334ef9d3f376eb1b2c68c2a40f748861a5368

Revert "Remove unused API BTM_RequestPeerSCA"

Reason for revert: This is needed for LE Audio

This reverts commit ddc067a95ef95fcdc888bfdefc07f784ecac2f1e.

Change-Id: I390a385ce8b5c2b0e76c7775ba3cae8db707b331

Revert "Remove unused API BTM_GetPeerSCA"

Reason for revert: This is needed for LE Audio

This reverts commit ab4932be3bad052d5a0f7b05cd59b21a88fe26e6.

Change-Id: I0a562e1bee8d3c4f3f7170e34ea92fe657123b96

Add btif/include/btif_hh::btif_hh_status_text

Toward loggable code

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

Change-Id: Iab6a4f33a3e498c33f4870abc5abd59e073d03f2

Use proper type tHCI_REASON btm_get_acl_disc_reason_code

Toward readable code

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

Change-Id: I7b2adfaa55feb96c620adfa5a6cb997055f1ce10

Merge "Use whole_static_library for cxx library deps."