From d3aa485f5b7c8f7c4d2c7d11294e0b6b7b97f2d5 Mon Sep 17 00:00:00 2001 From: Edgar Arriaga Date: Tue, 13 Dec 2022 18:13:57 +0000 Subject: [PATCH] Remove spammy on compaction request trace point This removes the spammy trace point that happens on compaction request and keep its only extra information into the existing trace point on actual compaction. Bug: 262417333 Test: Verified Trace Change-Id: I6f257476b61d4f96b57656514cc3ba7ff7dfbd3d --- .../core/java/com/android/server/am/CachedAppOptimizer.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/services/core/java/com/android/server/am/CachedAppOptimizer.java b/services/core/java/com/android/server/am/CachedAppOptimizer.java index 26891931a5512..913f151496467 100644 --- a/services/core/java/com/android/server/am/CachedAppOptimizer.java +++ b/services/core/java/com/android/server/am/CachedAppOptimizer.java @@ -742,9 +742,6 @@ public final class CachedAppOptimizer { "compactApp " + app.mOptRecord.getReqCompactSource().name() + " " + app.mOptRecord.getReqCompactProfile().name() + " " + processName); } - Trace.instantForTrack(Trace.TRACE_TAG_ACTIVITY_MANAGER, ATRACE_COMPACTION_TRACK, - "compactApp " + app.mOptRecord.getReqCompactSource().name() + " " - + app.mOptRecord.getReqCompactProfile().name() + " " + processName); app.mOptRecord.setHasPendingCompact(true); app.mOptRecord.setForceCompact(force); mPendingCompactionProcesses.add(app); @@ -1820,7 +1817,8 @@ public final class CachedAppOptimizer { try { Trace.traceBegin(Trace.TRACE_TAG_ACTIVITY_MANAGER, "Compact " + resolvedAction.name() + ": " + name - + " lastOomAdjReason: " + oomAdjReason); + + " lastOomAdjReason: " + oomAdjReason + + " source: " + compactSource.name()); long zramUsedKbBefore = getUsedZramMemory(); long startCpuTime = threadCpuTimeNs(); mProcessDependencies.performCompaction(resolvedAction, pid);