| commit | 24d897ed5f672296f19b69c84d05d63910933696 | [log] [tgz] |
|---|---|---|
| author | Cheney Ni <cheneyni@google.com> | Wed Jul 10 16:24:31 2019 +0800 |
| committer | Cheney Ni <cheneyni@google.com> | Wed Nov 27 10:15:23 2019 +0800 |
| tree | 9b7b21eecda03d3dbfb1407ad07b42e380c99025 | |
| parent | b1d73762ef9072cfb8cbd1c53ee270b540e11f35 [diff] |
Hearing Aid: Remove std::promise from suspend / resume calls, and stop audio ticks by no devices The Hearing Aid uses the stack main thread to be the media task thread. This thread handles events / feeds audio data sequentially, and won't have a race condition within the single thread. The promise is a bit redundant, but wasted time to cause a potential deadlock that the caller waits for the task thread finished which needed the caller's resource. This CL also stops the audio ticks when no devices available, and no need to wait for next timer expired. Besides, move all timer controls into the media task thread, so timer is started / stopped at the same thread, too. Bug: 134996542 Test: add big delay within functions, and check no abnormal manually Change-Id: If0b279a191d86b30db41a682dc7f0ad9470cb10e Merged-In: If0b279a191d86b30db41a682dc7f0ad9470cb10e (cherry picked from commit 90ba872a726b4b66c0b6af29e602dfee65edb710)
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”