On Tue, Nov 19, 2024 at 04:49:49PM +0100, Christoph Hellwig wrote: > xfs_bmap_add_extent_hole_delay works entirely on delalloc extents, for > which xfs_bmap_same_rtgroup doesn't make sense. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> Oooooops :( Reviewed-by: "Darrick J. Wong" <djwong@xxxxxxxxxx> --D > --- > fs/xfs/libxfs/xfs_bmap.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c > index 9052839305e2..5255f93bae31 100644 > --- a/fs/xfs/libxfs/xfs_bmap.c > +++ b/fs/xfs/libxfs/xfs_bmap.c > @@ -2620,8 +2620,7 @@ xfs_bmap_add_extent_hole_delay( > */ > if ((state & BMAP_LEFT_VALID) && (state & BMAP_LEFT_DELAY) && > left.br_startoff + left.br_blockcount == new->br_startoff && > - left.br_blockcount + new->br_blockcount <= XFS_MAX_BMBT_EXTLEN && > - xfs_bmap_same_rtgroup(ip, whichfork, &left, new)) > + left.br_blockcount + new->br_blockcount <= XFS_MAX_BMBT_EXTLEN) > state |= BMAP_LEFT_CONTIG; > > if ((state & BMAP_RIGHT_VALID) && (state & BMAP_RIGHT_DELAY) && > @@ -2629,8 +2628,7 @@ xfs_bmap_add_extent_hole_delay( > new->br_blockcount + right.br_blockcount <= XFS_MAX_BMBT_EXTLEN && > (!(state & BMAP_LEFT_CONTIG) || > (left.br_blockcount + new->br_blockcount + > - right.br_blockcount <= XFS_MAX_BMBT_EXTLEN)) && > - xfs_bmap_same_rtgroup(ip, whichfork, new, &right)) > + right.br_blockcount <= XFS_MAX_BMBT_EXTLEN))) > state |= BMAP_RIGHT_CONTIG; > > /* > -- > 2.45.2 > >
