Cert: implement assert function with event buffer

Test: run gd/cert/run_cert.sh
Change-Id: I237ecdee7e70b860f3e73d4825dd79b0235ae22b

Merge "Module: Add TestModule for dependency injection"

Module: Add TestModule for dependency injection

Test: HciLayerTest
Change-Id: I32dbfaa051be5cb42adb7a081b37966479df5268

Cert: add common helper for event stream on the py side

* add common helper for event stream on the py side, implement
  assert_event_occurs in the helper
* add event stream helpers for hci event, sco and acl in GdDevice
* add take_for() function in BlockingQueue for take data with timeout

Test: run gd/cert/run_cert.sh
Change-Id: Ia8168159ee47441ec332046627124e0ed3811d6b

Make SnoopLogger a Module

* Add API SnoopLogger::SetFilePath to set output file path
* Add return code check for GdDevice backing process and rootcanal

Test: cert/run_cert.sh
Change-Id: Iac33070d4ac880d50a0dd78ad75f7ac7ccc239b2

Rootcanal: Exit local loopback mode on HciReset

In Bluetooth code spec v5.1 Vol 2 Part E 7.6.2, it states "the commands
HCI_Reset and HCI_Write_Loopback_Mode can be used to exit Local Loopback
mode".

In HAL cert test, enter local loopback mode, reset, and check whether
local loopback mode.

Test: cert/run_cert.sh
Bug: 131625008
Change-Id: I79546b202affe8ccd48b10e230cf73465324b409

Rootcanal HCI HAL: Set read from socket blocking

In HciSocketDevice::SendHci, rootcanal first writes to HCI socket one
byte HCI packet type first, then HCI packet body. When the stack
receives the first byte, it tries to read the entire packet, but
sometimes the rest of packet was not delivered yet, so the stack will
crash. Instead, the stack should be blocked until all bytes are read,
with a 3 second timeout when rootcanal can't deliver all bytes to stack.

Test: try cert/run_cert.sh multiple times
Bug: 131625008
Change-Id: I99df53c658717a5c931ebc075eb75df447e10332

Merge changes I471a49f6,If5879a76

* changes:
  Clean up facade naming and paths
  Add enforcement of module dependency declaration

Merge "Create handlers for every module upon module start."

Merge "PDL: Check sizes before validating constraints"

Clean up facade naming and paths

Remove "cert" from build rules that apply to stack under test

hal/facade/* -> hal/
api.proto -> facade.proto

In proto -> [x]Facade
Module -> [x]FacadeModule
Grpc service -> [x]FacadeService

A few straggling references to HciCmd and HciEvt

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh

Change-Id: I471a49f65a216d6669b3cb7f5ffebc7c069e2d6d

Create handlers for every module upon module start.

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh
Change-Id: I360556ef3bb2ccf34d5658f00b30cc33949a77c1

Add enforcement of module dependency declaration

Move dependency access into module, enforce dependency was listed
in ListDependencies.

Remove ModuleRegistry from Start/Stop.

Test: atest --host bluetooth_test_gd and cert/run_cert.sh
Change-Id: If5879a76aae764286f1b5bd7df2b497ea8f51998

Gracefully shutdown SUT in cert

* After each test suite, shutdown the stack before closing rootcanal
* Don't stop gRPC server twice
* Crash the stack when can't build gRPC server

Test: cert/run_cert.sh
Change-Id: Ie4c8d749079e99b46d63fb70d157db26d9aa2474

HAL: Add facade for fetching incoming HCI packets

Adding the event stream primitives. General event stream request is
defined in facade/common.proto. "grpc/grpc_event_stream.h" provides
helper class templates GrpcEventStreamCallback and GrpcEventStream to
help user handle EventStream request without writing boilerplate code.

Test: cert/run_cert.sh
Change-Id: I00b51fc7b1faefb7c97cc647876f41854872a415

Fix BlockingQueue bug when using vector

return value instead of reference when take

Test: run unittest
Change-Id: I268f5022306353493a348a52ed363cf60a125661

Remove redundant Bluetooth from internal class names

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh
Change-Id: I55b2c1ecc1ec254a197d6fde70968a9aa8c06c00

Add a persistent key to overwrite MIN_CE_LEN for Hearing Aids

To help in debugging the noise issues on Hearing Aids, add a new
persistent key, persist.bluetooth.hearingaidmincelen, to overwrite the
MIN_CE_LEN parameter that are used for the Connection Parameter Update
operations.

Bug: 130916579
Test: Manual testing
Change-Id: I1c52b2f99235f8d88e0a1d278dbf3174f1c60709

Merge changes Ibcb21966,I026b9e89

* changes:
  Migrate GD code to module
  Add basic concept of module and module registry + dependency graph

Migrate GD code to module

Test: atest --host bluetooth_test_gd
Change-Id: Ibcb219666ff5552efc602bf063e69a6e6b7e4bd8

Add basic concept of module and module registry + dependency graph

Modules have protected lifecycle functions.
Modules are identified by a factory object that knows how to construct them.
Modules can indicate which dependencies they have.

Dependencies are started before their dependent modules.
Don't support incremental module stop for the moment - it's all or nothing.

Test: atest --host bluetooth_test_gd:ModuleTest
Change-Id: I026b9e893501506a6cea8387941b3a9424a9bec9

Implement blocking queue

Test: run unittest
Change-Id: I93c59a2d8c9bbe9bee762412b368773c4b35456f

PDL: Check sizes before validating constraints

Test: bluetooth_packet_parser_test  --gtest_filter=*WayTooSmall*
Change-Id: Ia43c066c11b83f1e9b651ffdd85c6e0196c45332

Add workaround for "Service Changed" indications breaking server-only remotes

Test: Connect with ruwido remote when magictether service is registered
Bug: 122994857
Change-Id: Ibb2c1272bb08c7a127c8f7d37ee6a05f15618fc2

Basic integration of GD with ACTS

* Add GD base test, which can parse config and bring up rootcanal.
* Add GD device controller, which is only optimized for host at this point.
* Add host config for running tests,
* Add basic hal test to test end to end flows.

Missing from this patch:
* GD cert device controller (should be similar to GD device)

Test: after setting up acts (gd/cert/set_up_acts.sh): gd/cert/run_cert.sh

Change-Id: Ibde7c7b0fe85b64643ed11fffe833e9ff53ed48c

Merge "Revert "Revert "Cert for HAL adapter"""

Revert "Revert "Cert for HAL adapter""

This reverts commit 865d9d138f1099a0236847e39329ee641c62b1ae.

Reason for revert: Re-landing CL I3cf2f54605ce2709d1c149b1221dedc8dc5dca92

Change-Id: I63d60ab99d3a4a1b369f77d6c48eff0f463506e4

Simple btsnoop logger implementation

Test: run unit test and check btsnoop log
Change-Id: Ib5ea3fc6440c48e1a7b5d5f3bcc3a0b1c23ac9c2

Merge "Revert "Cert for HAL adapter""

Merge "HearingAid: add addToWhiteList NativeInterface API"

Revert "Cert for HAL adapter"

This reverts commit 7b003eb97a032ae16fb5e793bd641635319724c4.

Reason for revert: aosp_x86_64-eng build error

Change-Id: I6e5484f960209b8486806ab21b8d694437e3c10f

HearingAid: add addToWhiteList NativeInterface API

Add a new native interface API, addToWhiteList, that adds a LE device to
the whitelist.

Bug: 129452236
Test: Manual testing with Hearing Aids
Change-Id: I57c327380f07243e484c5c3afedfe60404f883fa

Merge "Cert for HAL adapter"

Merge "PDL: Use underscores for field names"

Cert for HAL adapter

Introducing cert framework and a simple test case for HCI HAL
transportation adapter

Test-prerequisite: gRPC python. Install from pip
(https://grpc.io/blog/installation)

Test: 1. start HCI rootcanal or HIDL transport
      $ANDROID_HOST_OUT/nativetest64/root-canal/root-canal
      2. start stack with facade
      $ANDROID_HOST_OUT/bin/stack_with_facade
      3. run a simple test case
      $ANDROID_HOST_OUT/nativetest64/bluetooth_cert_test/bluetooth_cert_test

Or: One-liner: python cert/run_cert.py

Change-Id: I3cf2f54605ce2709d1c149b1221dedc8dc5dca92

PDL: Use underscores for field names

field_name : FieldType,

Test: bluetooth_packet_parser_test
Change-Id: I3b6d673111c7c4d7b5814563b132a3b48d3fdfcf

Root-canal: Add missing payload size field to loopback evt

Bug: 131168063
Test: 1. Download AOSP CL929617 and 948661
      2. mma. Run python cert/run_cert.py
      3. Check btsnoop log

Change-Id: I1e31a1323f4aa8cb549af7545029b9d21eaabe9c

Merge "HFP: Fix null pointer dereference in bta_ag_codec_negotiate()."

Call gatt_disconnect if the LE connection is not established and no APP use it.

When the LE connection hold APP is removed, if the LE connection is not
established and no APP use it, we never free transport control block
(p_tcb) or close it. For BLE device that have not established and no APP
use it, will call gatt_disconnect to free p_tcb and go to close state.

Bug: 127908699
Test: GattConnectTest
Change-Id: Ifed10c67141cb4c9221478db759c5614714fff30

Merge "Use GATT_WRITE_NO_RSP for "state change" updates"

Merge "HCI: Use fixed-width types"

Merge changes I321e12bf,If7e6e586,Ie145d1dd

* changes:
  l2cap: Add FCS support
  packet/parser: Add tests for payload size modifier
  packet/parser: Add Checksum support

Use GATT_WRITE_NO_RSP for "state change" updates

For Hearing Aid, replaces the GATT_WRITE with GATT_WRITE_NO_RSP for the
"state change" updates that are send to the devices. Using no response
write is faster and these informational updates does not need a
confirmation from the devices.

Bug: 130332748
Test: 1.run unit test
2.Forget/Repair HearingAid device from Bluetooth UI.
3.Disconnect/Reconnect HearingAid device from Bluetooth UI.
4.HearingAid device power off/on, check them can reconnect.
5.Multiple click connect/disconnect from Bluetooth UI within 1 sec,
check them can reconnect.
6.One side HearingAid device power off/on when playing music, check it
can reconnect and music is work fine.

Change-Id: I55e9b4b0b8c76412e5810e1cec5d96819527d099

Merge "rootcanal: Check return values from callbacks"

HCI: Use fixed-width types

Test: run bluetooth_test_gd
Change-Id: I9a9ef272d92635d7d56b8515a1889b7922ea3b87

l2cap: Add FCS support

Test: bluetooth_gd_test --gtest_filter=*Fcs*
Change-Id: I321e12bfb3ec8b5652d26d2b2e5066dc6a80895c

packet/parser: Add tests for payload size modifier

Test: bluetooth_packet_parser_test --gtest_filter=*SizeMod*
Change-Id: If7e6e586477e6580adc540127f82500642658aea

packet/parser: Add Checksum support

Make TypeDef represent Enum, Custom, and Checksum types.
Add SimpleSum as a test checksum.

Test: bluetooth_test_gd --gtest_filter=*SimpleSum*
Change-Id: Ie145d1dd17a710b50e98de8714102945b3c65c99

BitInserter: Add byte observer for checksums

Test: bluetooth_test_gd --gtest_filter=*BitInserter*
Change-Id: I9ef5bc5951244279fc142127a5c3c9a54c4eaf5c

rootcanal: Check return values from callbacks

Bug: 130624816
Test: kill the Bluetooth stack
Change-Id: Ie1d0e45f5847b4b4c6e6ee86061fa1092156d5db

HFP: Fix null pointer dereference in bta_ag_codec_negotiate().

BTM_ReadRemoteFeatures function will return NULL if ACL link to the remote
device is disconnected. Need to check whether p_rem_feat is nullptr
before dereferencing it.

Bug: 129736080
Test: make, force BTM_ReadRemoteFeatures() to return null and perform
MT/MO call tests.

Change-Id: I28d90823340d8721ee2349d3f55f07d1e823fee3

packet: Support fixed-width custom types

Test: bluetooth_packet_parser_test --gtest_filter=*testChildWithAddress*
Change-Id: I73ff7962d99f668b4cbf1b4068bf74d069b9f599

Send connected callback immediately when attempting to connect to already connected device

Instead of adding the device to the connection manager, just send the
successfull connection callback.

This patch is restoring behaviour from before Connection Manager
refactor. During the refactor this behaviour was broken.

Bug: 130273570
Test: sl4a GattConnectTest
Change-Id: Ic4fa7089a8262f6f792206496a7cfb9c83a16eb2

Merge "Mark these yacc files as creating headers"

Merge "Implement Queue"

Merge "Change storage metadata type to byte array"

RootCanal: Open network ports only if there is a console

Bug: 128355308
Test: toggle airplane mode and check for opening sockets
Change-Id: I8c4581b708c05a8c72dba3604c02adea97c1653b

Mark these yacc files as creating headers

So that they're properly represented in the build graph.

Test: treehugger
Change-Id: If7141171117260b870dc6a0c6608dcae696b3189

Implement Queue

* Implement Queue which provides a flow control mechanism for data
  transmission between modules. This queue streaming data with
  EnqueueCallback and DequeueCallback based on the reactor pattern.
  Enqueue end should register callback when data is ready to be sent
  and unregister when no data ready to send. Dequeue end should
  register callback when ready to handle data and unregister when not
  ready to handle data.

* Implement ReactiveSemaphore, a wrapper for an event_fd work in
  non-blocking and Semaphore mode used by Queue to handle readable
  signal in the reactor pattern.

Test: run bluetooth_test_gd
Change-Id: Ia7019cdbe271d193c92f1a0b405ecced41a2d84b

Simple stack bring up

Just bring up the HAL transportation layer

Test: unit test
Change-Id: I94641d3008ce155d13df6ffe78f3710b3efca645

HCI HAL adapter: separate initialization and set_callback

Test: run bluetooth_test_gd

Change-Id: I1275be3020e34c56e0ef439e0f748db127d9f8e2

Add .gitignore

Test: compile
Change-Id: I0e4dd26c07ba5ed5e6ea4acf0fa9c9f831cba534

Merge "packet: Enforce View inheritance and validation"

Merge "PDL: SMP packet definitions"

packet: Enforce View inheritance and validation

Add was_validated_ flag to Views that can be constructed from bytes.
Check was_validated_ in every generated Get* function.

Test: bluetooth_packet_parser_test --gtest_filter=*Validate*Death*
Change-Id: I1de8d4e5e60bdd7b51562ffd258300e808db4adb

Merge "Remove Ajay from owners"

Remove Ajay from owners

Moved to a different team.

Test: none
Change-Id: I2df8efa1dd59a98ae5c0d9bcd801955e5c85f8d7

Bluetooth HCI HAL adapter for Android HIDL

Test: unit test
Change-Id: I60339d3f0845437e64705edc744e6091a2c3a171

Merge "Switch to clang-r353983b."

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

PDL: SMP packet definitions

Change-Id: Ia6e2da10a959e9476c4f3f32398ac5e815996669

Merge "hci: CreateConnectionRoleSwitch is 8 bits"

hci: CreateConnectionRoleSwitch is 8 bits

Test: Manual comparison with the spec
      Vol 2 Part E 7.1.5 Command Parameter Allow_Role_Switch
Change-Id: Iaa7df5c3c946e3740115df79f14bfae0ddf872b6

DO NOT MERGE -- Merge PPRL.190405.003 into master

Bug: 130175407
Change-Id: I643bb187b1555598121d8b4a15b0acd73fe1f741

Merge "packet/parser: Add pragma once to generated files"

Merge "no forward_list in hci/acl_builder_test.cc"

no forward_list in hci/acl_builder_test.cc

Test: bluetooth_test_gd --gtest_filter="*Acl*"
Change-Id: I6a764ec276102df59897102d6380110b5dd266ce

packet/parser: Add pragma once to generated files

Test: bluetooth_packet_parser_test
Change-Id: Ie0a2717cdbf74859fd457ee68533caded16a5d20

Adapter layer for Bluetooth HCI HAL

Add adapter layer for HIDL HAL and host rootcanal HAL

Test: with cert and rootcanal
Change-Id: I1a84076d3df3176b75a499ac160e503e51fad8b3

Snap for 5310204 from 817facf4bc2a956fafa8a38c15ab5963f2d0bc0e to pi-platform-release

Change-Id: I704240f3f6bd8da8e23865ff7a6eb85b2233bcb1

Switch to clang-r353983b.

This is the current toolchain version being used in
build/soong/cc/config/global.go, so it should be used here as well.
Ideally, this python file would be updated to pick up the value directly
from the build configuration.

Test: N/A
Change-Id: I1c615a0d0c6dc4dedd82d1ced0381bab273f9e61

Modernise code to use override specifier

Generated by clang-tidy.

Test: m checkbuild
Change-Id: Icba58009f6db2a0e9e1360326b0b66fc10d04a5a

Merge changes Ica38a448,I526fb846

* changes:
  Add namespace in bluetooth_packetgen
  Add multi file support for bluetooth_packetgen

Merge "Add packet parser tests to TEST_MAPPING"

Merge "Fix handling of startup_timer_expired"

Add namespace in bluetooth_packetgen

::bluetooth is our top level namespace. Each subdirectory is a
namespace.

Test: bluetooth_packet_parser_test
Change-Id: Ica38a448c692396e6a34dd66eabc3eaa8df884e0

Add multi file support for bluetooth_packetgen

Test: bluetooth_packet_parser_test
Change-Id: I526fb8462f67f2ce099aab4e63ca695797f36723

Add packet parser tests to TEST_MAPPING

Test: atest bluetooth_packet_parser_test
Change-Id: If82370a834825cd265fac3c7e100176b94c79bb0

PDL: L2CAP packet definition

Test: Comparison with the SPEC
      bluetooth_test_gd --gtest_filter="*L2cap*"
Change-Id: If29036fa7147b27d5a79458773c5eb772eb15817

PDL: HCI packet definition

Test: Comparison with the SPEC
      bluetooth_test_gd --gtest_filter="*Acl*"
Change-Id: I48fcd2655edb54d831f8b395de9a209c46049189

packet: Add the PDL Packet Generator

Test: bluetooth_packet_parser_test
Change-Id: Ia741e608aebff7b55d005a37e0f0eaeae63767a8

Fix memory leak of reactor and reactor_unittest

* Fix memory leak happen in below two cases

Case 1

1. Reactable1 register
2. Reactable2 register
3. Reactable2 unregister during the callback event of Reactable2
4. Reactable1 unregister

In step 3, reactable_removed_ will set to true due to unregister while
reactable->is_executing_ is true, it makes sure we delete Reactable2
after the callback is executed. But it will cause Reactable1 to not be
deleted in step 4.

To avoid this, we can reset reactable_removed_ to false after Reactable
deleted.

Case 2

1. Reactable1 register
2. Reactable2 register
3. Reactable2 unregister during the callback event of Reactable2
4. Reactable1 unregister from different thread during step 3 processing

In step 3, although we reset reactable_removed_ to false after Reactable2
deleted immediately for case 1, if other thread unregister Reactable1
before Reactable2 deleted, Reactable1 will fail to be deleted.

To avoid thie, we add a local variable to check if the Reacable is
executing. If not, deleted the Reactable directly.

* Add unittest for these two cases
* Fix memory leak in reactor_unittest
* Turn on cfi and address flag

Test: sudo ./bluetooth_test_gd
Change-Id: I0a0ca79b439fd3a1bf3ec0fa2b2a43a88e037fbb

PacketBuilderTest: Modify member array

Test: sudo ./bluetooth_test_gd
Change-Id: Ibf015899c9430a25d7ae64f5bcc82a13e8f80871

Merge "Handle Bluetooth HAL service died"

Handle Bluetooth HAL service died

Bug: 129320921
Test: kill Bluetooth HAL
Change-Id: I83c897f5a9def52b2050a3e73d97f99b1b71aa3b

Fix handling of startup_timer_expired

1. The startup_timer needs to be less than
3 sec to allow the wakelock to be held by
the stack and needs to be less than the
BLE_START_TIMEOUT of framework.
2. On expiry of startup_timer, just abort
as there is no proper recovery steps here.

Test: "adb shell cat dev/ttyHS0" when Bluetooth is OFF
      Turn on Bluetooth
Bug: 129394377
Change-Id: I1a397bab5c88fe7ef8ccbc369897903ee5d0a36a
Merged-In: I1a397bab5c88fe7ef8ccbc369897903ee5d0a36a

Merge "RootCanal: Add beacons by default"

Merge "packet: Add Packet classes and tests"

packet: Add Packet classes and tests

Test: atest bluetooth_test_gd
Change-Id: I8e124d5123cff3354f5c451b3806904a2d60b38d

Add service change handle for Hearing Aids

When receive service changed indication from Hearing Aid (which indicates
Hearing Aid service changed), it will refresh the hearing aid attribute
handle to ensure the attribute handle is correct.

Bug: 122008481
Test: 1.run unit test
2.After Pair old version FW HearingAid, Disconnect/Reconnect new version
FW HearingAId, then check whether HearingAid is working fine.
3.After Pair old version FW HearingAid, Bluetooth off/on to reconnect
new version FW HearingAId, then check whether HearingAid is working
fine.
Change-Id: I48eae10a3016429f35f4f904752be93bb419d515