summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2020-05-19 14:47:19 +0200
committerGerrit Code Review <noreply-gerritcodereview@google.com>2020-05-19 14:47:19 +0200
commit082bea6325125376a5f2d5a750657ca9791a901d (patch)
tree4e7099064d574237230ed47055be46ce585f3ff6
parentMerge "Add EthernetDevice to manage ethernet connection." (diff)
parentrecovery: fastbootd: retry opening graphics (diff)
downloadandroid_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.cpp21
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;
}