]> git.baikalelectronics.ru Git - kernel.git/commit
btrfs: defrag: don't try to defrag extents which are under writeback
authorQu Wenruo <wqu@suse.com>
Tue, 8 Feb 2022 06:54:05 +0000 (14:54 +0800)
committerDavid Sterba <dsterba@suse.com>
Wed, 9 Feb 2022 17:53:24 +0000 (18:53 +0100)
commit9bdf4d8fdc61af0c1e66937e668e028f38fe92e1
tree3199cb2040911a016d333106b853209e5d8919f0
parentdb121470733425d14c9d652f3c05b510e354245e
btrfs: defrag: don't try to defrag extents which are under writeback

Once we start writeback (have called btrfs_run_delalloc_range()), we
allocate an extent, create an extent map point to that extent, with a
generation of (u64)-1, created the ordered extent and then clear the
DELALLOC bit from the range in the inode's io tree.

Such extent map can pass the first call of defrag_collect_targets(), as
its generation is (u64)-1, meets any possible minimal generation check.
And the range will not have DELALLOC bit, also passing the DELALLOC bit
check.

It will only be re-checked in the second call of
defrag_collect_targets(), which will wait for writeback.

But at that stage we have already spent our time waiting for some IO we
may or may not want to defrag.

Let's reject such extents early so we won't waste our time.

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