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
Bump RLIMIT_NOFILE #6629
Bump RLIMIT_NOFILE #6629
Conversation
Wayland compositors handle many file descriptors: client connections, DMA-BUFs, sync_files, wl_data_device pipes, and so on. Bump the limit to the max. Closes: swaywm#6285
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm running sway with this patch, and can no longer reproduce #6642.
Without this patch and the with my usual amount of windows (just restoring my Firefox session), this reproduces 100% of the times.
With this patch and more-than-usual amount of windows, the issue is no longer reproduced (I did 4 attempts).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks :)
Thanks! |
We should probably also log the nofile limit when setrlimit succeeds. The user might still have a low hard limit. |
Showing an error when the limit is reached might be a good idea. Someone mentioned it's handled in a dependency/library, but I can't find the reference to that. |
Wayland compositors handle many file descriptors: client
connections, DMA-BUFs, sync_files, wl_data_device pipes, and so
on. Bump the limit to the max.
Closes: #6285