16e112b1861d8f1147325372608c905627e00e88 |
|
12-Sep-2016 |
Gurpreet Ghai <gghai@codeaurora.org> |
Serialize stack shutdown, state change callback and cleanup Use Case: Bluetooth On/Off scenario, User switch Steps: 1. Turn ON Bluetooth 2. Switch user Also observed on occurrence of enable/disable timeout during On/Off testing. Failure: Crash while sending state change HAL callback. Root Cause: In the event when stack cleanup is queued to stack manager thread when stack shutdown is being processed, stack cleanup starts execution as soon as stack shutdown is finished. If the function event_signal_stack_down posted to btif thread at the end of stack shutdown is executed after stack cleanup crash occurs. Fix: Serialized the execution of stack shutdown, state change callback execution and stack cleanup to happen in same order. Test: code compilation. Change-Id: Ic96205b5c304acb44eab53f4e2cb150726643bda
/system/bt/btif/src/stack_manager.cc
|
6bd442f543972b072ef2cbbcf2f7c91202de1045 |
|
19-Oct-2016 |
Myles Watson <mylesgw@google.com> |
btif: Apply clang-format clang-format doesn't understand block quotes of this form: /* This is not handled well ** because there are two asterisks **/ cd btif/ # Replace '**' at the beginning of the line with ' *' sed 's/^[*][*]/ \*/' -i include/* src/* test/* co/* clang-format --style=file -i src/* include/* test/* co/* Test: mma -j32 Change-Id: I2477eae5480602d5b2fee5ec89c9ed7888022341
/system/bt/btif/src/stack_manager.cc
|
b2a292b5d8df2f359c38b0787bc01181225a9bc9 |
|
15-Oct-2016 |
Pavlin Radoslavov <pavlin@google.com> |
Renamed most C files to C++: *.c to *.cc Also: - Fixed C++ related compilation errors. - Added missing 'extern "C"' guards in some of the header files. - Added missing LOCAL_CPP_EXTENSION to Android.mk files. - Added-back btif/src/btif_mce.cc and bta/mce/bta_mce_* to btif/Android.mk and bta/Android.mk respectively. - Fixed the alphabetical ordering of the *.cc files in some of the Android.mk files. - Added missing Copyright header to "osi/include/list.h" - Updated "osi/src/wakelock.cc" to use C++ std::string instead of dynamic allocation of C-style strings. Test: code compilation, unit tests, and A2DP streaming Change-Id: Ia2f7215ed9df32775c701b68fc86b09875b942c7
/system/bt/btif/src/stack_manager.cc
|
713993d1784ab7c23aee1fa3cf1ab8676cc0aa69 |
|
21-Apr-2016 |
Jakub Pawlowski <jpawlowski@google.com> |
Convert BTIF code from C to C++ Modifications required: * added proper casting * moved variable definitions before goto statements * added 'extern "C"' markers where needed * renamed 'operator' to 'operator_name' Bug: 28485365 Change-Id: I903357967387207e678866c02e008f047f8263f6
/system/bt/btif/src/stack_manager.cc
|