OSDN Git Service
Sharvil Nanavati [Thu, 16 Oct 2014 01:30:49 +0000 (18:30 -0700)]
Use correct service ID for A2DP source to make way for A2DP sink.
Sharvil Nanavati [Wed, 15 Oct 2014 01:46:32 +0000 (18:46 -0700)]
L2CAP: expose a function to return the RCID and ACL handle for a given LCID.
Chris Manton [Fri, 31 Oct 2014 21:54:51 +0000 (14:54 -0700)]
Move controller module to device directory
Chris Manton [Fri, 31 Oct 2014 17:41:37 +0000 (10:41 -0700)]
Unused function
Chris Manton [Wed, 29 Oct 2014 04:30:03 +0000 (21:30 -0700)]
Remove unused inquiry DB functions
These are not used for the inquiry database.
Chris Manton [Fri, 24 Oct 2014 16:12:41 +0000 (09:12 -0700)]
Remove always true definition BTM_EIR_SERVER_INCLUDED
Chris Manton [Thu, 23 Oct 2014 23:28:28 +0000 (16:28 -0700)]
Removing unused function BTA_DmSetScanParam()
Chris Manton [Thu, 23 Oct 2014 19:36:14 +0000 (12:36 -0700)]
Remove always true BTM_USE_INQ_RESULTS_FILTER
Chris Manton [Thu, 23 Oct 2014 19:30:15 +0000 (12:30 -0700)]
Remove always false flag BTM_BYPASS_EVENT_FILTERING
Chris Manton [Tue, 21 Oct 2014 20:55:24 +0000 (13:55 -0700)]
Instrument data flow a bit
Use counters to grab some select data metrics.
Sharvil Nanavati [Mon, 27 Oct 2014 23:56:42 +0000 (16:56 -0700)]
Don't dispatch connections in the connection queue until stack comes up.
Bug:
18139425
Chris Manton [Wed, 13 Aug 2014 23:38:57 +0000 (16:38 -0700)]
counter implementation
Chris Manton [Thu, 16 Oct 2014 22:53:13 +0000 (15:53 -0700)]
Remove duplicate functionality
Now that bdtest depends on btcore we can
remove the bdaddr parsing functionality.
Zach Johnson [Mon, 20 Oct 2014 23:33:08 +0000 (16:33 -0700)]
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.
Chris Manton [Thu, 16 Oct 2014 17:54:20 +0000 (10:54 -0700)]
Remove unused functions BTM_RegForLstoEvt/btm_acl_set_discing
Chris Manton [Wed, 15 Oct 2014 23:31:49 +0000 (16:31 -0700)]
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.
Chris Manton [Wed, 15 Oct 2014 21:12:08 +0000 (14:12 -0700)]
Remove always true BTM_BUSY_LEVEL_CHANGE_INCLUDED
Chris Manton [Tue, 30 Sep 2014 04:37:44 +0000 (21:37 -0700)]
Remove data_types.h
Chris Manton [Wed, 15 Oct 2014 16:35:14 +0000 (09:35 -0700)]
Actually remove bd.[c|h]
Chris Manton [Wed, 1 Oct 2014 16:14:06 +0000 (09:14 -0700)]
Removal of bd.[c|h]
Consolidate legacy types into bt_types.h
June R. Tate-Gans [Fri, 19 Sep 2014 16:54:19 +0000 (09:54 -0700)]
Moving btif_sock_sdp to stdint and stdbool.
Zach Johnson [Tue, 16 Sep 2014 05:29:54 +0000 (22:29 -0700)]
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.
June R. Tate-Gans [Thu, 11 Sep 2014 00:05:21 +0000 (17:05 -0700)]
Refactoring btif_sock_sdp.c for clarity
Zach Johnson [Thu, 26 Feb 2015 00:46:34 +0000 (16:46 -0800)]
Revert "BT MAP: added support for email sharing over BT"
Another change being backed out until the megamerge is over.
This reverts commit
c331807e3c3a0e15180a11325e1635ae5268aa39.
Zach Johnson [Thu, 26 Feb 2015 00:44:31 +0000 (16:44 -0800)]
Revert "MAP: SDP corruption"
Backing out so our rewrite merges cleanly.
This reverts commit
54e24af853a77a057995ee4f234bae4d490b32e5.
Andre Eisenbach [Thu, 26 Feb 2015 00:06:15 +0000 (16:06 -0800)]
Revert "Forgot device/Android.mk"
Broken build...
This reverts commit
d935ef9446990c17412d2b0d5ce158df54377871.
Andre Eisenbach [Thu, 26 Feb 2015 00:01:07 +0000 (16:01 -0800)]
Revert "Remove unused inquiry variables"
Still used by BTA API BTA_DmSetScanParam
This reverts commit
d84f0c8c5eb67a45cb2e0993324d1dff8b499b1b.
Chris Manton [Fri, 31 Oct 2014 22:21:48 +0000 (15:21 -0700)]
Forgot device/Android.mk
Slipped by...
Chris Manton [Fri, 24 Oct 2014 16:18:57 +0000 (09:18 -0700)]
Remove always undefined BTM_EIR_UUID_LKUP_TBL
Chris Manton [Fri, 24 Oct 2014 16:00:13 +0000 (09:00 -0700)]
Remove always true definition BTM_EIR_CLIENT_INCLUDED
Chris Manton [Thu, 23 Oct 2014 23:31:49 +0000 (16:31 -0700)]
Remove unused inquiry variables
Sharvil Nanavati [Wed, 29 Oct 2014 06:43:59 +0000 (23:43 -0700)]
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
Andre Eisenbach [Wed, 25 Feb 2015 23:36:07 +0000 (15:36 -0800)]
Fix bdtool compile errors
Sharvil Nanavati [Sat, 20 Sep 2014 07:48:47 +0000 (00:48 -0700)]
Add Android.mk to tools/ directory.
Sharvil Nanavati [Mon, 22 Sep 2014 04:17:15 +0000 (21:17 -0700)]
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).
Sharvil Nanavati [Mon, 22 Sep 2014 06:43:22 +0000 (23:43 -0700)]
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.
Chris Manton [Wed, 24 Sep 2014 18:49:42 +0000 (11:49 -0700)]
Directory layout documentation
Sharvil Nanavati [Tue, 30 Sep 2014 23:56:10 +0000 (16:56 -0700)]
Document network ports used by bluedroid.
Sharvil Nanavati [Tue, 30 Sep 2014 23:50:31 +0000 (16:50 -0700)]
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.
Sharvil Nanavati [Tue, 14 Oct 2014 00:41:19 +0000 (17:41 -0700)]
avdt: Make sure LCID field is filled out even on incoming connections.
Zach Johnson [Wed, 24 Sep 2014 20:57:45 +0000 (13:57 -0700)]
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.
Zach Johnson [Wed, 24 Sep 2014 20:41:33 +0000 (13:41 -0700)]
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.
Sharvil Nanavati [Wed, 3 Sep 2014 18:47:48 +0000 (11:47 -0700)]
Add system tests for RFCOMM sockets.
Sharvil Nanavati [Mon, 22 Sep 2014 07:47:50 +0000 (00:47 -0700)]
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.
Sharvil Nanavati [Sat, 20 Sep 2014 17:43:01 +0000 (10:43 -0700)]
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.
Sharvil Nanavati [Tue, 24 Feb 2015 01:31:48 +0000 (17:31 -0800)]
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.
June R. Tate-Gans [Mon, 20 Oct 2014 19:41:43 +0000 (12:41 -0700)]
Removing a duplicate call to socket_register.
Chris Manton [Thu, 9 Oct 2014 23:04:05 +0000 (16:04 -0700)]
First entry for bdtool
Zach Johnson [Wed, 8 Oct 2014 16:18:48 +0000 (09:18 -0700)]
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.
Sharvil Nanavati [Sun, 4 May 2014 00:32:05 +0000 (17:32 -0700)]
Make hci/ depend on bdroid_CFLAGS
Zach Johnson [Wed, 29 Oct 2014 00:59:13 +0000 (17:59 -0700)]
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.
Zach Johnson [Tue, 28 Oct 2014 00:23:49 +0000 (17:23 -0700)]
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.
Sharvil Nanavati [Tue, 21 Oct 2014 09:38:01 +0000 (02:38 -0700)]
Remove excessive log message from btu_task.
Zach Johnson [Mon, 20 Oct 2014 22:40:01 +0000 (15:40 -0700)]
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
Zach Johnson [Mon, 20 Oct 2014 20:22:43 +0000 (13:22 -0700)]
Remove a bunch of always-false macros
Referenced missing header files, so they wouldn't even
compile if set to true.
Zach Johnson [Mon, 20 Oct 2014 20:06:52 +0000 (13:06 -0700)]
Remove definitions for DUN
Missing header files, etc so it wouldn't compile anyway.
Zach Johnson [Mon, 20 Oct 2014 18:43:02 +0000 (11:43 -0700)]
Remove always false TCS_INCLUDED macro & friends
Referenced non existent headers, so would not compile if set to true.
Zach Johnson [Sat, 18 Oct 2014 01:25:18 +0000 (18:25 -0700)]
Remove remaining references to GOEP
FALSE by bt_target. Header files never used. Missing header files.
Zach Johnson [Sat, 18 Oct 2014 01:17:20 +0000 (18:17 -0700)]
Remove unused header utfc.h
Zach Johnson [Sat, 18 Oct 2014 00:56:28 +0000 (17:56 -0700)]
Remove a bunch of unused functions and definitions from HCIC
Left the BLE stuff alone for now.
Zach Johnson [Sat, 18 Oct 2014 00:07:30 +0000 (17:07 -0700)]
Remove unused macros LMP_TEST and BTISE
Zach Johnson [Fri, 17 Oct 2014 18:28:46 +0000 (11:28 -0700)]
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.
Zach Johnson [Wed, 15 Oct 2014 00:56:37 +0000 (17:56 -0700)]
Remove unused code in GAP
Left the BLE stuff for now, since some of it appears to
be used at least right now.
Zach Johnson [Thu, 16 Oct 2014 22:52:45 +0000 (15:52 -0700)]
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.
Zach Johnson [Wed, 15 Oct 2014 20:43:26 +0000 (13:43 -0700)]
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.
Chris Manton [Wed, 15 Oct 2014 05:00:32 +0000 (22:00 -0700)]
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.
Zach Johnson [Tue, 14 Oct 2014 23:41:21 +0000 (16:41 -0700)]
Add comments to the device features and event mask structs
Zach Johnson [Wed, 8 Oct 2014 20:51:06 +0000 (13:51 -0700)]
Remove L2CAP_CORRUPT_ERTM_PKTS macro
In general, test code and production code should not coexist in the same file.
Zach Johnson [Wed, 8 Oct 2014 03:21:37 +0000 (20:21 -0700)]
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.
Zach Johnson [Mon, 6 Oct 2014 20:15:00 +0000 (13:15 -0700)]
Remove duplicate local bdaddr management functions
Zach Johnson [Sat, 27 Sep 2014 04:14:34 +0000 (21:14 -0700)]
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
Zach Johnson [Wed, 1 Oct 2014 02:10:56 +0000 (19:10 -0700)]
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.
Zach Johnson [Mon, 29 Sep 2014 21:03:20 +0000 (14:03 -0700)]
Remove <BT 2.0 workarounds
Zach Johnson [Tue, 30 Sep 2014 07:09:55 +0000 (00:09 -0700)]
Prune BTM_SetSecurityMode and GAP_SetSecurityMode
BTM_SetSecurityMode was only used from GAP_SetSecurityMode,
and GAP_SetSecurityMode was never used.
Zach Johnson [Sun, 28 Sep 2014 22:46:25 +0000 (15:46 -0700)]
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.
June R. Tate-Gans [Wed, 24 Sep 2014 22:25:02 +0000 (15:25 -0700)]
Removing unnecessary BTAPI #defines.
Zach Johnson [Sun, 28 Sep 2014 05:10:32 +0000 (22:10 -0700)]
Add a futured version of the HCI transmit_command
This will allow the controller restart sequence to be serialized into a single function.
Zach Johnson [Wed, 1 Oct 2014 02:38:44 +0000 (19:38 -0700)]
Remove some < BT 2.1 code & repeated init
Simple pairing is already enabled and the BLE event mask
is already set by this point.
Zach Johnson [Wed, 1 Oct 2014 02:23:09 +0000 (19:23 -0700)]
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.
Chris Manton [Mon, 29 Sep 2014 23:35:34 +0000 (16:35 -0700)]
Remove duplicate L2CAP_NUM_FIXED_CHNLS
Zach Johnson [Fri, 26 Sep 2014 21:03:13 +0000 (14:03 -0700)]
Remove unused hci_cmd_cb in btu
An earlier change obviated the need for it, but I
forgot to remove it at that time.
Zach Johnson [Thu, 25 Sep 2014 00:27:25 +0000 (17:27 -0700)]
Modulizification of bt_utils
Zach Johnson [Wed, 24 Sep 2014 19:04:01 +0000 (12:04 -0700)]
Modulizification of btif_config
Zach Johnson [Wed, 24 Sep 2014 03:53:41 +0000 (20:53 -0700)]
Remove BTTRC_INCLUDED
It used functions that didn't exist, so there's no way it
would compile if it was defined to true.
Also removes some defines that would nop without it.
Zach Johnson [Wed, 24 Sep 2014 03:25:47 +0000 (20:25 -0700)]
Modulizification of GKI
Zach Johnson [Wed, 24 Sep 2014 01:30:45 +0000 (18:30 -0700)]
Modulizification of the HCI layer
Shutdown is managed by the stack manager now, so we don't need to lock
around it. The fetching of the device address is a bit hacky right now,
until it becomes part of its own module.
Chris Manton [Sat, 27 Sep 2014 04:31:38 +0000 (21:31 -0700)]
Add paired cleanup routine for logmsg
Zach Johnson [Sat, 27 Sep 2014 04:16:20 +0000 (21:16 -0700)]
Move module_management_stop to clean_up
Module management isn't done when the stack is shut_down (bluetooth disabled)
Zach Johnson [Tue, 23 Sep 2014 05:11:55 +0000 (22:11 -0700)]
Refactor btsnoop and stack config into modules
Moves stack config out of the combined bte_config, and into
its own module.
Makes btsnoop more self sufficient and removes uneccessary
levels of indirection.
Refactor logging slightly into a (temporary) module to disassociate
from the direct calls from config. Eliminates some useless stuff in
the module as well.
Zach Johnson [Tue, 23 Sep 2014 05:14:04 +0000 (22:14 -0700)]
First pass at implementing modules
This first step creates the notion of a module and corresponding
lifecycle functions, with helpers to simplify working with them.
Once everything is converted over to this module format, then we
can make the stack manager automagically find the correct order for
init/start_up/shut_down/clean_up
Zach Johnson [Sat, 27 Sep 2014 00:04:51 +0000 (17:04 -0700)]
Unregister BTU queues upon shutdown
This ensures that the next run doesn't try to access memory
that is no longer valid.
Chris Manton [Fri, 26 Sep 2014 20:31:41 +0000 (13:31 -0700)]
Remove HCI access macros
And some other minor bt_target cleanup.
Chris Manton [Fri, 26 Sep 2014 18:42:20 +0000 (11:42 -0700)]
Remove GKI task code
Chris Manton [Fri, 26 Sep 2014 18:41:53 +0000 (11:41 -0700)]
Replace bt media GKI task with thread_t
Chris Manton [Fri, 5 Sep 2014 02:48:49 +0000 (19:48 -0700)]
Remove btu_task and replace with bt_workqueue reactor
Chris Manton [Wed, 24 Sep 2014 22:18:41 +0000 (15:18 -0700)]
Logging cleanup
Chris Manton [Wed, 24 Sep 2014 17:27:52 +0000 (10:27 -0700)]
Remove unused definitions from bt_target.h
Sharvil Nanavati [Mon, 22 Sep 2014 06:50:54 +0000 (23:50 -0700)]
Eliminate memory leaks and unsafe callbacks in BTA and BTE.
Alarms were being added to a hash map but were not freed during
teardown. This results in both memory leaks and potential access
of torn-down data (in case the still-armed alarm fires after
teardown completes).
Chris Manton [Fri, 19 Sep 2014 16:08:32 +0000 (09:08 -0700)]
Add string hashing function using djb2
Zach Johnson [Mon, 8 Sep 2014 18:39:49 +0000 (11:39 -0700)]
Disassociate hash algorithm with hash function name
This will make it easier to select the correct one for the job, and also
make it possible to substitute a better one in the future if need be without
having to change the function name.
kunth is duplicated, once for integer hashing and once for shallow pointer hashing, to
make usage more readable.