diff options
| author | 2019-04-24 11:46:43 -0700 | |
|---|---|---|
| committer | 2019-05-10 00:16:39 +0000 | |
| commit | 4b76bc1ce1a730b1a65f4a64b33ed198b2c7e46e (patch) | |
| tree | 6d4da3c361db6a3b99aa28672b7b8582827d0bf6 | |
| parent | 332d34e30dbd7b7088e180f3f9b0b016fe455808 (diff) | |
Process start optimization
Remove the check for valid process start, as it's redundant.
(Contributed by Huawei)
Bug: 111815164
Test: am perf tests
Change-Id: I53f60b854ccc45f5f00bc3964fa4f46209f92e72
| -rw-r--r-- | services/core/java/com/android/server/am/ProcessList.java | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/services/core/java/com/android/server/am/ProcessList.java b/services/core/java/com/android/server/am/ProcessList.java index 696697eddf25..b394eea95a88 100644 --- a/services/core/java/com/android/server/am/ProcessList.java +++ b/services/core/java/com/android/server/am/ProcessList.java @@ -1658,23 +1658,15 @@ public final class ProcessList { app.killed = false; final long startSeq = app.startSeq = ++mProcStartSeqCounter; app.setStartParams(uid, hostingRecord, seInfo, startTime); + app.setUsingWrapper(invokeWith != null + || SystemProperties.get("wrap." + app.processName) != null); + mPendingStarts.put(startSeq, app); + if (mService.mConstants.FLAG_PROCESS_START_ASYNC) { if (DEBUG_PROCESSES) Slog.i(TAG_PROCESSES, "Posting procStart msg for " + app.toShortString()); mService.mProcStartHandler.post(() -> { try { - synchronized (mService) { - final String reason = isProcStartValidLocked(app, startSeq); - if (reason != null) { - Slog.w(TAG_PROCESSES, app + " not valid anymore," - + " don't start process, " + reason); - app.pendingStart = false; - return; - } - app.setUsingWrapper(invokeWith != null - || SystemProperties.get("wrap." + app.processName) != null); - mPendingStarts.put(startSeq, app); - } final Process.ProcessStartResult startResult = startProcess(app.hostingRecord, entryPoint, app, app.startUid, gids, runtimeFlags, mountExternal, app.seInfo, requiredAbi, instructionSet, invokeWith, app.startTime); |