Merge "Retry pairing if the device did not respond (page timeout)" into lmp-dev
automerge: 25249f7

* commit '25249f7663d75244b5a2abcac08e301d584fcd8d':
  Retry pairing if the device did not respond (page timeout)

A2DP: Lowering maximum PCM frame limit to avoid audio breaks
automerge: 01d102b

* commit '01d102b3b187ab0d24050c1e12f96e6ee1ce50be':
  A2DP: Lowering maximum PCM frame limit to avoid audio breaks

Merge "Retry pairing if the device did not respond (page timeout)" into lmp-dev

A2DP: Lowering maximum PCM frame limit to avoid audio breaks

Some car kits (notably Renesas car kits used in Nissan/Infiniti models)
will throttle incoming A2DP frames if too many frames are sent at once.
This patch lowers the maximum number of frames sent per tick to not
overwhelm the remote device.

Bug: 17880920
Change-Id: I7bfbba625405c96159d1c94eebd8dc498f100daa

Retry pairing if the device did not respond (page timeout)

Implements a "blacklist" identifying troublesome remote devices which
may require multiple pairing attempts to succeed.

Change-Id: I11b6b7da7230e37fbd5ec4c07cd38b30dc12b40e

am 9d8df2d1: Merge "LE: Serialize multiple connection parameter update requests" into lmp-dev

* commit '9d8df2d1c4ad40fa49e7e1c1031421d8a194d927':
  LE: Serialize multiple connection parameter update requests

am d8586177: Merge "Implement GATT service cache" into lmp-dev

* commit 'd85861779fb900b1c8ac2c6597a3cc111f161dcb':
  Implement GATT service cache

Merge "LE: Serialize multiple connection parameter update requests" into lmp-dev

Merge "Implement GATT service cache" into lmp-dev

Implement GATT service cache

When trying to reconnect a previously paired device, notifications may
be sent by the remote device before the characteristic handles have been
discovered. Thus notifications will be lost.

This patch adds a non-volatile attribute cache that allows attribute
handles to be restored before notifications are received.

Bug: 17999991
Change-Id: I97faefbc6a2fed86cbce7f64d620ed03944d89b2

am f777d24c: am 595bcb4c: (-s ours) DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

* commit 'f777d24c24b3c1ff309151cf2bc21e98559cbf56':
  DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

am 595bcb4c: (-s ours) DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

* commit '595bcb4c72aaa2a9a54a23b127dd5fa9842876ca':
  DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

DO NOT MERGE Do not set sig_id in gerneral reject message per AVDTP SPEC 1.2

AVDTP SPEC 1.3 set sig_id in gerneral reject message. But current stack
is on AVDTP 1.2, we need set the field to be 0.
Setting sig_id would fail PTS test TP/SIG/SMG/BI-28-C

Bug: 17892440
Change-Id: I2037aae299eab15864fc7b53ada7f49c0f34b9c5

am 4a35476c: am 98ad505f: Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

* commit '4a35476c4ade75b985fa46f1ab82dad92a6b0810':
  Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

am 98ad505f: Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

* commit '98ad505f577599870cf0ae281a5d035e2e2453f9':
  Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

LE: Serialize multiple connection parameter update requests

Bluetooth controllers do not allow multiple connection parameter
updates where there is one outstanding. This requires serializing
all connection parameter update requests and send the latest
request when the pending one is finished.

Bug: 17979666
Change-Id: Ic49bd89a4ded28014d74b40e67f2a3ea490b9ef7

Increment seq before adding to packet in avdt_scb_hdl_write_req_no_frag

Bluetooth Certification Test case TP/TRA/BTR/BV-01-C for Basic
Transport Service Support fails. The failure is because the PTS
is expecting the first RTP Media Frame to have a sequence number
of 1. But metallica sends it as 0.
bug 17893538

Change-Id: Idfe7ae382c3764616dab4733cdf83b1627b8b3c9

am 12ca0ec5: Merge "Added missing A2DP priority on java alarm service" into lmp-dev

* commit '12ca0ec54796e31405a880de6675015c88857158':
  Added missing A2DP priority on java alarm service

am 1fbddc8f: Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

* commit '1fbddc8faacde61190df31a8e6ea9142feb4bc88':
  Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

Merge "Added missing A2DP priority on java alarm service" into lmp-dev

Added missing A2DP priority on java alarm service

Running GKI timer engine via JAVA alarm service was causing severe
timer preemption at times which affects A2DP media task timer. This in
turn was stressing the compensation logic towards UIPC resulting in
very bursty reads of up to 40 UIPC frames at a time. This led to
instabilities on the UIPC pipe and trickled upwards in audio framework.
Inclusion of java alarm thread in A2DP high priority thread list results
in a much smoother data flow for entire A2DP data path and improved
audio quality.

Bug: 17520043
Change-Id: I7cea6ec64aa5b973b5fff767389a8827e681619e

Revert "Unable to connect to a Motorola s705 until Bluetooth was toggled of and back on"

This reverts commit a934f018b6c6a7eda4662a60f2f04828afc3ab77.

This patch caused various headset audio break issues and slow
reconnections on headsets like the Bose AE2W.

Based on the original bug under which this patch was merged
(b/10648781) it is unlikely that this bug even fixed the
issue it was meant to address.

Bug: 17520043
Change-Id: I39bb9abbf5735abff25b5257425c34972287dc27

am ebc38ac1: am ba2f6d99: (-s ours) Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

* commit 'ebc38ac1acf8629d5a641c4687dd97ff1cf768e0':
  Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

am ba2f6d99: (-s ours) Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

* commit 'ba2f6d995d058bc9e77d860d0d755e4ae6086deb':
  Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

Revert "anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer"

This reverts commit c1ccb0d9854cea4b2cd09bbd57fc1e1081eb96b9.

This change looks like a likely candidate for the culprit behind
a wide range of BT issues we're seeing across many devices.
Reverting this in order to test.

am 5f0196cc: am 46bfe63d: (-s ours) DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

* commit '5f0196cc32e07f3db5969101d5b0da1e0434d84b':
  DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

am 9c9c4bf7: am c1ccb0d9: (-s ours) anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

* commit '9c9c4bf7b879c5f5580cba02ffe6649ff433b806':
  anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

am 46bfe63d: (-s ours) DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

* commit '46bfe63d383fd077e521d69321e011aff824bd39':
  DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

am c1ccb0d9: (-s ours) anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

* commit 'c1ccb0d9854cea4b2cd09bbd57fc1e1081eb96b9':
  anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

am de898558: Merge "Fix an "array index out of bound" bug for RPA." into lmp-dev

* commit 'de898558badd0d575829c4011fdffd33c548c1d6':
  Fix an "array index out of bound" bug for RPA.

Merge "Fix an "array index out of bound" bug for RPA." into lmp-dev

am 19a8f456: Merge "Raise A2DP threat priority to avoid music breaks" into lmp-dev

* commit '19a8f456d4a38694eaec270eabc5ff9aa639c86d':
  Raise A2DP threat priority to avoid music breaks

am d5aa2475: LE: Peer device\'s latest RPA is not being applied

* commit 'd5aa24750c2202ce944560c2c27ead0214e8350d':
  LE: Peer device's latest RPA is not being applied

Merge "Raise A2DP threat priority to avoid music breaks" into lmp-dev

LE: Peer device's latest RPA is not being applied

The latest discovered address is not used when reconnecting to devices
using resolvable private addresses, if RPA resolution offloadin is not
supported in the controller.
This patch also address the exception case where BT off is memseting
all control value while the IRK list is still being cleaned up.

Bug: 17187661
Change-Id: Ifeb5b8297e7eadb0e34ec1afbd41f46bdd836cf3
Author: Chaojing Sun <cjsun@broadcom.com>

Raise A2DP threat priority to avoid music breaks

* Ensure all a2dp audiopath threads are configured to high
  priority when actively streaming.
* For now set bt hc worker thread always to URGENT_AUDIO
  until new thread api can adjust thread priority dynamically.

Bug: 17520043
Change-Id: I10e314085329278bcfcc4e8bc5ce899b03d22e2b

DO NOT MERGE AVRCP: Proper handling of Fragmented AVCTP packet

- AVCTP : Allocate bigger buffer for reassembly
- AVRCP : Handle getelimentattr to eliminate duplicates
          and invalid attributes, this also fixes BOF.

Bug: 17669579
CRs-Fixed: 576831
Change-Id: I778c3df1386339ff8a00a755ffc9f24881b5167d

Fix an "array index out of bound" bug for RPA.

Also returns no more than BTM_BLE_MULTI_ADV_MAX for max advertise
instances.

Bug:17810619
Change-Id: If655e207b641fd4cd1590ae47b28185dc9f7cbbb

am 4baec1a8: Merge "Remove HOGP device from background connection list" into lmp-dev

* commit '4baec1a8b7987f6d46351ea17d359e23d0e17eb2':
  Remove HOGP device from background connection list

am 1cd92393: Null check for irk entry

* commit '1cd92393c162afe68b01aafcf7a1d41ab1d21e7b':
  Null check for irk entry

Merge "Remove HOGP device from background connection list" into lmp-dev

Null check for irk entry

When BT is turning off btm_ble_vendor control block is cleaned up.
So irk_entry which points to the first element of irk_list is NULL
Included a null check for irk_entry to prevent stack crash.

Bug 17399097

Change-Id: Ib129f7dd06f9db61cc54257aa4a563fe2c9f41f2

anthias:17614185:Bluetooth: Getbuf returning invalid buffer pointer

b/17623865

Getbuff is checking for the pool which can accommodate
requested size, but it changing the pool with a restricted
check and not checking for the free memory size in that pool.

Added a check to avoid return the wrong pool free buf pointer
and avoid gki get_buffer overflow.

Change-Id: I142870c989fc37f49c2d9b411d5eddbfba59bd9d

am 03cb1ad1: Merge "Enable flow control when tx buffer is full and peer stuck in receiving" into lmp-dev

* commit '03cb1ad1e063f7353c5098527bbf13ae7d563a7e':
  Enable flow control when tx buffer is full and peer stuck in receiving

Merge "Enable flow control when tx buffer is full and peer stuck in receiving" into lmp-dev

Remove HOGP device from background connection list

This adds logic to remove the device from whitelist when user unpairs or
disconnects device.
Bug: 17318494
Change-Id: I43c06dd16c665f9ce2e71a0e031c4f0f11a4012e

am 8b52f5f2: Merge "Suppress the warning compiled with gcc-4.9"

* commit '8b52f5f2f0eb1e3748a19af4438147872cba1cd2':
  Suppress the warning compiled with gcc-4.9

Merge "Suppress the warning compiled with gcc-4.9"

am 94c31e55: Merge "Fix of memset usage"

* commit '94c31e5518de3cfb610aae74835d5cc0a6d092d2':
  Fix of memset usage

Merge "Fix of memset usage"

am 3f24f460: All 00 mac address with advt

* commit '3f24f460ec3efedde25723e6a4297efc10873cd2':
  All 00 mac address with advt

Fix of memset usage

Build process failed following error (GCC 5.0):
btif_storage.c:1503:9: error: 'memset' used with constant zero length parameter; this could be due to transposed parameters [-Werror=memset-transposed-args]

Change-Id: I46e0d8bd6fcd9439b50b4f0aed5391c808b9eb73
Signed-off-by: Arseniy Antonov <arseniy.antonov@intel.com>

Suppress the warning compiled with gcc-4.9

Fix the warning below:

  gatt_sr.c: In function 'gatt_sr_process_app_rsp':
  gatt_sr.c:175:27: warning: 'p_rsp' may be used uninitialized in this function [-Wmaybe-uninitialized]
                     p_rsp = (tGATTS_RSP *)GKI_getnext (p_rsp);
                           ^
  gatt_sr.c:125:23: note: 'p_rsp' was declared here
     tGATTS_RSP       *p_rsp;
                       ^

Change-Id: I894495ed7893e958f009778e629525b85d56b2f6

All 00 mac address with advt

Read the controller features before local name read

Bug 17505545
Change-Id: I7995a8c714ebec6e1751af5371edfd29c74b12ea

am d1e6922c: Check userial fd before reading

* commit 'd1e6922ce963c90d19e81abdc1d305ccd0e65d7e':
  Check userial fd before reading

Enable flow control when tx buffer is full and
peer stuck in receiving

bug 17525124
Change-Id:Ic67e9c0aab7e64afc9679733e6c3de10a28078d7

Check userial fd before reading

The userial read loop uses one global flag
to condition the serial port read and a
second global file descriptor variable that
may change from another thread underneath it.

Check the validity of the serial port file
descriptor by making a copy immediately before
the read and only proceeding if it's a valid fd.

bug: 17552690

Change-Id: Iaaf1b8476fdee462ddc178d2a31ef8f55cb33e52

am 13e916d7: Merge "Add ability to preserve existing snoop log" into lmp-dev

* commit '13e916d7e120796a45f3bf109ff959d7f4c42f0f':
  Add ability to preserve existing snoop log

Merge "Add ability to preserve existing snoop log" into lmp-dev

Add ability to preserve existing snoop log

Currently a stack restart or device reboot will overwrite existing snoop
logs when the stack restarts. This patch adds a new configuration file
parameter "BtSnoopSaveLog" that, if enabled, will cause an existing
snoop log to be renamed to "<snoop_file_name>.last" before a new snoop
log is created.

This will allow for easier debugging of crashes that involve a reboot or
stack restart.

Change-Id: I4f56204d31a6bf2e96058938e8616eb6dee7360b

am 303d4ac5: Merge "For pairing over le transport, set default device type and address type" into lmp-dev

* commit '303d4ac5ff7758ab2cdde4040cb8db6cf37552f5':
  For pairing over le transport, set default device type and address type

Merge "For pairing over le transport, set default device type and address type" into lmp-dev

For pairing over le transport, set default device type and address type

When the LE pairing is initiated over NFC, skipping device scan, there is
no device type or address type info. We set the default as le and public.
bug 17497942
Change-Id: I3dcbb9029ead6ea0f7c48e62f46c8a055f1bc293

am c918d087: Document LE IO capabilities and use BR/EDR constant for LE

* commit 'c918d08763ec547f7f226553ab75ca885db357ac':
  Document LE IO capabilities and use BR/EDR constant for LE

Document LE IO capabilities and use BR/EDR constant for LE

This removes a hack to set LE IO capabilities and reuses the existing
constant defined in bt_target.h.

Change-Id: I3c97629bd092ac78bd673afc29458775b2bce836

am 24adb37e: host cannot recover when connection exceed controller capability

* commit '24adb37e4106bf8544c7729d34451fdf2777c4dc':
  host cannot recover when connection exceed controller capability

host cannot recover when connection exceed controller capability

Host can not initiate new connection when exceed controller capability
and connection then drop back to lower than the max. Controller will not
generate connection complete event when initaite LE ll connection request
failed, and host is dead loop waiting for the connection complete
then can not recover. Included a connection complete callback with failure
status when controller capability is exceeded.

bug 17326529
Change-Id: I9b863578dc868456e092f1d060e50d2c9fdcb441

am e3b1d0da: Merge "Correct the OBEX OPP Profile version"

* commit 'e3b1d0da533fbb01da67f7ccfa976a9e3d54ed94':
  Correct the OBEX OPP Profile version

Merge "Correct the OBEX OPP Profile version"

am 7ae25156: Avoid premature LE connection notifaction to app

* commit '7ae25156f4250af9d0e8cd8d559d916cfb847feb':
  Avoid premature LE connection notifaction to app

am 976990f8: Merge "Fix issues with manufacturer ID prefix scan filter." into lmp-dev

* commit '976990f872bb0873fedbb74649d0bf704bb70d05':
  Fix issues with manufacturer ID prefix scan filter.

am 1a5f8105: Merge "Fix ADV data exceeding max adv packet length" into lmp-dev

* commit '1a5f8105337cd41efad09d4ec8812311e8adaaec':
  Fix ADV data exceeding max adv packet length

Avoid premature LE connection notifaction to app

To avoid premature connection notification to app, always wait for
LE read remote feature complete with success before the connection
event is sent to application.

This patch also deprecates the BTA_SKIP_BLE_READ_REMOTE_FEAT flag.
Since the remote feature request is now serialized and start encryption
won't be called until the connection is up, this flag is no longer
necessary and interferes with the new behaviour.

This is an extension of previous CL

Bug: 17326529
Change-Id: Icfd4c5dfdd9f89d1318ef429e132eb005abb1f64

Merge "Fix issues with manufacturer ID prefix scan filter." into lmp-dev

Merge "Fix ADV data exceeding max adv packet length" into lmp-dev

am a015a834: LE: Use random address when initiating background connections

* commit 'a015a8348b9e09caf5261689f4c288a4d2ff6acd':
  LE: Use random address when initiating background connections

LE: Use random address when initiating background connections

When re-connecting to HID devices, the first reconnection immediately
after a reboot incorrectly uses the static, public Bluetooth address of
the phone/tablet as the own address.

This patch fixes the Bluetooth initialization sequence to make sure
privacy is enabled before any Bluetooth services are started.

Bug: 16952502
Change-Id: I43e595ddc7e347fab972b059382b512fb03e9459

Fix ADV data exceeding max adv packet length

Fixed the 128-bit UUID handling during setting up
of adv data/scan response

Bug:16799331
Change-Id: I81e4ab0a80a5c1b7abf012dccc29eead54229b21

am 53c9fd05: Merge "LE link_xmit_quota is not correctly allocated" into lmp-dev

* commit '53c9fd05c28d035cf3d2530b586d3d02b68a9866':
  LE link_xmit_quota is not correctly allocated

am 8dedc069: Merge "SMP doesn\'t detect the event of local processing taking more than SMP timeout (30 sec)" into lmp-dev

* commit '8dedc069c0b9183ae386d2cc413b0116720ef879':
  SMP doesn't detect the event of local processing taking more than SMP timeout (30 sec)

Merge "LE link_xmit_quota is not correctly allocated" into lmp-dev

Merge "SMP doesn't detect the event of local processing taking more than SMP timeout (30 sec)" into lmp-dev

am 000abb5e: Merge "LE: Remove stored link key if encryption fails due to missing key (remote)" into lmp-dev

* commit '000abb5e7be8c60c66d5ac7319362be448078121':
  LE: Remove stored link key if encryption fails due to missing key (remote)

LE link_xmit_quota is not correctly allocated

LE link transmit quota should be calculated based on the total LE
buffer available and number of LE link that shares the buffer. Currently
it's been calculated based on the ACL buffer number and total physical
links(BR and LE), that cause the LE link transmit quota not being
done right.
Modify the link transmit quota allocation, and keep track of LE link
seperately, also keep track of round robin scheme for LE links
solving the problem.

Bug: 16560957
Change-Id: I70f91827b96f2c10421e91311ea34cc81a5ef4b8

Merge "LE: Remove stored link key if encryption fails due to missing key (remote)" into lmp-dev

am d8f8e85d: Merge "If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection" into lmp-dev

* commit 'd8f8e85df529864286cfc31be8bd34dc2dcb4dae':
  If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection

Merge "If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection" into lmp-dev

am 98283bb8: Fixed SMP pairing failure due to race condition

* commit '98283bb879d26d0c29a76988e1bffebd6f730fc5':
  Fixed SMP pairing failure due to race condition

Fixed SMP pairing failure due to race condition

SMP pairing failure if remote send security request before master
side receive LE link connection callback. The racing condition will
cause master side bonding failure upon ignoring remote security request
and flush out all SMP information. Retain SMP information when a
locally initiated pairing is in process solves the problem.

Bug: 17412687
Change-Id: Ia2256160c866413f93c6f49e5db82b4c6489d9d2

SMP doesn't detect the event of local processing taking more than SMP timeout (30 sec)

Bug: 16560957
Change-Id: I1113f367187a30e01fa3d5d2a5ccb0243580e06b

If L2CA_ConnectFixedChnl fails, free L2CAP lcb for the attempted connection

Bug: 16560957
Change-Id: I91fd2cc90435d0b899474b555b4a4ab00306c156

am f5140468: Merge "LE: Disable background connections if RPA offloading is not supported" into lmp-dev

* commit 'f5140468dc83dafcd75ffe607e1a56c514b9a8a4':
  LE: Disable background connections if RPA offloading is not supported

Merge "LE: Disable background connections if RPA offloading is not supported" into lmp-dev

LE: Disable background connections if RPA offloading is not supported

If the local device does not support RPA offloading, background
connections to a resolvable random address will not work. Added callback
to indicate to the application writer that the requested operation is
not supported.

Bug: 16831527
Change-Id: I083d66b16704e997f52a40ea4daced3056b50ad5

Fix issues with manufacturer ID prefix scan filter.

Handled the case of no manufacturer data in filter configuration

Bug:17324059
Change-Id: I87c2c44644f808bd88f4d8681ff53e2d92815658

LE: Remove stored link key if encryption fails due to missing key (remote)

Remove the link key if encryption fails due to missing key or
authentication error. When a remote device loses the bond, all
HOGP services cached should be discarded and all client configuration
descriptors needs to be re-enabled. This will ensure successful
re-connection.

Bug: 15022622
Change-Id: Ie1730830c0cbbc294e2cd579d7dca69d16cf8218

am 7bf480e3: Notify that link is up after LE read remote features completes

* commit '7bf480e3e3e1df04c0be6d466d8dffac5abaae40':
  Notify that link is up after LE read remote features completes

Notify that link is up after LE read remote features completes

ACL link up notifaction was not sent after LE read remote features was successful.
Thus, total ACL link up count was not updated. Included a link up notification
after LE read remote features is successfull.

Bug: 17410697
Change-Id: I41fc640e76fa99354cb63d9b0b3031582178c057

am c8eddb5a: Make sure conn_id is valid before using it

* commit 'c8eddb5ae46da76552d3cd3d9fff8d33c1880a52':
  Make sure conn_id is valid before using it

Make sure conn_id is valid before using it

Bug: 17401266
Change-Id: I5f36e13b3ccb327f191db33123fff98858f07c5f