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

Commit de6a99d7 authored by Howard Ro's avatar Howard Ro Committed by Gerrit Code Review
Browse files

Merge changes from topic "libmetricslogger"

* changes:
  Deprecate libmetricslogger
  Remove libmetricslogger dependency from bootstats
parents a6c834f9 32724019
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -30,8 +30,8 @@ cc_defaults {
        "libbase",
        "libcutils",
        "liblog",
        "libmetricslogger",
    ],
    static_libs: ["libgtest_prod"],
}

// bootstat static library
+19 −12
Original line number Diff line number Diff line
@@ -44,7 +44,6 @@
#include <android/log.h>
#include <cutils/android_reboot.h>
#include <cutils/properties.h>
#include <metricslogger/metrics_logger.h>
#include <statslog.h>

#include "boot_event_record_store.h"
@@ -57,9 +56,10 @@ void LogBootEvents() {
  BootEventRecordStore boot_event_store;

  auto events = boot_event_store.GetAllBootEvents();
  for (auto i = events.cbegin(); i != events.cend(); ++i) {
    android::metricslogger::LogHistogram(i->first, i->second);
  }
  // TODO(b/148575354): Replace with statsd.
  // for (auto i = events.cbegin(); i != events.cend(); ++i) {
  //   android::metricslogger::LogHistogram(i->first, i->second);
  // }
}

// Records the named boot |event| to the record store. If |value| is non-empty
@@ -1212,13 +1212,17 @@ void RecordBootReason() {
  const auto reason = android::base::GetProperty(bootloader_reboot_reason_property, "");

  if (reason.empty()) {
    // TODO(b/148575354): Replace with statsd.
    // Log an empty boot reason value as '<EMPTY>' to ensure the value is intentional
    // (and not corruption anywhere else in the reporting pipeline).
    android::metricslogger::LogMultiAction(android::metricslogger::ACTION_BOOT,
                                           android::metricslogger::FIELD_PLATFORM_REASON, "<EMPTY>");
    // android::metricslogger::LogMultiAction(android::metricslogger::ACTION_BOOT,
    //                                        android::metricslogger::FIELD_PLATFORM_REASON,
    //                                        "<EMPTY>");
  } else {
    android::metricslogger::LogMultiAction(android::metricslogger::ACTION_BOOT,
                                           android::metricslogger::FIELD_PLATFORM_REASON, reason);
    // TODO(b/148575354): Replace with statsd.
    // android::metricslogger::LogMultiAction(android::metricslogger::ACTION_BOOT,
    //                                        android::metricslogger::FIELD_PLATFORM_REASON,
    //                                        reason);
  }

  // Log the raw bootloader_boot_reason property value.
@@ -1246,9 +1250,10 @@ void RecordFactoryReset() {
  time_t current_time_utc = time(nullptr);

  if (current_time_utc < 0) {
    // TODO(b/148575354): Replace with statsd.
    // UMA does not display negative values in buckets, so convert to positive.
    android::metricslogger::LogHistogram("factory_reset_current_time_failure",
                                         std::abs(current_time_utc));
    // android::metricslogger::LogHistogram("factory_reset_current_time_failure",
    //                                      std::abs(current_time_utc));

    // Logging via BootEventRecordStore to see if using android::metricslogger::LogHistogram
    // is losing records somehow.
@@ -1256,7 +1261,8 @@ void RecordFactoryReset() {
                                           std::abs(current_time_utc));
    return;
  } else {
    android::metricslogger::LogHistogram("factory_reset_current_time", current_time_utc);
    // TODO(b/148575354): Replace with statsd.
    // android::metricslogger::LogHistogram("factory_reset_current_time", current_time_utc);

    // Logging via BootEventRecordStore to see if using android::metricslogger::LogHistogram
    // is losing records somehow.
@@ -1276,7 +1282,8 @@ void RecordFactoryReset() {
  // Calculate and record the difference in time between now and the
  // factory_reset time.
  time_t factory_reset_utc = record.second;
  android::metricslogger::LogHistogram("factory_reset_record_value", factory_reset_utc);
  // TODO(b/148575354): Replace with statsd.
  // android::metricslogger::LogHistogram("factory_reset_record_value", factory_reset_utc);

  // Logging via BootEventRecordStore to see if using android::metricslogger::LogHistogram
  // is losing records somehow.

libmetricslogger/Android.bp

deleted100644 → 0
+0 −64
Original line number Diff line number Diff line
// Copyright 2017 The Android Open Source Project

metricslogger_lib_src_files = [
    "metrics_logger.cpp",
]

cc_defaults {
    name: "metricslogger_defaults",

    host_supported: true,

    export_include_dirs: ["include"],
    local_include_dirs: ["include"],
    shared_libs: [
        "libbase",
        "liblog",
        "libstatssocket",
    ],
    whole_static_libs: ["libgtest_prod"],

    cflags: [
        "-Wall",
        "-Wextra",
        "-Werror",
    ],
}

// metricslogger shared library
// -----------------------------------------------------------------------------
cc_library {
    name: "libmetricslogger",
    srcs: metricslogger_lib_src_files,
    defaults: ["metricslogger_defaults"],
    export_shared_lib_headers: ["libstatssocket"],
}

// metricslogger shared library, debug
// -----------------------------------------------------------------------------
cc_library_shared {
    name: "libmetricslogger_debug",
    srcs: metricslogger_lib_src_files,
    defaults: ["metricslogger_defaults"],

    target: {
        host: {
            cflags: ["-UNDEBUG"],
        },
    },
}

// Native tests
// -----------------------------------------------------------------------------
cc_test {
    name: "metricslogger_tests",
    isolated: true,
    defaults: ["metricslogger_defaults"],
    shared_libs: [
        "libbase",
        "libmetricslogger_debug",
    ],
    srcs: [
        "metrics_logger_test.cpp",
    ],
}

libmetricslogger/OWNERS

deleted100644 → 0
+0 −2
Original line number Diff line number Diff line
cwren@google.com
jhawkins@google.com
+0 −159
Original line number Diff line number Diff line
/*
 * Copyright (C) 2017 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 <log/log_event_list.h>
#include <stats_event_list.h>
#include <cstdint>
#include <string>

namespace android {
namespace metricslogger {

// Logs a Tron histogram metric named |event| containing |data| to the Tron log
// buffer.
void LogHistogram(const std::string& event, int32_t data);

// Logs a Tron counter metric named |name| containing |val| count to the Tron
// log buffer.
void LogCounter(const std::string& name, int32_t val);

// Logs a Tron multi_action with category|category| containing the string
// |value| in the field |field|.
void LogMultiAction(int32_t category, int32_t field, const std::string& value);

// Logs a Tron complex event.
//
// A complex event can include data in a structure not suppored by the other
// log event types above.
//
// Note that instances of this class are single use. You must call Record()
// to write the event to the event log.
class ComplexEventLogger {
  private:
    android_log_event_list logger;
    stats_event_list stats_logger;

  public:
    // Create a complex event with category|category|.
    explicit ComplexEventLogger(int category);
    // Set the package name that this event originates from.
    void SetPackageName(const std::string& package_name);
    // Add tagged data to the event, with the given tag and integer value.
    void AddTaggedData(int tag, int32_t value);
    // Add tagged data to the event, with the given tag and string value.
    void AddTaggedData(int tag, const std::string& value);
    // Add tagged data to the event, with the given tag and integer value.
    void AddTaggedData(int tag, int64_t value);
    // Add tagged data to the event, with the given tag and float value.
    void AddTaggedData(int tag, float value);
    // Record this event. This method can only be used once per instance
    // of ComplexEventLogger. Do not made any subsequent calls to AddTaggedData
    // after recording an event.
    void Record();
};

// TODO: replace these with the metric_logger.proto definitions
enum {
    LOGBUILDER_CATEGORY = 757,
    LOGBUILDER_TYPE = 758,
    LOGBUILDER_NAME = 799,
    LOGBUILDER_BUCKET = 801,
    LOGBUILDER_VALUE = 802,
    LOGBUILDER_COUNTER = 803,
    LOGBUILDER_HISTOGRAM = 804,
    LOGBUILDER_PACKAGENAME = 806,

    ACTION_BOOT = 1098,
    FIELD_PLATFORM_REASON = 1099,

    FIELD_DURATION_MILLIS = 1304,

    FIELD_END_BATTERY_PERCENT = 1308,

    ACTION_HIDDEN_API_ACCESSED = 1391,
    FIELD_HIDDEN_API_ACCESS_METHOD = 1392,
    FIELD_HIDDEN_API_ACCESS_DENIED = 1393,
    FIELD_HIDDEN_API_SIGNATURE = 1394,

    ACTION_USB_CONNECTOR_CONNECTED = 1422,
    ACTION_USB_CONNECTOR_DISCONNECTED = 1423,
    ACTION_USB_AUDIO_CONNECTED = 1424,
    FIELD_USB_AUDIO_VIDPID = 1425,
    ACTION_USB_AUDIO_DISCONNECTED = 1426,
    ACTION_HARDWARE_FAILED = 1427,
    FIELD_HARDWARE_TYPE = 1428,
    FIELD_HARDWARE_FAILURE_CODE = 1429,
    ACTION_PHYSICAL_DROP = 1430,
    FIELD_CONFIDENCE_PERCENT = 1431,
    FIELD_ACCEL_MILLI_G = 1432,
    ACTION_BATTERY_HEALTH = 1433,
    FIELD_BATTERY_HEALTH_SNAPSHOT_TYPE = 1434,
    FIELD_BATTERY_TEMPERATURE_DECI_C = 1435,
    FIELD_BATTERY_VOLTAGE_UV = 1436,
    FIELD_BATTERY_OPEN_CIRCUIT_VOLTAGE_UV = 1437,
    ACTION_BATTERY_CHARGE_CYCLES = 1438,
    FIELD_BATTERY_CHARGE_CYCLES = 1439,

    ACTION_SLOW_IO = 1442,
    FIELD_IO_OPERATION_TYPE = 1443,
    FIELD_IO_OPERATION_COUNT = 1444,
    ACTION_SPEAKER_IMPEDANCE = 1445,
    FIELD_SPEAKER_IMPEDANCE_MILLIOHMS = 1446,
    FIELD_SPEAKER_LOCATION = 1447,
    FIELD_BATTERY_RESISTANCE_UOHMS = 1448,
    FIELD_BATTERY_CURRENT_UA = 1449,
    FIELD_HARDWARE_LOCATION = 1450,
    ACTION_BATTERY_CAUSED_SHUTDOWN = 1451,
};

enum {
    TYPE_ACTION = 4,
};

enum {
    ACCESS_METHOD_NONE = 0,
    ACCESS_METHOD_REFLECTION = 1,
    ACCESS_METHOD_JNI = 2,
    ACCESS_METHOD_LINKING = 3,
};

enum HardwareType {
    HARDWARE_UNKNOWN = 0,
    HARDWARE_MICROPHONE = 1,
    HARDWARE_CODEC = 2,
    HARDWARE_SPEAKER = 3,
    HARDWARE_FINGERPRINT = 4,
};

enum HardwareFailureCode {
    HARDWARE_FAILURE_UNKNOWN = 0,
    HARDWARE_FAILURE_COMPLETE = 1,
    HARDWARE_FAILURE_SPEAKER_HIGH_Z = 2,
    HARDWARE_FAILURE_SPEAKER_SHORT = 3,
    HARDWARE_FAILURE_FINGERPRINT_SENSOR_BROKEN = 4,
    HARDWARE_FAILURE_FINGERPRINT_TOO_MANY_DEAD_PIXELS = 5,
};

enum IoOperation {
    IOOP_UNKNOWN = 0,
    IOOP_READ = 1,
    IOOP_WRITE = 2,
    IOOP_UNMAP = 3,
    IOOP_SYNC = 4,
};

}  // namespace metricslogger
}  // namespace android
Loading