xfs: simplify and optimize the RT allocation fallback cascade

There are currently multiple levels of fall back if an RT allocation
can not be satisfied:

 1) xfs_rtallocate_extent extends the minlen and reduces the maxlen due
    to the extent size hint.  If that can't be done, it return -ENOSPC
    and let's xfs_bmap_rtalloc retry, which then not only drops the
    extent size hint based alignment, but also the minlen adjustment
 2) if xfs_rtallocate_extent gets -ENOSPC from the underlying functions,
    it only drops the extent size hint based alignment and retries
 3) if that still does not succeed, xfs_rtallocate_extent drops the
    extent size hint (which is a complex no-op at this point) and the
    minlen using the same code as (1) above
 4) if that still doesn't success and the caller wanted an allocation
    near a blkno, drop that blkno hint.

The handling in 1 is rather inefficient as we could just drop the
alignment and continue, and 2/3 interact in really weird ways due to
the duplicate policy.

Move aligning the min and maxlen out of xfs_rtallocate_extent and into
a helper called directly by xfs_bmap_rtalloc.  This allows just
continuing with the allocation if we have to drop the alignment instead
of going through the retry loop and also dropping the perfectly usable
minlen adjustment that didn't cause the problem, and then just use
a single retry that drops both the minlen and alignment requirement
when we really are out of space, thus consolidating cases (2) and (3)
above.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
Signed-off-by: Chandan Babu R <chandanbabu@kernel.org>

diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c
index ff2d7b237ef6493358c5d2fead6051045d50af4b..d1fc64a8161fcfdf239ec7fb156bf676e70818e5 100644 (file)
--- a/fs/xfs/xfs_rtalloc.c
+++ b/fs/xfs/xfs_rtalloc.c
@@ -1094,21 +1094,6 @@ xfs_rtallocate_extent(
        ASSERT(xfs_isilocked(args.mp->m_rbmip, XFS_ILOCK_EXCL));
        ASSERT(minlen > 0 && minlen <= maxlen);
 
-       /*
-        * If prod is set then figure out what to do to minlen and maxlen.
-        */
-       if (prod > 1) {
-               xfs_rtxlen_t    i;
-
-               if ((i = maxlen % prod))
-                       maxlen -= i;
-               if ((i = minlen % prod))
-                       minlen += prod - i;
-               if (maxlen < minlen)
-                       return -ENOSPC;
-       }
-
-retry:
        if (start == 0) {
                error = xfs_rtallocate_extent_size(&args, minlen,
                                maxlen, len, prod, rtx);
@@ -1117,13 +1102,8 @@ xfs_rtallocate_extent(
                                maxlen, len, prod, rtx);
        }
        xfs_rtbuf_cache_relse(&args);
-       if (error) {
-               if (error == -ENOSPC && prod > 1) {
-                       prod = 1;
-                       goto retry;
-               }
+       if (error)
                return error;
-       }
 
        /*
         * If it worked, update the superblock.
@@ -1354,6 +1334,35 @@ xfs_rtpick_extent(
        return 0;
 }
 
+static void
+xfs_rtalloc_align_minmax(
+       xfs_rtxlen_t            *raminlen,
+       xfs_rtxlen_t            *ramaxlen,
+       xfs_rtxlen_t            *prod)
+{
+       xfs_rtxlen_t            newmaxlen = *ramaxlen;
+       xfs_rtxlen_t            newminlen = *raminlen;
+       xfs_rtxlen_t            slack;
+
+       slack = newmaxlen % *prod;
+       if (slack)
+               newmaxlen -= slack;
+       slack = newminlen % *prod;
+       if (slack)
+               newminlen += *prod - slack;
+
+       /*
+        * If adjusting for extent size hint alignment produces an invalid
+        * min/max len combination, go ahead without it.
+        */
+       if (newmaxlen < newminlen) {
+               *prod = 1;
+               return;
+       }
+       *ramaxlen = newmaxlen;
+       *raminlen = newminlen;
+}
+
 int
 xfs_bmap_rtalloc(
        struct xfs_bmalloca     *ap)
@@ -1436,10 +1445,13 @@ xfs_bmap_rtalloc(
         * perfectly aligned, otherwise it will just get us in trouble.
         */
        div_u64_rem(ap->offset, align, &mod);
-       if (mod || ap->length % align)
+       if (mod || ap->length % align) {
                prod = 1;
-       else
+       } else {
                prod = xfs_extlen_to_rtxlen(mp, align);
+               if (prod > 1)
+                       xfs_rtalloc_align_minmax(&raminlen, &ralen, &prod);
+       }
 
        error = xfs_rtallocate_extent(ap->tp, start, raminlen, ralen, &ralen,
                        ap->wasdel, prod, &rtx);