]> www.infradead.org Git - users/hch/xfs.git/commitdiff
xfs: make the rtalloc start hint a xfs_rtblock_t
authorChristoph Hellwig <hch@lst.de>
Thu, 1 Aug 2024 18:40:48 +0000 (20:40 +0200)
committerChristoph Hellwig <hch@lst.de>
Mon, 5 Aug 2024 20:26:37 +0000 (13:26 -0700)
0 is a valid start RT extent, and with pending changes it will become
both more common and non-unique.  Switch to pass a xfs_rtblock_t instead
so that we can use NULLRTBLOCK to determine if a hint was set or not.

Signed-off-by: Christoph Hellwig <hch@lst.de>
fs/xfs/xfs_rtalloc.c

index 9b9123e584225f53eedd7d6e9eed3684ee4962b2..c53f4d0dfac365b0ca851ddc3c336e3a65e43014 100644 (file)
@@ -1219,7 +1219,7 @@ xfs_rtalloc_align_minmax(
 static int
 xfs_rtallocate(
        struct xfs_trans        *tp,
-       xfs_rtxnum_t            start,
+       xfs_rtblock_t           bno_hint,
        xfs_rtxlen_t            minlen,
        xfs_rtxlen_t            maxlen,
        xfs_rtxlen_t            prod,
@@ -1233,6 +1233,7 @@ xfs_rtallocate(
                .mp             = tp->t_mountp,
                .tp             = tp,
        };
+       xfs_rtxnum_t            start = 0;
        xfs_rtxnum_t            rtx;
        xfs_rtxlen_t            len = 0;
        int                     error = 0;
@@ -1250,7 +1251,9 @@ xfs_rtallocate(
         * For an allocation to an empty file at offset 0, pick an extent that
         * will space things out in the rt area.
         */
-       if (!start && initial_user_data)
+       if (bno_hint)
+               start = xfs_rtb_to_rtx(args.mp, bno_hint);
+       else if (initial_user_data)
                start = xfs_rtpick_extent(args.mp, tp, maxlen);
 
        if (start) {
@@ -1363,15 +1366,16 @@ int
 xfs_bmap_rtalloc(
        struct xfs_bmalloca     *ap)
 {
-       struct xfs_mount        *mp = ap->ip->i_mount;
        xfs_fileoff_t           orig_offset = ap->offset;
-       xfs_rtxnum_t            start = 0; /* allocation hint rtextent no */
        xfs_rtxlen_t            prod = 0;  /* product factor for allocators */
        xfs_rtxlen_t            ralen = 0; /* realtime allocation length */
+       xfs_rtblock_t           bno_hint = NULLRTBLOCK;
        xfs_extlen_t            orig_length = ap->length;
        xfs_rtxlen_t            raminlen;
        bool                    rtlocked = false;
        bool                    noalign = false;
+       bool                    initial_user_data =
+               ap->datatype & XFS_ALLOC_INITIAL_USER_DATA;
        int                     error;
 
 retry:
@@ -1380,10 +1384,10 @@ retry:
                return error;
 
        if (xfs_bmap_adjacent(ap))
-               start = xfs_rtb_to_rtx(mp, ap->blkno);
+               bno_hint = ap->blkno;
 
-       error = xfs_rtallocate(ap->tp, start, raminlen, ralen, prod, ap->wasdel,
-                       ap->datatype & XFS_ALLOC_INITIAL_USER_DATA, &rtlocked,
+       error = xfs_rtallocate(ap->tp, bno_hint, raminlen, ralen, prod,
+                       ap->wasdel, initial_user_data, &rtlocked,
                        &ap->blkno, &ap->length);
        if (error == -ENOSPC) {
                if (!noalign) {