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

Commit 9abe3b4a authored by Alex Buynytskyy's avatar Alex Buynytskyy
Browse files

Defer the app data fixup until we are done with app data clearing above. 

It is already deferred and executed in another thread.
This avoids lock contention on clearAppData vs fixupAppData.

Bug: 216206114
Test: presubmit
Change-Id: Ib85db2789b300211ff780f0423cda07b961efe37
parent 7a53fdf6

services/core/java/com/android/server/pm/PackageManagerService.java

+3 −2
Original line number Diff line number Diff line
@@ -2055,8 +2055,6 @@ public class PackageManagerService extends IPackageManager.Stub 
                }

            }



            mPrepareAppDataFuture = mAppDataHelper.fixAppsDataOnBoot();



            // If this is first boot after an OTA, and a normal boot, then

            // we need to clear code cache directories.

            // Note that we do *not* clear the application profiles. These remain valid
@@ -2077,6 +2075,9 @@ public class PackageManagerService extends IPackageManager.Stub 
                ver.fingerprint = PackagePartitions.FINGERPRINT;

            }



            // Defer the app data fixup until we are done with app data clearing above.

            mPrepareAppDataFuture = mAppDataHelper.fixAppsDataOnBoot();



            // Legacy existing (installed before Q) non-system apps to hide

            // their icons in launcher.

            if (!mOnlyCore && mIsPreQUpgrade) {