82402753815ff4633cc572713ae490a17d9129e5 - platform/frameworks/base

commit	82402753815ff4633cc572713ae490a17d9129e5	[log] [tgz]
author	Fyodor Kupolov <fkupolov@google.com>	Wed Oct 28 14:54:51 2015 -0700
committer	Fyodor Kupolov <fkupolov@google.com>	Tue Nov 03 11:56:33 2015 -0800
tree	d3e711b013910f4b899df61b84f8bae51f518322
parent	4f16073556f7978708fb71c87628cfe1692412d5 [diff]

Introduced short-term lock for UMS internal state

Added mUsersLock - short-term lock for internal state, when interaction and
synchronization with PM is not required. Modifications to mUsers and
mRemovingUserIds must be guarded by 3 locks: mInstallLock, mPackagesLock and
mUsersLock. While reads can use mUsersLock.

Testing revealed that the following methods in UMS often cause contention:
- exists
- getUserInfo
- getProfileParent

They all now use a short-term lock mUsersLock for reads.

Bug: 24979571
Change-Id: Ie3a22ea7cbb450c7969800fe2a4a2b2516165e5b

core/java/android/os/UserManagerInternal.java[diff]
services/core/java/com/android/server/pm/PackageManagerService.java[diff]
services/core/java/com/android/server/pm/UserManagerService.java[diff]
services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java[diff]
services/tests/servicestests/src/com/android/server/pm/UserManagerServiceTest.java[diff]

5 files changed

tree: d3e711b013910f4b899df61b84f8bae51f518322