Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

render: choose DMA-BUF texture target via eglQueryDmaBufModifiersEXT #2255

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

emersion
Copy link
Member

@emersion emersion commented Jun 5, 2020

EGL_EXT_image_dma_buf_import_modifiers tells us whether we should use
GL_TEXTURE_2D or GL_TEXTURE_EXTERNAL_OES. Using the right texture target
can fix some failures and/or improve performance on some drivers.

This does the same as a Weston commit 1.

Closes: #2173


Breaking change: wlr_egl_create_image_from_dmabuf now takes an additional external_only argument.

EGL_EXT_image_dma_buf_import_modifiers tells us whether we should use
GL_TEXTURE_2D or GL_TEXTURE_EXTERNAL_OES. Using the right texture target
can fix some failures and/or improve performance on some drivers.

This does the same as a Weston commit [1].

[1]: https://gitlab.freedesktop.org/wayland/weston/commit/40c519a3e613

Closes: swaywm#2173
@escalade escalade mentioned this pull request Jun 5, 2020
@emersion emersion merged commit a3ba828 into swaywm:master Jun 6, 2020
@emersion emersion deleted the egl-texture-target branch June 6, 2020 06:59
@emersion emersion added the breaking Breaking change in public API label Jun 9, 2020
emersion added a commit to emersion/glider that referenced this pull request Jun 9, 2020
@emersion emersion added this to the 0.11.0 milestone Jul 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking Breaking change in public API
Development

Successfully merging this pull request may close these issues.

Use GL_TEXTURE_2D instead of GL_TEXTURE_EXTERNAL_OES when possible
2 participants