Merge "Stop reporting the duration including sleep for background DexOpt" (365a8d33) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/pm/BackgroundDexOptService.java

+1 −12

Original line number	Diff line number	Diff line
		@@ -152,8 +152,6 @@ public final class BackgroundDexOptService {

		@GuardedBy("mLock") @Status private int mLastExecutionStatus = STATUS_OK;

		@GuardedBy("mLock") private long mLastExecutionStartTimeMs;
		@GuardedBy("mLock") private long mLastExecutionDurationIncludingSleepMs;
		@GuardedBy("mLock") private long mLastExecutionStartUptimeMs;
		@GuardedBy("mLock") private long mLastExecutionDurationMs;

		@@ -234,10 +232,6 @@ public final class BackgroundDexOptService {
		writer.println(mDisableJobSchedulerJobs);
		writer.print("mLastExecutionStatus:");
		writer.println(mLastExecutionStatus);
		writer.print("mLastExecutionStartTimeMs:");
		writer.println(mLastExecutionStartTimeMs);
		writer.print("mLastExecutionDurationIncludingSleepMs:");
		writer.println(mLastExecutionDurationIncludingSleepMs);
		writer.print("mLastExecutionStartUptimeMs:");
		writer.println(mLastExecutionStartUptimeMs);
		writer.print("mLastExecutionDurationMs:");
		@@ -564,8 +558,6 @@ public final class BackgroundDexOptService {
		private boolean runIdleOptimization(
		PackageManagerService pm, List<String> pkgs, boolean isPostBootUpdate) {
		synchronized (mLock) {
		mLastExecutionStartTimeMs = SystemClock.elapsedRealtime();
		mLastExecutionDurationIncludingSleepMs = -1;
		mLastExecutionStartUptimeMs = SystemClock.uptimeMillis();
		mLastExecutionDurationMs = -1;
		}
		@@ -574,8 +566,6 @@ public final class BackgroundDexOptService {
		logStatus(status);
		synchronized (mLock) {
		mLastExecutionStatus = status;
		mLastExecutionDurationIncludingSleepMs =
		SystemClock.elapsedRealtime() - mLastExecutionStartTimeMs;
		mLastExecutionDurationMs = SystemClock.uptimeMillis() - mLastExecutionStartUptimeMs;
		}

		@@ -979,10 +969,9 @@ public final class BackgroundDexOptService {
		synchronized (mLock) {
		status = mLastExecutionStatus;
		durationMs = mLastExecutionDurationMs;
		durationIncludingSleepMs = mLastExecutionDurationIncludingSleepMs;
		}

		mStatsLogger.write(status, params.getStopReason(), durationMs, durationIncludingSleepMs);
		mStatsLogger.write(status, params.getStopReason(), durationMs);
		}

		/** Injector pattern for testing purpose */

services/core/java/com/android/server/pm/dex/ArtStatsLogUtils.java

+7 −4

Original line number	Diff line number	Diff line
		@@ -320,12 +320,15 @@ public class ArtStatsLogUtils {
		public static class BackgroundDexoptJobStatsLogger {
		/** Writes background dexopt job stats to statsd. */
		public void write(@BackgroundDexOptService.Status int status,
		@JobParameters.StopReason int cancellationReason, long durationMs,
		long durationIncludingSleepMs) {
		ArtStatsLog.write(ArtStatsLog.BACKGROUND_DEXOPT_JOB_ENDED,
		@JobParameters.StopReason int cancellationReason,
		long durationMs) {
		ArtStatsLog.write(
		ArtStatsLog.BACKGROUND_DEXOPT_JOB_ENDED,
		STATUS_MAP.getOrDefault(status,
		ArtStatsLog.BACKGROUND_DEXOPT_JOB_ENDED__STATUS__STATUS_UNKNOWN),
		cancellationReason, durationMs, durationIncludingSleepMs);
		cancellationReason,
		durationMs,
		0); // deprecated, used to be durationIncludingSleepMs
		}
		}
		}