gfs2: Remove unused RGRP_RSRV_MINBYTES definition
authorAndreas Gruenbacher <agruenba@redhat.com>
Tue, 25 Sep 2018 11:59:31 +0000 (12:59 +0100)
committerBob Peterson <rpeterso@redhat.com>
Fri, 12 Oct 2018 12:29:59 +0000 (07:29 -0500)
This definition is only used to define RGRP_RSRV_MINBLKS, with no
benefit over defining RGRP_RSRV_MINBLKS directly.

In addition, instead of forcing RGRP_RSRV_MINBLKS to be of type u32,
cast it to that type where that type is required.

Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Steven Whitehouse <swhiteho@redhat.com>
fs/gfs2/rgrp.c
fs/gfs2/rgrp.h

index 3b17a4e..22c73ce 100644 (file)
@@ -1566,7 +1566,7 @@ static void rg_mblk_search(struct gfs2_rgrpd *rgd, struct gfs2_inode *ip,
                extlen = 1;
        else {
                extlen = max_t(u32, atomic_read(&ip->i_sizehint), ap->target);
-               extlen = clamp(extlen, RGRP_RSRV_MINBLKS, free_blocks);
+               extlen = clamp(extlen, (u32)RGRP_RSRV_MINBLKS, free_blocks);
        }
        if ((rgd->rd_free_clone < rgd->rd_reserved) || (free_blocks < extlen))
                return;
index e90478e..6bb5ee1 100644 (file)
@@ -18,8 +18,7 @@
  * By reserving 32 blocks at a time, we can optimize / shortcut how we search
  * through the bitmaps by looking a word at a time.
  */
-#define RGRP_RSRV_MINBYTES 8
-#define RGRP_RSRV_MINBLKS ((u32)(RGRP_RSRV_MINBYTES * GFS2_NBBY))
+#define RGRP_RSRV_MINBLKS 32
 #define RGRP_RSRV_ADDBLKS 64
 
 struct gfs2_rgrpd;