]> git.baikalelectronics.ru Git - kernel.git/commit
btrfs: do not allow compression on nodatacow files
authorChung-Chiang Cheng <cccheng@synology.com>
Fri, 15 Apr 2022 08:04:06 +0000 (16:04 +0800)
committerDavid Sterba <dsterba@suse.com>
Wed, 27 Apr 2022 20:19:33 +0000 (22:19 +0200)
commitf4a8b61d3e2057f5bf2422a94391e8b79ea08c74
tree2f839b148579f16ae7954934d76db2300c424d4d
parentaa9338c51028df3b3a933d25588eee4425d0c747
btrfs: do not allow compression on nodatacow files

Compression and nodatacow are mutually exclusive. A similar issue was
fixed by commit 47184016f0071 ("btrfs: add missing check for nocow and
compression inode flags"). Besides ioctl, there is another way to
enable/disable/reset compression directly via xattr. The following
steps will result in a invalid combination.

  $ touch bar
  $ chattr +C bar
  $ lsattr bar
  ---------------C-- bar
  $ setfattr -n btrfs.compression -v zstd bar
  $ lsattr bar
  --------c------C-- bar

To align with the logic in check_fsflags, nocompress will also be
unacceptable after this patch, to prevent mix any compression-related
options with nodatacow.

  $ touch bar
  $ chattr +C bar
  $ lsattr bar
  ---------------C-- bar
  $ setfattr -n btrfs.compression -v zstd bar
  setfattr: bar: Invalid argument
  $ setfattr -n btrfs.compression -v no bar
  setfattr: bar: Invalid argument

When both compression and nodatacow are enabled, then
btrfs_run_delalloc_range prefers nodatacow and no compression happens.

Reported-by: Jayce Lin <jaycelin@synology.com>
CC: stable@vger.kernel.org # 5.10.x: aa9338c51028: btrfs: export a helper for compression hard check
CC: stable@vger.kernel.org # 5.10.x
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Chung-Chiang Cheng <cccheng@synology.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/props.c
fs/btrfs/props.h
fs/btrfs/xattr.c