75a4e3f38c52366d59447cc865b65e4d51622697 - platform_frameworks_base

commit	75a4e3f38c52366d59447cc865b65e4d51622697	[log] [tgz]
author	Jing Ji <jji@google.com>	Fri Mar 19 22:20:29 2021 -0700
committer	Jing Ji <jji@google.com>	Fri Mar 19 22:32:10 2021 -0700
tree	96f877519d8626875ce675f9374a86b6688e5b3d
parent	1c9ac73a7828f49e44718dca0da900d520c5426a [diff]

DO NOT MERGE: Don't take the AM lock in cached app compaction handler

A single oomAdjUpdate pass could trigger quite a few app compactions,
each of them would need to acquire the AM lock during the handling,
results in lock contentions. Now use its own lock. Also skip the
scheduling if the given app is pending with compaction already.

Bug: 163297662
Test: atest CachedAppOptimizerTest
Test: Manual - Boot & check the logs
Change-Id: I4618a3ae78838cd9783a431e7364130270ebb4d6

services/core/java/com/android/server/am/ActivityManagerService.java[diff]
services/core/java/com/android/server/am/CachedAppOptimizer.java[diff]
services/core/java/com/android/server/am/OomAdjuster.java[diff]
services/core/java/com/android/server/am/ProcessRecord.java[diff]
services/tests/mockingservicestests/src/com/android/server/am/CachedAppOptimizerTest.java[diff]

5 files changed