| commit | 5974d7c456d63b90c3e84b1d58d50efa4d639913 | [log] [tgz] |
|---|---|---|
| author | Cheney Ni <cheneyni@google.com> | Thu Aug 01 19:21:39 2019 +0800 |
| committer | Cheney Ni <cheneyni@google.com> | Wed Aug 07 14:51:32 2019 +0800 |
| tree | 332fd74eae449a16bfb7d603dec0f7d16c87d6ab | |
| parent | 8df347224386d6536e5dbdb2fdb18fed489e20a8 [diff] |
BluetoothAudioHAL: Fix the latency and the number of frames were inconsistent The Bluetooth Audio HAL uses 3 APIs to share the audio latency to audioserver, so they can do the audio / video synchronization. Those API's results have to be consistent in the presented number of frames, and should be reset only when the output was re-opened. Because the HAL queried information from the Bluetooth which might be reset after the Bluetooth session restarted, and caused a conflict between current and previous results. The media frameworks needed more works to be A/V synchronous again, and users would see the video was frozen. This CL checks the stack data, and there must be no big delta between the stack and HAL, or one of them was ever reset, and needs to use local counters instead. Bug: 137978401 Test: Switch active device and codec manually Change-Id: I35bffa834c0de2e8b36e99a96b4a70738cc2b639
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"