Commit ec380e54 authored by Boris Burkov's avatar Boris Burkov Committed by Peng Zhang
Browse files

btrfs: don't clear qgroup reserved bit in release_folio 

stable inclusion
from stable-v6.6.8
commit d3cf024353e213989a48399ed9ed24775e6472f0
bugzilla: https://gitee.com/openeuler/kernel/issues/I99K53

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=d3cf024353e213989a48399ed9ed24775e6472f0



--------------------------------

commit a86805504b88f636a6458520d85afdf0634e3c6b upstream.

The EXTENT_QGROUP_RESERVED bit is used to "lock" regions of the file for
duplicate reservations. That is two writes to that range in one
transaction shouldn't create two reservations, as the reservation will
only be freed once when the write finally goes down. Therefore, it is
never OK to clear that bit without freeing the associated qgroup
reserve. At this point, we don't want to be freeing the reserve, so mask
off the bit.

CC: stable@vger.kernel.org # 5.15+
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>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarZhangPeng <zhangpeng362@huawei.com>
parent 737b70b2

fs/btrfs/extent_io.c

+2 −1
Original line number Diff line number Diff line
@@ -2303,7 +2303,8 @@ static int try_release_extent_state(struct extent_io_tree *tree, 
		ret = 0;

	} else {

		u32 clear_bits = ~(EXTENT_LOCKED | EXTENT_NODATASUM |

				   EXTENT_DELALLOC_NEW | EXTENT_CTLBITS);

				   EXTENT_DELALLOC_NEW | EXTENT_CTLBITS |

				   EXTENT_QGROUP_RESERVED);



		/*

		 * At this point we can safely clear everything except the