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

Commit 8b27f9a6 authored by Chen Chen's avatar Chen Chen Committed by Automerger Merge Worker
Browse files

Merge "BluetoothMetrics: Upload Android device HCI and LMP versions" am: 770cd232 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/2268142



Change-Id: Iaab933de26cc598eb20e6d76b8a949459680e89a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0fd9d486 770cd232

system/gd/hci/controller.cc

+7 −0
Original line number Original line Diff line number Diff line
@@ -24,6 +24,7 @@ 
#include "common/init_flags.h"

#include "common/init_flags.h"

#include "hci/hci_layer.h"

#include "hci/hci_layer.h"

#include "hci_controller_generated.h"

#include "hci_controller_generated.h"

#include "os/metrics.h"





namespace bluetooth {

namespace bluetooth {

namespace hci {

namespace hci {
@@ -247,6 +248,12 @@ struct Controller::impl { 
    ASSERT_LOG(status == ErrorCode::SUCCESS, "Status 0x%02hhx, %s", status, ErrorCodeText(status).c_str());

    ASSERT_LOG(status == ErrorCode::SUCCESS, "Status 0x%02hhx, %s", status, ErrorCodeText(status).c_str());





    local_version_information_ = complete_view.GetLocalVersionInformation();

    local_version_information_ = complete_view.GetLocalVersionInformation();

    bluetooth::os::LogMetricBluetoothLocalVersions(

        local_version_information_.manufacturer_name_,

        static_cast<uint8_t>(local_version_information_.lmp_version_),

        local_version_information_.lmp_subversion_,

        static_cast<uint8_t>(local_version_information_.hci_version_),

        local_version_information_.hci_revision_);

  }

  }





  void read_local_supported_commands_complete_handler(CommandCompleteView view) {

  void read_local_supported_commands_complete_handler(CommandCompleteView view) {

system/gd/os/android/metrics.cc

+21 −0
Original line number Original line Diff line number Diff line
@@ -424,6 +424,27 @@ void LogMetricBluetoothHalCrashReason( 
  }

  }

}

}





void LogMetricBluetoothLocalVersions(

    uint32_t lmp_manufacturer_name,

    uint8_t lmp_version,

    uint32_t lmp_subversion,

    uint8_t hci_version,

    uint32_t hci_revision) {

  int ret = stats_write(

      BLUETOOTH_LOCAL_VERSIONS_REPORTED, lmp_manufacturer_name, lmp_version, lmp_subversion, hci_version, hci_revision);

  if (ret < 0) {

    LOG_WARN(

        "Failed for LogMetricBluetoothLocalVersions, "

        "lmp_manufacturer_name %d, lmp_version %hhu, lmp_subversion %d, hci_version %hhu, hci_revision %d, error %d",

        lmp_manufacturer_name,

        lmp_version,

        lmp_subversion,

        hci_version,

        hci_revision,

        ret);

  }

}



void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {

void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {

  int ret = stats_write(BLUETOOTH_CODE_PATH_COUNTER, key, count);

  int ret = stats_write(BLUETOOTH_CODE_PATH_COUNTER, key, count);

  if (ret < 0) {

  if (ret < 0) {

system/gd/os/host/metrics.cc

+7 −0
Original line number Original line Diff line number Diff line
@@ -103,6 +103,13 @@ void LogMetricA2dpPlaybackEvent(const Address& address, int playback_state, int 
void LogMetricBluetoothHalCrashReason(

void LogMetricBluetoothHalCrashReason(

    const Address& address, uint32_t error_code, uint32_t vendor_error_code) {}

    const Address& address, uint32_t error_code, uint32_t vendor_error_code) {}





void LogMetricBluetoothLocalVersions(

    uint32_t lmp_manufacturer_name,

    uint8_t lmp_version,

    uint32_t lmp_subversion,

    uint8_t hci_version,

    uint32_t hci_reversion) {}



void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {}

void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {}

}  // namespace os

}  // namespace os

}  // namespace bluetooth
 
}  // namespace bluetooth

system/gd/os/linux/metrics.cc

+7 −0
Original line number Original line Diff line number Diff line
@@ -103,6 +103,13 @@ void LogMetricA2dpPlaybackEvent(const Address& address, int playback_state, int 
void LogMetricBluetoothHalCrashReason(

void LogMetricBluetoothHalCrashReason(

    const Address& address, uint32_t error_code, uint32_t vendor_error_code) {}

    const Address& address, uint32_t error_code, uint32_t vendor_error_code) {}





void LogMetricBluetoothLocalVersions(

    uint32_t lmp_manufacturer_name,

    uint8_t lmp_version,

    uint32_t lmp_subversion,

    uint8_t hci_version,

    uint32_t hci_revision) {}



void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {}

void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count) {}





}  // namespace os

}  // namespace os

system/gd/os/metrics.h

+7 −0
Original line number Original line Diff line number Diff line
@@ -268,6 +268,13 @@ void LogMetricBluetoothHalCrashReason( 
    uint32_t error_code,

    uint32_t error_code,

    uint32_t vendor_error_code);

    uint32_t vendor_error_code);





void LogMetricBluetoothLocalVersions(

    uint32_t lmp_manufacturer_name,

    uint8_t lmp_version,

    uint32_t lmp_subversion,

    uint8_t hci_version,

    uint32_t hci_revision);



void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count);

void LogMetricBluetoothCodePathCounterMetrics(int32_t key, int64_t count);

}  // namespace os

}  // namespace os