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

Require INVALID for implicit format modifiers #3231

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

emersion
Copy link
Member

@emersion emersion commented Oct 1, 2021

See this for motivation: https://gitlab.freedesktop.org/wayland/wayland-protocols/-/merge_requests/75

  • This breaks DRM cursors when modifiers aren't supported: the plane has only { LINEAR } supported and receives a buffer with INVALID.

drmModeAddFB2 doesn't support explicit modifiers. Only accept INVALID
which indicates an implicit modifier and LINEAR which may indicate
that GBM_BO_USE_LINEAR has been used.
This was used to make the intersection of INVALID and LINEAR result
in LINEAR. We can now just require LINEAR to be in both lists.
@emersion emersion marked this pull request as draft October 1, 2021 15:29
@emersion emersion added the breaking Breaking change in public API label Oct 1, 2021
@emersion
Copy link
Member Author

emersion commented Nov 1, 2021

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:

https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3231

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.

1 participant