Skip to content

Commit 851a678

Browse files
Christoph HellwigDarrick J. Wong
authored andcommitted
xfs: remove duplicate asserts in xfs_defer_extent_free
The bno/len verification is already done by the calls to xfs_verify_rtbext / xfs_verify_fsbext, and reporting a corruption error seem like the better handling than tripping an assert anyway. Signed-off-by: Christoph Hellwig <[email protected]> Reviewed-by: Darrick J. Wong <[email protected]> Signed-off-by: Darrick J. Wong <[email protected]>
1 parent 81927e6 commit 851a678

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

fs/xfs/libxfs/xfs_alloc.c

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2567,23 +2567,10 @@ xfs_defer_extent_free(
25672567
{
25682568
struct xfs_extent_free_item *xefi;
25692569
struct xfs_mount *mp = tp->t_mountp;
2570-
#ifdef DEBUG
2571-
xfs_agnumber_t agno;
2572-
xfs_agblock_t agbno;
25732570

2574-
ASSERT(bno != NULLFSBLOCK);
2575-
ASSERT(len > 0);
25762571
ASSERT(len <= XFS_MAX_BMBT_EXTLEN);
25772572
ASSERT(!isnullstartblock(bno));
2578-
agno = XFS_FSB_TO_AGNO(mp, bno);
2579-
agbno = XFS_FSB_TO_AGBNO(mp, bno);
2580-
ASSERT(agno < mp->m_sb.sb_agcount);
2581-
ASSERT(agbno < mp->m_sb.sb_agblocks);
2582-
ASSERT(len < mp->m_sb.sb_agblocks);
2583-
ASSERT(agbno + len <= mp->m_sb.sb_agblocks);
2584-
#endif
25852573
ASSERT(!(free_flags & ~XFS_FREE_EXTENT_ALL_FLAGS));
2586-
ASSERT(xfs_extfree_item_cache != NULL);
25872574
ASSERT(type != XFS_AG_RESV_AGFL);
25882575

25892576
if (XFS_IS_CORRUPT(mp, !xfs_verify_fsbext(mp, bno, len)))

0 commit comments

Comments
 (0)