-
Notifications
You must be signed in to change notification settings - Fork 245
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
Enhancement: compress-force xattr #696
Comments
We have only the plain string in the properties and enhancing tha with anything, like level or the forcing flag would be an incompatible change, though easy to handle due to the string value. Just that kernel and progs could refuse the values and either behave unexpectedly or crash. As the NOCOMPRESS flag is set automatically, this would have to be overridden in the code according the the property, but this would complicate the compression decisions which are already hard to follow in some cases. Can you please describe the use case, e.g. for which type of files or if it's based on some external tool that picks which files would need it. Thanks. |
Hi Since we can already set level in an xattr, just that the kernel doesn't yet read the level part, could we further add a f to the level part? Something like:
The
|
Yeah something like that could work, the future-extensible syntax for the compression value would be "read from left as far as you can parse", which now would be the name, then ":number" and then the "f" eventually, but we can do the ":f" at once. Curretnly kernel verifies the name, so this should work in a backward compatible way only ignoring the level. |
As a complement to
btrfs property set <file> compression [algo]
, it could be useful to have aforce
option of the same.The text was updated successfully, but these errors were encountered: