Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c6f5f1e6 authored by Łukasz Rymanowski's avatar Łukasz Rymanowski
Browse files

leaudio: Add more logs for TA in dumpsys 

Bug: 264225824
Test: atest BluetoothInstrumentationTests
Tag: #feature
Merged-In: Iba4ba6959d477f145b5756297e9aede984e1b888
Change-Id: Iba4ba6959d477f145b5756297e9aede984e1b888
(cherry picked from commit fba7f74e)
parent 59613bcd

system/stack/gatt/connection_manager.cc

+20 −2
Original line number Diff line number Diff line
@@ -42,6 +42,8 @@ 


#define DIRECT_CONNECT_TIMEOUT (30 * 1000) /* 30 seconds */



constexpr char kBtmLogTag[] = "TA";



struct closure_data {

  base::OnceClosure user_task;

  base::Location posted_from;
@@ -172,6 +174,8 @@ static void target_announcement_observe_results_cb(tBTM_INQ_RESULTS* p_inq, 
  LOG_INFO("Found targeted announcement for device %s",

           addr.ToString().c_str());



  BTM_LogHistory(kBtmLogTag, addr, "Found TA from");



  if (it->second.is_in_accept_list) {

    LOG_INFO("Device %s is already connecting", addr.ToString().c_str());

    return;
@@ -192,6 +196,9 @@ static void target_announcement_observe_results_cb(tBTM_INQ_RESULTS* p_inq, 


void target_announcements_filtering_set(bool enable) {

  LOG_DEBUG("enable %d", enable);

  BTM_LogHistory(kBtmLogTag, RawAddress::kEmpty,

                 (enable ? "Start filtering" : "Stop filtering"));



  /* Safe to call as if there is no support for filtering, this call will be

   * ignored. */

  bluetooth::shim::set_target_announcements_filter(enable);
@@ -247,6 +254,10 @@ bool background_connect_targeted_announcement_add(tAPP_ID app_id, 
  }



  bgconn_dev[address].doing_targeted_announcements_conn.insert(app_id);

  if (bgconn_dev[address].doing_targeted_announcements_conn.size() == 1) {

    BTM_LogHistory(kBtmLogTag, address, "Allow connection from");

  }



  if (num_of_targeted_announcements_users() == 1) {

    target_announcements_filtering_set(true);

  }
@@ -336,13 +347,20 @@ bool background_connect_remove(uint8_t app_id, const RawAddress& address) { 
  auto num_of_targeted_announcements_before_remove =

      it->second.doing_targeted_announcements_conn.size();



  if (!it->second.doing_bg_conn.erase(app_id) &&

      !it->second.doing_targeted_announcements_conn.erase(app_id)) {

  bool removed_from_bg_conn = (it->second.doing_bg_conn.erase(app_id) > 0);

  bool removed_from_ta =

      (it->second.doing_targeted_announcements_conn.erase(app_id) > 0);

  if (!removed_from_bg_conn && !removed_from_ta) {

    LOG_WARN("Failed to remove background connection app %d for address %s",

             static_cast<int>(app_id), address.ToString().c_str());

    return false;

  }



  if (removed_from_ta &&

      it->second.doing_targeted_announcements_conn.size() == 0) {

    BTM_LogHistory(kBtmLogTag, address, "Ignore connection from");

  }



  if (is_anyone_connecting(it)) {

    LOG_DEBUG("some device is still connecting, app_id=%d, address=%s",

              static_cast<int>(app_id), address.ToString().c_str());

system/stack/test/gatt_connection_manager_test.cc

+3 −0
Original line number Diff line number Diff line
@@ -78,6 +78,9 @@ void BTM_BleTargetAnnouncementObserve(bool enable, 
  localAcceptlistMock->EnableTargetedAnnouncements(enable, p_results_cb);

}



void BTM_LogHistory(const std::string& tag, const RawAddress& bd_addr,

                    const std::string& msg){};



namespace bluetooth {

namespace shim {

bool is_gd_l2cap_enabled() { return false; }