From f7eb401770b7d6492a5f4914e6cdf27602a28add Mon Sep 17 00:00:00 2001 From: Rafal Slawik Date: Wed, 8 Feb 2023 13:49:26 +0000 Subject: [PATCH] Add counter metrics for tracking failures Test: build Bug: 268323312 Change-Id: I7d0f7348c271390f6cbbb07be2d72365d91cd018 --- .../internal/os/KernelSingleProcessCpuThreadReader.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/core/java/com/android/internal/os/KernelSingleProcessCpuThreadReader.java b/core/java/com/android/internal/os/KernelSingleProcessCpuThreadReader.java index 4d2a08a4bcf35..e2c096c11feed 100644 --- a/core/java/com/android/internal/os/KernelSingleProcessCpuThreadReader.java +++ b/core/java/com/android/internal/os/KernelSingleProcessCpuThreadReader.java @@ -20,6 +20,7 @@ import android.annotation.Nullable; import android.util.Slog; import com.android.internal.annotations.VisibleForTesting; +import com.android.internal.expresslog.Counter; import java.io.IOException; import java.util.Arrays; @@ -117,13 +118,16 @@ public class KernelSingleProcessCpuThreadReader { public void startTrackingThreadCpuTimes() { if (!mIsTracking) { if (!startTrackingProcessCpuTimes(mPid, mCpuTimeInStateReader)) { - Slog.e(TAG, "Failed to start tracking process CPU times for " + mPid); + Slog.wtf(TAG, "Failed to start tracking process CPU times for " + mPid); + Counter.logIncrement("cpu.value_process_tracking_start_failure_count"); } if (mSelectedThreadNativeTids.length > 0) { if (!startAggregatingThreadCpuTimes(mSelectedThreadNativeTids, mCpuTimeInStateReader)) { - Slog.e(TAG, "Failed to start tracking aggregated thread CPU times for " + Slog.wtf(TAG, "Failed to start tracking aggregated thread CPU times for " + Arrays.toString(mSelectedThreadNativeTids)); + Counter.logIncrement( + "cpu.value_aggregated_thread_tracking_start_failure_count"); } } mIsTracking = true;