diff options
author | Mustaavalkosta <niko.hyrynsalmi@gmail.com> | 2016-02-06 14:14:20 +0200 |
---|---|---|
committer | Michael Bestas <mikeioannina@cyanogenmod.org> | 2016-02-25 13:16:07 -0800 |
commit | 8c74aa8471af7f086b8d0e4c7798cdcc241dcab6 (patch) | |
tree | 8c1aeef1208107070529b8a138e28944b7eeddb3 | |
parent | 0299788b7a974841aa3324a573fbf04f49a4f23c (diff) | |
download | art-stable/cm-13.0-ZNH0E.tar.gz art-stable/cm-13.0-ZNH0E.tar.bz2 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.cc | 6 |
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_; |