Commit 3c6f0c5e authored by Boris Burkov's avatar Boris Burkov Committed by David Sterba
Browse files

btrfs: make btrfs_clear_delalloc_extent() free delalloc reserve 



Currently, this call site in btrfs_clear_delalloc_extent() only converts
the reservation. We are marking it not delalloc, so I don't think it
makes sense to keep the rsv around.  This is a path where we are not
sure to join a transaction, so it leads to incorrect free-ing during
umount.

Helps with the pass rate of generic/269 and generic/475.

Reviewed-by: default avatarQu Wenruo <wqu@suse.com>
Signed-off-by: default avatarBoris Burkov <boris@bur.io>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 211de933

fs/btrfs/inode.c

+1 −1
Original line number Diff line number Diff line
@@ -2533,7 +2533,7 @@ void btrfs_clear_delalloc_extent(struct btrfs_inode *inode, 
		 */

		if (bits & EXTENT_CLEAR_META_RESV &&

		    root != fs_info->tree_root)

			btrfs_delalloc_release_metadata(inode, len, false);

			btrfs_delalloc_release_metadata(inode, len, true);



		/* For sanity tests. */

		if (btrfs_is_testing(fs_info))