Skip to content

fcntl: F_SEAL_FUTURE_WRITE constant #114061

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

Closed
emanuele6 opened this issue Jan 14, 2024 · 2 comments
Closed

fcntl: F_SEAL_FUTURE_WRITE constant #114061

emanuele6 opened this issue Jan 14, 2024 · 2 comments
Labels
extension-modules C modules in the Modules dir type-feature A feature request or enhancement

Comments

@emanuele6
Copy link

emanuele6 commented Jan 14, 2024

Feature or enhancement

Proposal:

#13694 added fcntl.F_ADD_SEALS/fcntl.F_GET_SEALS and the fcntl.F_SEAL_* constants in the fcntl module, but the "new" fcntl.F_SEAL_FUTURE_WRITE constant from Linux 5.1 is missing.

Also, the documentation of the fcntl module only mentions the constants in:

Changed in version 3.8: The fcntl module now contains F_ADD_SEALS, F_GET_SEALS, and F_SEAL_* constants for sealing of os.memfd_create() file descriptors

Instead of listing out "F_SEAL_WRITE, F_SEAL_SHRINK, F_SEAL_GROW, and F_SEAL_SEAL", so I was not expecting F_SEAL_FUTURE_WRITE to be missing.

It would be awesome if F_SEAL_FUTURE_WRITE (0x10 by the way) was also defined in fcntl! :-)

Thank you!

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

@emanuele6 emanuele6 added the type-feature A feature request or enhancement label Jan 14, 2024
@Eclips4 Eclips4 added the extension-modules C modules in the Modules dir label Jan 14, 2024
@ronaldoussoren
Copy link
Contributor

Do you want to provide a PR for this?

@serhiy-storchaka
Copy link
Member

I included it in #114322.

@serhiy-storchaka serhiy-storchaka closed this as not planned Won't fix, can't repro, duplicate, stale Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension-modules C modules in the Modules dir type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

4 participants