diff options
author | Suprabh Shukla <suprabh@google.com> | 2021-07-20 22:59:18 -0700 |
---|---|---|
committer | Suprabh Shukla <suprabh@google.com> | 2021-07-20 23:05:22 -0700 |
commit | ef2636a43a5da876def5c0096c4c2f2c507f5feb (patch) | |
tree | 867828319ce52be6fffd5e5fadc1b7e16ade676a /apex | |
parent | 4e05326a55ce38e4f5ea4b2de263128c3bd05727 (diff) |
Logging the caller uid state with AlarmScheduled atom
Test: statsd_testdrive 368
Test: atest FrameworksMockingServicesTests:AlarmManagerServiceTest
Test: atest CtsAlarmManagerTestCases
Test: atest CtsStatsdAtomHostTestCases:AlarmStatsTests
Bug: 194249875
Change-Id: Ia55306b3a4839aadc4bc2fdd873878b1a4e591fb
Diffstat (limited to 'apex')
-rw-r--r-- | apex/jobscheduler/service/java/com/android/server/alarm/AlarmManagerService.java | 3 | ||||
-rw-r--r-- | apex/jobscheduler/service/java/com/android/server/alarm/MetricsHelper.java | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/apex/jobscheduler/service/java/com/android/server/alarm/AlarmManagerService.java b/apex/jobscheduler/service/java/com/android/server/alarm/AlarmManagerService.java index 9f529548833d..16a3eefec4e9 100644 --- a/apex/jobscheduler/service/java/com/android/server/alarm/AlarmManagerService.java +++ b/apex/jobscheduler/service/java/com/android/server/alarm/AlarmManagerService.java @@ -2052,10 +2052,11 @@ public class AlarmManagerService extends SystemService { + " -- package not allowed to start"); return; } + final int callerProcState = mActivityManagerInternal.getUidProcessState(callingUid); removeLocked(operation, directReceiver, REMOVE_REASON_UNDEFINED); incrementAlarmCount(a.uid); setImplLocked(a); - MetricsHelper.pushAlarmScheduled(a); + MetricsHelper.pushAlarmScheduled(a, callerProcState); } /** diff --git a/apex/jobscheduler/service/java/com/android/server/alarm/MetricsHelper.java b/apex/jobscheduler/service/java/com/android/server/alarm/MetricsHelper.java index 4e7311f10252..4c2f8d124566 100644 --- a/apex/jobscheduler/service/java/com/android/server/alarm/MetricsHelper.java +++ b/apex/jobscheduler/service/java/com/android/server/alarm/MetricsHelper.java @@ -22,6 +22,7 @@ import static com.android.internal.util.FrameworkStatsLog.ALARM_SCHEDULED__EXACT import static com.android.internal.util.FrameworkStatsLog.ALARM_SCHEDULED__EXACT_ALARM_ALLOWED_REASON__PERMISSION; import static com.android.server.alarm.AlarmManagerService.INDEFINITE_DELAY; +import android.app.ActivityManager; import android.app.AlarmManager; import android.app.StatsManager; import android.content.Context; @@ -93,7 +94,7 @@ class MetricsHelper { } } - static void pushAlarmScheduled(Alarm a) { + static void pushAlarmScheduled(Alarm a, int callerProcState) { FrameworkStatsLog.write( FrameworkStatsLog.ALARM_SCHEDULED, a.uid, @@ -103,7 +104,8 @@ class MetricsHelper { a.alarmClock != null, a.repeatInterval != 0, reasonToStatsReason(a.mExactAllowReason), - AlarmManagerService.isRtc(a.type)); + AlarmManagerService.isRtc(a.type), + ActivityManager.processStateAmToProto(callerProcState)); } static void pushAlarmBatchDelivered(int numAlarms, int wakeups) { |