summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMustaavalkosta <niko.hyrynsalmi@gmail.com>2016-02-06 14:14:20 +0200
committerMichael Bestas <mikeioannina@cyanogenmod.org>2016-02-25 13:16:07 -0800
commit8c74aa8471af7f086b8d0e4c7798cdcc241dcab6 (patch)
tree8c1aeef1208107070529b8a138e28944b7eeddb3
parent0299788b7a974841aa3324a573fbf04f49a4f23c (diff)
downloadandroid_art-stable/cm-13.0-ZNH0E.tar.gz
android_art-stable/cm-13.0-ZNH0E.tar.bz2
android_art-stable/cm-13.0-ZNH0E.zip
art: fix dex2oat watchdog timeoutstable/cm-13.0-ZNH0E
Merging of Android 6.0.1 release 3 (Change-Id: I23fd56f2c1a3e8e8b993a151a794e18f3569912e) reverted the watchdog timeout back to old short one. PackageManagerService in frameworks/base/ is still using 60 minutes as watchdog timeout (Change-Id: Ia8f089ea340f21023abb8a58ddb3aede8b2b728c) and I think this should match that or revert PackageManagerService side back to also using 10-minute timeout as documented here on dex2oat. I modified the old long timeout to match the change from 6.0.1_r3 merge (Change-Id: I425b19ab305cfaa43f6bddc3a892be892acaf513) which introduced slightly shorter (30 seconds) timeout here to make sure that "dex2oat kills itself before the system server watchdog kills the system because of the long installation". RM-208 Change-Id: I79e4bed34df7d964b9c4cac0d1c344cf773961b7 (cherry picked from commit fdbc58301fef858cb7f8db682729f1b8d8d5f97f)
-rw-r--r--dex2oat/dex2oat.cc6
1 files changed, 3 insertions, 3 deletions
diff --git a/dex2oat/dex2oat.cc b/dex2oat/dex2oat.cc
index 03679fa628..aaf31da51d 100644
--- a/dex2oat/dex2oat.cc
+++ b/dex2oat/dex2oat.cc
@@ -436,10 +436,10 @@ class WatchDog {
// Debug builds are slower so they have larger timeouts.
static constexpr int64_t kSlowdownFactor = kIsDebugBuild ? 5U : 1U;
- // 9.5 minutes scaled by kSlowdownFactor. This is slightly smaller than the Package Manager
- // watchdog (PackageManagerService.WATCHDOG_TIMEOUT, 10 minutes), so that dex2oat will abort
+ // 59.5 minutes scaled by kSlowdownFactor. This is slightly smaller than the Package Manager
+ // watchdog (PackageManagerService.WATCHDOG_TIMEOUT, 60 minutes), so that dex2oat will abort
// itself before that watchdog would take down the system server.
- static constexpr int64_t kWatchDogTimeoutSeconds = kSlowdownFactor * (9 * 60 + 30);
+ static constexpr int64_t kWatchDogTimeoutSeconds = kSlowdownFactor * (59 * 60 + 30);
bool is_watch_dog_enabled_;
bool shutting_down_;