]> 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)
commit2fa05f7b317a3c465c7dd089becd66c6e8e9e263
tree2f839b148579f16ae7954934d76db2300c424d4d
parent4e3bc39e8441875214965469aa31e641639afce6
btrfs: do not allow compression on nodatacow files

Compression and nodatacow are mutually exclusive. A similar issue was
fixed by commit 2c704c98ff512 ("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: 4e3bc39e8441: 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