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

Commit ea88e351 authored by Michael Sun's avatar Michael Sun Committed by Gerrit Code Review
Browse files

Merge changes I1a331784,Iaf5d53ca,Iee60ac5e,I1008239d 

* changes:
  floss: metrics: HID profiles connection state handling
  floss: metrics: introduce event parsing for profile connection state
  floss: metrics: make profile connection logging skinnier
  btif: hh: enhance the dis(connect) call error returns
parents d89f5e09 f2e9257b

system/btif/src/btif_hh.cc

+49 −16
Original line number Diff line number Diff line
@@ -1320,16 +1320,35 @@ static bt_status_t init(bthh_callbacks_t* callbacks) { 
 *

 ******************************************************************************/

static bt_status_t connect(RawAddress* bd_addr) {

  if (btif_hh_cb.status != BTIF_HH_DEV_CONNECTING) {

    btif_transfer_context(btif_hh_handle_evt, BTIF_HH_CONNECT_REQ_EVT,

                          (char*)bd_addr, sizeof(RawAddress), NULL);

    return BT_STATUS_SUCCESS;

  } else if ((btif_hh_cb.pending_conn_address == *bd_addr) &&

       (btif_hh_cb.status == BTIF_HH_DEV_CONNECTING)) {

    LOG(INFO) << __func__ << ": already connecting " << *bd_addr;

  btif_hh_device_t* p_dev;



  if (btif_hh_cb.status == BTIF_HH_DEV_CONNECTING) {

    BTIF_TRACE_WARNING("%s: Error, HH status = %d", __func__,

                       btif_hh_cb.status);

    return BT_STATUS_BUSY;

  } else if (btif_hh_cb.status == BTIF_HH_DISABLED ||

             btif_hh_cb.status == BTIF_HH_DISABLING) {

    BTIF_TRACE_WARNING("%s: Error, HH status = %d", __func__,

                       btif_hh_cb.status);

    return BT_STATUS_NOT_READY;

  }



  p_dev = btif_hh_find_connected_dev_by_bda(*bd_addr);

  if (p_dev) {

    if (p_dev->dev_status == BTHH_CONN_STATE_CONNECTED ||

        p_dev->dev_status == BTHH_CONN_STATE_CONNECTING) {

      BTIF_TRACE_ERROR("%s: Error, device %s already connected.", __func__,

                       bd_addr->ToString().c_str());

      return BT_STATUS_DONE;

    } else if (p_dev->dev_status == BTHH_CONN_STATE_DISCONNECTING) {

      BTIF_TRACE_ERROR("%s: Error, device %s is busy with (dis)connecting.",

                       __func__, bd_addr->ToString().c_str());

      return BT_STATUS_BUSY;

    }

  return BT_STATUS_FAIL;

  }



  return btif_transfer_context(btif_hh_handle_evt, BTIF_HH_CONNECT_REQ_EVT,

                               (char*)bd_addr, sizeof(RawAddress), NULL);

}



/*******************************************************************************
@@ -1346,19 +1365,33 @@ static bt_status_t disconnect(RawAddress* bd_addr) { 
  BTIF_TRACE_EVENT("BTHH: %s", __func__);

  btif_hh_device_t* p_dev;



  if (btif_hh_cb.status == BTIF_HH_DISABLED) {

  if (btif_hh_cb.status == BTIF_HH_DISABLED ||

      btif_hh_cb.status == BTIF_HH_DISABLING) {

    BTIF_TRACE_WARNING("%s: Error, HH status = %d", __func__,

                       btif_hh_cb.status);

    return BT_STATUS_FAIL;

    return BT_STATUS_UNHANDLED;

  }



  p_dev = btif_hh_find_connected_dev_by_bda(*bd_addr);

  if (p_dev != NULL) {

  if (!p_dev) {

    BTIF_TRACE_ERROR("%s: Error, device %s not opened.", __func__,

                     bd_addr->ToString().c_str());

    return BT_STATUS_UNHANDLED;

  }



  if (p_dev->dev_status == BTHH_CONN_STATE_DISCONNECTED ||

      p_dev->dev_status == BTHH_CONN_STATE_DISCONNECTING) {

    BTIF_TRACE_ERROR("%s: Error, device %s already disconnected.", __func__,

                     bd_addr->ToString().c_str());

    return BT_STATUS_DONE;

  } else if (p_dev->dev_status == BTHH_CONN_STATE_CONNECTING) {

    BTIF_TRACE_ERROR("%s: Error, device %s is busy with (dis)connecting.",

                     __func__, bd_addr->ToString().c_str());

    return BT_STATUS_BUSY;

  }



  return btif_transfer_context(btif_hh_handle_evt, BTIF_HH_DISCONNECT_REQ_EVT,

                               (char*)bd_addr, sizeof(RawAddress), NULL);

  } else {

    BTIF_TRACE_WARNING("%s: Error, device  not opened.", __func__);

    return BT_STATUS_FAIL;

  }

}



/*******************************************************************************

system/gd/metrics/chromeos/metrics.cc

+31 −3
Original line number Diff line number Diff line
@@ -170,10 +170,38 @@ void LogMetricsDeviceInfoReport( 
      .Record();

}



void LogMetricsProfileConnectionAttempt(RawAddress* addr, uint32_t intent, uint32_t profile) {}

void LogMetricsProfileConnectionStateChanged(RawAddress* addr, uint32_t profile, uint32_t status, uint32_t state) {

  int64_t boot_time;

  std::string addr_string;

  std::string boot_id;



  if (!GetBootId(&boot_id)) return;



  addr_string = addr->ToString();

  boot_time = bluetooth::common::time_get_os_boottime_us();



  ProfileConnectionEvent event = ToProfileConnectionEvent(addr_string, profile, status, state);



  if (Profile::UNKNOWN == (Profile)event.profile) return;



void LogMetricsProfileConnectionStateChanged(

    RawAddress* addr, uint32_t intent, uint32_t profile, uint32_t status, uint32_t state) {}

  LOG_DEBUG(

      "ProfileConnectionStateChanged: %s, %d, %s, %d, %d, %d",

      boot_id.c_str(),

      boot_time,

      addr_string.c_str(),

      event.type,

      event.profile,

      event.state);



  ::metrics::structured::events::bluetooth::BluetoothProfileConnectionStateChanged()

      .SetBootId(boot_id)

      .SetSystemTime(boot_time)

      .SetDeviceId(addr_string)

      .SetStateChangeType((int64_t)event.type)

      .SetProfile((int64_t)event.profile)

      .SetProfileConnectionState((int64_t)event.state)

      .Record();

}



}  // namespace metrics

}  // namespace bluetooth

system/gd/metrics/chromeos/metrics_event.cc

+244 −40
Original line number Diff line number Diff line
@@ -15,63 +15,87 @@ 
 */

#include "gd/metrics/chromeos/metrics_event.h"



#include <map>

#include <utility>



#include "hci/hci_packets.h"

#include "include/hardware/bluetooth.h"

#include "include/hardware/bt_hh.h"



namespace bluetooth {

namespace metrics {



// ENUM definitaion for Bluetooth Bond State in sync with topshim::btif::BtBondState

enum class BtBondState {

  NotBonded = 0,

  Bonding,

  Bonded,

// topshim::btif::BtBondState is a copy of hardware/bluetooth.h:bt_bond_state_t

typedef bt_bond_state_t BtBondState;

// topshim::btif::BtStatus is a copy of hardware/bluetooth.h:bt_status_t

typedef bt_status_t BtStatus;

// topshim::profile::hid_host::BthhConnectionState is a copy of hardware/bluetooth.h:bthh_connection_state_t

typedef bthh_connection_state_t BthhConnectionState;



// A normalized connection state ENUM definition all profiles

enum class ProfilesConnectionState {

  DISCONNECTED = 0,

  CONNECTING,

  CONNECTED,

  DISCONNECTING,

  UNKNOWN,

};



// ENUM definitaion for Bluetooth action status in sync with topshim::btif::BtStatus

enum class BtStatus {

  Success = 0,

  Fail,

  NotReady,

  NoMemory,

  Busy,

  Done,

  Unsupported,

  InvalidParam,

  Unhandled,

  AuthFailure,

  RemoteDeviceDown,

  AuthRejected,

  JniEnvironmentError,

  JniThreadAttachError,

  WakeLockError,



  // Any statuses that couldn't be cleanly converted

  Unknown = 0xff,

// ENUM definition for Bluetooth profiles in sync with ::uuid::Profiles

enum class ProfilesFloss {

  A2dpSink = 0,

  A2dpSource,

  AdvAudioDist,

  Hsp,

  HspAg,

  Hfp,

  HfpAg,

  AvrcpController,

  AvrcpTarget,

  ObexObjectPush,

  Hid,

  Hogp,

  Panu,

  Nap,

  Bnep,

  PbapPce,

  PbapPse,

  Map,

  Mns,

  Mas,

  Sap,

  HearingAid,

  LeAudio,

  Dip,

  VolumeControl,

  GenericMediaControl,

  MediaControl,

  CoordinatedSet,

};



static PairingState StatusToPairingState(uint32_t status) {

  switch ((BtStatus)status) {

    case BtStatus::Success:

    case BtStatus::BT_STATUS_SUCCESS:

      return PairingState::PAIR_SUCCEED;

    case BtStatus::Fail:

    case BtStatus::BT_STATUS_FAIL:

      return PairingState::PAIR_FAIL_FAILED;

    case BtStatus::NoMemory:

    case BtStatus::BT_STATUS_NOMEM:

      return PairingState::PAIR_FAIL_NO_RESOURCES;

    case BtStatus::Busy:

    case BtStatus::BT_STATUS_BUSY:

      return PairingState::PAIR_FAIL_BUSY;

    case BtStatus::Unsupported:

    case BtStatus::BT_STATUS_UNSUPPORTED:

      return PairingState::PAIR_FAIL_NOT_SUPPORTED;

    case BtStatus::InvalidParam:

    case BtStatus::BT_STATUS_PARM_INVALID:

      return PairingState::PAIR_FAIL_INVALID_PARAMS;

    case BtStatus::AuthFailure:

    case BtStatus::BT_STATUS_AUTH_FAILURE:

      return PairingState::PAIR_FAIL_AUTH_FAILED;

    case BtStatus::RemoteDeviceDown:

    case BtStatus::BT_STATUS_RMT_DEV_DOWN:

      return PairingState::PAIR_FAIL_ESTABLISH_CONN;

    case BtStatus::AuthRejected:

    case BtStatus::BT_STATUS_AUTH_REJECTED:

      return PairingState::PAIR_FAIL_AUTH_FAILED;

    case BtStatus::NotReady:

    case BtStatus::Done:

    case BtStatus::Unhandled:

    case BtStatus::BT_STATUS_NOT_READY:

    case BtStatus::BT_STATUS_DONE:

    case BtStatus::BT_STATUS_UNHANDLED:

    default:

      return PairingState::PAIR_FAIL_UNKNOWN;

  }
@@ -175,10 +199,10 @@ PairingState ToPairingState(uint32_t status, uint32_t bond_state, int32_t fail_r 
  PairingState pairing_state = PairingState::PAIR_FAIL_UNKNOWN;



  // The Bonding is a transitional state during the pairing process. Ignore it by returning the starting again.

  if ((BtBondState)bond_state == BtBondState::Bonding) return PairingState::PAIR_STARTING;

  if ((BtBondState)bond_state == BtBondState::BT_BOND_STATE_BONDING) return PairingState::PAIR_STARTING;



  if ((BtStatus)status == BtStatus::Success && (hci::ErrorCode)fail_reason == hci::ErrorCode::SUCCESS) {

    if ((BtBondState)bond_state == BtBondState::Bonded) {

  if ((BtStatus)status == BtStatus::BT_STATUS_SUCCESS && (hci::ErrorCode)fail_reason == hci::ErrorCode::SUCCESS) {

    if ((BtBondState)bond_state == BtBondState::BT_BOND_STATE_BONDED) {

      return PairingState::PAIR_SUCCEED;

    } else {

      return PairingState::PAIR_FAIL_CANCELLED;
@@ -193,5 +217,185 @@ PairingState ToPairingState(uint32_t status, uint32_t bond_state, int32_t fail_r 
  return pairing_state;

}



int64_t StatusToProfileConnectionState(uint32_t status, StateChangeType type) {

  int64_t state;

  if (StateChangeType::STATE_CHANGE_TYPE_CONNECT == type) {

    switch ((BtStatus)status) {

      case BtStatus::BT_STATUS_SUCCESS:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_SUCCEED;

        break;

      case BtStatus::BT_STATUS_BUSY:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_BUSY_CONNECTING;

        break;

      case BtStatus::BT_STATUS_DONE:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_ALREADY_CONNECTED;

        break;

      case BtStatus::BT_STATUS_UNSUPPORTED:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_PROFILE_NOT_SUPPORTED;

        break;

      case BtStatus::BT_STATUS_PARM_INVALID:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_UNKNOWN_ERROR;

        break;

      case BtStatus::BT_STATUS_AUTH_FAILURE:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_CONNECTION_REFUSED;

        break;

      case BtStatus::BT_STATUS_RMT_DEV_DOWN:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_REMOTE_UNAVAILABLE;

        break;

      case BtStatus::BT_STATUS_AUTH_REJECTED:

      case BtStatus::BT_STATUS_FAIL:

      case BtStatus::BT_STATUS_NOT_READY:

      case BtStatus::BT_STATUS_NOMEM:

      case BtStatus::BT_STATUS_UNHANDLED:

      default:

        state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_UNKNOWN_ERROR;

        break;

    }

  } else {

    switch ((BtStatus)status) {

      case BtStatus::BT_STATUS_SUCCESS:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_SUCCEED;

        break;

      case BtStatus::BT_STATUS_BUSY:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_BUSY_DISCONNECTING;

        break;

      case BtStatus::BT_STATUS_DONE:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_ALREADY_DISCONNECTED;

        break;

      case BtStatus::BT_STATUS_UNSUPPORTED:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_UNKNOWN_ERROR;

        break;

      case BtStatus::BT_STATUS_PARM_INVALID:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_INVALID_PARAMS;

        break;

      case BtStatus::BT_STATUS_AUTH_FAILURE:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_DISCONNECTION_REFUSED;

        break;

      case BtStatus::BT_STATUS_RMT_DEV_DOWN:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_UNKNOWN_ERROR;

        break;

      case BtStatus::BT_STATUS_AUTH_REJECTED:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_DISCONNECTION_REFUSED;

        break;

      case BtStatus::BT_STATUS_FAIL:

      case BtStatus::BT_STATUS_NOT_READY:

      case BtStatus::BT_STATUS_NOMEM:

      case BtStatus::BT_STATUS_UNHANDLED:

      default:

        state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_UNKNOWN_ERROR;

        break;

    }

  }



  return state;

}



static std::pair<uint32_t, uint32_t> ToProfileConnectionState(uint32_t profile, uint32_t state) {

  std::pair<uint32_t, uint32_t> output;



  switch ((ProfilesFloss)profile) {

    // case ProfilesFloss::A2dpSink:

    // case ProfilesFloss::A2dpSource:

    // case ProfilesFloss::AdvAudioDist:

    // case ProfilesFloss::Hsp:

    // case ProfilesFloss::HspAg:

    // case ProfilesFloss::Hfp:

    // case ProfilesFloss::HfpAg:

    // case ProfilesFloss::AvrcpController:

    // case ProfilesFloss::AvrcpTarget:

    // case ProfilesFloss::ObexObjectPush:

    case ProfilesFloss::Hid:

    case ProfilesFloss::Hogp:

      output.first = (uint32_t)Profile::HID;

      switch ((BthhConnectionState)state) {

        case BthhConnectionState::BTHH_CONN_STATE_CONNECTED:

          output.second = (uint32_t)ProfilesConnectionState::CONNECTED;

          break;

        case BthhConnectionState::BTHH_CONN_STATE_CONNECTING:

          output.second = (uint32_t)ProfilesConnectionState::CONNECTING;

          break;

        case BthhConnectionState::BTHH_CONN_STATE_DISCONNECTED:

          output.second = (uint32_t)ProfilesConnectionState::DISCONNECTED;

          break;

        case BthhConnectionState::BTHH_CONN_STATE_DISCONNECTING:

          output.second = (uint32_t)ProfilesConnectionState::DISCONNECTING;

          break;

        case BthhConnectionState::BTHH_CONN_STATE_UNKNOWN:

          output.second = (uint32_t)ProfilesConnectionState::UNKNOWN;

          break;

      }

      break;

    // case ProfilesFloss::Panu:

    // case ProfilesFloss::Nap:

    // case ProfilesFloss::Bnep:

    // case ProfilesFloss::PbapPce:

    // case ProfilesFloss::PbapPse:

    // case ProfilesFloss::Map:

    // case ProfilesFloss::Mns:

    // case ProfilesFloss::Mas:

    // case ProfilesFloss::Sap:

    // case ProfilesFloss::HearingAid:

    // case ProfilesFloss::LeAudio:

    // case ProfilesFloss::Dip:

    // case ProfilesFloss::VolumeControl:

    // case ProfilesFloss::GenericMediaControl:

    // case ProfilesFloss::MediaControl:

    // case ProfilesFloss::CoordinatedSet:

    default:

      output = std::make_pair((uint32_t)Profile::UNKNOWN, state);

      break;

  }



  return output;

}



ProfileConnectionEvent ToProfileConnectionEvent(std::string addr, uint32_t profile, uint32_t status, uint32_t state) {

  ProfileConnectionEvent event;

  // A map stores the pending StateChangeType used to match a (dis)connection event with unknown type.

  // map<std::pair<address, profile>, type>

  static std::map<std::pair<std::string, uint32_t>, StateChangeType> pending_type;



  auto profile_state_pair = ToProfileConnectionState(profile, state);

  auto key = std::make_pair(addr, profile_state_pair.first);

  event.profile = (int64_t)profile_state_pair.first;



  switch ((ProfilesConnectionState)profile_state_pair.second) {

    case ProfilesConnectionState::CONNECTED:

      event.type = (int64_t)StateChangeType::STATE_CHANGE_TYPE_CONNECT;

      event.state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_SUCCEED;

      pending_type.erase(key);

      break;

    case ProfilesConnectionState::CONNECTING:

      event.type = (int64_t)StateChangeType::STATE_CHANGE_TYPE_CONNECT;

      event.state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_STARTING;

      pending_type[key] = StateChangeType::STATE_CHANGE_TYPE_CONNECT;

      break;

    case ProfilesConnectionState::DISCONNECTED:

      event.type = pending_type.find(key) != pending_type.end()

                       ? (int64_t)pending_type[key]

                       : (int64_t)StateChangeType::STATE_CHANGE_TYPE_DISCONNECT;

      // If the profile successfully disconnected for a connect intent, i.e., a connection is attempted but received a

      // disconnection state update. Report this as an unknown error.

      if (StateChangeType::STATE_CHANGE_TYPE_CONNECT == (StateChangeType)event.type &&

          BtStatus::BT_STATUS_SUCCESS == (BtStatus)status) {

        event.state = (int64_t)MetricProfileConnectionStatus::PROFILE_CONN_STATE_UNKNOWN_ERROR;

      } else {

        event.state = StatusToProfileConnectionState(status, (StateChangeType)event.type);

      }

      pending_type.erase(key);

      break;

    case ProfilesConnectionState::DISCONNECTING:

      event.type = (int64_t)StateChangeType::STATE_CHANGE_TYPE_DISCONNECT;

      event.state = (int64_t)MetricProfileDisconnectionStatus::PROFILE_DISCONN_STATE_STARTING;

      pending_type[key] = StateChangeType::STATE_CHANGE_TYPE_DISCONNECT;

      break;

    default:

      event.profile = (int64_t)Profile::UNKNOWN;

      break;

  }



  return event;

}



}  // namespace metrics

}  // namespace bluetooth
 
 No newline at end of file

system/gd/metrics/chromeos/metrics_event.h

+66 −3
Original line number Diff line number Diff line
@@ -16,15 +16,16 @@ 
#pragma once



#include <cstdint>

#include <string>



namespace bluetooth {

namespace metrics {



// ENUM definitaion for adapter state that in sync with ChromeOS strcutured metrics

// ENUM definition for adapter state that in sync with ChromeOS structured metrics

// BluetoothAdapterStateChanged/AdapterState.

enum class AdapterState : int64_t { OFF = 0, ON = 1 };



// ENUM definitaion for pairing state that in sync with ChromeOS strcutured metrics

// ENUM definition for pairing state that in sync with ChromeOS structured metrics

// BluetoothPairingStateChanged/PairingState and BlueZ metrics_pair_result.

enum class PairingState : int64_t {

  PAIR_STARTING = 0,
@@ -68,11 +69,73 @@ enum class PairingState : int64_t { 
  PAIR_FAIL_END = 19,

};



// ENUM definition for pairing state that in sync with ChromeOS structured metrics

// BluetoothProfileConnectionStateChanged/Profile and BlueZ metrics_bluetooth_profile.

enum class Profile : int64_t {

  UNKNOWN = 0,

  HSP = 1,

  HFP = 2,

  A2DP = 3,

  AVRCP = 4,

  HID = 5,

  HOG = 6,

  GATT = 7,

  GAP = 8,

  DEVICE_INFO = 9,

  BATTERY = 10,

  NEARBY = 11,

  PHONEHUB = 12,

};



// ENUM definition for profile connection state that in sync with ChromeOS structured metrics

// MetricProfileConnectionStatus and BlueZ's metrics_profile_conn_state.

enum class MetricProfileConnectionStatus : int64_t {

  // Common states for connecting and disconnecting a profile

  PROFILE_CONN_STATE_STARTING = 0,

  PROFILE_CONN_STATE_SUCCEED = 1,

  PROFILE_CONN_STATE_ALREADY_CONNECTED = 2,

  PROFILE_CONN_STATE_BUSY_CONNECTING = 3,

  PROFILE_CONN_STATE_CONNECTION_REFUSED = 4,

  PROFILE_CONN_STATE_CONNECTION_CANCELED = 5,

  PROFILE_CONN_STATE_REMOTE_UNAVAILABLE = 6,

  PROFILE_CONN_STATE_PROFILE_NOT_SUPPORTED = 7,

  PROFILE_CONN_STATE_UNKNOWN_ERROR = 8,



};



// ENUM definition for profile disconnection state that in sync with ChromeOS structured metrics

// MetricProfileDisconnectionStatus and BlueZ's metrics_profile_disconn_state.

enum class MetricProfileDisconnectionStatus : int64_t {

  // Common states for connecting and disconnecting a profile

  PROFILE_DISCONN_STATE_STARTING = 0,

  PROFILE_DISCONN_STATE_SUCCEED = 1,

  PROFILE_DISCONN_STATE_ALREADY_DISCONNECTED = 2,

  PROFILE_DISCONN_STATE_BUSY_DISCONNECTING = 3,

  PROFILE_DISCONN_STATE_DISCONNECTION_REFUSED = 4,

  PROFILE_DISCONN_STATE_DISCONNECTION_CANCELED = 5,

  PROFILE_DISCONN_STATE_BT_IO_CONNECT_ERROR = 6,

  PROFILE_DISCONN_STATE_INVALID_PARAMS = 7,

  PROFILE_DISCONN_STATE_UNKNOWN_ERROR = 8,

};



// A binary ENUM defines the metrics event is logged for: either for an attempt to connect or to disconnect.

enum class StateChangeType : int64_t { STATE_CHANGE_TYPE_DISCONNECT = 0, STATE_CHANGE_TYPE_CONNECT = 1 };



// A struct holds the parsed profile connection event.

struct ProfileConnectionEvent {

  int64_t type;

  int64_t profile;

  int64_t state;

};



// Convert topshim::btif::BtState to AdapterState.

AdapterState ToAdapterState(uint32_t state);



// Convert topshim::btif::bond_state info (status, addr, bond_state, and fail_reason) to PairingState

PairingState ToPairingState(uint32_t status, uint32_t bond_state, int32_t fail_reason);



// Convert Floss profile connection info to ProfileConnectionEvent

ProfileConnectionEvent ToProfileConnectionEvent(std::string addr, uint32_t profile, uint32_t status, uint32_t state);



}  // namespace metrics

}  // namespace bluetooth

system/gd/metrics/linux/metrics.cc

+1 −4
Original line number Diff line number Diff line
@@ -37,10 +37,7 @@ void LogMetricsDeviceInfoReport( 
    uint32_t product_id,

    uint32_t version) {}



void LogMetricsProfileConnectionAttempt(RawAddress* addr, uint32_t intent, uint32_t profile) {}



void LogMetricsProfileConnectionStateChanged(

    RawAddress* addr, uint32_t intent, uint32_t profile, uint32_t status, uint32_t state) {}

void LogMetricsProfileConnectionStateChanged(RawAddress* addr, uint32_t profile, uint32_t status, uint32_t state) {}



}  // namespace metrics

}  // namespace bluetooth