commit | ee5c25732350e156dcf986961f4ed0c366ebc0ae | [log] [tgz] |
---|---|---|
author | Jack He <siyuanh@google.com> | Fri Jan 20 11:17:44 2017 -0800 |
committer | Jack He <siyuanh@google.com> | Tue Jan 24 11:47:24 2017 -0800 |
tree | d34676bf0dc71c686333fc0fd0a6b81107afffed | |
parent | 6d75adcc03975cf6c9cae95349233aa6c3a19da2 [diff] |
Corrected default value assignments in Bluetooth A2DP Metrics * In certain cases btif_a2dp_source_stop_audio_req() could be called before btif_a2dp_source_start_audio_req() is called, resulting in session_start_us to be 0. In this case, audio_duration_ms = time_get_os_boottime_us() - 0 will be a very large number that is not the actual audio duration. This CL marks audio_duration_ms as -1 in the above situation so that we can differentiate between valid and invalid audio durations * Set default timer and counter values to 0 when tx_queue_dequeue_stats.total_updates > 1 * Move update_scheduling_stats for tx_queue_enqueue_stats to btif_a2dp_source_audio_handle_timer as we intend to capture the time intervals for enqueue scheduling instead of time intervals between frame enqueue (i.e. one scheduling event can enqueue multiple frames) * Use tx_queue_enqueue_stats for metrics instead of *dequeue* since only enqueue is triggered by timer event Bug: 33694310 Test: Code compilation, BtFunhausMetricsTest Change-Id: I10984920afd4d77f07a5ac75736f8dcd69b13af8 (cherry picked from commit 5a9925784a4a34811ae29608b67dee4a580c226b)
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/libchrome git clone https://android.googlesource.com/platform/external/modp_b64 git clone https://android.googlesource.com/platform/external/tinyxml2 git clone https://android.googlesource.com/platform/hardware/libhardware
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/libchrome libchrome ln -s ../../../external/modp_b64 modp_b64 ln -s ../../../external/tinyxml2 tinyxml2 ln -s ../../../hardware/libhardware libhardware 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"