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

libct/cg/sd: use systemd v240+ new MAJOR:* syntax #3843

Merged
merged 1 commit into from
Jul 30, 2023

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented Apr 25, 2023

Since systemd v240 (systemd commit 8e8b5d2e6d91180a), one can use
/dev/{char,block}-MAJOR syntax to specify that all MAJOR:*
devices of the type are allowed.

Use it, if available, since it's more straightforward, plus
we can skip somewhat expensive parsing of /proc/devices
(which also reminds me we need to merge #3357).

@kolyshkin kolyshkin marked this pull request as draft April 25, 2023 00:14
@kolyshkin kolyshkin marked this pull request as ready for review April 26, 2023 03:47
@kolyshkin kolyshkin mentioned this pull request Apr 26, 2023
@kolyshkin kolyshkin added this to the 1.2.0 milestone Apr 27, 2023
@kolyshkin
Copy link
Contributor Author

@opencontainers/runc-maintainers PTAL

@kolyshkin
Copy link
Contributor Author

@opencontainers/runc-maintainers PTAL

@kolyshkin
Copy link
Contributor Author

rebased

@kolyshkin
Copy link
Contributor Author

@lifubang PTAL

Since systemd v240 (commit 8e8b5d2e6d91180a), one can use
/dev/{char,block}-MAJOR syntax to specify that all MAJOR:*
devices are allowed.

Use it, if available, since it's more straightforward, plus
we can skip somewhat expensive parsing of /proc/devices.

Signed-off-by: Kir Kolyshkin <[email protected]>
@lifubang lifubang merged commit 14c7ab7 into opencontainers:main Jul 30, 2023
@cyphar cyphar mentioned this pull request Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants