RootCanal: Add more scripts

Test: Connect to HCI sockets and LinkLayer sockets
Change-Id: Ice4cee21f295bbb3a3fe35cfe44073028e65d811

RootCanal: Desktop simulation envrionment

Test: nativetest64/root-canal/root-canal [TEST_PORT] [HCI_PORT] [LINK_PORT]
      python scripts/test_channel.py

Change-Id: I6e57981182c392366d7d97249d837694b49dfa4e

RootCanal: New Directory Structure

model/controller/ contains the simulated chip
model/devices/ contains simulated devices
model/setup/ contains the test model
packets/link_layer/ contains the fake LMP packets

Test: cts-verifier Insecure RFCOMM test
      LinkLayerPacketBuilderTest
      rootcanal-packets_test_host
Change-Id: Ieae0cb21f7d57c03797f800797cedae59dd70e49

Merge "Run code coverage only on host target"

Run code coverage only on host target

Bug: 129421924
Test: run code coverage
Change-Id: I1dc681c72d8af29e43ccb1441ffd218059b8056c

Merge "BQR: Fix for the issue that Bluetooth keeps on crashing if the controller firmware does not support the BQR feature."

Merge "Only use the new format in gd/"

Merge "Copy reactor classes to new directory"

Only use the new format in gd/

Test: Upload a change with long lines
Change-Id: Iba4a08f9556219bfada9a13272463acbf7ddbdce

Merge "A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled"

Copy reactor classes to new directory

Test: compile and run unit test
Change-Id: Ie704bf6b80f24bcae86d557ca351a893dcf8bdfa

Merge "Hearing aid: use new common/Timer"

A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled

When using BluetoothA2dp / BluetoothAudio HAL, the UIPC won't be used,
and is no need to do the initialization. If it is up, there will be an
uipc-main thread running, and we have to release by the UIPC_Close API
before the A2DP source restarting. If we are acquiring a new one before
released, some of its resource will be leaked, and causes the stack
abnormal.

Bug: 128256722
Test: Check uipc-main is running when using the legacy HAL only
Change-Id: Icc8ea102d29c92b58c77099979d17e85e5cb9a83

Add test about post/clear task from callback for handler

Test: sudo ./bluetooth_test_common
Change-Id: Ibbf8622a705d3523638ef6cc9e6a62130b877637

Hearing aid: use new common/Timer

Previously osi/alarm cannot be cancelled or freed while in a callback.
Replace it with common/Timer. It's safe to be cancelled in a callback
and well tested.

Bug: 119533256
Test: manual
Change-Id: I73c7c98d683c85ec8f0f4256c3e4bccf3e11b56a

Merge "Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported"

BQR: Fix for the issue that Bluetooth keeps on crashing if the
controller firmware does not support the BQR feature.

Considering for the Treble case, it might upgrade the system image
only and the BQR feature will be enabled on a device whose Bluetooth
controller firmware does not support the BQR feature. The change is
not to crash the system if the controller firmware does not handle the
BQR VSC.

Bug: 129037162
Test: Enabled the BQR feature on the device whose Bluetooth controller
firmware does not support the BQR feature. The Bluetooth can be turned
on and works normally.

Change-Id: I6e9310a2b604679ba300e558ae4482887501e022

Merge "Refine the log messages of audio_bluetooth_hal"

Merge "Add a workaround to play A2DP SBC Mono"

Merge "Add event_value to BluetoothClassicPairingEventReported"

Merge "Use PLOG rather than LOG strerror(errno)."

Refine the log messages of audio_bluetooth_hal

This CL follows the review suggestions at aosp/920718 and aosp/925233 to
refine logs.

Bug: none
Test: check logs manually
Change-Id: I3c57afaf6d416752687a7a697a310fe98c8ee495

Add a workaround to play A2DP SBC Mono

There is a similar WAR of aosp/522661 at A2DP legacy HAL. In order to
suport MONO channel mode, the PCM audio is pulled as STEREO and mixed
into MONO by the Bluetooth Audio HAL.

Test: Playing SBC mono with Headset
Bug: 127593318
Change-Id: I78f3973ba6c8c733dc18122288a915daed97be65

Add event_value to BluetoothClassicPairingEventReported

* Add an event_value field to log status value such as
 - encryption enabled state
 - simple pairing mode
 - secure connection host support
 - delete all flag
* Use the event_value field to log above information during
  classic pairing process instead

Bug: 128966402
Test: make
Change-Id: I2a87c3837754bfc5bcd55f72325400c428c2ca25

Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported

Because the stack may run without the new BluetoothAudio HAL like GSI
under old devices, it will be nullptr to getService from the
IBluetoothAudioProvidersFactory in such condition. We take nullptr as
unsupported, and fallback to the legacy HAL.

Bug: 128419724
Test: Manually running A2DP and hearing aid with / without the HAL
Change-Id: I606abc3e5b63b7857c3307c879fd4cbe46dd05d9

Add Rx RSSI logs for Hearing Aids

When there is a data buffer flush (which indicates data congestion),
the received RSSI will be queried for this connection multiple times and
logged in the dumpsys.

Bug: 124331686
Test: Manual test
Change-Id: I686f4e34bda3f8067b42d6b41ca8bf316a5bf6f1

Use PLOG rather than LOG strerror(errno).

Test: builds
Change-Id: I75c3311ce00fccc79b8efc198ce69ffed96e805a

Merge "Allow to disable the rootcanal test console with a property"

Merge "No need to abort the audio HAL / Bluetooth when failed to suspend"

Merge "Move controller_properties.json to the vendor image"

Merge "compiler based array initialization"

Merge "Recover from HandsFree client connection collision"

No need to abort the audio HAL / Bluetooth when failed to suspend

When the headset was disconnecting, the audio HAL may not be able to
suspend the stream successfully, and it is no need to abort the process
for such acceptable failure. This change also adds extra log messages
about HIDL status.

Bug: 127654107
Test: Play / pause music via the Bluetooth manually
Change-Id: I335fcf75708343c7971ebd97514fea5db50f17a0

Merge "Release encoder state when cleaning up the hearing aids instance"

Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"

Allow to disable the rootcanal test console with a property

Rootcanal's test console is in a tcp socket which is forbiden for HALs
by the Android security policy.

Bug: 128355308
Test: run in cuttlefish, ensure no 'avc: denied' messages show up
Change-Id: I74bf07c34166d2df399f33194cbb41e57178edc2

Move controller_properties.json to the vendor image

Bug: 128355308
Test: build
Change-Id: I3cb9140b34af18569a983122eef3bd642516440c

compiler based array initialization

Have the caller null out the array rather than rely on the callee
performing a memset.

Bug: 121194976
Test: compiles. No runtime tests performed.
Change-Id: I1ce3f0530080769b97be0af313e822b74fd375ce

Recover from HandsFree client connection collision

If a HF client fails to connect (due to collision), clean up state such
that subsequent connection attempts will pass.

Bug: 118422992
Test: Connect HF client to AG
Change-Id: I093cf3d229956c66d9e8501eb882071600321555

Release encoder state when cleaning up the hearing aids instance

When the Bluetooth state changes from ON to BLE_ON, the hearing aid
instance was cleaned up, but not encoder state. Since the Bluetooth
process is kept at BLE_ON but not exited, the new instance of the
hearing aids would see the encoder as initialized without starting a new
Bluetooth audio session, and caused the audio HAL to be unable to talk
to the stack. We now reset the encoder state when cleaning up the
instance, so it will start a session next time during the first
connection of a new hearing aid instance.

Bug: 127610666
Test: ON / OFF BT with BLE_ON and switch active device manually
Change-Id: I426fed4ea22c0b858bee273727fca6e2e7481e84

Merge "Hearing Aid Dump Audio Tool: Add No Start Cmd feature"

DO NOT MERGE - Merge PPRL.190305.001 into master

Bug: 127812889
Change-Id: I733a17b7dd1e24a6f4a19aac2d45e312b15e0199

Hearing Aid Dump Audio Tool: Add No Start Cmd feature

When the HCI Snoop logs wraparound, the Hearing Aid Audio Control
Command "Start" is lost. For the case, we can enable No Start Cmd
feature to set a fake "Start" to extract audio data.

Bug: 127745964
Test: ./dump_hearingaid_audio.py -c1 1 -c2 3 -ns true btsnoop_hci.log
Change-Id: Id7f67da0d5476faf38352ab57339fed8e358bac6

Merge "Use a weak pointer to deliver updates to AVRCP devices."

Merge "osi: explicitly release wakelock during turn off"

Snap for 5240760 from b9dd3863033facdd8608904e4dd59ff3cc52871b to pi-platform-release

Change-Id: I8098c6f610e684be5b84a984b163088d255420b8

Use a weak pointer to deliver updates to AVRCP devices.

If a device disconnects right before a update message gets queued, the
device becomes null and there is a crash when the callback for the
update executes on the disconnected device. This patch switches the
device reference from being Unretained to using a weak pointer so that
the callback just doesn't execute if the device is disconnected.

Bug: 120431125
Test: Use the same test as b/120477414 as that bug causes a disconnect
at the same time as a media update.

Change-Id: I1dcc08e5c9866106e7ec0dad52505e34b42da600

osi: explicitly release wakelock during turn off

Ensures we release the wakelock when turning off if any module hasn't stopped timers properly

Bug: 123289721
Test: atest net_test_bluetooth
Change-Id: Ibec5c262d2ec33ce22f2c8bb2a9b838e8d6b90a7

AVRCP Controller Play Position Changed

Implement code to register for the AVRCP notification
EVENT_PLAYBACK_POS_CHANGED such that playback position gets update on
skip to beginning, and during audio scrubbing from phone.

Bug: 119119967
Test: Play music, scrub audio to another position on phone, observe
playback position is in sync.

Change-Id: Ib8459dece7629622649ba6ed340dcd697c700b09

Fix for connection attempt after connection cancel

When connection attempt is canceled, we put the transport control block
(p_tcb) into CLOSING state, but we never close or free it.
For LE devices that have not established the connection, it's safe to
go straight to CLOSED state and free the p_tcb.

This issue was introduced during connection manager refactor.

Bug: 127242749
Test: GattConnectTest
Change-Id: I8e45ead9b071c60e95d2e60dcea8afdbaefb92b1

Merge "Clear the link key when releasing security records"

Merge "Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails"

Clear the link key when releasing security records

Test: compilation
Bug: 127479372
Change-Id: Ia571a2048fa5ca85c45b03b39d398e480aadb55a

Add btm_free() to clean up btm properly

Test: Run AdapterRepeatedEnableDisable in system/bt/test with
kTestRepeatCount set to 1000.

Change-Id: I6d6650e735697bf97098827b5a9d39552c6616d8

Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails

If the cuttlefish device does not have an rtc device (such as the crosvm
VMM) the bt osi layer can promote crashes due to it not being able to
create a CLOCK_BOOTTIME_ALARM timer. Bring back a fallback but enable it
at runtime instead of compile time.

Bug: 126955943
Test: run with cuttlefish
Change-Id: I3ab0282b3e8fde776aa7b37d5772c8f62cf957bf

Merge "bta_jv_act: Avoid writing through NULL/freed pointer"

Merge "bta_jv_act: Avoid potential NULL dereference"

Merge "When stopping Hearing Aids, send Stop cmd to devices"

When stopping Hearing Aids, send Stop cmd to devices

This fixes the bug where the Stop command is not send to the Hearing
Aids devices which switching audio away from Hearing Aid.

Bug: 126465843
Test: Manual switching between A2DP and Hearing Aids while
playing audio.

Change-Id: Ifd8eaed7496e7e16c3948683c518d7627824d3e8

Merge "Reland BluetoothAudio HAL: flip to be enabled by default"

bta_jv_act: Avoid writing through NULL/freed pointer

Before writing to 't->init_called', we know that 't' might be
NULL (there's an explicit check), so we add another check here
to assure it's not NULL.  Furthermore, we're possibly freeing
't' prior to writing to 't->init_called'.  So we set 't' NULL
in that case, so our new NULL check will let us avoid corrupting
memory we no longer own.

Test: TreeHugger
Change-Id: Iaa246d5190f5f99610dace24707e74a846df3cf3

bta_jv_act: Avoid potential NULL dereference

When 'tc' is NULL (a condition we check for), 't' is never
set to a non-NULL value, yet is still dereferenced.  We move
our NULL check of 't' to make this code safe.

Test: TreeHugger
Change-Id: Ibfea74963dd763401ea18ccdd24a51ecd0baefc5

Merge "Reland Start the media encoder for a successful BTA_AV_START_EVT"

Reland BluetoothAudio HAL: flip to be enabled by default

Bug: 126304229
Test: Switch BluetoothAudio HAL and A2DP offload manully
Change-Id: I936e6c9e9fa3e35508ec435dcff8fa54b3f4d883
Merged-In: I936e6c9e9fa3e35508ec435dcff8fa54b3f4d883

Reland Start the media encoder for a successful BTA_AV_START_EVT

When phone is A2DP source and receives BTA_AV_START_EVT, we need to
proceed to start the encoder of software or offload, and ack back to
audio HAL. When issue happened, the remote as the AVDTP_START initiator
triggered BTA_AV_START_EVT, and caused we did not start the encoder to
stream the audio. This change simplified the handler of BTA_AV_START_EVT
to always start the encoder for a successful event, and ack to audio HAL
if we are AVDTP initiator.

Bug: 126136429
Test: A2DP play and paused manually on Pixel 2 and Pixel 3
Change-Id: Ieb479fd6f42da1bf37f8f32af7794d86e04cac1b
Merged-In: Ieb479fd6f42da1bf37f8f32af7794d86e04cac1b

Merge "Fix a bunch of incorrect syntax statements"

Fix a bunch of incorrect syntax statements

Found by the upcoming compiler update.

Test: m checkbuild
Change-Id: Ieafcddde7c91de990aaf7bf42de58039ba5555cd

Merge "Audio HAL: Check if ProviderFactory is null before printing debugging"

Audio HAL: Check if ProviderFactory is null before printing debugging

Test: Build and flash on auto hardware to make sure the error went away
Change-Id: I90a8f3a0e0cbc23e7e4a2edcead46402f44ef3a4

Reactor-based alarm implementation

* Use Reactor+Thread (common/thread.h) to replace existing
  libchrome-based message_loop_thread
* Use Alarm/RepeatingAlarm to replace existing Timer, by using
  kernel-based timerfd, to make all components unified

Bug: 110303473
Test: run unit test, and run benchmark
Change-Id: I6d6bc8dc3897649d0f6cc00ce0aa7054a3ddc09d

Reactor-based threading model

* Use Reactor+Thread (common/thread.h) to replace existing
  libchrome-based message_loop_thread
* Use Handler to implement multiple message queue per thread, by using
  kernel-based eventfd

Test: run unit test, and run benchmark
Change-Id: Idd2e4ef99fb9a7b2c0956de0e372c67a1098f1b6

Merge "clang: Set to 120 characters per line"

clang: Set to 120 characters per line

Modern terminals and text editors can display 120 characters

Test: compile
Change-Id: I799de73e622d045ba997e749eb15d5f786764fdd

Don't require A2DP codec information when initialize audio_hal_interface

Before set active device, it is normal that can't get the current codec
information, and will be setup later. If we query the codec and break
the init, it will cause the stack can't use Bluetooth Audio HAL for
A2DP, so remove here.

Bug: 125245738
Test: Connect / disconnect multiple A2DP manually when playing
Change-Id: Ic5493141cfba60115783c03fac4faf56a7a64b07

Fix crash when more than 2 PHYs specified for connection

Bug: 124793834
Change-Id: Ide304e22d4b390f252ba9577aeb5d9a67fa1a1cd

Merge changes I58ceea50,I73319018

* changes:
  Check A2DP UIPC state before feeding data
  Break the control early when the stream state of Bluetooth Audio changed

Check A2DP UIPC state before feeding data

When using the new Bluetooth Audio HAL, A2DP UIPC won't be initialized
and can't be accessed. This change checks the UIPC is valid before
accessing, and prevents unexpected SIGSEGV.

Bug: 124645960
Test: Start / pause A2DP manually
Change-Id: I58ceea503d0c159817167bb3ca3b99b6f933bf23

Break the control early when the stream state of Bluetooth Audio changed

When switching active device, the Bluetooth stack would first stop the
datapath while the audio HAL was still streaming, and the failure by
busy was returned to HAL. Since the control status was returned, HAL
should not ignore the result or kept waiting, because the stack wouldn't
queue the control.

Bug: 124838461
Test: switch the codec and active device manually
Change-Id: I73319018f726590c52c07547af90fac57114b58a

DO NOT MERGE - Merge PPRL.190205.001 into master

Bug: 124234733
Change-Id: I260dc329f1d83ef8a9cab7e9caee32ba47277477

Merge "Add support for a2dp offload HAL ver2.0"
am: 10cad005f5

Change-Id: Ic77b64e6703e89599399c8f857647d58c29d3310

Merge "Add support for a2dp offload HAL ver2.0"

Merge "Rename a2dp_software_encoding to a2dp_encoding"
am: ad392e481f

Change-Id: I24ca935afc634a93f62ed9b72ea23bbe244cece6

Merge "Rename a2dp_software_encoding to a2dp_encoding"

Add support for a2dp offload HAL ver2.0

Test: manual
Change-Id: Id5323b3c22d6aff6c194e3835eefeb22e32145c9

Rename a2dp_software_encoding to a2dp_encoding

Change-Id: Ief1c8cb1a768902a719c4956947d16081ad20add

Remove unnecessary unlock
am: 9ddb2adeb6

Change-Id: Ibba705841e26a19698ce8405e3335dd375be2b31

Remove unnecessary unlock

Bug: 124480689
Test: manual A2DP test
Change-Id: Id4cd4e2c97a9c4ed7219d8f1b4c15bf82b1031be

DO NOT MERGE - Merge pi-platform-release (PPRL.190205.001) into stage-aosp-master

Bug: 124234733
Change-Id: I2ad9e0da67801687409b5e755d2436ba3c99770a

Merge "Flush buffer to filesystem before fsync"
am: 2cc25459dd

Change-Id: Ic12f4a62777d89a4bc01442ab1864c1566535614

Merge "Flush buffer to filesystem before fsync"

Flush buffer to filesystem before fsync

If power is lost immediately after updating a config file it is possible
to save an empty file in its place.  By flushing the stream buffer to
the filesystem before flushing the filesystem to the disk this risk is
mitigated;

Bug: 122607444
Test: Update Bluetooth Device name and interrupt power to device.
Change-Id: Iaf89c4d9c594e925507e713a9febbf23a7cdae43

Merge "Metrics: Log manfuacturer information from Device ID profile to statsd"
am: 3894d1f0e6

Change-Id: Ia1b71faa25a6f1544089e7f228363f5555292925

Merge "Metrics: Log manfuacturer information from Device ID profile to statsd"

Merge "Metrics: Log L2CAP and RFCOMM socket connection state changes to statsd"
am: b8f9808ea5

Change-Id: I12a2985a5c5f5d93c65c077de6abb0ad00adb7d8

Merge "Metrics: Log L2CAP and RFCOMM socket connection state changes to statsd"

Snap for 5180536 from 6f2759c64311b505b1ddc73e2bfc9f0330845780 to pi-platform-release

Change-Id: Ia9fad0673602c58b504e41515cfe515e751fa848

Metrics: Log manfuacturer information from Device ID profile to statsd

Bug: 112969790
Test: make, test drive with statsd
Change-Id: Ie22aaad07037f97017186ad1f1d9ad124e4756c0

Metrics: Log L2CAP and RFCOMM socket connection state changes to statsd

* Log L2CAP LE Coc, L2CAP BREDR, and RFCOMM socket connection state
  changes, including port number, socket type, connection states,
  number of bytes sent and received, server port number, uid of socket
  owner
* Address is empty if this is a server port

Bug: 112969790
Test: make, test drive with statsd
Change-Id: Ic0ee93a6d9e4fa4109ddb89dea5e92907c49f2fc

Update owners
am: 5f896dff62

Change-Id: I6b148cf88d294200eb8df4d5e068092fe6b5d4b2

Update owners

remove folks not on team anymore, add new folks

Change-Id: Ib5834ecb068b8149b8b8f53373100786d4533ea0