On Tue, Oct 11, 2022 at 10:50:01AM -0400, Sasha Levin wrote: > From: Omar Sandoval <osandov@xxxxxx> > > [ Upstream commit 48ff70830bec1ccc714f4e31059df737f17ec909 ] > > struct btrfs_caching_ctl::progress and struct > btrfs_block_group::last_byte_to_unpin were previously needed to ensure > that unpin_extent_range() didn't return a range to the free space cache > before the caching thread had a chance to cache that range. However, the > commit "btrfs: fix space cache corruption and potential double > allocations" made it so that we always synchronously cache the block > group at the time that we pin the extent, so this machinery is no longer > necessary. > > Reviewed-by: Filipe Manana <fdmanana@xxxxxxxx> > Signed-off-by: Omar Sandoval <osandov@xxxxxx> > Signed-off-by: David Sterba <dsterba@xxxxxxxx> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> > --- > fs/btrfs/block-group.c | 13 ------------ > fs/btrfs/block-group.h | 2 -- > fs/btrfs/extent-tree.c | 9 ++------- > fs/btrfs/free-space-tree.c | 8 -------- > fs/btrfs/transaction.c | 41 -------------------------------------- > fs/btrfs/zoned.c | 1 - > 6 files changed, 2 insertions(+), 72 deletions(-) Hi, Sasha, This commit is a cleanup. Please drop it from 6.0 and 5.19. Thanks, Omar