From 920de9aa8696f41c0088f941f01d51467d5f7702 Mon Sep 17 00:00:00 2001 From: Alex Buynytskyy Date: Wed, 2 Mar 2022 13:30:00 -0800 Subject: [PATCH] Remove blocking GC on package installation. It was introduced a long time ago and most probably not needed now. Bug: 220083428 Test: presubmit Change-Id: I4f19f06ec26b995a96ab95e2a37a7ddb2f4b740f --- .../java/com/android/server/pm/DeletePackageHelper.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/pm/DeletePackageHelper.java b/services/core/java/com/android/server/pm/DeletePackageHelper.java index 5b7cf2dc85592..87a83effee1ae 100644 --- a/services/core/java/com/android/server/pm/DeletePackageHelper.java +++ b/services/core/java/com/android/server/pm/DeletePackageHelper.java @@ -66,6 +66,8 @@ import com.android.server.pm.pkg.PackageStateInternal; import com.android.server.pm.pkg.PackageUserState; import com.android.server.wm.ActivityTaskManagerInternal; +import dalvik.system.VMRuntime; + import java.util.Collections; import java.util.List; @@ -245,8 +247,11 @@ final class DeletePackageHelper { info.sendSystemPackageUpdatedBroadcasts(disabledSystemPs.getAppId()); } } - // Force a gc here. - Runtime.getRuntime().gc(); + + // Force a gc to clear up things. + // Ask for a background one, it's fine to go on and not block here. + VMRuntime.getRuntime().requestConcurrentGC(); + // Delete the resources here after sending the broadcast to let // other processes clean up before deleting resources. synchronized (mPm.mInstallLock) {