commit | 6d110527c8363f244530ee8e10f85c325f2bcb0e | [log] [tgz] |
---|---|---|
author | Jack He <siyuanh@google.com> | Wed Jul 25 21:43:01 2018 -0700 |
committer | Jack He <siyuanh@google.com> | Thu Aug 02 10:57:48 2018 -0700 |
tree | a272c16ea19a8505968263819a91085f6f944c7a | |
parent | c27d25789bc0e59f421ad432d8ccca39a16b0a63 [diff] |
Use MessageLoopThread in A2DP source, JNI, BTA, HCI and module bring-up * Replace OSI thread with MessageLoopThread in - btif_a2dp_source (A2DP encoding thread) - btif_core (JNI thread) - btu_init (BTA thread) - hci_layer (HCI thread) - module (module bring up thread) - various unit test threads * Code that uses reactor from OSI thread is not replaced in this change because they need to move from using reactor to message loop first * Main difference: - HCI layer no longer has access to mutex protecting message loop set up and tear down. Messages posted to thread after ShutDown() is called become no-op - In all cases QuitClosure() is used instead of QuitWhenIdleClosure(). This means that we will never force kill a thread. An infinite loop will delay the shutdown of Bluetooth stack. * do_in_hci_thread is actually do_in_bta_thread * Move the definition of do_in_bta_thread from bta to btu since btu is where the thread actually lives * Remove bta_closure_api.h and replace it with stack/include/btu.h * Remove header inclusion in bta_sys.h and include stack/include/btu.h in individual compilation units * Fixed a bug in btif_cleanup_bluetooth where btif_jni_disassociate() was not called on the JNI thread * Make setting real time priority a requirement * Crash during the following scenario: - When btif_profile_queue cleanup failed to be scheduled on JNI thread - When A2DP encoder thread failed to gain real time priority - When BTA thread failed to start up - When BTA thread failed to gain real time priority - When BTA thread failed to schedule its first callback * Turn off Bluetooth in the following scenario: - When HCI thread failed to gain real time priority - When bt_workqueue thread failed to gain real time priority Bug: 110303473 Test: make, native and Java unit tests, Connect to headset and listen to music through A2DP, testplans/details/184455/3975 Change-Id: Ib448992fc0ba5af82c2d117dd65f1abd45d2acb6
Just build AOSP - Fluoride is there by default.
Instructions for Ubuntu, tested on 14.04 with Clang 3.5.0 and 16.10 with Clang 3.8.0
mkdir ~/fluoride cd ~/fluoride git clone https://android.googlesource.com/platform/system/bt
Install dependencies (require sudo access):
cd ~/fluoride/bt build/install_deps.sh
Then fetch third party dependencies:
cd ~/fluoride/bt mkdir third_party cd third_party git clone https://github.com/google/googletest.git git clone https://android.googlesource.com/platform/external/aac git clone https://android.googlesource.com/platform/external/libchrome git clone https://android.googlesource.com/platform/external/libldac git clone https://android.googlesource.com/platform/external/modp_b64 git clone https://android.googlesource.com/platform/external/tinyxml2
And third party dependencies of third party dependencies:
cd fluoride/bt/third_party/libchrome/base/third_party mkdir valgrind cd valgrind curl https://chromium.googlesource.com/chromium/src/base/+/master/third_party/valgrind/valgrind.h?format=TEXT | base64 -d > valgrind.h curl https://chromium.googlesource.com/chromium/src/base/+/master/third_party/valgrind/memcheck.h?format=TEXT | base64 -d > memcheck.h
NOTE: If system/bt is checked out under AOSP, then create symbolic links instead of downloading sources
cd system/bt mkdir third_party cd third_party ln -s ../../../external/aac aac ln -s ../../../external/libchrome libchrome ln -s ../../../external/libldac libldac ln -s ../../../external/modp_b64 modp_b64 ln -s ../../../external/tinyxml2 tinyxml2 ln -s ../../../external/googletest googletest
cd ~/fluoride/bt gn gen out/Default
cd ~/fluoride/bt ninja -C out/Default all
This will build all targets (the shared library, executables, tests, etc) and put them in out/Default. To build an individual target, replace "all" with the target of your choice, e.g. ninja -C out/Default net_test_osi
.
cd ~/fluoride/bt/out/Default LD_LIBRARY_PATH=./ ./bluetoothtbd -create-ipc-socket=fluoride
Follows the Chromium project Eclipse Setup Instructions until "Optional: Building inside Eclipse" section (don't do that section, we will set it up differently)
Generate Eclipse settings:
cd system/bt gn gen --ide=eclipse out/Default
In Eclipse, do File->Import->C/C++->C/C++ Project Settings, choose the XML location under system/bt/out/Default
Right click on the project. Go to Preferences->C/C++ Build->Builder Settings. Uncheck "Use default build command", but instead using "ninja -C out/Default"
Goto Behaviour tab, change clean command to "-t clean"