diff --git a/services/core/java/com/android/server/pm/BackgroundDexOptService.java b/services/core/java/com/android/server/pm/BackgroundDexOptService.java index faa06f70d4114..9bca9f0192b2a 100644 --- a/services/core/java/com/android/server/pm/BackgroundDexOptService.java +++ b/services/core/java/com/android/server/pm/BackgroundDexOptService.java @@ -404,7 +404,7 @@ public final class BackgroundDexOptService { "BackgroundDexOptService_" + (isPostBootUpdateJob ? "PostBoot" : "Idle"), () -> { TimingsTraceAndSlog tr = - new TimingsTraceAndSlog(TAG, Trace.TRACE_TAG_PACKAGE_MANAGER); + new TimingsTraceAndSlog(TAG, Trace.TRACE_TAG_DALVIK); tr.traceBegin("jobExecution"); boolean completed = false; boolean fatalError = false; @@ -494,6 +494,8 @@ public final class BackgroundDexOptService { @GuardedBy("mLock") private void waitForDexOptThreadToFinishLocked() { TimingsTraceAndSlog tr = new TimingsTraceAndSlog(TAG, Trace.TRACE_TAG_PACKAGE_MANAGER); + // This tracing section doesn't have any correspondence in ART Service - it never waits for + // cancellation to finish. tr.traceBegin("waitForDexOptThreadToFinishLocked"); try { // Wait but check in regular internal to see if the thread is still alive. diff --git a/services/core/java/com/android/server/pm/DexOptHelper.java b/services/core/java/com/android/server/pm/DexOptHelper.java index 0fd81ac0bc415..a9d4115b4b79f 100644 --- a/services/core/java/com/android/server/pm/DexOptHelper.java +++ b/services/core/java/com/android/server/pm/DexOptHelper.java @@ -16,7 +16,7 @@ package com.android.server.pm; -import static android.os.Trace.TRACE_TAG_PACKAGE_MANAGER; +import static android.os.Trace.TRACE_TAG_DALVIK; import static com.android.server.LocalManagerRegistry.ManagerNotFoundException; import static com.android.server.pm.ApexManager.ActiveApexInfo; @@ -470,11 +470,11 @@ public final class DexOptHelper { @DexOptResult private int performDexOptTraced(DexoptOptions options) { - Trace.traceBegin(TRACE_TAG_PACKAGE_MANAGER, "dexopt"); + Trace.traceBegin(TRACE_TAG_DALVIK, "dexopt"); try { return performDexOptInternal(options); } finally { - Trace.traceEnd(TRACE_TAG_PACKAGE_MANAGER); + Trace.traceEnd(TRACE_TAG_DALVIK); } } @@ -605,7 +605,7 @@ public final class DexOptHelper { throw new IllegalArgumentException("Can't dexopt APEX package: " + packageName); } - Trace.traceBegin(TRACE_TAG_PACKAGE_MANAGER, "dexopt"); + Trace.traceBegin(TRACE_TAG_DALVIK, "dexopt"); // Whoever is calling forceDexOpt wants a compiled package. // Don't use profiles since that may cause compilation to be skipped. @@ -615,7 +615,7 @@ public final class DexOptHelper { @DexOptResult int res = performDexOptInternalWithDependenciesLI(pkg, packageState, options); - Trace.traceEnd(TRACE_TAG_PACKAGE_MANAGER); + Trace.traceEnd(TRACE_TAG_DALVIK); if (res != PackageDexOptimizer.DEX_OPT_PERFORMED) { throw new IllegalStateException("Failed to dexopt: " + res); } diff --git a/services/core/java/com/android/server/pm/PackageDexOptimizer.java b/services/core/java/com/android/server/pm/PackageDexOptimizer.java index f33813759e4f8..0a90e7a30db63 100644 --- a/services/core/java/com/android/server/pm/PackageDexOptimizer.java +++ b/services/core/java/com/android/server/pm/PackageDexOptimizer.java @@ -386,7 +386,7 @@ public class PackageDexOptimizer { options.getCompilationReason()); // OTAPreopt doesn't have stats so don't report in that case. if (packageStats != null) { - Trace.traceBegin(Trace.TRACE_TAG_PACKAGE_MANAGER, "dex2oat-metrics"); + Trace.traceBegin(Trace.TRACE_TAG_DALVIK, "dex2oat-metrics"); try { long sessionId = sRandom.nextLong(); ArtStatsLogUtils.writeStatsLog( @@ -403,7 +403,7 @@ public class PackageDexOptimizer { dexCodeIsa, path); } finally { - Trace.traceEnd(Trace.TRACE_TAG_PACKAGE_MANAGER); + Trace.traceEnd(Trace.TRACE_TAG_DALVIK); } } diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java index a42e78b697091..6557ec565343f 100644 --- a/services/core/java/com/android/server/pm/PackageManagerService.java +++ b/services/core/java/com/android/server/pm/PackageManagerService.java @@ -6690,9 +6690,9 @@ public class PackageManagerService implements PackageSender, TestUtilityService } synchronized (mInstallLock) { - Trace.traceBegin(TRACE_TAG_PACKAGE_MANAGER, "dump profiles"); + Trace.traceBegin(Trace.TRACE_TAG_DALVIK, "dump profiles"); mArtManagerService.dumpProfiles(pkg, dumpClassesAndMethods); - Trace.traceEnd(TRACE_TAG_PACKAGE_MANAGER); + Trace.traceEnd(Trace.TRACE_TAG_DALVIK); } }