]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
xfs: don't allocate the xfs_extent_busy structure for zoned RTGs
authorChristoph Hellwig <hch@lst.de>
Wed, 16 Jul 2025 12:54:01 +0000 (14:54 +0200)
committerCarlos Maiolino <cem@kernel.org>
Fri, 18 Jul 2025 15:42:31 +0000 (17:42 +0200)
commit5948705adbf1a7afcecfe9a13ff39221ef61e16b
tree199d9b2453346191985684d37501c82d88911583
parent9b027aa3e8c44ea826fab1928f5d02a186ff1536
xfs: don't allocate the xfs_extent_busy structure for zoned RTGs

Busy extent tracking is primarily used to ensure that freed blocks are
not reused for data allocations before the transaction that deleted them
has been committed to stable storage, and secondarily to drive online
discard.  None of the use cases applies to zoned RTGs, as the zoned
allocator can't overwrite blocks before resetting the zone, which already
flushes out all transactions touching the RTGs.

So the busy extent tracking is not needed for zoned RTGs, and also not
called for zoned RTGs.  But somehow the code to skip allocating and
freeing the structure got lost during the zoned XFS upstreaming process.
This not only causes these structures to unnecessarily allocated, but can
also lead to memory leaks as the xg_busy_extents pointer in the
xfs_group structure is overlayed with the pointer for the linked list
of to be reset zones.

Stop allocating and freeing the structure to not pointlessly allocate
memory which is then leaked when the zone is reset.

Fixes: 080d01c41d44 ("xfs: implement zoned garbage collection")
Signed-off-by: Christoph Hellwig <hch@lst.de>
Cc: <stable@vger.kernel.org> # v6.15
[cem: Fix type and add stable tag]
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
fs/xfs/libxfs/xfs_group.c
fs/xfs/xfs_extent_busy.h