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

Merge "Add an explicit reason for the compilation of shared apks" into oc-mr1-dev

Browse Source
This commit is contained in:
Calin Juravle
2017-09-19 21:56:41 +00:00
committed by Android (Google) Code Review
parent 7b911fbdaf f53201f8c7
commit d78d269650
3 changed files with 18 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
services/core/java/com/android/server/pm/PackageDexOptimizer.java
View File

@@ -474,8 +474,9 @@ public class PackageDexOptimizer {
}
if (isProfileGuidedCompilerFilter(targetCompilerFilter) && isUsedByOtherApps) {
// If the dex files is used by other apps, we cannot use profile-guided compilation.
return getNonProfileGuidedCompilerFilter(targetCompilerFilter);
// If the dex files is used by other apps, apply the shared filter.
return PackageManagerServiceCompilerMapping.getCompilerFilterForReason(
PackageManagerService.REASON_SHARED);
}
return targetCompilerFilter;

3
services/core/java/com/android/server/pm/PackageManagerService.java
View File

@@ -577,8 +577,9 @@ public class PackageManagerService extends IPackageManager.Stub
public static final int REASON_BACKGROUND_DEXOPT = 3;
public static final int REASON_AB_OTA = 4;
public static final int REASON_INACTIVE_PACKAGE_DOWNGRADE = 5;
public static final int REASON_SHARED = 6;
public static final int REASON_LAST = REASON_INACTIVE_PACKAGE_DOWNGRADE;
public static final int REASON_LAST = REASON_SHARED;
/** All dangerous permission names in the same order as the events in MetricsEvent */
private static final List<String> ALL_DANGEROUS_PERMISSIONS = Arrays.asList(

14
services/core/java/com/android/server/pm/PackageManagerServiceCompilerMapping.java
View File

@@ -26,14 +26,19 @@ import dalvik.system.DexFile;
public class PackageManagerServiceCompilerMapping {
// Names for compilation reasons.
static final String REASON_STRINGS[] = {
"first-boot", "boot", "install", "bg-dexopt", "ab-ota", "inactive"
"first-boot", "boot", "install", "bg-dexopt", "ab-ota", "inactive", "shared"
};
static final int REASON_SHARED_INDEX = 6;
// Static block to ensure the strings array is of the right length.
static {
if (PackageManagerService.REASON_LAST + 1 != REASON_STRINGS.length) {
throw new IllegalStateException("REASON_STRINGS not correct");
}
if (!"shared".equals(REASON_STRINGS[REASON_SHARED_INDEX])) {
throw new IllegalStateException("REASON_STRINGS not correct because of shared index");
}
}
private static String getSystemPropertyName(int reason) {
@@ -52,11 +57,18 @@ public class PackageManagerServiceCompilerMapping {
!DexFile.isValidCompilerFilter(sysPropValue)) {
throw new IllegalStateException("Value \"" + sysPropValue +"\" not valid "
+ "(reason " + REASON_STRINGS[reason] + ")");
} else if (!isFilterAllowedForReason(reason, sysPropValue)) {
throw new IllegalStateException("Value \"" + sysPropValue +"\" not allowed "
+ "(reason " + REASON_STRINGS[reason] + ")");
}
return sysPropValue;
}
private static boolean isFilterAllowedForReason(int reason, String filter) {
return reason != REASON_SHARED_INDEX || !DexFile.isProfileGuidedCompilerFilter(filter);
}
// Check that the properties are set and valid.
// Note: this is done in a separate method so this class can be statically initialized.
static void checkProperties() {