Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 09379073 authored by Misha Wagner's avatar Misha Wagner Committed by Automerger Merge Worker
Browse files

Fix race condition when using mPreserveTopNApps. am: f81d50e2 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15684473

Change-Id: If6958378d4533f44597a562a01314b183baf1f47
parents 289a799c f81d50e2

services/core/java/com/android/server/am/CacheOomRanker.java

+7 −5
Original line number Diff line number Diff line
@@ -243,6 +243,7 @@ public class CacheOomRanker { 
        float lruWeight;

        float usesWeight;

        float rssWeight;

        int preserveTopNApps;

        int[] lruPositions;

        RankedProcessRecord[] scoredProcessRecords;
@@ -250,6 +251,7 @@ public class CacheOomRanker { 
            lruWeight = mLruWeight;

            usesWeight = mUsesWeight;

            rssWeight = mRssWeight;

            preserveTopNApps = mPreserveTopNApps;

            lruPositions = mLruPositions;

            scoredProcessRecords = mScoredProcessRecords;

        }
@@ -276,19 +278,19 @@ public class CacheOomRanker { 
            ++numProcessesEvaluated;

        }



        // Count how many apps we're not re-ranking (up to mPreserveTopNApps).

        // Count how many apps we're not re-ranking (up to preserveTopNApps).

        int numProcessesNotReRanked = 0;

        while (numProcessesEvaluated < lruProcessServiceStart

                && numProcessesNotReRanked < mPreserveTopNApps) {

                && numProcessesNotReRanked < preserveTopNApps) {

            ProcessRecord process = lruList.get(numProcessesEvaluated);

            if (appCanBeReRanked(process)) {

                numProcessesNotReRanked++;

            }

            numProcessesEvaluated++;

        }

        // Exclude the top `mPreserveTopNApps` apps from re-ranking.

        if (numProcessesNotReRanked < mPreserveTopNApps) {

            numProcessesReRanked -= mPreserveTopNApps - numProcessesNotReRanked;

        // Exclude the top `preserveTopNApps` apps from re-ranking.

        if (numProcessesNotReRanked < preserveTopNApps) {

            numProcessesReRanked -= preserveTopNApps - numProcessesNotReRanked;

            if (numProcessesReRanked < 0) {

                numProcessesReRanked = 0;

            }