Global counter API for bluedroid

Added encapulsated iterator to hashmap

Also added back hash_map_test which seemed
to have been dropped somewhere along the way.

Modify eager reader to support multi-byte reads

H4: -1 -> INVALID_FD; make uart_fd invalid on close

Implement an HCI HAL for MCT

Unit tested. Needs verification on actual device (i.e. HDP profile).

Scope the lock_count variable in the alarm test harness

Unspecial-case the zero queue in the data dispatcher

Since hash map supports zero keys now this is no longer necessary.

Refactor HCI layer

Refactor the HCI layer for better testability.
A future CL will add a HAL implementation for multi channel transport.

Since btsnoop and vendor are modified, it's not easy to incrementally introduce these changes.

Add eager reader

Add stub testing macros

Refactor alarm tests to use a test harness

This improves the test writing experience for tests that rely on alarm code.

Add some stream skip macros for code clarity

Add peek + dequeue registratation to fixed queue

Add a join function to thread

Make hash map support NULL or 0 keys

Add an allocator interface + default malloc allocator

Switch to an epoll-based reactor implementation.

epoll is a much nicer interface that very closely matches the
reactor interface. It's also thread-safe which makes it a more
suitable choice for bluedroid. As a result of this change,
reactor_register and reactor_unregister are both thread-safe without
introducing any synchronization in user-space.

Fix build after enabling assertions: variable reactor is no longer defined.

Add a function to check if a string represents a bdaddr

Const-correctness for str2bd.

Add a method to create a config object without loading from disk.

Add thread_is_self function

This is useful to compare the current thread against another one.

Add a tool to inject HCI commands into bluedroid.

This tool is very primitive in a number of ways. First, each new
HCI command needs to be carefully crafted. Second, it is entirely
stateless and operates in "fire-and-forget" mode. In other words,
there's no "read-modify-write" capability yet. That being said,
this tool will grow along with the infrastructure in bluedroid.

btpan thread signal function workaround

btsock does not provide a reliable teardown
procedure.  Stale file descriptors may continue
to actively invoke signal callbacks from the
btsock thread.  Extra logic to handle gracefully.

Extend add/subtract operations to atomics

Going to be used for counters

Add 64 bit atomic types

Increase thread queue size above default

The UART enqueues work requests in addition to
the actual work onto the rx_q.  Many small byte
packets received by the UART cause many work requests
which may fill the work queue causing deadlock.

Add fixed queue size option to thread creation

Expose fixed queue capacity

Fix reactor to use prepend, fix reactor tests

Fix socket to use safe reactor registration

Implement a data dispatcher.

This will be useful at layer boundaries.

Hide thread's reactor

Add functions on fixed_queue to check empty state

Replace magic number used instead of MAX_RFC_CHANNEL number.

osi: Support for atomic data types

Fix build: include osi.h from semaphore to get definition of INVALID_FD

Define INVALID_FD to improve readability and reduce likelihood of errors

Allow iterating over sections in a config file and writing config to disk.

These two additional operations will allow us to replace the existing
XML-based config files and switch to a single file format.

Switch over duplicated timer_create routines to new alarm code.

Add a mechanism for debug tools to inject HCI commands into the HCI stream.

Expose thread's reactor so we can register new fds with it.

This change allows a thread's reactor loop to manage additional
file descriptors, e.g. sockets and queues, entirely within that
thread's context.

hci: Renaming of callback functions

Changing name to conform to *_cb convention already
in place.  Further, current names will be re-used
for osi buffer resource management.

hash_map header uses standard integral types so it must include stdint.h.

Wake up reactor whenever a new object is registered/unregistered.

Before this change, if the reactor was blocked waiting for an event
and a new object was registered, it would not wait for events on the
new object until it woke up on its own. This change forces the reactor
to re-evaluate the file descriptor set whenever the object list is
modified.

Introduce a socket class

This socket implementation is limited to TCP server sockets and
provides no mechanism to connect to a remote host. It's primarily
intended to be used to interface with debugging tools. It's designed
to work seamlessly with the reactor (event-driven) model.

Remove some unused code in L2CAP

osi: hash map implementation

Fix gatt_main.c compile error due to log macro change

rfcomm: Check for null pointers on DLC request

The function to get the port control block from the DLCI
explicitly mentions it will return NULL if there is no
active rfcomm connection for the given DLCI.

Release GATT resources upon teardown

Add a system property for libbt-vendor.so initialization timeout

Use a system property, bluetooth.enable_timeout_ms, to specify in ms
how long to wait for libbt-vendor.so to initialize.

am 45085bf6: Merge "Ignore service UUID length if no UUID filter is specified" into lmp-mr1-dev

* commit '45085bf6ed93ea9398047c21bc4b94c8516f9daa':
  Ignore service UUID length if no UUID filter is specified

Merge "Ignore service UUID length if no UUID filter is specified" into lmp-mr1-dev

am 114c9ce4: Merge "Increase SDP buffer from 750 bytes to 4096 bytes" into lmp-mr1-dev

* commit '114c9ce459e9d9e40d55586da1d18d0d65c45c8d':
  Increase SDP buffer from 750 bytes to 4096 bytes

Merge "Increase SDP buffer from 750 bytes to 4096 bytes" into lmp-mr1-dev

Ignore service UUID length if no UUID filter is specified

Bug: 19016326
Change-Id: I6156fee1d6079fec7879f47b6120befe2c4d35cd

am dfb3b2f2: Add flag indicating encrypted connections to btif_dm_get_connection_state()

* commit 'dfb3b2f2595dc16fe7f576a4b0c2f5e73db0327f':
  Add flag indicating encrypted connections to btif_dm_get_connection_state()

Add flag indicating encrypted connections to btif_dm_get_connection_state()

Add a check to btif_dm_get_connection_state() to see if the currently
active connection is encrypted.

Bug: 19186961
Change-Id: I27bcc6e296d272ffd17c0bbe97631ffa4df57ccd

am d6f2fa93: (-s ours) am d6419cd6: am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'd6f2fa93052c1d9bdf6035ace6b32345013516db':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am e791f732: (-s ours) am ae9d174b: am 408dec19: am 977acabd: (-s ours) am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'e791f73202a2832b4d20dbc1ecd4b520adbb75d3':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 0ad6cfde: (-s ours) am ce5ae4ab: am a708936d: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '0ad6cfdee8857a7bc54d4c3eb98bf1cdc82dc9a7':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am d6419cd6: am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'd6419cd6309bd20a2ab095c783bb6ad1f208bf4f':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '0a8c803a48fe5afa71d1f8660539012e9b1f9642':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '7f34538ac0e447a88d1f54ff819c28b3574f0aa4':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '65aec3a3f62c004d7073470f18330007b7ecf230':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am ae9d174b: am 408dec19: am 977acabd: (-s ours) am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'ae9d174bc0afe0a7ed0bb4fb82836338fbb6adb1':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 408dec19: am 977acabd: (-s ours) am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '408dec1991255be919b1605e5cfe983b7f7154e9':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 977acabd: (-s ours) am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '977acabd3744a3308057b1485c7ac8658698e96b':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '6d429d9d0a1b537600c652dda2bc98d4c4ebd783':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'eebc8ab9c5e362d721f8f606f0a564ca9aff89d9':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '3efb9e2e11471a67329ad17d48ffa79118ea76c6':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am ce5ae4ab: am a708936d: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'ce5ae4abddaab8865bb44c83d2d44e14b0d7536b':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am a708936d: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'a708936da347b32f2a0811b5f33d9068c2b0de6f':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

Increase SDP buffer from 750 bytes to 4096 bytes

Bug 18670169

Change-Id: Ida1a6a57dcce94339a190974af4565f14157cdc2

DO NOT MERGE Change pairing_cb to assume temporary pairing by default

When pairing takes place, the pairing_cb.is_temp flag indicates whether
a pairing is temporary or permanent. Link keys are not stored for
temporary pairings. Since this is a "positive" flag, resetting the
pairing_cb control block (ex. memset to 0), it will assume persistent
pairing by default. Under certain circumstances, this can lead to a link
key being stored for temporarily secured connection.

This patch reverses the flag to be a "negative" flag. Renamed to
"persistent_bond", the default 0 meaning is now used to indicate a
temporary bond. If the lag is not properly set now, it will default to a
temporary bond and will not save the link key erronously.

Bug: 18345373
Change-Id: I06b1ba9331a70ebc29f4437bf836164658dec5ae

DO NOT MERGE Change pairing_cb to assume temporary pairing by default

When pairing takes place, the pairing_cb.is_temp flag indicates whether
a pairing is temporary or permanent. Link keys are not stored for
temporary pairings. Since this is a "positive" flag, resetting the
pairing_cb control block (ex. memset to 0), it will assume persistent
pairing by default. Under certain circumstances, this can lead to a link
key being stored for temporarily secured connection.

This patch reverses the flag to be a "negative" flag. Renamed to
"persistent_bond", the default 0 meaning is now used to indicate a
temporary bond. If the lag is not properly set now, it will default to a
temporary bond and will not save the link key erronously.

Bug: 18345373
Change-Id: I8558abc9eace609b1ba02485b0ff775fb6f3c6af

DO NOT MERGE Change pairing_cb to assume temporary pairing by default

When pairing takes place, the pairing_cb.is_temp flag indicates whether
a pairing is temporary or permanent. Link keys are not stored for
temporary pairings. Since this is a "positive" flag, resetting the
pairing_cb control block (ex. memset to 0), it will assume persistent
pairing by default. Under certain circumstances, this can lead to a link
key being stored for temporarily secured connection.

This patch reverses the flag to be a "negative" flag. Renamed to
"persistent_bond", the default 0 meaning is now used to indicate a
temporary bond. If the lag is not properly set now, it will default to a
temporary bond and will not save the link key erronously.

Bug: 18345373
Change-Id: Ifb61e7a23db07888a625db9ab851548600183136

Merge "bta_dm_search_cancel sends dm_search state machine complete events" into lmp-mr1-dev

bta_dm_search_cancel sends dm_search state machine complete events

bta_dm_search_cancel moves dm_search state from active search to
search_canceling state. The function is also responsible of moving
the state out of canceling state.
Bug: 19017339

Change-Id: Idea0cb0e2987892a5fc0f66db8eed19f57c68b89

Change pairing_cb to assume temporary pairing by default

When pairing takes place, the pairing_cb.is_temp flag indicates whether
a pairing is temporary or permanent. Link keys are not stored for
temporary pairings. Since this is a "positive" flag, resetting the
pairing_cb control block (ex. memset to 0), it will assume persistent
pairing by default. Under certain circumstances, this can lead to a link
key being stored for temporarily secured connection.

This patch reverses the flag to be a "negative" flag. Renamed to
"persistent_bond", the default 0 meaning is now used to indicate a
temporary bond. If the lag is not properly set now, it will default to a
temporary bond and will not save the link key erronously.

Bug: 18345373
Change-Id: I6e821595877ff30d64dc6e33602ac049cab3dd1e

Merge "Relax max PCM frames per tick a bit" into lmp-mr1-dev

LE: Add error status to failed write operation

Bug: 18833375
Change-Id: Ib91c8e92658a276d813954264897b02863f1148b

Relax max PCM frames per tick a bit

Also made the value overwrite-able in bt_target.h.

Bug: 18913091
Change-Id: I64bbe453b35fdce6f127b3c23c0552b14d4ac49b

Added check if the device address is random

A check if the device address is random is needed since in a
public device address the two most significatant bits can also be
equal to 01 (see Bluetooth specification Core_v4.1.pdf, page 2021
and page 2500).

Bug: 18784040
Change-Id: I4f967a426ebb825731461e202c0741ae0540ef6a

Allow connection parameters updates by slave

GATT service discovery in the slave role does not update to fast connection
parameters when connected. Accordin to the 4.0 spec, the connection
parameter disable/enable call is used for only by the masster role.
The 4.1 and later spec allows the slave role to control the connection
parameter update so that a slave role GATT client can perform the service
discovery using fast connection parameters.

Also switched to requestiong 7.5ms connection interval by default during
service discovery.

Bug: 18266904
Change-Id: Ie6300587306be3d694937e3888069e252592be42

Merge "LE: Register for service change indication" into lmp-mr1-dev

Do not log LE encryption keys or passkey

Bug: 18687167
Change-Id: I2a2a7113e6bc7cfb22a6bd6f16d16ce2b7c84cff

LE: Register for service change indication

Some bonded remote devices require the client to register for service
change indication and actually set the client configuration descriptor
before service change indications can be removed.

This change add an additional step after device bonding has occured to
register for service change indication if possible.

Bug: 18173911
Change-Id: I25386faec0d58834ee2b0a9d1db2d2e052311264

LE: Ignore non-volatile GATT service cache for non-bonded devices

If a device is not bonded, the non-volatile GATT service cache should
not be used. Only bonded devices have the capability to invalidate the
cache using the service changed indication.

Bug: 18571592
Change-Id: Ia75aa14f1c862a18237e78d24b63c22cf1dc82a8

am cf7f6a15: Prevent Java call for acquiring already held wakelock

* commit 'cf7f6a154aed48659260d391994dc59151be937b':
  Prevent Java call for acquiring already held wakelock

Merge "Remove unnecessary check of multiple advertise instance count." into lmp-mr1-dev

Prevent Java call for acquiring already held wakelock

Change-Id: Icf455b6aa7c0c1bf7abf7ea9700cfde17f6c7c2b

Merge "PAN TAP fd is not monitored in all cases" into lmp-mr1-dev

Merge "Make GKI_TIMER_INTERVAL_FOR_WAKELOCK configurable for different target" into lmp-mr1-dev

Make GKI_TIMER_INTERVAL_FOR_WAKELOCK configurable for different target

Set the default value as 3000 ms
Bug: 18549150

Change-Id: I1d5b4fca627c22fdd8a32a41d5beac6837b6dff4

Add server-side callback for change in MTU (2/4)

When a client requests to update the LE transport MTU, the server
currently does not get notified and can therefor not properly size
notifications appropriate to the current MTU.

Bug: 18388114
Change-Id: Ice355f8919d4f69876574df1d90ce7d31f42bc4f

Remove unnecessary check of multiple advertise instance count.

Bug:18539121
Change-Id: I841e4323ed4995bd9f8117681a759f52995d071f

PAN TAP fd is not monitored in all cases

Bug: 18271987
Change-Id: I348f3851a955b5277e7cdf0102583eaaef2ef0f3

am 254b5ba2: gki_buffer: add missing mutex unlock

* commit '254b5ba2e3f4c44b82a0369579b66bafcd0d6aa2':
  gki_buffer: add missing mutex unlock