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

Merge changes I87740397,I0cc4538b,Iaceb2b7c am: 56ba9447cf

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

Change-Id: I4bc1321f1e1c687ce6d29376d73531919da6b545
This commit is contained in:
Treehugger Robot
2021-05-11 16:38:32 +00:00
committed by Automerger Merge Worker
parent 8cdf84ddf6 56ba9447cf
commit f98d2cc9fc
1 changed files with 50 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

88
services/core/java/com/android/server/pm/BackgroundDexOptService.java
View File

@@ -16,7 +16,6 @@
package com.android.server.pm; package com.android.server.pm;
import static com.android.server.pm.PackageManagerService.DEBUG_DEXOPT;
import static com.android.server.pm.PackageManagerService.PLATFORM_PACKAGE_NAME; import static com.android.server.pm.PackageManagerService.PLATFORM_PACKAGE_NAME;
import android.annotation.Nullable; import android.annotation.Nullable;
@@ -24,8 +23,11 @@ import android.app.job.JobInfo;
import android.app.job.JobParameters; import android.app.job.JobParameters;
import android.app.job.JobScheduler; import android.app.job.JobScheduler;
import android.app.job.JobService; import android.app.job.JobService;
import android.content.BroadcastReceiver;
import android.content.ComponentName; import android.content.ComponentName;
import android.content.Context; import android.content.Context;
import android.content.Intent;
import android.content.IntentFilter;
import android.content.pm.PackageInfo; import android.content.pm.PackageInfo;
import android.os.BatteryManagerInternal; import android.os.BatteryManagerInternal;
import android.os.Environment; import android.os.Environment;
@@ -35,6 +37,7 @@ import android.os.UserHandle;
import android.os.storage.StorageManager; import android.os.storage.StorageManager;
import android.util.ArraySet; import android.util.ArraySet;
import android.util.Log; import android.util.Log;
import android.util.Slog;
import com.android.internal.util.ArrayUtils; import com.android.internal.util.ArrayUtils;
import com.android.internal.util.FrameworkStatsLog; import com.android.internal.util.FrameworkStatsLog;
@@ -63,9 +66,7 @@ public class BackgroundDexOptService extends JobService {
private static final int JOB_IDLE_OPTIMIZE = 800; private static final int JOB_IDLE_OPTIMIZE = 800;
private static final int JOB_POST_BOOT_UPDATE = 801; private static final int JOB_POST_BOOT_UPDATE = 801;
private static final long IDLE_OPTIMIZATION_PERIOD = DEBUG private static final long IDLE_OPTIMIZATION_PERIOD = TimeUnit.DAYS.toMillis(1);
? TimeUnit.MINUTES.toMillis(1)
: TimeUnit.DAYS.toMillis(1);
private static ComponentName sDexoptServiceName = new ComponentName( private static ComponentName sDexoptServiceName = new ComponentName(
"android", "android",
@@ -113,14 +114,24 @@ public class BackgroundDexOptService extends JobService {
return; return;
} }
JobScheduler js = (JobScheduler) context.getSystemService(Context.JOB_SCHEDULER_SERVICE); final JobScheduler js = context.getSystemService(JobScheduler.class);
// Schedule a one-off job which scans installed packages and updates // Schedule a one-off job which scans installed packages and updates
// out-of-date oat files. // out-of-date oat files. Schedule it 10 minutes after the boot complete event,
js.schedule(new JobInfo.Builder(JOB_POST_BOOT_UPDATE, sDexoptServiceName) // so that we don't overload the boot with additional dex2oat compilations.
.setMinimumLatency(TimeUnit.MINUTES.toMillis(10)) context.registerReceiver(new BroadcastReceiver() {
.setOverrideDeadline(TimeUnit.MINUTES.toMillis(60)) @Override
.build()); public void onReceive(Context context, Intent intent) {
js.schedule(new JobInfo.Builder(JOB_POST_BOOT_UPDATE, sDexoptServiceName)
.setMinimumLatency(TimeUnit.MINUTES.toMillis(10))
.setOverrideDeadline(TimeUnit.MINUTES.toMillis(60))
.build());
context.unregisterReceiver(this);
if (DEBUG) {
Slog.i(TAG, "BootBgDexopt scheduled");
}
}
}, new IntentFilter(Intent.ACTION_BOOT_COMPLETED));
// Schedule a daily job which scans installed packages and compiles // Schedule a daily job which scans installed packages and compiles
// those with fresh profiling data. // those with fresh profiling data.
@@ -130,8 +141,8 @@ public class BackgroundDexOptService extends JobService {
.setPeriodic(IDLE_OPTIMIZATION_PERIOD) .setPeriodic(IDLE_OPTIMIZATION_PERIOD)
.build()); .build());
if (DEBUG_DEXOPT) { if (DEBUG) {
Log.i(TAG, "Jobs scheduled"); Slog.d(TAG, "BgDexopt scheduled");
} }
} }
@@ -151,7 +162,7 @@ public class BackgroundDexOptService extends JobService {
@SuppressWarnings("deprecation") @SuppressWarnings("deprecation")
final long lowThreshold = StorageManager.from(context).getStorageLowBytes(mDataDir); final long lowThreshold = StorageManager.from(context).getStorageLowBytes(mDataDir);
if (lowThreshold == 0) { if (lowThreshold == 0) {
Log.e(TAG, "Invalid low storage threshold"); Slog.e(TAG, "Invalid low storage threshold");
} }
return lowThreshold; return lowThreshold;
@@ -198,13 +209,12 @@ public class BackgroundDexOptService extends JobService {
long usableSpace = mDataDir.getUsableSpace(); long usableSpace = mDataDir.getUsableSpace();
if (usableSpace < lowThreshold) { if (usableSpace < lowThreshold) {
// Rather bail than completely fill up the disk. // Rather bail than completely fill up the disk.
Log.w(TAG, "Aborting background dex opt job due to low storage: " + Slog.w(TAG, "Aborting background dex opt job due to low storage: " +
usableSpace); usableSpace);
break; break;
} }
if (DEBUG) {
if (DEBUG_DEXOPT) { Slog.i(TAG, "Updating package " + pkg);
Log.i(TAG, "Updating package " + pkg);
} }
// Update package if needed. Note that there can be no race between concurrent // Update package if needed. Note that there can be no race between concurrent
@@ -236,13 +246,13 @@ public class BackgroundDexOptService extends JobService {
public void run() { public void run() {
int result = idleOptimization(pm, pkgs, BackgroundDexOptService.this); int result = idleOptimization(pm, pkgs, BackgroundDexOptService.this);
if (result == OPTIMIZE_PROCESSED) { if (result == OPTIMIZE_PROCESSED) {
Log.i(TAG, "Idle optimizations completed."); Slog.i(TAG, "Idle optimizations completed.");
} else if (result == OPTIMIZE_ABORT_NO_SPACE_LEFT) { } else if (result == OPTIMIZE_ABORT_NO_SPACE_LEFT) {
Log.w(TAG, "Idle optimizations aborted because of space constraints."); Slog.w(TAG, "Idle optimizations aborted because of space constraints.");
} else if (result == OPTIMIZE_ABORT_BY_JOB_SCHEDULER) { } else if (result == OPTIMIZE_ABORT_BY_JOB_SCHEDULER) {
Log.w(TAG, "Idle optimizations aborted by job scheduler."); Slog.w(TAG, "Idle optimizations aborted by job scheduler.");
} else { } else {
Log.w(TAG, "Idle optimizations ended with unexpected code: " + result); Slog.w(TAG, "Idle optimizations ended with unexpected code: " + result);
} }
if (result != OPTIMIZE_ABORT_BY_JOB_SCHEDULER) { if (result != OPTIMIZE_ABORT_BY_JOB_SCHEDULER) {
// Abandon our timeslice and do not reschedule. // Abandon our timeslice and do not reschedule.
@@ -256,7 +266,7 @@ public class BackgroundDexOptService extends JobService {
// Optimize the given packages and return the optimization result (one of the OPTIMIZE_* codes). // Optimize the given packages and return the optimization result (one of the OPTIMIZE_* codes).
private int idleOptimization(PackageManagerService pm, ArraySet<String> pkgs, private int idleOptimization(PackageManagerService pm, ArraySet<String> pkgs,
Context context) { Context context) {
Log.i(TAG, "Performing idle optimizations"); Slog.i(TAG, "Performing idle optimizations");
// If post-boot update is still running, request that it exits early. // If post-boot update is still running, request that it exits early.
mExitPostBootUpdate.set(true); mExitPostBootUpdate.set(true);
mAbortIdleOptimization.set(false); mAbortIdleOptimization.set(false);
@@ -331,11 +341,15 @@ public class BackgroundDexOptService extends JobService {
final long lowStorageThresholdForDowngrade = LOW_THRESHOLD_MULTIPLIER_FOR_DOWNGRADE final long lowStorageThresholdForDowngrade = LOW_THRESHOLD_MULTIPLIER_FOR_DOWNGRADE
* lowStorageThreshold; * lowStorageThreshold;
boolean shouldDowngrade = shouldDowngrade(lowStorageThresholdForDowngrade); boolean shouldDowngrade = shouldDowngrade(lowStorageThresholdForDowngrade);
Log.d(TAG, "Should Downgrade " + shouldDowngrade); if (DEBUG) {
Slog.d(TAG, "Should Downgrade " + shouldDowngrade);
}
if (shouldDowngrade) { if (shouldDowngrade) {
Set<String> unusedPackages = Set<String> unusedPackages =
pm.getUnusedPackages(mDowngradeUnusedAppsThresholdInMillis); pm.getUnusedPackages(mDowngradeUnusedAppsThresholdInMillis);
Log.d(TAG, "Unsused Packages " + String.join(",", unusedPackages)); if (DEBUG) {
Slog.d(TAG, "Unsused Packages " + String.join(",", unusedPackages));
}
if (!unusedPackages.isEmpty()) { if (!unusedPackages.isEmpty()) {
for (String pkg : unusedPackages) { for (String pkg : unusedPackages) {
@@ -407,7 +421,9 @@ public class BackgroundDexOptService extends JobService {
*/ */
private boolean downgradePackage(PackageManagerService pm, String pkg, private boolean downgradePackage(PackageManagerService pm, String pkg,
boolean isForPrimaryDex) { boolean isForPrimaryDex) {
Log.d(TAG, "Downgrading " + pkg); if (DEBUG) {
Slog.d(TAG, "Downgrading " + pkg);
}
boolean dex_opt_performed = false; boolean dex_opt_performed = false;
int reason = PackageManagerService.REASON_INACTIVE_PACKAGE_DOWNGRADE; int reason = PackageManagerService.REASON_INACTIVE_PACKAGE_DOWNGRADE;
int dexoptFlags = DexoptOptions.DEXOPT_BOOT_COMPLETE int dexoptFlags = DexoptOptions.DEXOPT_BOOT_COMPLETE
@@ -529,7 +545,7 @@ public class BackgroundDexOptService extends JobService {
long usableSpace = mDataDir.getUsableSpace(); long usableSpace = mDataDir.getUsableSpace();
if (usableSpace < lowStorageThreshold) { if (usableSpace < lowStorageThreshold) {
// Rather bail than completely fill up the disk. // Rather bail than completely fill up the disk.
Log.w(TAG, "Aborting background dex opt job due to low storage: " + usableSpace); Slog.w(TAG, "Aborting background dex opt job due to low storage: " + usableSpace);
return OPTIMIZE_ABORT_NO_SPACE_LEFT; return OPTIMIZE_ABORT_NO_SPACE_LEFT;
} }
@@ -568,8 +584,8 @@ public class BackgroundDexOptService extends JobService {
@Override @Override
public boolean onStartJob(JobParameters params) { public boolean onStartJob(JobParameters params) {
if (DEBUG_DEXOPT) { if (DEBUG) {
Log.i(TAG, "onStartJob"); Slog.i(TAG, "onStartJob");
} }
// NOTE: PackageManagerService.isStorageLow uses a different set of criteria from // NOTE: PackageManagerService.isStorageLow uses a different set of criteria from
@@ -577,17 +593,13 @@ public class BackgroundDexOptService extends JobService {
// restart with a period of ~1 minute. // restart with a period of ~1 minute.
PackageManagerService pm = (PackageManagerService)ServiceManager.getService("package"); PackageManagerService pm = (PackageManagerService)ServiceManager.getService("package");
if (pm.isStorageLow()) { if (pm.isStorageLow()) {
if (DEBUG_DEXOPT) { Slog.i(TAG, "Low storage, skipping this run");
Log.i(TAG, "Low storage, skipping this run");
}
return false; return false;
} }
final ArraySet<String> pkgs = pm.getOptimizablePackages(); final ArraySet<String> pkgs = pm.getOptimizablePackages();
if (pkgs.isEmpty()) { if (pkgs.isEmpty()) {
if (DEBUG_DEXOPT) { Slog.i(TAG, "No packages to optimize");
Log.i(TAG, "No packages to optimize");
}
return false; return false;
} }
@@ -603,8 +615,8 @@ public class BackgroundDexOptService extends JobService {
@Override @Override
public boolean onStopJob(JobParameters params) { public boolean onStopJob(JobParameters params) {
if (DEBUG_DEXOPT) { if (DEBUG) {
Log.i(TAG, "onStopJob"); Slog.d(TAG, "onStopJob");
} }
if (params.getJobId() == JOB_POST_BOOT_UPDATE) { if (params.getJobId() == JOB_POST_BOOT_UPDATE) {
@@ -625,7 +637,7 @@ public class BackgroundDexOptService extends JobService {
private void notifyPinService(ArraySet<String> updatedPackages) { private void notifyPinService(ArraySet<String> updatedPackages) {
PinnerService pinnerService = LocalServices.getService(PinnerService.class); PinnerService pinnerService = LocalServices.getService(PinnerService.class);
if (pinnerService != null) { if (pinnerService != null) {
Log.i(TAG, "Pinning optimized code " + updatedPackages); Slog.i(TAG, "Pinning optimized code " + updatedPackages);
pinnerService.update(updatedPackages, false /* force */); pinnerService.update(updatedPackages, false /* force */);
} }
} }
@@ -660,7 +672,7 @@ public class BackgroundDexOptService extends JobService {
final String sysPropKey = "pm.dexopt.downgrade_after_inactive_days"; final String sysPropKey = "pm.dexopt.downgrade_after_inactive_days";
String sysPropValue = SystemProperties.get(sysPropKey); String sysPropValue = SystemProperties.get(sysPropKey);
if (sysPropValue == null || sysPropValue.isEmpty()) { if (sysPropValue == null || sysPropValue.isEmpty()) {
Log.w(TAG, "SysProp " + sysPropKey + " not set"); Slog.w(TAG, "SysProp " + sysPropKey + " not set");
return Long.MAX_VALUE; return Long.MAX_VALUE;
} }
return TimeUnit.DAYS.toMillis(Long.parseLong(sysPropValue)); return TimeUnit.DAYS.toMillis(Long.parseLong(sysPropValue));