Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WaylandBackend: Fix hotkeys failing to bind on Wayland desktops #1666

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matte-schwartz
Copy link

@matte-schwartz matte-schwartz commented Dec 15, 2024

Per Wayland protocol specsheet regarding keymapping: "From version 7 onwards, the fd must be mapped with MAP_PRIVATE by the recipient, as MAP_SHARED may fail."
https://wayland.app/protocols/wayland#wl_keyboard:event:keymap

This matches up exactly with what we're seeing with this error: [gamescope] [Error] xdg_backend: Failed to map keymap fd.

Changing MAP_SHARED to MAP_PRIVATE per the spec addresses this error.

Fixes: #1658
(hopefully) fixes: #1637

Per Wayland protocol specsheet regarding keymapping:
"From version 7 onwards, the fd must be mapped with MAP_PRIVATE by the
recipient, as MAP_SHARED may fail."

This matches up exactly with what we're seeing with this error:
[gamescope] [Error] xdg_backend: Failed to map keymap fd.

Changing MAP_SHARED to MAP_PRIVATE per the spec addresses this error.

Fixes: ValveSoftware#1658
(hopefully) fixes: ValveSoftware#1637
@matte-schwartz matte-schwartz mentioned this pull request Dec 15, 2024
6 tasks
@HanabishiRecca
Copy link

Seems to work.

@shawarden
Copy link

Works as expected, resolving the Super + hotkey issue #1658. Can't confirm the Super-less hotkey.

@naxil
Copy link

naxil commented Dec 15, 2024

Works as expected, resolving the Super + hotkey issue #1658. Can't confirm the Super-less hotkey.
how to change this on linux for fix it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Super key not working F key toggles fullscreen mode
4 participants