Linux Btrfs filesystem development
 help / color / mirror / Atom feed
* [PATCH v3 0/5] btrfs: add per-inode compression levels in xattrs
@ 2026-08-09 18:52 koraynilay
  2026-08-09 18:52 ` [PATCH v3 1/5] btrfs: fix -o compress= level getting inherited by compression xattr property koraynilay
                   ` (4 more replies)
  0 siblings, 5 replies; 8+ messages in thread
From: koraynilay @ 2026-08-09 18:52 UTC (permalink / raw)
  To: Chris Mason, David Sterba
  Cc: Qu Wenruo, Zygo Blaxell, linux-btrfs, koraynilay

Add per-inode compression levels using
btrfs property set /path/to/file compression "algo:level", using the
same syntax as the compress mount option.
If set on folders, all new children will inherit the setting, while
already existing children will be unaffected.

This patch series also fixes a small bug: before, when setting
btrfs.compression, it would keep the mount option level, even if the
algo was different (!!), so with e.g. compress=zstd:15 and
btrfs.compression=zlib the data gets compressed at zlib:9 (because it
would still get clamped at the right range); now if the level isn't
explicitly set it will use the default level for the algo.

I also wanna thank Zygo for helping me by explaining stuff and for
noticing this bug.

Changes in this v3:
* the first patch contains only the bugfix for the mentioned bug, so it
  can be backported to older kernels if needed;
* fix a bisection and compilation error.

koraynilay (5):
  btrfs: fix -o compress= level getting inherited by compression xattr
    property
  btrfs: export btrfs_match_compress_type(), move it to compression.h
  btrfs: also validate compression levels in
    btrfs_compress_is_valid_type()
  btrfs: add per-inode compression levels in xattrs
  btrfs: support inheritance for per-inode compression levels

 fs/btrfs/btrfs_inode.h |  1 +
 fs/btrfs/compression.c | 35 ++++++++++++++++++++--
 fs/btrfs/compression.h |  3 +-
 fs/btrfs/inode.c       |  2 ++
 fs/btrfs/props.c       | 66 ++++++++++++++++++++++++++++++++++--------
 fs/btrfs/super.c       |  8 -----
 6 files changed, 91 insertions(+), 24 deletions(-)


base-commit: a13307e97d5c54b65720bb71fa379960ded1e51a
-- 
2.55.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-08-09 22:16 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-09 18:52 [PATCH v3 0/5] btrfs: add per-inode compression levels in xattrs koraynilay
2026-08-09 18:52 ` [PATCH v3 1/5] btrfs: fix -o compress= level getting inherited by compression xattr property koraynilay
2026-08-09 20:15   ` Zygo Blaxell
2026-08-09 22:16     ` Qu Wenruo
2026-08-09 18:53 ` [PATCH v3 2/5] btrfs: export btrfs_match_compress_type(), move it to compression.h koraynilay
2026-08-09 18:53 ` [PATCH v3 3/5] btrfs: also validate compression levels in btrfs_compress_is_valid_type() koraynilay
2026-08-09 18:53 ` [PATCH v3 4/5] btrfs: add per-inode compression levels in xattrs koraynilay
2026-08-09 18:53 ` [PATCH v3 5/5] btrfs: support inheritance for per-inode compression levels koraynilay

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox