Revert "OS: Construct Handler using Reactor"

Reactors are an implementation-specific detail.

This reverts commit 0a2f52e692bb65cea7ecff7ce1b1e839b786d5c7.
Test: bluetooth_gd_test

Change-Id: If74a9e0f32671ba96a723e51eee495f2aebf1e7c

Merge "BLE: [IOT] Initiate disconnection when encryption fails during pairing"

Merge "GD SMP ECC: Add const qualifier to ECC_PointMult"

GD SMP: use fixed size arrays in packets

Test: compilation
Bug: 126463795
Change-Id: Iad1fd5004efab67294c6926475625e4c4b35a38c

PDL: fixed size type arrays

Currently attempt to add fixed size array is causing segfault. This
patch fixes that. It also adds a test to verify those arrays are
properly encoded and decoded.

Test: bluetooth_packet_parser_test
Change-Id: I09c7e5595a7d1819a43e9203846bb487fb23aca3

HCI Layer: Register CommandComplete and CommandStatus on Start()

Module.GetHandler() works only after a Module is started. Move
RegisterEventHandler() for CommandComplete and CommandStatus to Start().

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

Merge "PDL: Add Arrays"

Merge "Make sure the Hearing Aid source is ready before resuming or suspending calls"

PDL: Add Arrays

Test: bluetooth_packet_parser_test
Change-Id: If7f057a16ab40066a53d2c66980c5579a480650f

Open connection if it doesn't exsist in bta_gattc_process_indicate

If Service Changed indication is received before the connection is open
, send the BTA_GATTC_INT_CONN_EVT to open it before processing the
indication.

Bug: 131814072
Test: CTS Bluetooth LE Secure Test. Unit test.

Change-Id: I1b3415ef6aafb52a6ac229409514a85f35de5842

Merge "OS: Hold the mutex when decreasing the semaphore"

Merge "OS: Protect invalidation_list_ with the mutex"

Make sure the Hearing Aid source is ready before resuming or suspending calls

Those calls to resume / suspend streaming are from the audio's IPC
threads like HwBinder or UIPC, and calls to start / end session are from
the stack main thread. There is timing that those threads may run in
parallel, and trys to change the streaming state when the Hearing Aid
source is not ready. This change checks the source state before resuming
or suspending streaming, and breaks the deadlock between IPC threads and
the stack main thread.

Bug: 132860565
Test: 1. Connected to a Hearing Aids and disable the Bluetooth
      2. Switch the active device manually

Change-Id: I6ae11faab3d530c65f55ffa9b8a9f3cae2bde5f4

GD SMP ECC: Add const qualifier to ECC_PointMult

Second parameter passed to ECC_PointMult is either remote device public
key, or the constant from the BT spec. Both of them should never be modified.

Assigning the "z" value inside ECC_PointMult had no effect on the rest
of computation.

Test: SmpEccValidationTest
Bug: 126463795
Change-Id: I0530d26e0e8841cb8b263ea948ae7a19850f0670

Merge changes I42c47d98,If38ad50d

* changes:
  Fix mac build
  bpfmt -w $(find . -name Android.bp)

OS: Hold the mutex when decreasing the semaphore

Test: bluetooth_test_gd # Run in a loop
Change-Id: I72e9f28c417986f63390cbbbe6467f64df6e9237

Merge "Add missing header in gd/packet/parser/checksum_type_checker.h"

OS: Protect invalidation_list_ with the mutex

Test: AclManagerTest.startup_teardown
Change-Id: I5a9467691ffd1bb6bc2bb9d1080dce5f5bc655fa

Add missing header in gd/packet/parser/checksum_type_checker.h

Test: compilation
Change-Id: I7a72cba471db5e59c8a6fa2ee1ba03cc258719ef

GD ECC: add const qualifiers

Test: SmpEccValidationTest
Bug: 126463795
Change-Id: I2e8b6027f959a181655286fb061bbb30b91e57f8

Merge "Disable absolute volume control for AirPods 2"

GD SMP ECC improvements part 1

* make curve parameters into constants
* separate multipoint from BT specific ECC curve

Test: SmpEccValidationTest
Bug: 126463795
Change-Id: If6401c3038b340926310eafb40c2b7c5dc40dba9

Merge "Eliptic Curve Cryptography for GD"

Merge changes I610e6f4f,Ib7dc12d8

* changes:
  OS: Construct Handler using Reactor
  Queue: Store callbacks in the runnable

Merge "Turn off the new pass manager for coverage builds"

Eliptic Curve Cryptography for GD

This patch moves the code and adds the necessary namespace.
Further improvements in upcoming patches.

Test: SmpEccValidationTest
Bug: 126463795
Change-Id: I2682ff96aa904dd89ca6b5f2487fe72047211374

OS: Construct Handler using Reactor

Store a Reactor instead of a Thread in Handler.  Allow construction from either.
Test: bluetooth_test_gd --gtest_filter=QueueTestSingleThread.no_unregister_enqueue_death_test

Change-Id: I610e6f4ffb48120d1fa47b4ff59b59375c7a11d8

Queue: Store callbacks in the runnable

Test: bluetooth_test_gd \
  --gtest_filter=QueueTest.pass_smart_pointer_and_unregister
Change-Id: Ib7dc12d8e767125e51d83810ade77c5187a4c174

Fix mac build

system/bt/gd is compiled for the host but does not work on mac,
disable it.

Fixes: 133439713
Test: mmma system/bt/gd
Change-Id: I42c47d987ebfa8bba498a438d742388d216cedd2

bpfmt -w $(find . -name Android.bp)

Test: none
Change-Id: If38ad50d848c9e40f07c6598272a354312d9efea

Disable absolute volume control for AirPods 2

Volume of AirPods 2 start out extremely loud when absolute volume
enabled. Interim response from remote show normal when register volume
changed notification. Disable absolute volume control for AirPods 2 to
avoid volume start out extremely loud.

Bug: 129602489
Test: Play music with AirPods 2
Change-Id: I555576be1929a42ae8e01f661572a89d1993dee9

Fix CryptoToolboxTest.bt_spec_test_d_1_test

When moving code, ElementsAreArray matcher was replaced by memcmp to
remove dependency, but "== 0" is missing to actually make sure elements
are same.

Bug: 132863815
Test: CryptoToolboxTest.bt_spec_test_d_1_test
Change-Id: I4069ee3baea9de095b23072820cef3bd0673608a

Turn off the new pass manager for coverage builds

The new pass manager crashes if `-fprofile-instr-generat` cflag is set
on the current stable Android compiler. Turn it off until we roll past
the revision with the fix.

Test: build
Bug: 131855431
Bug: 131132095
Change-Id: I92f36b73747cca898e104118a2721a4796511282

Get rid of KEY_LENGTH_DWORDS_P192

Only KEY_LENGTH_DWORDS_P256 is used.

Test: compilation
Bug: 126463795
Change-Id: I32a80e4b012c4606a70994fa9d0b5c2602707bb3

Merge "PDL: Strengthen CamelCase checks"

PDL: Strengthen CamelCase checks

Test: bluetooth_packet_parser_test
Change-Id: I1f85a0435aeab79cee8b938061c3961b6a21c49b

crypto_toolbox

This is copy of the crypto_toolbox for GD usage.
Only necessary changes were made:
* removed dependency on bt_types.h
* removed dependency on libbase
* reformatted code
* added namespace

Further improvements will be stacked on top of this patch.

Test: CryptoToolboxTest
Bug: 132863815
Change-Id: I639ba42705b23218a14085abcf431d744f87b913

Merge "Fix use-after-free in ReactorPerformanceTest::TearDown."

Fix use-after-free in ReactorPerformanceTest::TearDown.

Bug: 128921871
Test: net_test_performance
Change-Id: I7b7fda7c437f75019564a0b56764d40704b0b33f

Merge "QueueTest: UnregisterDequeue before promise set value"

Merge "Hearing Aid Dump Audio Tool: Add dump debug buffer feature"

Merge "Remove Address dependency from PacketParserTest"

Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start the 2nd AVDT stream"

Remove Address dependency from PacketParserTest

Test: bluetooth_packet_parser_test
Change-Id: I769e0ea2feacbc8c14598762f01746c76efe1079

Hearing Aid Dump Audio Tool: Add dump debug buffer feature

Add dump HearingAid debug buffer feature to let the HearingAid debug
information dump to debug file and make it readable.

Bug: 130781854
Test: ./dump_hearingaid_audio.py -d true btsnoop_hci.log
      ./dump_hearingaid_audio.py -sd true btsnoop_hci.log

Change-Id: I150fd897e6f197d5b853554734926ad613de4e6b

Import ::testing::DoAll

It is used in this file but didn't have corresponding using statement.
It fails compile in new googletest.

Test: build
Change-Id: I1ac9d04c1bc8c46b57ecfb7859ee0e3c31bbb092

QueueTest: UnregisterDequeue before promise set value

For some tests, ~Queue will() be trigger right after we get promise
we should UnregisterDequeue before promise set value to prevent
race condition

Test: atest bluetooth_test_gd
Change-Id: I9409b6829cce61ef95c766a1d3b3679197b8f9f4

Merge changes I2526f4bc,Iafe4a46d

* changes:
  Allow Handler synchronization in test code
  Remove testing code from ModuleRegistry

Merge "crypto_toolbox: add const qualifiers to f4, f5, and f6 parameters"

Merge "Add SMP PDL definition."

Add SMP PDL definition.

Bug: 132863815
Test: N/A
Change-Id: I5d762432be133a264db1f9b9de74acad2c60b429

crypto_toolbox: add const qualifiers to f4, f5, and f6 parameters

Bug: 132863815
Change-Id: I58de98b9087c81a27054eb8302212a1bf0595093

Make AES control block into a thread local variable

When computing ACES-CMAC, we use static control block that's shared
between multiple threads. That means that having computation done in
different threads can interfere and cause bad value to be returned.
This patch fixes that by making the control block into thread local
variable.

Bug: 132863815
Change-Id: Ibe2aaac54b93f77d644564d23694846c0c078bd6

Merge "DO NOT MERGE - Skip pi-platform-release (PPRL.190505.001) in stage-aosp-master" into stage-aosp-master

Merge "L2CAP: Always use SUCCESS"
am: f256ea9c02

Change-Id: If78bace42679e10a74b8b3df98feda1970e29450

Merge "L2CAP: Always use SUCCESS"

Require a semicolon after ASSERT
am: 49496a3d06

Change-Id: I15734a820b5a67733babd0aae0263f8f97edc508

Require a semicolon after ASSERT

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

L2CAP: Always use SUCCESS

Test: bluetooth_test_gd --gtest_filter=*L2*
Change-Id: I6070168e2d9055b6d7f54726a773ab646dcb567c

Allow Handler synchronization in test code

SynchronizeModuleHandler() blocks until all previous closures are done.

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

Remove testing code from ModuleRegistry

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

While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start the 2nd AVDT stream

There were 2 cases to start the 2nd stream automatically by the stack:

* When the 2nd AVDT opened, the stack would check the 1st stream state
  to determinate the 2nd should be starting or not.
* While the AVRC timer which was fired after AVDT opened was expired,
  the stack would open the AVRC and start the 2nd stream if the 1st was
  started.

Both of them are unnecessary since all the stream must be controlled by
the upper layer. We currently support an active device only, and the
2nd stream will be tracked as remote triggered and suspended immediately.
To samplize the behavior, it is better to not start the 2nd stream
automatically by the stack.

Bug: 132146974
Test: manual
Change-Id: I75f39801e9779ee55fa574e30051e01966c61ea3

Merge "HCI: Add classic interface definitions"
am: b8b18593c6

Change-Id: If2710b9c0327ac8456984722524d464512abfe81

Merge "HCI: Add classic interface definitions"

HCI: Add classic interface definitions

Test: bluetooth_test_gd --gtest_filter=*Hci*
Change-Id: I4206c818084302341d4e99f2cce8aad1c12ce4ae

DO NOT MERGE - Skip pi-platform-release (PPRL.190505.001) in stage-aosp-master

Bug: 132622481
Change-Id: Iec8c8a55b0f442d3ce16c86872e59c68f61cf36b

[automerger skipped] Merge "DO NOT MERGE - Merge Pie Bonito/Sargo into "
am: 7bd0d9e0c3 -s ours
am skip reason: subject contains skip directive

Change-Id: I0a6eb0daaa03161d899b838f88945cc06bac056f

[automerger skipped] DO NOT MERGE - Merge Pie Bonito/Sargo into 
am: e39adc7f03 -s ours
am skip reason: subject contains skip directive

Change-Id: Ie61608aea918084208a6cc25b5f39c9567ee611e

Merge "DO NOT MERGE - Merge Pie Bonito/Sargo into "

Merge "Add Delay reporting for Hearing Aids"
am: 122ce6871f

Change-Id: If98f6b3faad73841924abdf92b2a4b09ac9411b9

Merge "Add Delay reporting for Hearing Aids"

DO NOT MERGE - Merge Pie Bonito/Sargo into master.

Bug: 131756210
Change-Id: I4009cad536b5c2d7a9f9d02769128dc5077e53b9

HCI: Rename ErrorCodes and consolidate Enable
am: 38ea5acca8

Change-Id: Ifa89f600275e8259c37fbddc02dd1e2f66b9112e

Add Delay reporting for Hearing Aids

Use the RenderDelay parameter from the Hearing Aid devices to implement
the Delay Reporting feature. This will helps in the lip-sync issues.
Please note that this change will requires a re-pair of the Hearing Aids
devices in order for the delay report to work well.

Bug: 132115153
Test: Manual test
Change-Id: If4bef049c18a3912817c2fa96b9bd890d5712c22

HCI: Rename ErrorCodes and consolidate Enable

Test: bluetooth_gd_test --gtest_filter=*HciTest*
Change-Id: Ic957968b5f0de1eac486b4e2ba103ec749819b0e

ModuleRegistry: Start() injected modules
am: 30fa258cf8

Change-Id: I096fdf186f33e9e690e5dd564f4b5ac28b71701b

ModuleRegistry: Start() injected modules

Call Start() on injected modules, and set registry_ and handler_
properly

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

Snap for 5450365 from f8a5ce9945240777b0d5cd2ba68fdda48e1484a8 to pi-platform-release

Change-Id: I93c7dd9dc1084fbe5f2f3cd8075b92fd05013d7e

Merge "Add benchmark for queue"
am: 7ec4793b50

Change-Id: I9586994386bbf51fae402424b66c3fa955c29a76

Merge "Add benchmark for queue"

Merge "Handling of incoming AVRCP Controller connections"
am: e7b536bc93

Change-Id: I7574cd7421edb6572472ffef685b70429f6d8d2a

Merge "Handling of incoming AVRCP Controller connections"

Merge "Close unexpected HearingAid open event if is not attempting to connect"
am: f612b732e1

Change-Id: Id6ca92d830e6e7f10d22260c15f6b47fea3b80fa

Merge "Close unexpected HearingAid open event if is not attempting to connect"

Merge "A2DP: Fix offload start VSC parameters"
am: da0b935bf4

Change-Id: I3425223bc0c0b49f49f04a164c10fdfa2a50bc57

Merge "A2DP: Fix offload start VSC parameters"

Close unexpected HearingAid open event if is not attempting to connect

When HearingAid bta layer receive unexpected HearingAid open event,
should close it if is not attempting to connect.

Bug: 131863877
Test: 1.Forget/Repair HearingAid device from Bluetooth UI.
2.Disconnect/Reconnect HearingAid device from Bluetooth UI.
3.HearingAid device power off/on, check them can reconnect.

Change-Id: I75d63d0441dcc6b7338b1e30453021eb271f4872

Add a signaling socket from facade/cert to tester
am: 663ee01503

Change-Id: I2f2dfb01de8ac8ad132ba7ec9b4e677bdb256277

Add grpc root service for test configuration
am: 30bf869b83

Change-Id: I9226c028d4a40b7aee39b714e19c9d33b94d36ab

Add a signaling socket from facade/cert to tester

The tester needs to wait for grpc server to start up before sending any
RPC. Use a UNIX domain socket to indicate when the grpc server is ready

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

Add benchmark for queue

Test: run bluetooth_benchmark_gd
Change-Id: I074f7a6f0a7927f9bf35f92cbc0bb4f3f10adfc8

Add grpc root service for test configuration

When facade_main and cert_main is started, it only starts grpc root
service, and listens for StartStack() request before brining up required
modules. Then, it listens for StopStack() request before cleaning up
modules. Defining new rootservice.proto for facade and cert.

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

Break GdDevice into GdDevice and GdCertDevice
am: 87ddd5b26f

Change-Id: I9fb0d8bdb0b718a8846de25f5aaa8fbdaa33c1e4

Consolidate facade and cert entry point naming
am: 4999504f61

Change-Id: Ia48e8a6f3b4c6849e2e5cb5c0751e8e156f5c306

Add BidiQueue
am: d5b7ba71ba

Change-Id: I00a6898e46cb7e8ad550944fe63bc0cd3a9105ca

Break GdDevice into GdDevice and GdCertDevice

We need distinct devices, since the gRPC capabilities are
different from each other and we need to tell them apart
from each other in tests.

Also, remove unused gd/cert/run_cert.py
Also, don't indicate rootcanal error at end of test if
return code indicates it was terminated by the SIGINT we sent it.
(-N return code indicates terminated by N signal, as per
https://docs.python.org/2/library/subprocess.html#subprocess.Popen.returncode)

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

Consolidate facade and cert entry point naming

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

Add BidiQueue

An interface for passing a pair of queues (an up and a down),

Also adds BidiQueueEnd, so you can pass the end around without worrying
about your clients using your end of the queue by mistake.

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

Cert stack bring up and test Inquiry with two stacks
am: 90ec7400f0

Change-Id: I85e10a876b52922ca0ce558292b0f54d87752405

Cert stack bring up and test Inquiry with two stacks

* Add main entry point for cert stack.
* Add a test case where cert stack enables page scan and inquiry scan
and DUT starts inquiry. Verify the inquiry result.

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