am 1ee74df8: am a0124571: Suspend AVDTP link in remote initiated AVDTP Start

* commit '1ee74df81106f6e5293d3216c1952d4629c4de14':
  Suspend AVDTP link in remote initiated AVDTP Start

am fadbccdb: am 96162b60: A2DP: Use mutex to only to check and set HAL state

* commit 'fadbccdbcf2d7a2a8871620d928f151fbacb2453':
  A2DP: Use mutex to only to check and set HAL state

am a0124571: Suspend AVDTP link in remote initiated AVDTP Start

* commit 'a0124571e3d5455448dbfcea7617f583b20210f9':
  Suspend AVDTP link in remote initiated AVDTP Start

am 96162b60: A2DP: Use mutex to only to check and set HAL state

* commit '96162b608ce2ad6d86214d7efd2bcc120d1f079c':
  A2DP: Use mutex to only to check and set HAL state

Suspend AVDTP link in remote initiated AVDTP Start

As media player is not started as part of remote initiated
AVDTP start, hence if link is kept in Started state when
remote initiated START is received it unncessarily blocks
power manager to trigger sniff on that link which increases
power number.
Related issue is seen with Moto HS820, Denso08.
These peer devices send AVDTP start on AVDTP connect and
does not suspend causing link not to go to sniff.
This change takes care of sending suspend in such scenarios
DUT initiated start gets triggered when local playback starts.
Moreover this change helps in suspending streaming if remote
initiates avdtp start during an ongoing call. This helps
remote and DUT's a2dp state to be in sync.

Bug: 22028876
Change-Id: I8ebee009c95148cf733e39d897bc8ffe5af7791e

A2DP: Use mutex to only to check and set HAL state

In multi-threaded env A2DP state gets updated to suspended
after checking and before acquiring mutex lock. A2DP State
should be checked inside mutex lock to avoid checking against
inconsistent state.
Moreover this change prevents overwriting of state
from AUDIO_A2DP_STATE_SUSPENDED to AUDIO_A2DP_STATE_STOPPED
which would start the data path when there is data inflow
from audio in suspended state, which is not intended.
This change helps avoiding above wrong actions.

Bug: 22028876
Change-Id: I4fb73c1a6851fa97d539860d5fc0b8022b019bc1

am 123d77a3: resolved conflicts for merge of 4197cfdb to mnc-dev-plus-aosp

* commit '123d77a395b12463186e24b09c0cebcb3fe9f185':
  Fix the file permissions of /data/misc/bluedroid/bt_config.conf

resolved conflicts for merge of 4197cfdb to mnc-dev-plus-aosp

Change-Id: I01e295c4f86e821f84e45fbf1985088dd1a5347a

Fix the file permissions of /data/misc/bluedroid/bt_config.conf

Fix the file access permissions and group ownership of
"/data/misc/bluedroid/bt_config.conf" so the file can be reused when
switching users on the device.
For that purpose, we need to do the following:

1. Set the set-group-ID (bit 02000) flag for directory "/data/misc/bluedroid"
   so the files created in that directory will have group-id of
   "net_bt_stack" .

2. Change the file's permissions of file "/data/misc/bluedroid/bt_config.conf"
   to Read/Write by User and Group.

Bug: 21493919
Change-Id: I48b73711789734b5221231729f8cd9ff9bb94be7

am c6da6770: am 6a27776c: Property based change to qualify TC_INT_SRC_SIG_SMG_BV_23_C

* commit 'c6da677026f58436ed8d9763f79662c0e88cc736':
  Property based change to qualify TC_INT_SRC_SIG_SMG_BV_23_C

am 6a27776c: Property based change to qualify TC_INT_SRC_SIG_SMG_BV_23_C

* commit '6a27776c488dd5ce2d29bce543ee42a12dfad60b':
  Property based change to qualify TC_INT_SRC_SIG_SMG_BV_23_C

Property based change to qualify TC_INT_SRC_SIG_SMG_BV_23_C

As Bluedroid AVDTP solution does not use ABORT command
in any of the usecases hence this property based change
is made to pass AVDTP ABORT mandatory TC using PTS.
Affected PTS TC: TC_INT_SRC_SIG_SMG_BV_23_C.
System Property to enable sending ABORT to pass the TC is:
bluetooth.pts.force_a2dp_abort

Bug: 21896912
Change-Id: I71bd11de07b9d8fb15a933712db25f0a93e10dd5

am a918eb91: am ee039728: Add deep copy of AVRCP metadata commands

* commit 'a918eb91348acf453c426f316146f8f1f464f632':
  Add deep copy of AVRCP metadata commands

am ee039728: Add deep copy of AVRCP metadata commands

* commit 'ee03972892ca31fe47a759f69be15dd46b0abefe':
  Add deep copy of AVRCP metadata commands

Add deep copy of AVRCP metadata commands

The data structure for RC metadata event contains pointers within.
Added deep copy of the data structures to avoid memory corruption.

Bug: 20017069
Change-Id: Ibc5bc75cfd62abda9611060b5b435bed9e472376

am b56ff2d0: am 90dcc56c: Disconnect Gatt during failure of service changed discovery process

* commit 'b56ff2d08212db6555ad20d7ea08515d3a47edb1':
  Disconnect Gatt during failure of service changed discovery process

am 803b381f: am 4508f24b: Not Implemented response for unsupported AVRCP commands.

* commit '803b381f61d0a034cfdce37a8a2c92ff945c2ec3':
  Not Implemented response for unsupported AVRCP commands.

am 90dcc56c: Disconnect Gatt during failure of service changed discovery process

* commit '90dcc56c1357f2f0b2fa03638033455d89a8a758':
  Disconnect Gatt during failure of service changed discovery process

am 4508f24b: Not Implemented response for unsupported AVRCP commands.

* commit '4508f24b72ed40b6d0cc320cae8cebade6521b2c':
  Not Implemented response for unsupported AVRCP commands.

Disconnect Gatt during failure of service changed discovery process

Gatt service changed discovery holds link till it completes discovery of service,
characteristics, descriptots and writing notification for characteristic descriptor,
this change takes care of removing  the link if service discovery fails during any
stage of this process.

Bug: 21896912
Change-Id: Ie97a039568fedc66403895e7e917e4d72a5419e1

Not Implemented response for unsupported AVRCP commands.

Ensure DUT responds as not implemented for AVRCP
pass-through commands which are not supported by DUT.
This gives right impression to peer device on the correct
set of pass-thorugh commands supported by DUT.

Bug: 21896912
Change-Id: Ie10e74e2a399bb420f55c55005600c945b53bfc2

am 94fbc040: am 00ba745e: Use correct UUID for Central Address Resolution characteristic

* commit '94fbc040f2f1d218c5f038966ed71ddbb7953a12':
  Use correct UUID for Central Address Resolution characteristic

am 00ba745e: Use correct UUID for Central Address Resolution characteristic

* commit '00ba745e3a5f76111db92026c0a2ff9097290fa7':
  Use correct UUID for Central Address Resolution characteristic

Use correct UUID for Central Address Resolution characteristic

This patch rectifies the UUID used for Central Address resolution
characteristic which was earlier assigned the same value as
UUID for service change indication

Bug: 21896912
Change-Id: I2b341413d0a4331584844b54971f7645fd7e438e

am f83beedf: am 19165cb5: Reject AVDTP START and SUSPEND request in invalid state

* commit 'f83beedf0212b12cc0d8b695a4ec95327e2a2c1e':
  Reject AVDTP START and SUSPEND request in invalid state

am 5920d62b: am fa8c7958: Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

* commit '5920d62b29baeeb2c056b30a707e9cca10d5a1e5':
  Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

am 19165cb5: Reject AVDTP START and SUSPEND request in invalid state

* commit '19165cb562df83f29f7403b9a0f38b199a2edfb3':
  Reject AVDTP START and SUSPEND request in invalid state

am fa8c7958: Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

* commit 'fa8c795867b284239eb88970889a145cfa026f78':
  Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

Reject AVDTP START and SUSPEND request in invalid state

Reject remote initiated AVDTP START and SUSPEND request
in invalid state. This helps passing the below PTS cases.
TC_ACP_SRC_SIG_SMG_BI_20
TC_ACP_SRC_SIG_SMG_BI_26

Change-Id: I177456389246df89e7d7560555ffffde0639e618

Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

This patch improves RFCOMM throughput and reduces CPU utilization.
Instead of using a counting semaphore to measure bytes and reading
one at a time from the eager reader's buffer, read in bulk based
on the incoming read request size.

Change-Id: I17046bfbc3ca49576a9c82b38911aeb84234881a

am d7c8b539: am 08019f3e: Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

* commit 'd7c8b539e6576057b44aa38a0bc5d3018ec88b76':
  Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

am 08019f3e: Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

* commit '08019f3e6a2430e0631f2524e577e497e420c04e':
  Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

am 2288c316: am 6efc8233: Fix insufficient buffer size allocation in hci_inject

* commit '2288c31626c17c87a2e3bf636c2620545f3288c8':
  Fix insufficient buffer size allocation in hci_inject

am 6efc8233: Fix insufficient buffer size allocation in hci_inject

* commit '6efc8233c745717562aa6d21bb395494d5d10a01':
  Fix insufficient buffer size allocation in hci_inject

Reduce CPU utilization from ~60% to ~10% on busy HCI reads.

This patch improves RFCOMM throughput and reduces CPU utilization.
Instead of using a counting semaphore to measure bytes and reading
one at a time from the eager reader's buffer, read in bulk based
on the incoming read request size.

Change-Id: I17046bfbc3ca49576a9c82b38911aeb84234881a

Fix insufficient buffer size allocation in hci_inject

A bug in the hci_inject when requesting a buffer to carry the
injected HCI packet. The allocated size should include the BT_HDR
header size.

Change-Id: Ic1fd41264249103140ce50dff843a99a6bd7dbd8

am dd895a61: am d5137a4b: Merge "Rename alarm->creation to alarm->creation_time for readability."

* commit 'dd895a617da78d26fedb30e09267fd2270d4246e':
  Rename alarm->creation to alarm->creation_time for readability.

am d5137a4b: Merge "Rename alarm->creation to alarm->creation_time for readability."

* commit 'd5137a4b82870b5384b748a1a70d26238d2883e0':
  Rename alarm->creation to alarm->creation_time for readability.

Merge "Rename alarm->creation to alarm->creation_time for readability."

am 0fc68b07: am 5e5bc23c: service: Fix clang warnings in uuid_test.cpp

* commit '0fc68b07de18604feb4941f3a300538112c3eba5':
  service: Fix clang warnings in uuid_test.cpp

am 82f1aa27: am a59e8d1f: Merge "Add bthost, a simple BLE GATT server."

* commit '82f1aa271938b8c9f106a513de004bc8f14e77f7':
  Add bthost, a simple BLE GATT server.

am 5de94fbd: am bb954527: build: Fix ptr <-> integer cast warnings

* commit '5de94fbdceb7af83ac6d0eabdbb4c0b67863072d':
  build: Fix ptr <-> integer cast warnings

am fbe40411: am ecb1b43c: osi: Introduce ptr <-> integer conversion macros

* commit 'fbe40411358a3f69601215a718374ca09d776acf':
  osi: Introduce ptr <-> integer conversion macros

am f4b0c924: am cab5fc17: gn-build: Remove all -Wno-unused-* flags from build files

* commit 'f4b0c9242b09316b228234219b67efcbffd7eaec':
  gn-build: Remove all -Wno-unused-* flags from build files

am 5e5bc23c: service: Fix clang warnings in uuid_test.cpp

* commit '5e5bc23ca797f9060e5bf382a0d4a80e4952ceba':
  service: Fix clang warnings in uuid_test.cpp

am a59e8d1f: Merge "Add bthost, a simple BLE GATT server."

* commit 'a59e8d1f46d9881ba3221afcca732241416e6804':
  Add bthost, a simple BLE GATT server.

am bb954527: build: Fix ptr <-> integer cast warnings

* commit 'bb95452741bfd65723ea417c25dbd326c7568ab7':
  build: Fix ptr <-> integer cast warnings

am ecb1b43c: osi: Introduce ptr <-> integer conversion macros

* commit 'ecb1b43c54458de894a570349f371eb4fb2c8b8c':
  osi: Introduce ptr <-> integer conversion macros

am cab5fc17: gn-build: Remove all -Wno-unused-* flags from build files

* commit 'cab5fc179730ce6910989307680b275195876c4d':
  gn-build: Remove all -Wno-unused-* flags from build files

service: Fix clang warnings in uuid_test.cpp

This CL fixes some clang compile warnings and errors and style issues in
service/uuid_test.cpp

Bug: none
Change-Id: I2d1ecf6e21fe4d6b1741f2a113ed5834518a29c6

Merge "Add bthost, a simple BLE GATT server."

Add bthost, a simple BLE GATT server.

This is accessible via Unix socket.
It only has a couple of interesting features:

 * Built in support for large blob attributes (>512 octets)
 * Attribute caching (avoid frame-level IO for IPC clients)

Some string utilies are taken from modp_b64 and Chromium base.

Bug: 21076037
Change-Id: I6a29959159de76f8dd68d6bbaabe2100daabb6fa

build: Fix ptr <-> integer cast warnings

This CL removes the -Wno-pointer-to-integer-cast and
-Wno-integer-to-pointer-cast flags from GN build files. The resulting errors
were fixed using the following:

  1. All ptr <-> integer casts are now done by using the new
     PTR_TO_INT/INT_TO_PTR macros defined in osi.h

  2. The TIMER_PARAM_TYPE macro, defined in gki/common/gki.h and
     include/bt_target.h have been redefined as void* rather than UINT32. This
     is better, since "void*" can act as a simple container without any
     precision loss that would be caused by a type such as UINT32 on 64-bit
     systems. void* inherently is a safer container for all pointer types but
     UINT32 isn't.

BUG=21570302

Change-Id: I4a82c4a40c91caa31e372382c40d424be220cbe3

osi: Introduce ptr <-> integer conversion macros

This CL introduces macros for safe pointer to integer conversion (and vice
versa). Also fixed a small style issue in log.h

Bug: 21570302
Change-Id: If76bf5e35970f9b33f9bef53fbd03a7effae08dc

gn-build: Remove all -Wno-unused-* flags from build files

This CL removes the -Wno-unused-variable, -Wno-unused-value, and
-Wno-unused-function flags from all GN build and Android.mk files
(except in test/ & tools/, which are not built using GN yet). Compile
warnings, caused by the newly enabled flags, have been fixed throughout
the code base.

Bug: 21570302

Change-Id: I1c128b62256faa17668124798341005bcd0f7315

am 9df517bb: am 74626115: Re-enable the OSI memory allocation tracker.

* commit '9df517bb2e81f29e51a4da8b8dd436c7649b6211':
  Re-enable the OSI memory allocation tracker.

am 74626115: Re-enable the OSI memory allocation tracker.

* commit '74626115db17478a12d1d39bcefd5af500fadfda':
  Re-enable the OSI memory allocation tracker.

am 56c96ff1: am 1ab6fd76: Revert previous commit and disable net_test_hci under sanitizers.

* commit '56c96ff15132fa0494d9553ba3594b5a2ba77e31':
  Revert previous commit and disable net_test_hci under sanitizers.

am 1ab6fd76: Revert previous commit and disable net_test_hci under sanitizers.

* commit '1ab6fd76ae670029fa3811c93878329dc793cde3':
  Revert previous commit and disable net_test_hci under sanitizers.

am bb355edb: am 506c366c: Fix ASan build.

* commit 'bb355edb31589397b3f15ac20ea0a228f08023f7':
  Fix ASan build.

am 506c366c: Fix ASan build.

* commit '506c366cc3a1b33bc785f5f007c16ae4bff784c8':
  Fix ASan build.

Revert previous commit and disable net_test_hci under sanitizers.

Change-Id: Ifd544ef241f622c647ecea1026025049a470bd16

Fix ASan build.

Not sure why, but with ASan I get missing symbol GKI_freebuf.
Indeed, it is defined in a library that's not in the target's deps.

Bug: 21785137
Change-Id: I595f3e3df6ad5dde3f4f8570426479050b1552fa

Re-enable the OSI memory allocation tracker.

Previously, the OSI memory allocation tracker was disabled as
a temporary workaround for Bug 21561735.
The root cause for the original issue is now addressed by using the
appropriate linker flags when generating the bluetooth.default.so
shared library:

  LOCAL_LDLIBS := -Wl,-Bsymbolic,-Bsymbolic-functions

Also, added missing #include in file "btif/src/btif_sock_sco.c"

Bug: 21561735
Change-Id: I384a6f3d1d03c74f8d1e3c96bf7e0b5961206feb

Rename alarm->creation to alarm->creation_time for readability.

Change-Id: I007b2211d9b52422b4442765bb97c0ba90613c50

am 979e3421: Fix mnc-dev <-> master merge errors

* commit '979e3421c8111a56ebfcf176f6383df58eed52ea':
  Fix mnc-dev <-> master merge errors

am 6fc48327: am 625ec341: gn-build: Fix GN build after the merges

* commit '6fc4832730341b0ba944e5de72ffe5171089f8b7':
  gn-build: Fix GN build after the merges

am afa8dc8b: am ab813119: (-s ours) Fix more BLE_INCLUDED == FALSE build breakage

* commit 'afa8dc8b5afe17e26c4e6a8200571bf177a4dd55':
  Fix more BLE_INCLUDED == FALSE build breakage

am 422a4c14: am 087d1acc: Add missing include.

* commit '422a4c147ceb4c726b7be978c0d9c868c3aa7a95':
  Add missing include.

Fix mnc-dev <-> master merge errors

Change-Id: If4d56229de7d47289560a12205feb7e33ac6fcf5

am 625ec341: gn-build: Fix GN build after the merges

* commit '625ec34106a70d9299c68593279a0d9c77bfd87f':
  gn-build: Fix GN build after the merges

am ab813119: (-s ours) Fix more BLE_INCLUDED == FALSE build breakage

* commit 'ab813119d61a3ca8ac8ecb5239b73bf77e2c05bb':
  Fix more BLE_INCLUDED == FALSE build breakage

am 087d1acc: Add missing include.

* commit '087d1acc3f41829e9ce982e6172260d35527e85e':
  Add missing include.

gn-build: Fix GN build after the merges

This CL fixes the GN build after the recent merges to aosp/master.

Change-Id: I0c9d8178986165d5a4a67261532e8a08b6923d22

Fix more BLE_INCLUDED == FALSE build breakage

Change-Id: I2da6dcdaced78d3a3bce6002fc3885c7480fba82

Add missing include.

osi_malloc was never included, so it was implicitly defined as
returning an integer (thank you, C). Clang caught the int to pointer
conversion error.

Change-Id: I0d8a3363d92cb72a22780d049a53e193d2e50ec5

am d0e14be8: am 53ef6df3: Check transport before subscribing for service changes

* commit 'd0e14be82bc0bdd54edc1f3830174cc67e27b86e':
  Check transport before subscribing for service changes

am 5196d7ca: am 2a527108: Add proper checks for PAN & BNEP in BD stack

* commit '5196d7caa474471370f735c375da157c147807b5':
  Add proper checks for PAN & BNEP in BD stack

am 2e58ca8c: am 4633fb8d: Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

* commit '2e58ca8c367717fda1d4831d15399069cc6f93e8':
  Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

am c3165ecd: am 335c2087: Ignore STOP command if A2DP stream is suspended

* commit 'c3165ecd2d6d9a94baa133f1472fa68a40bbc8de':
  Ignore STOP command if A2DP stream is suspended

am 0d4b8f6a: am 5e45edb7: Remove check for stream type in a2dp hal_callbacks.

* commit '0d4b8f6a2fae051cf341b24c81961fb85cc4db6c':
  Remove check for stream type in a2dp hal_callbacks.

am ac65d8b3: am fc0a9f61: Clear remote BD address information.

* commit 'ac65d8b3cd99e6dc1ef745b132157d355d5b1da4':
  Clear remote BD address information.

am 158b9760: am c1c26aae: Don\'t stop sniff timer before checking for service ID entry

* commit '158b97603dad919f2376f961af64dc46da99db2f':
  Don't stop sniff timer before checking for service ID entry

am a969944b: am b3997013: Don\'t initiate sniff in sniff mode

* commit 'a969944bba8e707790ab358cf32be9a28b1e9a5a':
  Don't initiate sniff in sniff mode

am 88542ba7: am 45d4c3e3: Updated ATD string parsing

* commit '88542ba796ed4db89116c50ebc9bdad7a0904d5e':
  Updated ATD string parsing

am 8786e15b: am e6f4ed85: Stop AG collision timer.

* commit '8786e15b222e759d98589e6497ec986903b9f327':
  Stop AG collision timer.

am bb85d173: am b0765000: Return ERROR for invalid CHLD command arguments

* commit 'bb85d173cf20f59e4d402b3adb0a6b6386d8badd':
  Return ERROR for invalid CHLD command arguments

am e63921d8: am c3aa28eb: Reject second AVRCP connection.

* commit 'e63921d870870aff552c356647cf09a358c5fb98':
  Reject second AVRCP connection.

am fa56d2d1: am b9851818: Disable SCMST feature for A2DP Sink

* commit 'fa56d2d1e47a1147075f39f6ece4991ca71b7ec5':
  Disable SCMST feature for A2DP Sink

am 45529f12: am 9b6dfa44: Restarting PM timer for AG while reading OR writing

* commit '45529f12d1d93291d641c0f236befed66c0b1e6a':
  Restarting PM timer for AG while reading OR writing

am 65c522c4: am b92ab7d8: Initialize btif_max_hf_clients with proper value

* commit '65c522c436a18f2294dff5fc8fc5a3a46bc779eb':
  Initialize btif_max_hf_clients with proper value

am fa8c203c: am 9c264e92: Set the proper authentication level for AvOpen

* commit 'fa8c203cdfd8b712e93147a7a1b87f5eb4606b48':
  Set the proper authentication level for AvOpen

am b376f951: am 6b359689: Don\'t open SCO for 2nd MO call if a call is active

* commit 'b376f9510bd883c855d7c7a79580b9c01b8d3eb8':
  Don't open SCO for 2nd MO call if a call is active

am 611d71c9: am 4f51fc36: Don\'t close SCO for MO calls

* commit '611d71c9be20e1c9057d75ab9cfd1b76531f15a8':
  Don't close SCO for MO calls

am 4e0353e8: am 684d1364: (-s ours) SCO connection should happen after SLC.

* commit '4e0353e8ebdbfe220bf2be47f17193e30f4eb73d':
  SCO connection should happen after SLC.

am 53ef6df3: Check transport before subscribing for service changes

* commit '53ef6df3e40846528a561f762515d1e628077c19':
  Check transport before subscribing for service changes

am 2a527108: Add proper checks for PAN & BNEP in BD stack

* commit '2a527108d22861aa4091143c740e09235b3692c5':
  Add proper checks for PAN & BNEP in BD stack

am 4633fb8d: Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

* commit '4633fb8d4b0376474f6d38c58ee5ee563a392ef5':
  Fix issues in A2dp, Avrcp, HF and AG reported by static analysis tool

am 335c2087: Ignore STOP command if A2DP stream is suspended

* commit '335c208713b3c9eadcc301acac947b2f96e31da6':
  Ignore STOP command if A2DP stream is suspended

am 5e45edb7: Remove check for stream type in a2dp hal_callbacks.

* commit '5e45edb7f50497436d00423327c55707588211eb':
  Remove check for stream type in a2dp hal_callbacks.