Remove dependency on GKI tasks from BTIF media task.

The previous code started a thread and posted a long-running function
to the thread's event loop. In essence, it created a thread and ran
a custom thread main that did its own event handling using GKI_wait.

The new code doesn't use GKI_wait or GKI_send_event at all. Instead,
it uses thread_post and the thread's reactor to enqueue and dispatch
events.

Don't call GKI_send_event on BTU task since it's not a GKI task anymore.

Remove GKI timers

Adding some documentation for the power management flow.

Add sniff changed log statement on the mode changed event

This will ensure we log accordingly when the other end
decides to put the connection into sniff mode.

Collapse BNEP #defines that don't need to be configurable.

In some instances, defining these macros to FALSE would result in
broken runtime behavior (e.g. UUID fields containing uninitialized
values). This change collapses the following defines:

BNEP_SUPPORTS_PROT_FILTERS
BNEP_SUPPORTS_MULTI_FILTERS
BNEP_SUPPORTS_DEBUG_DUMP
BNEP_SUPPORTS_ALL_UUID_LENGTHS

Add a new l2cap_client class.

The code to negotiate an L2CAP connection is currently scattered
and duplicated throughout the codebase. Each profile that uses
L2CAP has its own explicit or implicit state machine to go establish
a connection and go through the handshake. This class is intended
to consolidate that duplicated code and provide a usable API for
L2CAP client connections.

Add connection-specific function L2CA_SetConnectionCallbacks.

This function allows a client to specify callback routines per-
connection instead of per-PSM.

property api unification, naming and testing

Add a reference counted buffer implementation.

This implementation introduces the notion of a 'slice'. A slice is
simply a sub-buffer of the original buffer. Slices refer to their
parent buffer so no memory is copied and slice creation becomes an
O(1) operation. Although I'm introducing a new concept, in practice,
a slice is indistinguishable from a buffer (they share a type and
all operations).

I expect slices to be used heavily during packet construction. For
example, an RFCOMM implementation would request a buffer from the
L2CAP layer (which may in turn request a buffer from the HCI layer).
The L2CAP layer would reserve some space in the buffer for its header
and return a slice to the RFCOMM layer.

Collapse more feature flags.

Start reviewing in include/bt_target.h.

Collapse RFCOMM_INCLUDED and GAP_INCLUDED guards.

Both of these features are always available in bluedroid.

Delete unused Broadcom-specific extensions API.

Eliminate unusable wcassert code in favor of plain old C asssert.

The WC_* macros only expand to something meaningful when _DEBUG is
also defined. However, on enabling _DEBUG, bluedroid fails to build
because the wc_assert function is not defined anywhere. We can get
what I imagine is the equivalent behavior by switching over to the
standard C assert macro.

Drop duplicate profile connection requests

If profile connection requests come in too quickly
for us to handle, clients can effectively DoS the bluetooth
stack.

We used to see out of buffer errors because the requests
were batched up and sitting on buffers the rest of the stack
needed. With the switch to pure allocation we didn't see that
specific problem anymore, but we were still running into the
sanity assert.

Bug: 18323666

Data type representing the bluetooth iac parameter

First pass at the bluedroid profile manager.

Rename legacy uuid_copy to not conflict

Add size_t inclusion

bt property object

Add bdcopy method

Remove some more unused functions in btm_acl

Remove unused read link policy + friends

Was never called, and the removal of which obviated
a lot of other code.

Remove unused link supervision timeout code

The btm callback was never set, so this code
did nothing useful.

uuid_test

Device class implementation

Add compile time checks

Add more uuid methods

Add a simple classic peer

Add a hash function for bluetooth addresses

Also includes simple tests for it + disambiguates
including hash_function.h throughout the stack.

Add key equality function option for hash_map

This will allow us to do deeper equality on things like
bluetooth addresses where the actual pointers are different
but the values of the bluetooth addresses are the same.

Clean up unused functions in btm_sec

Remove unused security abort callback

The callback in BTA was null, so it never was actually
used.

Remove link key request callback

The core security manager stores whether we have link keys
for each peer. This other call up to the bta device manager
to see if it has link keys is a dead end and does nothing.

Canonicalize HCI logtags with bt_ prefix

Add bash script for running unit tests

This script helps simplify pushing and running bluedroid unit tests.
You can run all of the known ones, or provide names of selected unit
test executables you wish to run.

Rename Bluetooth binaries to use the net_ and net_test_ prefixes.

Sometimes I forget stopping shell before running bdtest.

Make this check automatic and print out a friendly message for reminder.

Remove unnecessary newlines at the end of log statements.

Add platform-independent logging macros to OSI.

These macros should replace ALOG* and the various trace macros
used throughout bluedroid. This change eliminates all uses of the
ALOG* macros in favor of the new ones.

Remove unused ACL packet size getters, remove unused timer entity

Extend bdtool to accept / initiate SCO connections.

This is a great way to run controlled audio quality experiments for
the Handsfree profile.

Remove unused shell script, gen-buildcfg.sh.

Add RFCOMM result code string for an unspecified error.

The code enumerating the list of possible error codes is not
a C enum and the original authors skipped a value. As a result, the
table mapping error codes to strings is off by one after that skipped
value. I'm inserting a string for that error code even though it's not
defined so the strings line up with the actual error.

Adding some logging for the close states for RFCOMM connections.

Fix bug in SDP server where it would get into an infinite loop.

If the SDP database is mutated while a client is fetching SDP records,
we can end up in an infinite request/response loop. Specifically, if
an SDP record is deleted after a client has received the first fragment
of a multi-fragment response, the server will not be able to complete
the request. Instead, it will return the same continuation token back
to the client which will request the next fragment and the server,
again, will return the same continuation token. This process repeats
forever, resulting in a large amount of unncessary BT traffic and
power costs on both devices.

Unfortunately, this seems to be a design flaw in the current SDP
server implementation. This change simply detects the above condition
and aborts the transaction entirely by returning an error to the
client. Future changes will eliminate this class of problem entirely.

Introduce a growing array class.

Arrays are useful for storing a collection of small objects where
linked list node allocations would be a lot of overhead. They are
also useful for random access through the container. Lastly, this
implementation assumes copy semantics which are intentionally
different from list's reference semantics (since objects are assumed
to be smaller).

The current interface is intentionally sparse.

Increase GKI buffer size

Bug: 18324148

Add counter unit tests

Explicitly add include paths

Allocation test harness included relatively local
header allocation_tracker.h.  Change to include
absolute header location from bluedroid root.

Enable allocation tracking for eng and userdebug builds.

This change will result in a constant increase in memory utilization
per allocation but will help us catch memory errors earlier.

Add a "setDiscoverable" command to hci.

Adding in some additional, but critical, logs.

Update module names to not use C preprocessor and to be at top of headers.

Replace BUFFER_Q with list_t #2

rcv_hold_q -> rcv_pending_q

Replace BUFFER_Q with list_t

soc_queue

General tidy up of L2CAP flow control

- Alphabetize include files
- Add asserts
- Ensure static funtions are marked as such
- Remove unnecessary logging

Properly iterate when removing from list

Ensure we hand off the node iterator before
proceeding to remove from the list.

Add error checking when getting property

Also added PAN API

Expand include pathnames and add bd root path

Hopefully this will squash the sem_t semaphore_t collision

Back GKI buffers with malloc

It looks like we're somehow getting NULL back from buffer allocation.
Backing GKI buffers with malloc should fix this.

Bug: 18324148

ungkibufferize the profile queue, use assert

ASSERTC isn't a real assert. It just prints a message and allows the
code to carry on. Switch to asserts so the exception condition is
evident in the crash log.

Make the profile queue use osi_malloc/free instead of gki buffers, so it
can't drain the buffer pool.

Print out every power management mode switch initiated from the host.

Fixed HAL layer ignore state handling

If the hci layer ever entered the ignore state,
synchronization death was immenent. This patch fixes that.
Also adds additional info print out when entering the ignored state.

Adds tests for the ignored state.

Doesn't solve the problem of why we were getting NULL back from
the buffer allocator, though. (The trigger for the ignored state.)

Bug: 18156298

Add a command line flag to skip sanity test suite.

bdtest: read the paired device's bdaddr from the stack config file.

Remove ptim and associated GKI timer functions

Remove some remaining references to message based timers

Some of this gets cleaned up in a later CL during the merge
but removing them now so the ptim removal change compiles.

Remove unused link key functions from devctl

The security manager looks for the link key notification event,
so it doesn't use these read/write link key functions at all.

Modify bdtest to enable running individual tests by their name.

Remove the afh functions from btm_devctl

The callers were removed in a previous CL, so these
are no longer necessary.

Remove BTA_CheckEirData, a direct redefinition of BTM_CheckEirData

Gentle move towards using more of BTM directly.

Remove some functionredefinitions from bta dm

These weren't used. And were another layer of indirection
for things that already existed. One reason for the initial
implementation is that they would ensure the actual calls
happen in a known thread.

Use correct service ID for A2DP source to make way for A2DP sink.

L2CAP: expose a function to return the RCID and ACL handle for a given LCID.

Move controller module to device directory

Unused function

Remove unused inquiry DB functions

These are not used for the inquiry database.

Remove always true definition BTM_EIR_SERVER_INCLUDED

Removing unused function BTA_DmSetScanParam()

Remove always true BTM_USE_INQ_RESULTS_FILTER

Remove always false flag BTM_BYPASS_EVENT_FILTERING

Instrument data flow a bit

Use counters to grab some select data metrics.

Don't dispatch connections in the connection queue until stack comes up.

Bug: 18139425

counter implementation

Remove duplicate functionality

Now that bdtest depends on btcore we can
remove the bdaddr parsing functionality.

Remove uncompiled unused PBAP support

Like OPP, the phone book access profile is implemented in
java for android. Cleaning up some references to uncompiled phone book
support in bluedroid.

Left the few definitions SDP relies on. We can consolidate that later.

Remove unused functions BTM_RegForLstoEvt/btm_acl_set_discing

Remove always true BTM_PWR_MGR_INCLUDED

There are two code paths to set the chip into the power modes.
The power manager aggregates access to the power modes.
Does not compile when set to false.

Remove always true BTM_BUSY_LEVEL_CHANGE_INCLUDED

Remove data_types.h

Actually remove bd.[c|h]

Removal of bd.[c|h]

Consolidate legacy types into bt_types.h

Moving btif_sock_sdp to stdint and stdbool.

Fix how add_sdp_by_uuid passes the uuid to SDP_AddSequence

SDP_AddSequence expects to dereference the passed
pointer and find a pointer there, and that pointer
pointing to UINT8s.

Before, we were passing &type_buf, which gives the
address of the first UINT8 in the array on the stack.
This meant SDP_AddSequence was treating the first few
bytes of the uuid as a pointer to somewhere, leading
to a segfault when it tried to follow it.

Refactoring btif_sock_sdp.c for clarity

Revert "BT MAP: added support for email sharing over BT"

Another change being backed out until the megamerge is over.

This reverts commit c331807e3c3a0e15180a11325e1635ae5268aa39.

Revert "MAP: SDP corruption"

Backing out so our rewrite merges cleanly.

This reverts commit 54e24af853a77a057995ee4f234bae4d490b32e5.

Revert "Forgot device/Android.mk"

Broken build...

This reverts commit d935ef9446990c17412d2b0d5ce158df54377871.

Revert "Remove unused inquiry variables"

Still used by BTA API BTA_DmSetScanParam

This reverts commit d84f0c8c5eb67a45cb2e0993324d1dff8b499b1b.