diff options
author | Bruno de Oliveira Abinader <bruno@mapbox.com> | 2016-11-02 16:46:59 +0200 |
---|---|---|
committer | Bruno de Oliveira Abinader <bruno@mapbox.com> | 2016-11-02 17:34:03 +0200 |
commit | d1723ce0403d8d2858c070f1ef732c8608915683 (patch) | |
tree | bdb3b2cfa5021500d0b89401fb65d0791c767ead | |
parent | 87449f08c592a73ecd67de2768b1c1be0abc552f (diff) | |
download | qtlocation-mapboxgl-upstream/travis-updates.tar.gz |
[linux] Refactor Headless EGL with latest headless displayupstream/travis-updates
- s/HeadlessView/HeadlessBackend/g
- Use EGL_DEFAULT_DISPLAY
m--------- | .mason | 0 | ||||
-rw-r--r-- | .travis.yml | 6 | ||||
-rw-r--r-- | include/mbgl/gl/implementation.hpp | 2 | ||||
-rw-r--r-- | include/mbgl/platform/default/headless_backend.hpp | 13 | ||||
-rw-r--r-- | include/mbgl/platform/default/headless_display.hpp | 9 | ||||
-rw-r--r-- | platform/default/headless_backend_egl.cpp | 79 | ||||
-rw-r--r-- | platform/default/headless_backend_glx.cpp | 67 | ||||
-rw-r--r-- | platform/default/headless_display.cpp | 21 | ||||
-rw-r--r-- | platform/linux/config.cmake | 1 | ||||
-rwxr-xr-x | scripts/travis_setup.sh | 4 |
10 files changed, 22 insertions, 180 deletions
diff --git a/.mason b/.mason -Subproject 48ea4761bd386f9113397a4366d63e83e358664 +Subproject 4bbaa3c2e331fcfd818116d5003493fb7fdf771 diff --git a/.travis.yml b/.travis.yml index 6c749acfdf..428db242ac 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,14 +6,14 @@ addons_shortcuts: addons_clang35: &clang35 apt: sources: [ 'ubuntu-toolchain-r-test', 'george-edison55-precise-backports' ] - packages: [ 'clang-3.5', 'libstdc++-4.9-dev', 'libstdc++6', 'libllvm3.4', 'cmake', 'cmake-data', - 'mesa-utils', 'libxi-dev', 'libglu1-mesa-dev', 'x11proto-randr-dev', 'x11proto-xext-dev', 'libxrandr-dev', + packages: [ 'clang-3.5', 'libstdc++-4.9-dev', 'libstdc++6', 'libllvm3.8v4', 'cmake', 'cmake-data', + 'mesa-utils', 'libxi-dev', 'libegl1-mesa-dev', 'libgbm-dev', 'x11proto-randr-dev', 'x11proto-xext-dev', 'libxrandr-dev', 'x11proto-xf86vidmode-dev', 'libxxf86vm-dev', 'libxcursor-dev', 'libxinerama-dev' ] addons_gcc5: &gcc5 apt: sources: [ 'ubuntu-toolchain-r-test', 'george-edison55-precise-backports' ] packages: [ 'gdb', 'g++-5', 'gcc-5', 'cmake', 'cmake-data', - 'mesa-utils', 'libxi-dev', 'libglu1-mesa-dev', 'x11proto-randr-dev', 'x11proto-xext-dev', 'libxrandr-dev', + 'mesa-utils', 'libxi-dev', 'libegl1-mesa-dev', 'libgbm-dev', 'x11proto-randr-dev', 'x11proto-xext-dev', 'libxrandr-dev', 'x11proto-xf86vidmode-dev', 'libxxf86vm-dev', 'libxcursor-dev', 'libxinerama-dev' ] addons_qt4: &qt4 apt: diff --git a/include/mbgl/gl/implementation.hpp b/include/mbgl/gl/implementation.hpp index 4e3a3e51c7..7275d22ea6 100644 --- a/include/mbgl/gl/implementation.hpp +++ b/include/mbgl/gl/implementation.hpp @@ -10,5 +10,5 @@ #define MBGL_USE_CGL 1 #endif #else - #define MBGL_USE_GLX 1 + #define MBGL_USE_EGL 1 #endif diff --git a/include/mbgl/platform/default/headless_backend.hpp b/include/mbgl/platform/default/headless_backend.hpp index 6b096b694f..f7e8ee7295 100644 --- a/include/mbgl/platform/default/headless_backend.hpp +++ b/include/mbgl/platform/default/headless_backend.hpp @@ -6,12 +6,6 @@ class QGLWidget; #elif MBGL_USE_CGL #include <OpenGL/OpenGL.h> -#elif MBGL_USE_GLX -#define MBGL_USE_EGL 1 -struct gbm_device {}; -typedef void* EGLContext; -typedef void* EGLDisplay; -typedef void* EGLConfig; #endif #include <mbgl/map/backend.hpp> @@ -60,13 +54,8 @@ private: CGLContextObj glContext = nullptr; #endif -#if MBGL_USE_EAGL - void *glContext = nullptr; -#endif #if MBGL_USE_EGL - EGLDisplay dpy; - EGLContext glContext = nullptr; - EGLConfig config; + void *glContext = nullptr; #endif std::function<void(MapChange)> mapChangeCallback; diff --git a/include/mbgl/platform/default/headless_display.hpp b/include/mbgl/platform/default/headless_display.hpp index 45646d754a..e0fc8e17ea 100644 --- a/include/mbgl/platform/default/headless_display.hpp +++ b/include/mbgl/platform/default/headless_display.hpp @@ -4,9 +4,10 @@ #if MBGL_USE_CGL #include <OpenGL/OpenGL.h> -#elif MBGL_USE_GLX -typedef struct _XDisplay Display; -typedef struct __GLXFBConfigRec* GLXFBConfig; +#endif + +#if MBGL_USE_EGL +#include <EGL/egl.h> #endif namespace mbgl { @@ -21,8 +22,6 @@ public: #endif #if MBGL_USE_EGL - int fd; - struct gbm_device *gbm = nullptr; EGLDisplay dpy = 0; EGLConfig config = 0; #endif diff --git a/platform/default/headless_backend_egl.cpp b/platform/default/headless_backend_egl.cpp index ee1c875531..bed6129f0e 100644 --- a/platform/default/headless_backend_egl.cpp +++ b/platform/default/headless_backend_egl.cpp @@ -12,13 +12,13 @@ namespace mbgl { -gl::glProc HeadlessView::initializeExtension(const char* name) { +gl::glProc HeadlessBackend::initializeExtension(const char* name) { return eglGetProcAddress(name); } -void HeadlessView::createContext() { - dpy = display->dpy; - config = display->config; +void HeadlessBackend::createContext() { + auto dpy = display->dpy; + auto config = display->config; assert(dpy != EGL_NO_DISPLAY); assert(glContext == EGL_NO_CONTEXT); @@ -34,8 +34,9 @@ void HeadlessView::createContext() { } } -void HeadlessView::destroyContext() { +void HeadlessBackend::destroyContext() { if (glContext) { + auto dpy = display->dpy; if (!eglDestroyContext(dpy, glContext)) { throw std::runtime_error("Failed to destroy context."); } @@ -44,75 +45,15 @@ void HeadlessView::destroyContext() { } } -void HeadlessView::resizeFramebuffer() { - const unsigned int w = dimensions[0] * pixelRatio; - const unsigned int h = dimensions[1] * pixelRatio; - - // Create depth/stencil buffer - MBGL_CHECK_ERROR(glGenRenderbuffersEXT(1, &fboDepthStencil)); - MBGL_CHECK_ERROR(glBindRenderbufferEXT(GL_RENDERBUFFER_EXT, fboDepthStencil)); - MBGL_CHECK_ERROR(glRenderbufferStorageEXT(GL_RENDERBUFFER_EXT, GL_DEPTH24_STENCIL8_EXT, w, h)); - MBGL_CHECK_ERROR(glBindRenderbufferEXT(GL_RENDERBUFFER_EXT, 0)); - - MBGL_CHECK_ERROR(glGenRenderbuffersEXT(1, &fboColor)); - MBGL_CHECK_ERROR(glBindRenderbufferEXT(GL_RENDERBUFFER_EXT, fboColor)); - MBGL_CHECK_ERROR(glRenderbufferStorageEXT(GL_RENDERBUFFER_EXT, GL_RGBA8, w, h)); - MBGL_CHECK_ERROR(glBindRenderbufferEXT(GL_RENDERBUFFER_EXT, 0)); - - MBGL_CHECK_ERROR(glGenFramebuffersEXT(1, &fbo)); - MBGL_CHECK_ERROR(glBindFramebufferEXT(GL_FRAMEBUFFER_EXT, fbo)); - - MBGL_CHECK_ERROR(glFramebufferRenderbufferEXT(GL_FRAMEBUFFER_EXT, GL_COLOR_ATTACHMENT0_EXT, GL_RENDERBUFFER_EXT, fboColor)); - MBGL_CHECK_ERROR(glFramebufferRenderbufferEXT(GL_FRAMEBUFFER_EXT, GL_DEPTH_STENCIL_ATTACHMENT, GL_RENDERBUFFER_EXT, fboDepthStencil)); - - GLenum status = MBGL_CHECK_ERROR(glCheckFramebufferStatusEXT(GL_FRAMEBUFFER_EXT)); - - if (status != GL_FRAMEBUFFER_COMPLETE_EXT) { - std::string error("Couldn't create framebuffer: "); - switch (status) { - case GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT_EXT: (error += "incomplete attachment"); break; - case GL_FRAMEBUFFER_INCOMPLETE_MISSING_ATTACHMENT_EXT: error += "incomplete missing attachment"; break; - case GL_FRAMEBUFFER_INCOMPLETE_DIMENSIONS_EXT: error += "incomplete dimensions"; break; - case GL_FRAMEBUFFER_INCOMPLETE_FORMATS_EXT: error += "incomplete formats"; break; - case GL_FRAMEBUFFER_INCOMPLETE_DRAW_BUFFER_EXT: error += "incomplete draw buffer"; break; - case GL_FRAMEBUFFER_INCOMPLETE_READ_BUFFER_EXT: error += "incomplete read buffer"; break; - case GL_FRAMEBUFFER_UNSUPPORTED: error += "unsupported"; break; - default: error += "other"; break; - } - throw std::runtime_error(error); - } - - MBGL_CHECK_ERROR(glViewport(0, 0, w, h)); -} - -void HeadlessView::clearBuffers() { - assert(active); - - MBGL_CHECK_ERROR(glBindFramebufferEXT(GL_FRAMEBUFFER_EXT, 0)); - - if (fbo) { - MBGL_CHECK_ERROR(glDeleteFramebuffersEXT(1, &fbo)); - fbo = 0; - } - - if (fboColor) { - MBGL_CHECK_ERROR(glDeleteRenderbuffersEXT(1, &fboColor)); - fboColor = 0; - } - - if (fboDepthStencil) { - MBGL_CHECK_ERROR(glDeleteRenderbuffersEXT(1, &fboDepthStencil)); - fboDepthStencil = 0; - } -} - -void HeadlessView::activateContext() { +void HeadlessBackend::activateContext() { + auto dpy = display->dpy; if (!eglMakeCurrent(dpy, EGL_NO_SURFACE, EGL_NO_SURFACE, glContext)) { throw std::runtime_error("Switching OpenGL context failed.\n"); } } -void HeadlessView::deactivateContext() { +void HeadlessBackend::deactivateContext() { + auto dpy = display->dpy; if (!eglMakeCurrent(dpy, EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT)) { throw std::runtime_error("Removing OpenGL context failed.\n"); } diff --git a/platform/default/headless_backend_glx.cpp b/platform/default/headless_backend_glx.cpp deleted file mode 100644 index bbfd19345b..0000000000 --- a/platform/default/headless_backend_glx.cpp +++ /dev/null @@ -1,67 +0,0 @@ -#include <mbgl/platform/default/headless_backend.hpp> -#include <mbgl/platform/default/headless_display.hpp> - -#include <mbgl/platform/log.hpp> - -// #include <cassert> - -#include <GL/glx.h> - -namespace mbgl { - -gl::glProc HeadlessBackend::initializeExtension(const char* name) { - return glXGetProcAddress(reinterpret_cast<const GLubyte*>(name)); -} - -void HeadlessBackend::createContext() { - xDisplay = display->xDisplay; - fbConfigs = display->fbConfigs; - - if (!glContext) { - // Try to create a legacy context - glContext = glXCreateNewContext(xDisplay, fbConfigs[0], GLX_RGBA_TYPE, None, True); - if (glContext) { - if (!glXIsDirect(xDisplay, glContext)) { - Log::Error(Event::OpenGL, "failed to create direct OpenGL Legacy context"); - glXDestroyContext(xDisplay, glContext); - glContext = nullptr; - } - } - } - - if (glContext == nullptr) { - throw std::runtime_error("Error creating GL context object."); - } - - // Create a dummy pbuffer. We will render to framebuffers anyway, but we need a pbuffer to - // activate the context. - int pbufferAttributes[] = { - GLX_PBUFFER_WIDTH, 8, - GLX_PBUFFER_HEIGHT, 8, - None - }; - glxPbuffer = glXCreatePbuffer(xDisplay, fbConfigs[0], pbufferAttributes); -} - -void HeadlessBackend::destroyContext() { - if (glxPbuffer) { - glXDestroyPbuffer(xDisplay, glxPbuffer); - glxPbuffer = 0; - } - - glXDestroyContext(xDisplay, glContext); -} - -void HeadlessBackend::activateContext() { - if (!glXMakeContextCurrent(xDisplay, glxPbuffer, glxPbuffer, glContext)) { - throw std::runtime_error("Switching OpenGL context failed.\n"); - } -} - -void HeadlessBackend::deactivateContext() { - if (!glXMakeContextCurrent(xDisplay, 0, 0, nullptr)) { - throw std::runtime_error("Removing OpenGL context failed.\n"); - } -} - -} // namespace mbgl diff --git a/platform/default/headless_display.cpp b/platform/default/headless_display.cpp index a4517327cb..4d7ef5c933 100644 --- a/platform/default/headless_display.cpp +++ b/platform/default/headless_display.cpp @@ -4,7 +4,6 @@ #include <mbgl/util/string.hpp> #include <EGL/egl.h> #include <fcntl.h> -#include <gbm.h> #include <unistd.h> #endif @@ -34,22 +33,7 @@ HeadlessDisplay::HeadlessDisplay() { } #endif #if MBGL_USE_EGL - for (int i = 128; i < (128 + 16); i++) { - auto device_name = std::string{ "/dev/dri/renderD" } + mbgl::util::toString(i); - fd = open(device_name.c_str(), O_RDWR); - if (fd > 0) - break; - } - if (fd < 0) { - throw std::runtime_error("Couldn't open drm device."); - } - - gbm = gbm_create_device(fd); - if (gbm == NULL) { - throw std::runtime_error("Couldn't create gbm device."); - } - - dpy = eglGetDisplay(reinterpret_cast<EGLNativeDisplayType>(gbm)); + dpy = eglGetDisplay(EGL_DEFAULT_DISPLAY); if (dpy == EGL_NO_DISPLAY) { throw std::runtime_error("eglGetDisplay() failed."); } @@ -79,11 +63,8 @@ HeadlessDisplay::~HeadlessDisplay() { #if MBGL_USE_CGL CGLDestroyPixelFormat(pixelFormat); #endif - #if MBGL_USE_EGL eglTerminate(dpy); - gbm_device_destroy(gbm); - close(fd); #endif } diff --git a/platform/linux/config.cmake b/platform/linux/config.cmake index c39a3da806..61ffdc008f 100644 --- a/platform/linux/config.cmake +++ b/platform/linux/config.cmake @@ -41,7 +41,6 @@ macro(mbgl_platform_core) PRIVATE platform/default/webp_reader.cpp # Headless view - PRIVATE platform/default/headless_backend_glx.cpp PRIVATE platform/default/headless_backend_egl.cpp PRIVATE platform/default/headless_backend.cpp PRIVATE platform/default/headless_display.cpp diff --git a/scripts/travis_setup.sh b/scripts/travis_setup.sh index ae8dda46b5..8b573c4d5d 100755 --- a/scripts/travis_setup.sh +++ b/scripts/travis_setup.sh @@ -49,8 +49,8 @@ touch package.json # Install and set up to load a more recent version of mesa mapbox_time "install_mesa" \ -mason install mesa 11.2.2 -export LD_LIBRARY_PATH="`mason prefix mesa 11.2.2`/lib:${LD_LIBRARY_PATH:-}" +mason install mesa 12.0.3 +export LD_LIBRARY_PATH="`mason prefix mesa 12.0.3`/lib:${LD_LIBRARY_PATH:-}" # Install and set up to load awscli pip install --user awscli |