Re: [PATCH] btrfs: zstd: add `zstd-fast` alias mount option for fast modes

From: Qu Wenruo
Date: Mon Apr 14 2025 - 23:50:35 EST




在 2025/4/14 16:23, Daniel Vacek 写道:
On Wed, 2 Apr 2025 at 16:31, Daniel Vacek <neelx@xxxxxxxx> wrote:
[...]
I'd still opt for keeping full range and functionality including
negative levels using the plain `zstd:N` option and having the other
just as an additional alias (for maybe being a bit nicer to some
humans, but not a big deal really and a matter of preference).
Checking the official documentation, it still mentions "negative
compression levels" being the fast option.

https://facebook.github.io/zstd/
https://facebook.github.io/zstd/zstd_manual.html

The deprecation part looks like just some gossip. It looks more about
the cli tool api and we are defining a kernel mount api - perfectly
unrelated.

Any feedback, Dave? I tend to drop this ida of `zstd-fast` alias.

Not Dave here, but if the future of "zstd-fast" is not that clear, we can definitely wait for a while.

It's always safer to adapt when the terminology is mature enough.

Thanks,
Qu


We can make this change before 6.15 final so it's not in any released
kernel and we don't have to deal with compatibility.