summaryrefslogtreecommitdiff
path: root/libs/gui/SyncFeatures.cpp
diff options
context:
space:
mode:
author Xin Li <delphij@google.com> 2023-08-25 13:45:39 -0700
committer Xin Li <delphij@google.com> 2023-08-25 13:45:39 -0700
commit1b988a4ee33de9cab9740ddc1ee70b1734c8e622 (patch)
treec4240b01ae5e973213d6532885fcc00fac6f9e0b /libs/gui/SyncFeatures.cpp
parent13da0f9a0f2d5fcb70418fcdceb0d184313d7f48 (diff)
parentb7bc7af7daca55a85bcc993c45c9be354d48f89a (diff)
Merge Android U (ab/10368041)
Bug: 291102124 Merged-In: I9b175092d433fc6d7b22b437a09d76d3d2e0ce14 Change-Id: If8a2897a99b111ba107f33c19537d40bcbdc802e
Diffstat (limited to 'libs/gui/SyncFeatures.cpp')
-rw-r--r--libs/gui/SyncFeatures.cpp10
1 files changed, 8 insertions, 2 deletions
diff --git a/libs/gui/SyncFeatures.cpp b/libs/gui/SyncFeatures.cpp
index 1a8fc1a00a..2d863c2585 100644
--- a/libs/gui/SyncFeatures.cpp
+++ b/libs/gui/SyncFeatures.cpp
@@ -36,8 +36,12 @@ SyncFeatures::SyncFeatures() : Singleton<SyncFeatures>(),
mHasFenceSync(false),
mHasWaitSync(false) {
EGLDisplay dpy = eglGetDisplay(EGL_DEFAULT_DISPLAY);
- // This can only be called after EGL has been initialized; otherwise the
- // check below will abort.
+ // eglQueryString can only be called after EGL has been initialized;
+ // otherwise the check below will abort. If RenderEngine is using SkiaVk,
+ // EGL will not have been initialized. There's no problem with initializing
+ // it again here (it is ref counted), and then terminating it later.
+ EGLBoolean initialized = eglInitialize(dpy, nullptr, nullptr);
+ LOG_ALWAYS_FATAL_IF(!initialized, "eglInitialize failed");
const char* exts = eglQueryString(dpy, EGL_EXTENSIONS);
LOG_ALWAYS_FATAL_IF(exts == nullptr, "eglQueryString failed");
if (strstr(exts, "EGL_ANDROID_native_fence_sync")) {
@@ -63,6 +67,8 @@ SyncFeatures::SyncFeatures() : Singleton<SyncFeatures>(),
mString.append(" EGL_KHR_wait_sync");
}
mString.append("]");
+ // Terminate EGL to match the eglInitialize above
+ eglTerminate(dpy);
}
bool SyncFeatures::useNativeFenceSync() const {