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

Commit 0f47cce8 authored by Michael Sun's avatar Michael Sun
Browse files

floss: add metrics logging for adapter state 

Complete the adapter state reporting call by creating a utility helper
to fetch boot_id from the kernel file and introducing metrics event
definition and conversion skeletons.

BUG: 232098615
Tag: #floss
Test: emerge-${BOARD} floss

Change-Id: Ied7a7552401b1c2409ce052bde5b242dac956f38
parent 98ac72fc

system/gd/metrics/BUILD.gn

+8 −2
Original line number Diff line number Diff line
@@ -15,7 +15,10 @@ 
#



source_set("BluetoothMetricsSources_chromeos") {

  sources = [ "chromeos/metrics.cc" ]

  sources = [

    "chromeos/metrics.cc",

    "chromeos/metrics_event.cc",

  ]

  configs += [ "//bt/system/gd:gd_defaults" ]

}
@@ -25,7 +28,10 @@ source_set("BluetoothMetricsSources_linux") { 
}



source_set("BluetoothMetricsSources") {

  sources = [ "counter_metrics.cc" ]

  sources = [

    "counter_metrics.cc",

    "utils.cc",

  ]



  configs += [ "//bt/system/gd:gd_defaults" ]

  deps = [ "//bt/system/gd:gd_default_deps" ]

system/gd/metrics/chromeos/metrics.cc

+18 −1
Original line number Diff line number Diff line
@@ -17,10 +17,27 @@ 


#include "gd/metrics/metrics.h"



#include <metrics/structured_events.h>



#include "common/time_util.h"

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

#include "gd/metrics/utils.h"



namespace bluetooth {

namespace metrics {



void LogMetricsAdapterStateChanged(uint32_t state){};

void LogMetricsAdapterStateChanged(uint32_t state) {

  std::string boot_id;



  if (!GetBootId(&boot_id)) return;



  ::metrics::structured::events::bluetooth::BluetoothAdapterStateChanged()

      .SetBootId(boot_id)

      .SetSystemTime(bluetooth::common::time_get_os_boottime_us())

      .SetIsFloss(true)

      .SetAdapterState((int64_t)ToAdapterState(state))

      .Record();

};



}  // namespace metrics

}  // namespace bluetooth

system/gd/metrics/chromeos/metrics_event.cc

0 → 100644
+26 −0
Original line number Diff line number Diff line 
/*

 * Copyright 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

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



namespace bluetooth {

namespace metrics {



AdapterState ToAdapterState(uint32_t state) {

  return state == 1 ? AdapterState::ON : AdapterState::OFF;

}



}  // namespace metrics

}  // namespace bluetooth
 
 No newline at end of file

system/gd/metrics/chromeos/metrics_event.h

0 → 100644
+31 −0
Original line number Diff line number Diff line 
/*

 * Copyright 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

#pragma once



#include <cstdint>



namespace bluetooth {

namespace metrics {



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

// BluetoothAdapterStateChanged/AdapterState.

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



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

AdapterState ToAdapterState(uint32_t state);



}  // namespace metrics

}  // namespace bluetooth
 
 No newline at end of file

system/gd/metrics/utils.cc

0 → 100644
+39 −0
Original line number Diff line number Diff line 
/*

 * Copyright 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



#include "metrics/utils.h"



#include <base/files/file_util.h>

#include <base/strings/string_util.h>



namespace bluetooth {

namespace metrics {



namespace {

// The path to the kernel's boot_id.

const char kBootIdPath[] = "/proc/sys/kernel/random/boot_id";

}  // namespace



bool GetBootId(std::string* boot_id) {

  if (!base::ReadFileToString(base::FilePath(kBootIdPath), boot_id)) {

    return false;

  }

  base::TrimWhitespaceASCII(*boot_id, base::TRIM_TRAILING, boot_id);

  return true;

}



}  // namespace metrics

}  // namespace bluetooth