Fixed a property naming mismatch.

Device configuration properties in ZygoteConfig.java used old names for
certain values.  These name strings have been updated to the correct
values and additional logging has been added to help catch future
issues.

Test: m
Test: boot and check log for USAP messages
Change-Id: I2fb92afe7edd9b1254dd3be659ec5063aa0b8e16
This commit is contained in:
Chris Wailes
2019-03-22 17:45:28 -07:00
parent c90040e0b3
commit da5b64e7f2
3 changed files with 10 additions and 6 deletions

View File

@@ -671,11 +671,13 @@ public class ZygoteProcess {
Boolean.parseBoolean(USAP_POOL_ENABLED_DEFAULT));
}
if (origVal != mUsapPoolEnabled) {
boolean valueChanged = origVal != mUsapPoolEnabled;
if (valueChanged) {
Log.i(LOG_TAG, "usapPoolEnabled = " + mUsapPoolEnabled);
}
return origVal != mUsapPoolEnabled;
return valueChanged;
}
private boolean mIsFirstPropCheck = true;

View File

@@ -24,14 +24,14 @@ package com.android.internal.os;
public class ZygoteConfig {
/** If {@code true}, enables the unspecialized app process (USAP) pool feature */
public static final String USAP_POOL_ENABLED = "blastula_pool_enabled";
public static final String USAP_POOL_ENABLED = "usap_pool_enabled";
/** The threshold used to determine if the pool should be refilled */
public static final String USAP_POOL_REFILL_THRESHOLD = "blastula_refill_threshold";
public static final String USAP_POOL_REFILL_THRESHOLD = "usap_refill_threshold";
/** The maximum number of processes to keep in the USAP pool */
public static final String USAP_POOL_SIZE_MAX = "blastula_pool_size_max";
public static final String USAP_POOL_SIZE_MAX = "usap_pool_size_max";
/** The minimum number of processes to keep in the USAP pool */
public static final String USAP_POOL_SIZE_MIN = "blastula_pool_size_min";
public static final String USAP_POOL_SIZE_MIN = "usap_pool_size_min";
}

View File

@@ -353,6 +353,8 @@ class ZygoteServer {
return null;
}
Log.i(TAG, "USAP Pool status change: " + (newStatus ? "ENABLED" : "DISABLED"));
mUsapPoolEnabled = newStatus;
if (newStatus) {