]> git.baikalelectronics.ru Git - kernel.git/commit
btrfs: defrag: remove an ambiguous condition for rejection
authorQu Wenruo <wqu@suse.com>
Fri, 28 Jan 2022 07:21:22 +0000 (15:21 +0800)
committerDavid Sterba <dsterba@suse.com>
Wed, 23 Feb 2022 16:42:55 +0000 (17:42 +0100)
commitf97d27d70b7e0c80cfbd3efc1bc7c16168ab61d3
treee38d765f7f95528cd16e3c97b1855fe772e6a515
parenta4e19827165af4029ed21cbf10497f35c223e0e0
btrfs: defrag: remove an ambiguous condition for rejection

From the very beginning of btrfs defrag, there is a check to reject
extents which meet both conditions:

- Physically adjacent

  We may want to defrag physically adjacent extents to reduce the number
  of extents or the size of subvolume tree.

- Larger than 128K

  This may be there for compressed extents, but unfortunately 128K is
  exactly the max capacity for compressed extents.
  And the check is > 128K, thus it never rejects compressed extents.

  Furthermore, the compressed extent capacity bug is fixed by previous
  patch, there is no reason for that check anymore.

The original check has a very small ranges to reject (the target extent
size is > 128K, and default extent threshold is 256K), and for
compressed extent it doesn't work at all.

So it's better just to remove the rejection, and allow us to defrag
physically adjacent extents.

CC: stable@vger.kernel.org # 5.16
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ioctl.c