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

Commit 7c8169b0 authored by jinchul.kim's avatar jinchul.kim Committed by Steve Kondik
Browse files

Kill process asynchronously on updateOomAdjLocked. 

Kill process takes time, especially on killProcessGroup. If it is called
by updateOomAdjLocked, it keeps lock of mServices for moments and other
requests would be delayed, like attachApplication.

If is called frequently, it may occur latency problem. Killing job is
needed to be done asynchronously in updateOomAdjLocked at least.

Change-Id: I745851943e8dbee7f823a8b4aa17c521e43e2e06
parent 383ec100

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

+13 −4
Original line number Diff line number Diff line
@@ -18606,19 +18606,19 @@ public final class ActivityManagerService extends ActivityManagerNative 
                        mNumCachedHiddenProcs++;

                        numCached++;

                        if (numCached > cachedProcessLimit) {

                            app.kill("cached #" + numCached, true);

                            postKillProc(app, "cached #" + numCached, true);

                        }

                        break;

                    case ActivityManager.PROCESS_STATE_CACHED_EMPTY:

                        if (numEmpty > ProcessList.TRIM_EMPTY_APPS

                                && app.lastActivityTime < oldTime) {

                            app.kill("empty for "

                            postKillProc(app, "empty for "

                                    + ((oldTime + ProcessList.MAX_EMPTY_TIME - app.lastActivityTime)

                                    / 1000) + "s", true);

                        } else {

                            numEmpty++;

                            if (numEmpty > emptyProcessLimit) {

                                app.kill("empty #" + numEmpty, true);

                                postKillProc(app, "empty #" + numEmpty, true);

                            }

                        }

                        break;
@@ -18634,7 +18634,7 @@ public final class ActivityManagerService extends ActivityManagerNative 
                    // definition not re-use the same process again, and it is

                    // good to avoid having whatever code was running in them

                    // left sitting around after no longer needed.

                    app.kill("isolated not needed", true);

                    postKillProc(app, "isolated not needed", true);

                }















                if (app.curProcState >= ActivityManager.PROCESS_STATE_HOME










@@ -18859,6 +18859,15 @@ public final class ActivityManagerService extends ActivityManagerNative













        }















    }



























    private postKillProc(final ProcessRecord app, final String reason, final boolean noisy) {













        mHandler.post(new Runnable() {













            @Override













            public void run() {













                app.kill(reason, noisy);













            }













        });













    }



























    final void trimApplications() {















        synchronized (this) {















            int i;