Raise alarm callback thread priority am: 41a91a52b0
am: ab6ec2bc93

* commit 'ab6ec2bc930e080aae9cb5c3d82650624f512a04':
  Raise alarm callback thread priority

Reduce persistence on aquiring master role am: 77e70b4b2d
am: 752c1cd5bc

* commit '752c1cd5bcc298e9daeacaf64244ffbb5059e6c7':
  Reduce persistence on aquiring master role

Raise alarm callback thread priority
am: 41a91a52b0

* commit '41a91a52b06171140938a1aa503bab8fe15e12e4':
  Raise alarm callback thread priority

Reduce persistence on aquiring master role
am: 77e70b4b2d

* commit '77e70b4b2d0b4bf1fc319a705701c1726589b2fc':
  Reduce persistence on aquiring master role

Raise alarm callback thread priority

Since alarms [/callbacks] are used in the critical audio path, raising
the thread priority is required to ensure timely encoding/transmission
of audio packets and avoiding thread pre-emption.

This patch indiscriminatly raises the (single) callback thread priority,
affecting all alarm callbacks. In the future, thread priotiry should be
adjusted based on the alarms pending in the queue.

Bug: 24570959
Change-Id: I64928fd7325f1d6f76815a185dbccf1e01ae30d2

Reduce persistence on aquiring master role

Whenever a device has more than 1 ACL link active and transferring
data on one of its links via PAN, HH or JV (RFCOMM) the sys busy/idle
state toggles frequently. To avoid triggering role switches for each
of these events we filter this out and let the other SYS events handle
it in combination with other role policy management code. Ideally we
should revert the toggling to properly reflect the busy/idle state of
each profile but to limit risk of side effects at this stage we will
make this intermittent change.

This also affects audio streaming in certain cases.

Bug: 24570959, 25129209
Change-Id: I141e17ee069c82624e153fd8de5db90ae93724b9

Merge "Fix broken test caused by misordered callback arguments." into cw-e-dev
am: bedd614381

* commit 'bedd6143818b91e47b44e4fe67835594d10b62e1':
  Fix broken test caused by misordered callback arguments.

Always update remote address type during LE scan am: 03d171170c
am: 02ddf25bd0

* commit '02ddf25bd0e7f5af6205957b8a63e06378141632':
  Always update remote address type during LE scan

Merge "Do not mask out secure connections (SC) bit for BT <4.2" into mnc-dr-dev am: 15c1f685db
am: 442e504115

* commit '442e5041158a6c2af28e570b245d2cccdc048a8e':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup am: 7312c6d913
am: 50d8df7a33

* commit '50d8df7a33e19e7fb753ffbfe7f6a982fc76488a':
  Fix bug in SDP 128-bit UUID lookup

Serialize remote version query over LE am: 864e7734f2
am: b2bf772af1

* commit 'b2bf772af125597ecc7ec36f434728ef8a942aff':
  Serialize remote version query over LE

Check remote LMP version before enabling secure connections am: 4f7c4f6732
am: 406a266f13  -s ours

* commit '406a266f137f88ab34ac3d6425614ba5350878df':
  Check remote LMP version before enabling secure connections

Fix flaky EagerReaderTest.test_large_data_multibyte am: b74ef93233
am: 03b1e30775

* commit '03b1e30775ced16c82a3ccfa5dd2f2ef8e35ea38':
  Fix flaky EagerReaderTest.test_large_data_multibyte

Fix flaky EagerReaderTest.test_large_data_multibyte
am: b74ef93233

* commit 'b74ef9323325068f43e7f7765cd8b858881ab197':
  Fix flaky EagerReaderTest.test_large_data_multibyte

Fix flaky EagerReaderTest.test_large_data_multibyte

eager_reader_read() makes no guarantee that the number of bytes read
equals the number of bytes to be read. Insisting that it must in the
test can lead to race conditions where not all data is read at once and
the test fails.

Adjusted test to match API promise.

Change-Id: I5df4b6d8df61d0601402511bb3d9f29f22378981

Pass unknown run_unit_tests long arguments to tests am: 6babef0456
am: bc2b435ab4

* commit 'bc2b435ab416d71ddcb0021ae9a9ab1da99aa43c':
  Pass unknown run_unit_tests long arguments to tests

Pass unknown run_unit_tests long arguments to tests
am: 6babef0456

* commit '6babef04564490d0843504f7abe9f5217570f5ef':
  Pass unknown run_unit_tests long arguments to tests

Add test filter option to run_unit_tests am: 753c074b44
am: 138f65e659

* commit '138f65e6596a734eac9c7dc81d74041bb22393e2':
  Add test filter option to run_unit_tests

Add test filter option to run_unit_tests
am: 753c074b44

* commit '753c074b449596f80224dec367816366fbd9bfe9':
  Add test filter option to run_unit_tests

Pass unknown run_unit_tests long arguments to tests

e.g. ./run_unit_tests.sh --v=2
     ./run_unit_tests.sh --gtest_filter=*Advertise*

Change-Id: I63e7549d850a15107ca56333868cc8ccd3eeb893

Give run_unit_tests an exit code am: 0ff191abbf
am: a3feca655c

* commit 'a3feca655cd2a6a38ddef02b8394655a248316cc':
  Give run_unit_tests an exit code

Add test filter option to run_unit_tests

e.g. ./run_unit_test.sh net_test_hci.HciLayerTest.*

Useful since this script wraps the hard part of pushing the binary
to the device and then shell to running it.

This also allows multiple individual tests to be run, since the filter
is per-target rather than global.

Change-Id: I74a6511ab226fb55054cbb840bb47815e0fdc97e

Give run_unit_tests an exit code
am: 0ff191abbf

* commit '0ff191abbfa3d23d4e8a81cb674509580873bdc2':
  Give run_unit_tests an exit code

Give run_unit_tests an exit code

So you can tell whether tests failed by checking $?

Change-Id: I0eb55c49946a4183206bd68cd3c0b1bdc69f6bb0

service/test: Reset command-line on SetUp am: 8531fc3571
am: 1c13443412

* commit '1c1344341213eac001eb0d5127f319c8515d8b60':
  service/test: Reset command-line on SetUp

service/test: Reset command-line on SetUp
am: 8531fc3571

* commit '8531fc357121b8e2fc0135b00f7703205e64e7e7':
  service/test: Reset command-line on SetUp

service/test: Reset command-line on SetUp

Changed SettingsTest to reset the global base::CommandLine on
SetUp so the the first test case that runs follows a clean state.

Bug: 25753839
Change-Id: I0d536e1f85036fccd0099e03f7f59289f5411c50

Merge "Memory overwrite due to HDP doesn't allocate enough buffer"

Add -s <specific device> to run_unit_tests am: 03e8be8b11
am: 5d384a505a

* commit '5d384a505ab85e6a0f9736a0c2077ac482103f4b':
  Add -s <specific device> to run_unit_tests

Add -s <specific device> to run_unit_tests
am: 03e8be8b11

* commit '03e8be8b11c4e277a7c4600847a49a4b152b40a7':
  Add -s <specific device> to run_unit_tests

Add -s <specific device> to run_unit_tests

For the developer with more than one Android or Brillo device plugged
into their workstation at once.

Change-Id: Ice273564f2debdc0708ce473be6119ff2ae33f39

Revert "Enable kernel wakelocks and timers"

This reverts commit 28bf007f7103ccd36d5fa2f87837947a27cc698d.

The revert is needed because of kernel wakelock related issues
on Fugu.

Change-Id: Id8383a12b12aad8c5fb66c0c2ddfe2d42a85c9c2

Merge "service/client: Add register|unregister-gatt" am: 6b881b1ff9
am: a269e0676d

* commit 'a269e0676d3d3e3961ec628434f868b1b877c1d5':
  service/client: Add register|unregister-gatt

Merge "service: Introduce bluetooth::GattClient" am: 3445e896b5
am: ecad06cb6c

* commit 'ecad06cb6cbdebfd1d399c73067984bc26ce3964':
  service: Introduce bluetooth::GattClient

Merge "service: Add IBluetoothGattClient IPC bindings" am: 3ee69e43c2
am: 83d02310c3

* commit '83d02310c31de4614ab19e4076d4580902170193':
  service: Add IBluetoothGattClient IPC bindings

Merge "service/client: Add register|unregister-gatt"
am: 6b881b1ff9

* commit '6b881b1ff9c062a5c35a41fb47018f3cec20832f':
  service/client: Add register|unregister-gatt

Merge "service: Introduce bluetooth::GattClient"
am: 3445e896b5

* commit '3445e896b5357a6f33110573f4743b23aaa9e7a9':
  service: Introduce bluetooth::GattClient

Merge "service: Add IBluetoothGattClient IPC bindings"
am: 3ee69e43c2

* commit '3ee69e43c29f2268a3b0961f126b9cd7a8efcbd1':
  service: Add IBluetoothGattClient IPC bindings

Merge "service/client: Add register|unregister-gatt"

Merge "service: Introduce bluetooth::GattClient"

Merge "service: Add IBluetoothGattClient IPC bindings"

Memory overwrite due to HDP doesn't allocate enough buffer

HDP doesn't allocate enough buffer, so L2CAP overwrite two bytes.
Allocation tracker trigger assert due to find that memory be overwrite.

Bug: 23981241

Change-Id: Ib2c27472b16de2188758ec521ef290d6c9a6c8f0

net_test_bluetooth: replace with gtest variant am: e27d3133c2
am: a16a0266e4

* commit 'a16a0266e4653ab9fcae620706c1c167cc4455c5':
  net_test_bluetooth: replace with gtest variant

net_test_bluetooth: replace with gtest variant
am: e27d3133c2

* commit 'e27d3133c26d299ac9c776670031957f2d9bad04':
  net_test_bluetooth: replace with gtest variant

net_test_bluetooth: replace with gtest variant

Bug: 25385380
Change-Id: Ia719363823d8d538caa6b42f076d5d6d737ae036

Fix SIGBUS crash when copying data am: 3f06e14fdd
am: c4203a27fc

* commit 'c4203a27fc0efc6879ab7e8b8d005cb342fb66bb':
  Fix SIGBUS crash when copying data

Fix SIGBUS crash when copying data
am: 3f06e14fdd

* commit '3f06e14fddc0e674347ae28ea4a25218d4ce1d2a':
  Fix SIGBUS crash when copying data

Fix SIGBUS crash when copying data

We have the following memory alignment-related issue, that seems
to be architecture/compiler/memcpy(3) specific.

Within struct tBTIF_CONTEXT_SWITCH_CBACK, the beginning of the
zero-length array "char p_param[]" is not aligned (because of the
struct internals).
However, this p_param pointer is casted within function
btif_gattc_deep_copy() to the struct pointer (btif_adv_data_t *).
By definition, the memory pointed to by such pointer is suppose
to be aligned:

    btif_adv_data_t *dst = (btif_adv_data_t*) p_dest;

It seems that on some architectures/compilers the executed memcpy()
instructions are optimized for such memory alignment.
If the memory was not aligned, we get SIGBUS.

Apparently, just using (void *) casting for the memcpy() destination,
avoids using the optimized memory aligned instructions:
  memcpy((void *)dst, src, ...);

The solutions are twofold:
 * Make sure that "char p_param[]" within struct
   tBTIF_CONTEXT_SWITCH_CBACK is aligned. Otherwise, the casting
   to "(btif_adv_data_t*)" can be problematic.
 * Add (void *) casting to all memcpy() calls which might be
   referring to such mis-aligned memory.
   This is done by using the new macro maybe_non_aligned_memcpy()
   in all places that such casting might be needed.

Either solution is sufficient to prevent the crash as identified in
this particular case. We need to apply both solutions, to reduce the
chance of running again into a similar issue.

Bug: 25601669
Change-Id: I6c49645c00f10c594a5d1e53a9fac202c506657c

Indicate failed tests in run_unit_tests.sh am: 7ab59096c2
am: 08a960d795

* commit '08a960d7952900c419b8afa107af9d1e00e8b03f':
  Indicate failed tests in run_unit_tests.sh

Indicate failed tests in run_unit_tests.sh
am: 7ab59096c2

* commit '7ab59096c26fe1175258f4cf40674736884ceeb0':
  Indicate failed tests in run_unit_tests.sh

Merge "Retry HID Polling when EINTR is returned" am: 95e0c57578
am: 3d1d6595ec

* commit '3d1d6595ec8a44b6db3f89c8f81dd3125d8ba65f':
  Retry HID Polling when EINTR is returned

Indicate failed tests in run_unit_tests.sh

Previously "run_unit_tests.sh --all" did not indicate to the user if a
unit test other than the last one failed.

Change-Id: I9b64e4280612cd9af18acc17f245b8760bea793f

Merge "Retry HID Polling when EINTR is returned"
am: 95e0c57578

* commit '95e0c57578f1bb59b999195e08d7dfba3e4957e1':
  Retry HID Polling when EINTR is returned

Merge "Retry HID Polling when EINTR is returned"

service/client: Add register|unregister-gatt

Added the register-gatt and unregister-gatt commands to bluetooth-cli.

Bug: 25128256
Change-Id: Ic2521a6297d277de6feb4cb22dbc8bb60794da5f

service: Introduce bluetooth::GattClient

Added the bluetooth::GattClient class that will provide a
per-application entry point to GATT client-role operations. Implemented
IBluetoothGattClient client registration.

Bug: 25128256
Change-Id: I14ec0ddfa7057eedeb5e4f330fcbc2d09f6f3765

Null-terminate hex string in uuid_new() am: addbe8f489
am: 5f2c3d86d2

* commit '5f2c3d86d25de232af9ff87e8504230996e0b7d8':
  Null-terminate hex string in uuid_new()

Null-terminate hex string in uuid_new()
am: addbe8f489

* commit 'addbe8f4895fdcac71c6ef2ea844612e3bb5938c':
  Null-terminate hex string in uuid_new()

Null-terminate hex string in uuid_new()

Change-Id: I28d8b9e6495992ab990e9bd618221373848957c8

service: Add IBluetoothGattClient IPC bindings

Added the Binder IPC bindings for the IBluetoothGattClient API.

Bug: 25128256
Change-Id: Idc37ac7d85c0d3a2aa17e2b95ee588d590a8bb8f

Retry HID Polling when EINTR is returned

While I'm here
 - use __func__ consistently in btif_hh_poll_event_thread.
 - remove braces around a single line

BUG: 24647083
Change-Id: I80b99ff0d3ba6336f5ab7a517269fdf63df59b37
Signed-off-by: Myles Watson <mylesgw@google.com>

Fix net_test_hci am: b0f59256c6
am: 0005a705cc

* commit '0005a705cc80132c78fc9e69c4a59087325f22aa':
  Fix net_test_hci

Fix net_test_hci
am: b0f59256c6

* commit 'b0f59256c6b3c465bc6721394f5e193f711b8384':
  Fix net_test_hci

Fix net_test_hci

vendor->send_command is not send during init(), it's called as a result
to calling enable().

Change-Id: I498870a63057e3644c0b889cd5be046cb41db88d

Merge "service/doc: Add Binder GATT client API definition" am: 88e68c2cd6
am: 6b0dadb3a3

* commit '6b0dadb3a39c2c01843575eb9e0e5d38982ef39f':
  service/doc: Add Binder GATT client API definition

Fix an error loading audio.a2dp.default.so library am: a3aed3bd53
am: 4402e3a447

* commit '4402e3a44766d33d961294cfb6ed5aae1bc6f72b':
  Fix an error loading audio.a2dp.default.so library

Merge "service/doc: Add Binder GATT client API definition"
am: 88e68c2cd6

* commit '88e68c2cd6d28a21c2ac7bdf5c1019d4bef98890':
  service/doc: Add Binder GATT client API definition

Fix an error loading audio.a2dp.default.so library
am: a3aed3bd53

* commit 'a3aed3bd53e326ca79d8d01479d8737cd85729c5':
  Fix an error loading audio.a2dp.default.so library

Merge "service/doc: Add Binder GATT client API definition"

Fix an error loading audio.a2dp.default.so library

Fixes an issue where a symbol is being stripped from the compiled
library preventing it from being loaded.

Bug: 21743601
Change-Id: I60347e83ee85ec282c25ae76f3cf80567bcd19a2

Fix broken test caused by misordered callback arguments. am: 1b4c8b4ebb
am: f5dee186f1

* commit 'f5dee186f188331045eb6a49bb2e9c92c1c4bb58':
  Fix broken test caused by misordered callback arguments.

Fix broken test caused by misordered callback arguments.
am: 1b4c8b4ebb

* commit '1b4c8b4ebbc2f529225737055ba030175c94c9ac':
  Fix broken test caused by misordered callback arguments.

Merge "Fix broken test caused by misordered callback arguments." into cw-e-dev

Fix broken test caused by misordered callback arguments.

Change-Id: I16976a2a680eab76af31090548c2efc53bebaa60

Fix broken test caused by misordered callback arguments.

Change-Id: I16976a2a680eab76af31090548c2efc53bebaa60

Treat empty/truncated GATT cache as error am: 1f656f15b1
am: 774defe11b

* commit '774defe11b4d58c8821f145a987b74e6ccfc8bd4':
  Treat empty/truncated GATT cache as error

Treat empty/truncated GATT cache as error
am: 1f656f15b1

* commit '1f656f15b146febdba8ed0ef0ee444a1e3bd9c0b':
  Treat empty/truncated GATT cache as error

Treat empty/truncated GATT cache as error

Bug: 23054535
Change-Id: I6f60ff556204dabca91ce9075d5802d75fd35857

service/doc: Add Binder GATT client API definition

Added AIDL-style docs for the GATT client Binder API.

Bug: 25128256
Change-Id: I03c4c5a371600b3946a6692a3d0c87f7e529d96a

Always update remote address type during LE scan am: 7b9fe757f5
am: 5c5d572459

* commit '5c5d572459b0e8a11b1bdb2c450bea752227511c':
  Always update remote address type during LE scan

Always update remote address type during LE scan
am: 7b9fe757f5

* commit '7b9fe757f5df8c529c506b1c41403f2f3f204f9e':
  Always update remote address type during LE scan

Always update remote address type during LE scan

This will ensure that devices which do not contain a device name in the
advertising record are stored with the correct address type to ensure
re-connection to the device succeeds.

Bug: 25499163
Change-Id: I85f18ce2de267844c5d5ed49beeb448b9b074d74

Removed libpower-related references. am: 45a94e198e
am: b8cde2c80d

* commit 'b8cde2c80dbf81c3e5f1aea00e9f6a25e38abc5e':
  Removed libpower-related references.

Removed libpower-related references.
am: 45a94e198e

* commit '45a94e198ec424c1f15d9735a605a6876fa6a493':
  Removed libpower-related references.

Always update remote address type during LE scan
am: 03d171170c

* commit '03d171170c7ad4e40454a9575cfd4919d2e1ef2f':
  Always update remote address type during LE scan

Always update remote address type during LE scan

This will ensure that devices which do not contain a device name in the
advertising record are stored with the correct address type to ensure
re-connection to the device succeeds.

Bug: 25499163
Change-Id: I85f18ce2de267844c5d5ed49beeb448b9b074d74

Removed libpower-related references.

Removed libpower-related references, because those are not
needed anymore. Now, the wakelock-related calls and references
are implemented within osi/src/alarm.c

Change-Id: Ifdb232b9724af216b9b414f8b65e85cd713c20d3

gn-build: Get net_test_bluetooth to compile am: 3e54b60e10
am: ee4744eed6

* commit 'ee4744eed62bc53f146174b247625f2e95fb9da1':
  gn-build: Get net_test_bluetooth to compile

gn-build: Get net_test_bluetooth to compile
am: 3e54b60e10

* commit '3e54b60e10093f48f1c4ea3414635bc942f9ecfe':
  gn-build: Get net_test_bluetooth to compile

Merge "Do not mask out secure connections (SC) bit for BT <4.2" into mnc-dr-dev
am: 15c1f685db

* commit '15c1f685db0453a3e20f81de94fca7040a52a39a':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup
am: 7312c6d913

* commit '7312c6d91376e533914eb2a108b65699ca54353a':
  Fix bug in SDP 128-bit UUID lookup

gn-build: Get net_test_bluetooth to compile

This makes net_test_bluetooth compile.

Bug: 21584091

Change-Id: Id0c3d6bdcfd6f2c75c749e7f5a306645371fbb25

Fix bug in SDP 128-bit UUID lookup am: 526f3cc7e9
am: 1a1c20dd86

* commit '1a1c20dd86883db239cddb82ff7d9b96e213a2f0':
  Fix bug in SDP 128-bit UUID lookup

Do not mask out secure connections (SC) bit for BT <4.2 am: 97305504e5
am: 5d30292aaa

* commit '5d30292aaa8390e77aaeeaa026acede6476aeaae':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup
am: 526f3cc7e9

* commit '526f3cc7e93214f4f3e2c7c3c648710ff7251446':
  Fix bug in SDP 128-bit UUID lookup

Do not mask out secure connections (SC) bit for BT <4.2
am: 97305504e5

* commit '97305504e5771fdf76d16ca8583d5fc1680face8':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup

Also re-factored the surrounding code slightly to where it would have
prevented this type of bug in the first place...

Change-Id: I7b7ed4459c4f377d97b608bfe2e0c13aac31680e

Merge "Do not mask out secure connections (SC) bit for BT <4.2" into mnc-dr-dev

Do not mask out secure connections (SC) bit for BT <4.2

Bug: 25467621
Change-Id: Ifdb52295fa5b8c5a39524a722dc044227c735240

Fix bug in SDP 128-bit UUID lookup

Bug: 24344122
Change-Id: I7b7ed4459c4f377d97b608bfe2e0c13aac31680e