GKI cleanup - Remove GKI pool related defines

 * Removed pool ID related defines
 * Updated / renamed / cleanup buffer size related defines
 * Added new #define BT_DEFAULT_BUFFER_SIZE
 * Removed unused defines

Change-Id: Ib1fbf463d1e6fda2c71631347de0c365a37ba5ce

GKI cleanup - Eliminate function GKI_get_pool_bufsize()

Change-Id: I87168157b9bb3e9d315c03b98a10d18b38cce3e1

GKI cleanup - Eliminated function GKI_poolcount

Refactored function l2c_link_adjust_chnl_allocation.
Now the buffer quota computation is done without using the
GKI poll counts.

Change-Id: I669492becc9024c12f3360ea58a06188caa6420a

GKI cleanup - Eliminate usage of pool buffers

* Replace usage of function GKI_getpoolbuf() with GKI_getbuf()
* Remove usage of function GKI_poolutilization()
* Remove usage of function GKI_poolfreecount()

Change-Id: Ide938192b878bbfb4912642c903fce548f2b5368

Fix PAN and AV role switch war

Bug: 23740164
Change-Id: Ib26a64b624c711443201adc4fde6b041ecb0dde1

Ensure thread priority is raised for media task / UIPC

Bug: 23978964
Change-Id: I36b6d05bb82eb0283eed3b465038d65890c4c245

Change UHID socket to be non-blocking

On some platforms, the logs indicate that the BTU task is stalled by
UHID driver when writing the HID descriptor to the kernel. This patch
converts the UHID socket to non-blocking to avoid stalling the main
BTU task.

Bug: 23978964
Change-Id: I5a7a3e106fb2c967d68f077faedcd4fe62bbd912

Check BR device count before initiating role switch

A case where DUT is acting as PANU device on BR/EDR link & also
acting as central on LE link. As LE link & PANU link need not to
be master & it's not a scatter-net scenario, because there is only
a BR link. So checking for BR device count before initiating role
switch to avoid continuous loop of role switches.

Cherry-picked from: https://partner-android-review.googlesource.com/#/c/376058/

Change-Id: I70e4041a71ba965f1f9e25729c3cfd20ae03d6b4
Bug: 23290136

GKI cleanup - Eliminated function GKI_delay

Removed function GKI_delay from the GKI module, and replaced
it with a local static function inside file btif/src/btif_rc.c - the
only place it is (still) used.

Change-Id: Id8f3f700efd22d6e31c70aa8b1724ffa9afbc631

Add libpower as a dependency to all device targets that include libosi.

Bug: 23902077

Change-Id: Icc7119a2b3199310e2210d46f599da00261fdff6

Allow BT inquiry while discovery is in progress.

BT Inquiry is disallowed when service discovery
is in progress. Innitially when GATT discovery
is in progress (crosskey pairing), it resulted
in inquiry requests being rejected. During an
insecure connection like OPP, bonding link key
is not stored. In an earlier patch (b820aec7),
BR/EDR discovery was bypassed if the bonding was
temporary. However, this was not done for LE.
Applying the same logic to LE, resolved the
inquiry issue reported in this bug.

Bug: 23468169
Change-Id: I6111c80a420aeb0bc36a0393eb215c112512d251

Eliminate recursive calling when handling GATT related errors

If there are errors when processing GATT related events (e.g.,
configuring the MTU), don't use recursive calls into
bta_gattc_sm_execute(), because it breaks the free-ing of some
of the memory.

Bug: 23756301
Change-Id: I3c685170e868ffbf4e488d2bb5a31904e3f7b39d

Fix issue where bluesleep idle timer is not started

Sometimes allow_device_sleep() is called before wake_assert, this result in transmit_done
does not start idle timer. This is because the "wake_state" does not match while
allow_device_sleep() is called before wake_assert() is called. So start idle timer
anyway when transmit_done is called.

Bug: 23754156
Change-Id: I5437aea0abc4b7f5cf847701e2d0a889c6823fcb

Fix btwrite timer timeout for Angler (2/2)

When calculating the appropriate LPM's idle timeout,
the local chip name was NULL. This is fixed by calculating
LMP's idle timeout when the chip is enabling.

Bug: 23248798
Change-Id: I8ee2b28fb4363359e035e242fdcff8a736ee448c

Implementation of net_test_bluedroid as a GTest.

Change-Id: If9fc9a26fe58858a107b8ad6ac9f1e757ba07f90

LE Multi-Advertising State Change Sub-event Wrong Variable Type

Wrong connection handle type in multi-adv event parsing,
causing random address (confirm value) mismatch and pairing failure.

Bug id: 23201007

Change-Id: I13ce231360937e711f61eb0777805b07bcde7074

Fix incorrect SetConnScanParam function mapping

BTA_DM_API_BLE_CONN_SCAN_PARAM_EVT is defined before
BTA_DM_API_BLE_SCAN_PARAM_EVT in the enum but the corresponding
entries in bta_dm_action were inverted. This resulted in incorrect
invocation of set_conn_scan_params while trying to set parameters
for scanning. The call was hence failing here and was not
resulting in setting of the new scan parameters in the Controller
irrespective of the scan mode selected from the App. This would
essentially result in the breakage of ScanSettings.SCAN_MODE_<xx>
Android SDK APIs.

Bug: 23602042
Change-Id: I17e1b14a32250e3ccb7631a67690ec2e0a6bc321

Enable clang compilation.

Files under system/bt/device do not use atomic operation.
Old setting of LOCAL_CLANG is unnecessary.

Other atomic operations in system/bt/bt/core were changed
and osi/include/atomic.h was removed, in
https://android-review.googlesource.com/#/c/166762

Change-Id: I807e1fd1792e801392f40ab92a8990c826a5785a

Send current RPA instead of pseudo-address on connect.

Bug: 23079777

Change-Id: If6da80392748cdaad2a4833dd31fe299b62c28e3

A2DP audio HAL: implement out_get_presentation_position()

And complete implementation of out_get_render_position()

Bug: 21199150
Change-Id: I95038ad68433e401b096a0159282a2ae372570e1

Load factory Bluetooth address from system property

Implemented a check to grab the ro.boot.btmacaddr property in case the
device fails to receive an address beforehand.

Bug: 22618015
Change-Id: Ie322888e114a732f8e85c81793e3e5e5eacffc7a

Add a SCO routing command to net_hci.

This allows us to change the SCO routing parameters at runtime while
debugging / doing device bringup.

Change-Id: I8bafb7c7fb7ad7470d378cd14a7ee4aad63f3b9b

Fix merge discrepencies and resulting compile errors

Change-Id: I29fbec9f668ee9218b90a222801c9ddcb9352017

service/doc: Fixed some post-review comments

Change-Id: I62e4f26cc55763f5880922bd06373db47329def9

service: Fix broken UUID advertising

The BTIF API expects a single 128-bit UUID in little-endian form to be
passed for its multi-adv service UUID field. The system service never
did this conversion so UUIDs were never properly advertised. This CL
fixes this.

Bug: 24247700
Change-Id: I541921da85bdabd907bfa9f75fbde7a3477ff593

service/client: Add start-adv -u flag

Added the -u flag that allows passing a UUID in string representation
(16-bit, 32-bit, and 128-bit) to be advertised as long as all fields
fit inside the data.

Bug: 24247700

Change-Id: I3e572db31b84f1e010ba19ab5a51d00c563bb3b7

service: Improve creating UUIDs from string

- When UUIDs are initialized from a 128-bit UUID, they are now parsed
  from the canonical UUID string representation.

- Invalid strings now cause the UUID to be marked as invalid, rather than
  causing a crash.

- Input such as "0x180d" is now valid.

- Added the GetShortestRepresentationSize() method to use in determining whether
  a UUID is within Base BT SIG UUID domain.

Bug: 24247700

Change-Id: Id28255ec963bc6da27a8e871598016d41920b688

Add net_admin group capability to bluetoothtbd

The net_admin group capability is needed for accessing the rfkill
device in the kernel.

Change-Id: Ic5c2c2717fbad6428fc75ff00a0a3afcd2c5dec6

service/client: Handle EOF

Added handling for EOF to the REPL so we gracefully exit if
Ctrl+D is pressed.

Change-Id: I706f3f46a7092a30b1e9d83c32805dbefc5180c9

service: Added Binder API documentation

Added the service/doc directory with documentation for the
currently supported Binder interfaces in AIDL format.

Change-Id: I81bec12b7da7fbee110b1df0ffafcc3dc7d6eec3

service: Create a client library for applications

This CL creates the libbluetooth-client static library that external
code can link against to interact with the Bluetooth daemon using
Binder. To this end, all service/ code that is common to the client
library and the daemon itself have been moved to service/common/bluetooth
and service/common is exported in LOCAL_EXPORT_C_INCLUDE_DIRS.

bluetooth-cli and bt-example-hr-server have been update to demonstrate
how an app should use this library (using library includes).

Bug: 23169160
Change-Id: Ie6841bd6c48846eecec824ff1e80a9971385d1fc

Allow parsing of UUID and service data for advertising

Added code so that bluetooth advertisement packets will now
parse UUID data and service data. Also added unit tests to
confirm data is being parsed correctly. Also added constants for
the type fields for the different service UUID and service data
types.

Bug: 24247700

Change-Id: I53f5b639adb30e781ef155c8d55004547a26672d

service: Enable all tests for target

Made some changes to the test structure following the instructions
at http://go/brillo/brillo-team/testing:

  - Renamed the existing host and target tests to bluetoothtbd-host_test
    and bluetoothtbd_test respectively.
  - The target suite now includes ALL daemon unit tests + binder tests.
  - Added custom test/main.cpp to set up logging and command-line. Now
    we can run tests with the -v flag to increase log verbosity level.

Bug: 24708686
Change-Id: Ia4ce5d7028ce6cb0efd4a1c51ffebdf3bbe6921e

Merge "brillo: run bluetoothtbd with correct permissions"

brillo: run bluetoothtbd with correct permissions

Adding net_bt_admin group grabs more correct permissions.

Change-Id: I3764e6d242fd79bb78cda620f4ec65f0b732aedd

am 2846af52: service: Add missing build includes

* commit '2846af5215215c12ec162b76075769f395e41434':
  service: Add missing build includes

service: Add missing build includes

Added missing files to service/Android.mk to fix a build breakage on
master.

Change-Id: Ic6a3e3511fa25cbc8f2c546e46a91f8665d2f654

am 1c162dc3: service/example: Complete HR simulation

* commit '1c162dc31bad03ee21a93e5afb9c7a572bf21ec7':
  service/example: Complete HR simulation

service/example: Complete HR simulation

Added the remaining logic for an example Heart Rate server app.
Also added more annotations throughout the code.

Bug: 24245347
Change-Id: I46184d2585eeb7683efdc5ea7122d0472dd7f05b

am cd644e3c: service: Support GattServer notifications/indications

* commit 'cd644e3c54a4321b7626b52531c59f88e24e87be':
  service: Support GattServer notifications/indications

am 4ebcbd9e: service: Support GATT server async write requests.

* commit '4ebcbd9ede4006500c3ead0685befe0a8e4aadc2':
  service: Support GATT server async write requests.

service: Support GattServer notifications/indications

Added outgoing notification/indication support to
bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I2d5cea90137e5d771f969163591b50aee1ad171c

am 514bf608: service: Support GATT server async read requests.

* commit '514bf6087093375351784b287cb29c5f4603273c':
  service: Support GATT server async read requests.

am b10f96fa: service: Add HAL wrappers for read_request_cb and connection_cb

* commit 'b10f96fa25266b69d0d31af166e93afe14134be7':
  service: Add HAL wrappers for read_request_cb and connection_cb

service: Support GATT server async write requests.

Added support for incoming characteristic/descriptor read requests
to bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I18202c3255f0fb8fc5338b1139c375e1439b6568

service: Support GATT server async read requests.

Added support for incoming characteristic/descriptor read requests
to bluetooth::GattServer and implemented the related Binder API.

Bug: 24245347
Change-Id: I88161c5bd3ca9ba5604c00aa4e569b3baf19137c

am c61ef610: service: Add helper to util/address_helper

* commit 'c61ef610feb8858db4917ac7f3d0e847f14c80f8':
  service: Add helper to util/address_helper

am e0fe3875: service: Added GATT server Heart Rate example

* commit 'e0fe387561141e049bb2fc748cacb0b56af4928f':
  service: Added GATT server Heart Rate example

service: Add HAL wrappers for read_request_cb and connection_cb

Added HAL wrappers for GATT server callbacks for incoming read
requests, GATT connection state, and the send_response function.

Bug: 24245347
Change-Id: I8605fc6548294f9e09416ca8474cf13619463c40

am d9b0552a: service: Implement IBluetoothGattServer.addDescriptor

* commit 'd9b0552a7f75737b5c8c60cb87f892b1b9323acf':
  service: Implement IBluetoothGattServer.addDescriptor

am 91613743: service: Implement IBluetoothGattServer.addCharacteristic

* commit '91613743c9db02dbe05d93d2afa163b78068d054':
  service: Implement IBluetoothGattServer.addCharacteristic

service: Add helper to util/address_helper

Added the BdAddrFromString function which populates a bt_bdaddr_t
from a string.

Bug: 24245347
Change-Id: I75f798956a796ba6a972fc31e336a1aff86af148

service: Added GATT server Heart Rate example

Added a simple example Heart Rate Service application that
demonstrates how to use the GATT server Binder APIs. The example
currently cannot respond to read/write requests or send
notifications as they are not supported in the API yet.

Bug: 24245347
Change-Id: Id10186a566559c9667adbd99b0484f2c90a422a0

service: Implement IBluetoothGattServer.addDescriptor

Declared and implemented the IBluetoothGattServer.addDescriptor
API.

Bug: 24245347
Change-Id: I7d1e992321c260b9d77521b1c8d7a1e7737b35cc

service: Implement IBluetoothGattServer.addCharacteristic

Declared and implemented the IBluetoothGattServer.addCharacteristic
Binder API.

Bug: 24245347
Change-Id: I8baaed8d6dbf8ac2209b8fd0c918cade78dcba64

am 3bdeebe1: service: Add IBluetoothGattServer service APIs

* commit '3bdeebe14fa2285aaf1bb2eee0a5734538d9048b':
  service: Add IBluetoothGattServer service APIs

service: Add IBluetoothGattServer service APIs

Declared and implemented the IBluetoothGattServer
beginServiceDeclaration/endServiceDeclaration APIs and the
IBluetoothGattServerCallback onServiceAdded API.

Bug: 24245347
Change-Id: I586660b586e125263f71d242f42b6e7bdec169c2

am ae43de62: service: Fix IBluetoothLowEnergy signatures to return bool

* commit 'ae43de627b9c91e54215d439149c0e01599249c0':
  service: Fix IBluetoothLowEnergy signatures to return bool

service: Fix IBluetoothLowEnergy signatures to return bool

Changed the IBluetoothLowEnergy API signatures that can
synchronously fail to return bool to report synchronous errors. Also
fixed a bug with lambda capture by value in advertising API
implementation.

Bug: 24245347
Change-Id: I9ce4bb44d155d74ba38916e2834d7e93bf49301c

am f92aeb96: service: Add Parcel utilities for UUID and GATT

* commit 'f92aeb96ae4dfe2e8cd7c27ca7b94a03518855cf':
  service: Add Parcel utilities for UUID and GATT

am df0b2714: service: Implement adding a service declaration

* commit 'df0b2714553cabc735222ce18a5f018c5a5fa138':
  service: Implement adding a service declaration

service: Add Parcel utilities for UUID and GATT

Added Parcel helpers for UUID and GattIdentifier. Also added the
util::IsAddressValid helper function and unit tests.

Bug: 24245347
Change-Id: I52d7274dd0a8e2806836efc4467eb4d11ac0c7e4

am 78a44cdb: service: Add HAL wrappers for adding a service

* commit '78a44cdb25616892087af2c92abd382878c2ccb4':
  service: Add HAL wrappers for adding a service

am 80826c9d: service: Add GattIdentifier.

* commit '80826c9d492b7d6c5d2a14b6e538510b8ce92dc5':
  service: Add GattIdentifier.

service: Implement adding a service declaration

This CL implements initiating and starting a GATT service
declaration in GattServer.

Bug: 24245347
Change-Id: I53281d29a052c1890616b6595881108ca0eedec3

service: Add HAL wrappers for adding a service

Added wrappers for HAL functions for adding and starting a service
declaration.

Bug: 24245347
Change-Id: Ie9bd09abb443abdfdc31f3439d563c58066961f2

service: Add GattIdentifier.

This CL introduces the GattIdentifier class which servers as an
opaque identifier for GATT API objects/attributes.

Bug: 24245347
Change-Id: I377f53e1ff5d51fa8581756522235d4a90e90233

am 7b54b0de: service: Implement IBluetoothGattServer register functions

* commit '7b54b0de26de5eb34579304e1ccf94e1d9b77701':
  service: Implement IBluetoothGattServer register functions

service: Implement IBluetoothGattServer register functions

This patch implements the IBluetoothGattServer interface
registerServer, unregisterServer, and unregisterAll functions
against bluetooth::GattServer.

Bug: 24245347
Change-Id: Ide114730a20f939a378a6a846161a6800a12c461

am 0f2d4897: service: Add new GattServer abstraction

* commit '0f2d4897046f037a9f181f47f3d349a9dd646478':
  service: Add new GattServer abstraction

service: Add new GattServer abstraction

Added bluetooth::GattServer and bluetooth::GattServerFactory,
which conform the BluetoothClientInterface and
BluetoothClientInterfaceFactory interfaces. A GattServer represents
a per-application server instance that was registered with the stack
(a GATT server API "client").

The new code was added to the gatt_server.h|.cpp files, replacing
the old code that was added for cast. The old code has been moved to
the gatt_server_old.h|.cpp files.

Bug: 24245347
Change-Id: I3ca2eb53ab8320c9dba656859a4a89683607bb22

am 42c89a0e: service: Fix crash in hal/

* commit '42c89a0e5f3e2771681ebd11281d6e3c20341894':
  service: Fix crash in hal/

service: Fix crash in hal/

Fixed crash in hal/ interfaces when the Bluetooth library
fails to load.

Bug: 24409298

Change-Id: I8841d4e568756ef976e40335ef4697645563aebc

am ed7d0e3c: service: build unit tests on non-Linux hosts

* commit 'ed7d0e3cea7ff7a66e9f13fffd5de554c425c7bf':
  service: build unit tests on non-Linux hosts

am e415c050: Rename IPCHandlerUnix to IPCHandlerLinux

* commit 'e415c050edbb2710e8807dd2602c851412953268':
  Rename IPCHandlerUnix to IPCHandlerLinux

service: build unit tests on non-Linux hosts

Use a stub for the Linux-specific IPC on non-Linux hosts.

BUG=24306825

Change-Id: I038c17ad43695be88399617687a1c3b77946daae

Rename IPCHandlerUnix to IPCHandlerLinux

Since it relies on the Linux-specific sequential packet socket type,
rename the "Unix" IPC Handler to "Linux".

Change-Id: I04c4d6d3f73061099ebffb1ee3adfd0a20430dd7

am 9e520536: service: Add server support to BluetoothGattInterface

* commit '9e5205368d61adf11e75c9e2bd9f6ff26b375bac':
  service: Add server support to BluetoothGattInterface

am 4624e1a6: service: Add IBluetoothGattServer stubs

* commit '4624e1a6d25a838dad2e72001e340dd77d68179e':
  service: Add IBluetoothGattServer stubs

service: Add server support to BluetoothGattInterface

Added HAL GATT server interface wrappers to
hal::BluetoothGattInterface. Extended the fake implementation to
allow mocking server calls.

Bug: 24245347
Change-Id: Ic3436b8a82ecd93fed3b61579662fe776eebef58

service: Add IBluetoothGattServer stubs

Added the IBluetoothGattServer and IBluetoothGattServerCallback
interface definitions with stub daemon implementation. Added
IBluetooth.getGattServerInterface to obtain a GATT server
interface handle.

Bug: 24245347
Change-Id: I2d361a55d2b21e270dc74484e678985df3d30306

am 08f80ebd: service: Move client registration into base class

* commit '08f80ebd5c714364cb76cc4e4a93454b42ed5669':
  service: Move client registration into base class

am 2fea18e5: service: Add missing include

* commit '2fea18e501ab98df3254a08fca981513bf098274':
  service: Add missing include

service: Move client registration into base class

This CL introduces InterfaceWithClientsBase, which is an abstract class
that implements common client registration logic, which will be reused
for Binder interfaces such as IBluetoothGattClient and
IBluetoothGattServer. This also introduces BluetoothClientInstance, so
that per-application API client instances (and their factories) conform
to a common interface.

Bug: 24245347
Change-Id: Ia4b464fa85f7a33c631bf63342631fce8d155b4a

service: Add missing include

Added missing "#include <atomic>" line to fix build breakages on
some build flavors.

Change-Id: I611bb63e84787217ff4bf774ceebc09ffd3c0476

am d19bc045: service/client: Add start-adv and stop-adv

* commit 'd19bc0457a9b6519acd6a79c3ac7de653894f5ec':
  service/client: Add start-adv and stop-adv

am 1a3e9aba: service: Implement IBluetoothLowEnergy advertising

* commit '1a3e9aba73551eeecdfeece04f1c39a00880d56c':
  service: Implement IBluetoothLowEnergy advertising

am 12338405: service: Add advertising support to LowEnergyClient

* commit '12338405e0d8d70573e650560ccc8e1ce7d8e8cd':
  service: Add advertising support to LowEnergyClient

am b2b1ecd3: service: Fix bug in RemoteCallbackMap

* commit 'b2b1ecd3c282412773f5ca70ee2798542bc2920f':
  service: Fix bug in RemoteCallbackMap

am 909b7e66: service: Add various LE advertising constants

* commit '909b7e6681101eb5e2d56209993134932284b50e':
  service: Add various LE advertising constants

service/client: Add start-adv and stop-adv

Added the start-adv and stop-adv commands to FCLI for testing
the IBluetoothLowEnergy advertising APIs.

Bug: 23793954
Change-Id: I023441e4db4a50608b4f9a90bf3365c1dab8d0d1

service: Implement IBluetoothLowEnergy advertising

This CL implements the IBluetoothLowEnergy Binder APIs for
multi-advertising.

Bug: 23793954
Change-Id: I6519f86b9e528d23d2c1f46618188d1123686856

service: Add advertising support to LowEnergyClient

This CL adds the StartAdvertising and StopAdvertising APIs to
LowEnergyClient. These internally call into the multi-advertising
APIs in HAL and report success or failure in an async callback.
The sequence of calls to enable advertising and setting advertising
and scan response data are serialized internally.

Bug: 23793954
Change-Id: Id093a98f3baca6328653b986857a064e8a722690

service: Fix bug in RemoteCallbackMap

Fixed a bug in RemoteCallbackMap that returned the wrong value
in one of the template functions that wasn't exercised before.

Bug: 23793954
Change-Id: Id6848bd08e6941d3c8c6d6e4d6e494a0b9dc4c69

service: Add various LE advertising constants

Added several constants that are used in LE advertising. Added
constants for the exposed EIR fields here as well so that clients
can use it.

Bug: 23793954
Change-Id: Ib6a97d6beb1a31c9c3ea7ed468167dab627d7d32

am 27650cd8: service: Fix build breakage.

* commit '27650cd8f2473356351f3279177486971a0f9054':
  service: Fix build breakage.

service: Fix build breakage.

Fixed build breakage due to missing changes.

Change-Id: Iac962fef2e9be979a2af90a43a66a5c66bbe4ad4

am d1ed1f53: service: Add HAL wrappers for multi-advertising

* commit 'd1ed1f53a5c0cce5ecb8d784acf5bef53211a20f':
  service: Add HAL wrappers for multi-advertising

service: Add HAL wrappers for multi-advertising

This CL adds the fake and real HAL wrappers for the multi-advertising
APIs in hardware/bluetooth/bt_gatt_client.h.

Bug: 23793954
Change-Id: I485e7cf87c4b6f42c3a702c62230a287443a75c1

am 52bfc606: service: Add binder stubs for advertising API

* commit '52bfc6060cec652a67c8989e0548225af0008be1':
  service: Add binder stubs for advertising API