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.

Forgot device/Android.mk

Slipped by...

Remove always undefined BTM_EIR_UUID_LKUP_TBL

Remove always true definition BTM_EIR_CLIENT_INCLUDED

Remove unused inquiry variables

A2DP sink: reply with success on Start command.

We've observed a Nexus 5 sending back-to-back START requests. If we
accept the first request and move to the STREAMING state, we'll reject
subsequent requests. The phone, however, ignores the first response
(since it has already issued another START) and will not route audio
to Glass.

This is a speculative fix. If we're in the STREAMING state and we
get an incoming START request, accept the request instead of rejecting
it to work around a buggy implementation on the peer.

Bug: 18165461

Fix bdtool compile errors

Add Android.mk to tools/ directory.

Don't touch alarm's contents if it has already been freed.

The current code potentially writes values to the alarm struct after
it has been freed. This change makes sure we check that the struct
is owned by the current thread before touching its contents (read or
write).

Fix NULL pointer dereference in bta_pan_co_tx_path.

This happens when the PAN connection terminates but the stack is
still sending data back to the system. There's a timing component
to this bug; the stack doesn't crash 100% of the time since it
depends on whether the TX path notices the PAN connection was torn
down or not before calling bta_pan_co_tx_path.

Directory layout documentation

Document network ports used by bluedroid.

Document the set of log tags used by bluedroid.

Going forward, we should clean up the log tags to use a consistent
naming scheme. All log tags should be lowercase and should use
underscores instead of hyphens.

avdt: Make sure LCID field is filled out even on incoming connections.

Make bdtest report the total number of tests (sanity + suite)

There are currently 11 tests, but if one failed it would report 1/9 rather than 1/11.

Change bdtest text color gray to the default color

My command prompt outputs white text, so this change makes bdtest
restore the default color after running, instead of leaving it gray.

Add system tests for RFCOMM sockets.

Add a watchdog timer to test suite.

If a test case doesn't make forward progress after some amount
of time, we terminate the process and assume failure. It's important
for test automation that the test suite does not hang forever.

Only colorize test result output if stdout is attached to a tty.

This will make sure redirected output won't have escape sequences
while still leaving the output pretty when running tests by hand.

Make build more strict by adding more warning flags.

This change introduces -Wunused-but-set-variable to help catch
programming errors. It also undefines NDEBUG so assertions are
fatal and defines LOG_NDEBUG=1 to suppress LOGV.

Removing a duplicate call to socket_register.

First entry for bdtool

Comment said BTM_PM_SNIFF_SLOT_WORK_AROUND would be removed someday. Today is that day.

Looks like this has been around for a while, and isn't/shouldn't be used.
restore_pkt_types then becomes obsolete, so removed.

Make hci/ depend on bdroid_CFLAGS

Fix BLE GAP handling after callback refactoring

BLE supported states was moved to controller during the callback
re-factoring CL, but in the commotion I forgot to remove the
supported states variable from devcb.

This meant I didn't realize GAP was looking at the wrong place
for BLE supported states. This CL fixes that.

Remove unused function BTM_ChangeLinkKey & friends

Not used in the codebase. Removed it along with
all its related code.

According to the spec the change link key event is only
sent in response to the change link key command, so
it should be completely safe to remove that logic.

Remove excessive log message from btu_task.

Remove some remaining references to OBX

Android implements the Object Push Profile in java
against RFCOMM sockets with its own implementation of
OBX there.

OBX in Bluedroid isn't compiled, and is missing header files required
to compile it anyway. Removed a number of OBX related headers and updated
a few references where they were used out of place.

Change-Id: Ifaecfb6f85b1dca9055932886af484b6dc107fc1

Remove a bunch of always-false macros

Referenced missing header files, so they wouldn't even
compile if set to true.

Remove definitions for DUN

Missing header files, etc so it wouldn't compile anyway.

Remove always false TCS_INCLUDED macro & friends

Referenced non existent headers, so would not compile if set to true.

Remove remaining references to GOEP

FALSE by bt_target. Header files never used. Missing header files.

Remove unused header utfc.h

Remove a bunch of unused functions and definitions from HCIC

Left the BLE stuff alone for now.

Remove unused macros LMP_TEST and BTISE

Disambiguate event mask setting for bluetooth DUT mode

The nuances of addressing a string literal are tricky, so it
looks like we're using a char ** when in reality it's a pointer to
the array containing the literal, which converts just fine.

Removing the address-of makes it easier to see correctness.

Remove unused code in GAP

Left the BLE stuff for now, since some of it appears to
be used at least right now.

Don't call null callback in vendor lib

Don't require a vendor library command transmit to provide a callback.
If the vendor library provides a null callback just ignore it.

Fix accidental hardcoding of 2+ extended feature page numbers

Happened to be ok on devices with 1 or 2 feature pages, but for
devices with 3 or more feature pages it was always loading the
page at index 1 for pages past the page at index 0, causing
an infinite loop, causing bluetooth enable to time out.

Move properties data primitive into btcore

This is a valuable data structure that could be
used widely within the stack.  It's used by
bdtest and bdtool, and could be used by any
other application built on top of the stack.

Add comments to the device features and event mask structs

Remove L2CAP_CORRUPT_ERTM_PKTS macro

In general, test code and production code should not coexist in the same file.

Refactor bta_jv functions

Remove unused L2CAP and SDP code to reduce distraction from unused code
path.

In general, if we need any of this behavior in the future, it's best
to talk directly to the apis instead of through another layer.

Remove duplicate local bdaddr management functions

Refactor btm_devctl reset sequence

* Controller bring up on blockable thread now
* Removed some duplicate and commands during controller bring up
* The code to make commands for controller bring up is smaller and better

Prune BTM_ReadLocalVersion

It was called once, but the version info read back was
never used. The status returned was used, but that was
implicitly getting whether the device was up or not so
replaced with a call to BTM_IsDeviceUp.

Remove <BT 2.0 workarounds

Prune BTM_SetSecurityMode and GAP_SetSecurityMode

BTM_SetSecurityMode was only used from GAP_SetSecurityMode,
and GAP_SetSecurityMode was never used.

Add temporary callbacked wrapper for module startup

When all is said and done with the module conversions, all lifecycle functions
will run on the lifecycle managment thread, which can be blocked while waiting
for futures.

This CL means new modules can depend on that fact during start_up, alllowing
newly converted modules to be spliced into the existing startup callback madness
without having to sacrifice clean implementation.

We can add other callbacked lifecycle function variants as necessary.

Removing unnecessary BTAPI #defines.

Add a futured version of the HCI transmit_command

This will allow the controller restart sequence to be serialized into a single function.

Remove some < BT 2.1 code & repeated init

Simple pairing is already enabled and the BLE event mask
is already set by this point.

Remove BTM_SetDeviceClass call in devctl

BTM_SetDeviceClass is called later on in bta_dm_sys_hw_cback
during BTA_SYS_HW_ON_EVT using the real device class. No need
to set an "init" device class as far as I know.

Remove duplicate L2CAP_NUM_FIXED_CHNLS

Remove unused hci_cmd_cb in btu

An earlier change obviated the need for it, but I
forgot to remove it at that time.

Modulizification of bt_utils