diff options
author | Treehugger Robot <treehugger-gerrit@google.com> | 2020-05-19 14:47:19 +0200 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2020-05-19 14:47:19 +0200 |
commit | 082bea6325125376a5f2d5a750657ca9791a901d (patch) | |
tree | 4e7099064d574237230ed47055be46ce585f3ff6 | |
parent | Merge "Add EthernetDevice to manage ethernet connection." (diff) | |
parent | recovery: fastbootd: retry opening graphics (diff) | |
download | android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar.gz android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar.bz2 android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar.lz android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar.xz android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.tar.zst android_bootable_recovery-082bea6325125376a5f2d5a750657ca9791a901d.zip |
-rw-r--r-- | recovery_ui/screen_ui.cpp | 21 |
1 files changed, 20 insertions, 1 deletions
diff --git a/recovery_ui/screen_ui.cpp b/recovery_ui/screen_ui.cpp index 6dcb161fa..b2c828f34 100644 --- a/recovery_ui/screen_ui.cpp +++ b/recovery_ui/screen_ui.cpp @@ -37,6 +37,7 @@ #include <unordered_map> #include <vector> +#include <android-base/chrono_utils.h> #include <android-base/logging.h> #include <android-base/properties.h> #include <android-base/stringprintf.h> @@ -881,10 +882,28 @@ bool ScreenRecoveryUI::LoadWipeDataMenuText() { return true; } +static bool InitGraphics() { + // Timeout is same as init wait for file default of 5 seconds and is arbitrary + const unsigned timeout = 500; // 10ms increments + for (auto retry = timeout; retry > 0; --retry) { + if (gr_init() == 0) { + if (retry < timeout) { + // Log message like init wait for file completion log for consistency. + LOG(WARNING) << "wait for 'graphics' took " << ((timeout - retry) * 10) << "ms"; + } + return true; + } + std::this_thread::sleep_for(10ms); + } + // Log message like init wait for file timeout log for consistency. + LOG(ERROR) << "timeout wait for 'graphics' took " << (timeout * 10) << "ms"; + return false; +} + bool ScreenRecoveryUI::Init(const std::string& locale) { RecoveryUI::Init(locale); - if (gr_init() == -1) { + if (!InitGraphics()) { return false; } |