diff --git a/services/core/java/com/android/server/am/CachedAppOptimizer.java b/services/core/java/com/android/server/am/CachedAppOptimizer.java index cee9f79285a76..347557cacda33 100644 --- a/services/core/java/com/android/server/am/CachedAppOptimizer.java +++ b/services/core/java/com/android/server/am/CachedAppOptimizer.java @@ -110,6 +110,8 @@ public final class CachedAppOptimizer { private static final String ATRACE_COMPACTION_TRACK = "Compaction"; + @VisibleForTesting static final boolean ENABLE_FILE_COMPACT = false; + // Defaults for phenotype flags. @VisibleForTesting static final Boolean DEFAULT_USE_COMPACTION = false; @VisibleForTesting static final Boolean DEFAULT_USE_FREEZER = true; @@ -515,9 +517,11 @@ public final class CachedAppOptimizer { @GuardedBy("mProcLock") void compactAppSome(ProcessRecord app, boolean force) { - app.mOptRecord.setReqCompactAction(COMPACT_PROCESS_SOME); - ++mSomeCompactRequest; - compactApp(app, force, "some"); + if (ENABLE_FILE_COMPACT) { + app.mOptRecord.setReqCompactAction(COMPACT_PROCESS_SOME); + ++mSomeCompactRequest; + compactApp(app, force, "some"); + } } // This method returns true only if requirements are met. Note, that requirements are different @@ -1278,6 +1282,15 @@ public final class CachedAppOptimizer { } } + if (!ENABLE_FILE_COMPACT) { + // Turn off file compaction + if (resolvedAction == COMPACT_ACTION_FULL) { + resolvedAction = COMPACT_ACTION_ANON; + } else if (resolvedAction == COMPACT_ACTION_FILE) { + resolvedAction = COMPACT_ACTION_NONE; + } + } + return resolvedAction; } @@ -1545,6 +1558,9 @@ public final class CachedAppOptimizer { } int resolvedAction = resolveCompactionAction(requestedAction); + if (resolvedAction == COMPACT_ACTION_NONE) { + return; + } action = compactActionIntToString(resolvedAction); try { diff --git a/services/tests/mockingservicestests/src/com/android/server/am/CachedAppOptimizerTest.java b/services/tests/mockingservicestests/src/com/android/server/am/CachedAppOptimizerTest.java index 2baa1ec6cdc24..8d9c592587d6a 100644 --- a/services/tests/mockingservicestests/src/com/android/server/am/CachedAppOptimizerTest.java +++ b/services/tests/mockingservicestests/src/com/android/server/am/CachedAppOptimizerTest.java @@ -1079,14 +1079,16 @@ public final class CachedAppOptimizerTest { mCachedAppOptimizerUnderTest.mLastCompactionStats.clear(); - // We force a some compaction - mCachedAppOptimizerUnderTest.compactAppSome(processRecord, true); - waitForHandler(); - // then process is compacted. - String executedCompactAction = + if (CachedAppOptimizer.ENABLE_FILE_COMPACT) { + // We force a some compaction + mCachedAppOptimizerUnderTest.compactAppSome(processRecord, true); + waitForHandler(); + // then process is compacted. + String executedCompactAction = compactActionIntToString(processRecord.mOptRecord.getLastCompactAction()); - assertThat(executedCompactAction) + assertThat(executedCompactAction) .isEqualTo(mCachedAppOptimizerUnderTest.mCompactActionSome); + } } private void setFlag(String key, String value, boolean defaultValue) throws Exception {