diff options
| author | Andreas Gampe <agampe@google.com> | 2015-09-14 15:34:38 -0700 |
|---|---|---|
| committer | Andreas Gampe <agampe@google.com> | 2015-09-30 18:38:19 -0700 |
| commit | 9d96bd65577bb6b7499627ecba8ea027f3299091 (patch) | |
| tree | 2ed82a5a2607ece196aa00f94e0d46a336af7093 | |
| parent | 39becb96f0c2ba1823ca0cf046e9f12344f9f3bd (diff) | |
| download | art-9d96bd65577bb6b7499627ecba8ea027f3299091.tar.gz art-9d96bd65577bb6b7499627ecba8ea027f3299091.tar.bz2 art-9d96bd65577bb6b7499627ecba8ea027f3299091.zip | |
ART: Decrease dex2oat watchdog timeout
Keep the dex2oat watchdog timeout lower than the package manager
timeout, so that dex2oat kills itself before the system server
watchdog kills the system because of the long installation.
Bug: 23629410
(cherry picked from commit 540138ae55ac1909606a436d7f52e20146c56657)
Change-Id: I5faa23a1715736a1c2ba3f302a6ee41130d3fbae
| -rw-r--r-- | dex2oat/dex2oat.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/dex2oat/dex2oat.cc b/dex2oat/dex2oat.cc index e913c20407..c51c31769e 100644 --- a/dex2oat/dex2oat.cc +++ b/dex2oat/dex2oat.cc @@ -433,8 +433,10 @@ class WatchDog { // Debug builds are slower so they have larger timeouts. static constexpr int64_t kSlowdownFactor = kIsDebugBuild ? 5U : 1U; - // 10 minutes scaled by kSlowdownFactor. - static constexpr int64_t kWatchDogTimeoutSeconds = kSlowdownFactor * 10 * 60; + // 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 + // itself before that watchdog would take down the system server. + static constexpr int64_t kWatchDogTimeoutSeconds = kSlowdownFactor * (9 * 60 + 30); bool is_watch_dog_enabled_; bool shutting_down_; |