Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Make caller wait for installd to become available." into tm-dev am: 9804076394

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18284939

Change-Id: Iaa5231b6dfd3f2bdde1b91481a3cdbfab48a7fb8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Alex Buynytskyy
2022-05-11 20:35:58 +00:00
committed by Automerger Merge Worker
parent 0f0d894760 9804076394
commit 08b9601342
1 changed files with 23 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
services/core/java/com/android/server/pm/Installer.java
View File

@@ -44,6 +44,9 @@ import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.concurrent.CompletableFuture;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;
public class Installer extends SystemService {
private static final String TAG = "Installer";
@@ -118,9 +121,13 @@ public class Installer extends SystemService {
public static final int FLAG_CLEAR_APP_DATA_KEEP_ART_PROFILES =
IInstalld.FLAG_CLEAR_APP_DATA_KEEP_ART_PROFILES;
private static final long CONNECT_RETRY_DELAY_MS = DateUtils.SECOND_IN_MILLIS;
private static final long CONNECT_WAIT_MS = 10 * DateUtils.SECOND_IN_MILLIS;
private final boolean mIsolated;
private volatile boolean mDeferSetFirstBoot;
private volatile IInstalld mInstalld;
private volatile IInstalld mInstalld = null;
private volatile CompletableFuture<IInstalld> mInstalldFuture = new CompletableFuture<>();
private volatile Object mWarnIfHeld;
public Installer(Context context) {
@@ -149,6 +156,7 @@ public class Installer extends SystemService {
public void onStart() {
if (mIsolated) {
mInstalld = null;
mInstalldFuture = null;
} else {
connect();
}
@@ -168,7 +176,9 @@ public class Installer extends SystemService {
}
if (binder != null) {
mInstalld = IInstalld.Stub.asInterface(binder);
IInstalld installd = IInstalld.Stub.asInterface(binder);
mInstalld = installd;
mInstalldFuture.complete(installd);
try {
invalidateMounts();
executeDeferredActions();
@@ -202,9 +212,18 @@ public class Installer extends SystemService {
if (mIsolated) {
Slog.i(TAG, "Ignoring request because this installer is isolated");
return false;
} else {
return true;
}
if (mInstalld == null && mInstalldFuture != null) {
try {
Slog.i(TAG, "installd not ready, waiting for: " + CONNECT_WAIT_MS + "ms");
mInstalld = mInstalldFuture.get(CONNECT_WAIT_MS, TimeUnit.MILLISECONDS);
} catch (InterruptedException | ExecutionException | TimeoutException e) {
Slog.e(TAG, "Ignoring request because this installer is not initialized", e);
}
}
return mInstalld != null;
}
// We explicitly do NOT set previousAppId because the default value should always be 0.