4ad88791222c4c8df72629faa84042a6602209de - platform_frameworks_base

commit	4ad88791222c4c8df72629faa84042a6602209de	[log] [tgz]
author	Jing Ji <jji@google.com>	Tue Nov 09 13:50:44 2021 -0800
committer	Jing Ji <jji@google.com>	Wed Nov 17 14:18:43 2021 -0800
tree	5e0513565284eba0505e623aba6a0ff0d3d7fb5f
parent	eb7f4c076f47df186cf21f8de0e4150e34e431df [diff]

Acquire the current timestamp within the procstats lock.

This CL fixes the race condition, where timestamps could be acquired
before going to the procstats-locked code section, while the ordering
of entering the code section isn't guaranteed anyhwere, thus a call
into procstats with an earlier timestamp could happen after a call
with a later timestamp, thus confuses the bookkeeping of procstats.

Bug: 194991626
Test: CtsAppTestCases
Change-Id: I8863c3738a97a4fcce881cb21f1db4df559a7b6b

services/core/java/com/android/server/am/ActiveServices.java[diff]
services/core/java/com/android/server/am/ActivityManagerService.java[diff]
services/core/java/com/android/server/am/AppProfiler.java[diff]
services/core/java/com/android/server/am/ConnectionRecord.java[diff]
services/core/java/com/android/server/am/ContentProviderConnection.java[diff]
services/core/java/com/android/server/am/OomAdjuster.java[diff]
services/core/java/com/android/server/am/ProcessProfileRecord.java[diff]
services/tests/mockingservicestests/src/com/android/server/am/MockingOomAdjusterTests.java[diff]

8 files changed