| commit | f92966049c67cae0edcea4e8d5648579512d79c1 | [log] [tgz] |
|---|---|---|
| author | Cheney Ni <cheneyni@google.com> | Fri Mar 27 11:54:28 2020 +0800 |
| committer | Cheney Ni <cheneyni@google.com> | Tue Apr 14 17:43:46 2020 +0800 |
| tree | b13d90b9bff34483f70924001bbfaae966468dcf | |
| parent | e134c6f179a319d648dcad2715f2e9c3a1e6c3b8 [diff] |
A2DP: Be more careful in allocating a BtifAvPeer instance While a new peer is coming either by in-coming or out-going connection, the stack first tries to find or create an associated BtifAvPeer object for it, but uses an unknown BTA handle if BTA_AV is still registering. This handle will be updated after BTA_AV registered, but causes a mismatch in the handle usage between BTIF and BTA layers since the register process just finished. This change disallows to use an unknown BTA handle, so it prevents such abnormal cases. Besides, it adds more log messages about BTA_AV registering, and also makes sure the disabling flag won't be activated after disabled. Bug: 135655859 Bug: 152597903 Test: Switch BT state quickly between STATE_BLE_ON and STATE_ON Change-Id: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2 Merged-In: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2 (cherry picked from commit 714e639cb7291ecfb9feb12acad9bc7c8f455ba0)
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"