|
 |
0bf83d |
From 9eb0d6eb9066daa621e710139c8c8d50fedbabcf Mon Sep 17 00:00:00 2001
|
|
|
0bf83d |
From: "Darrick J. Wong" <darrick.wong@oracle.com>
|
|
|
0bf83d |
Date: Fri, 20 Nov 2020 17:03:27 -0500
|
|
|
0bf83d |
Subject: [PATCH] mkfs: enable the inode btree counter feature
|
|
|
0bf83d |
|
|
|
0bf83d |
Teach mkfs how to enable the inode btree counter feature.
|
|
|
0bf83d |
|
|
|
0bf83d |
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
|
|
|
0bf83d |
Reviewed-by: Brian Foster <bfoster@redhat.com>
|
|
|
0bf83d |
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
|
|
|
0bf83d |
---
|
|
|
0bf83d |
|
|
|
0bf83d |
diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
|
|
|
0bf83d |
index 9d6f315..45b150f 100644
|
|
|
0bf83d |
--- a/man/man8/mkfs.xfs.8
|
|
|
0bf83d |
+++ b/man/man8/mkfs.xfs.8
|
|
|
0bf83d |
@@ -188,6 +188,21 @@ option set. When the option
|
|
|
0bf83d |
.B \-m crc=0
|
|
|
0bf83d |
is used, the free inode btree feature is not supported and is disabled.
|
|
|
0bf83d |
.TP
|
|
|
0bf83d |
+.BI inobtcount= value
|
|
|
0bf83d |
+This option causes the filesystem to record the number of blocks used by
|
|
|
0bf83d |
+the inode btree and the free inode btree.
|
|
|
0bf83d |
+This can be used to reduce mount times when the free inode btree is enabled.
|
|
|
0bf83d |
+.IP
|
|
|
0bf83d |
+By default,
|
|
|
0bf83d |
+.B mkfs.xfs
|
|
|
0bf83d |
+will not enable this option.
|
|
|
0bf83d |
+This feature is only available for filesystems created with the (default)
|
|
|
0bf83d |
+.B \-m finobt=1
|
|
|
0bf83d |
+option set.
|
|
|
0bf83d |
+When the option
|
|
|
0bf83d |
+.B \-m finobt=0
|
|
|
0bf83d |
+is used, the inode btree counter feature is not supported and is disabled.
|
|
|
0bf83d |
+.TP
|
|
|
0bf83d |
.BI uuid= value
|
|
|
0bf83d |
Use the given value as the filesystem UUID for the newly created filesystem.
|
|
|
0bf83d |
The default is to generate a random UUID.
|
|
|
0bf83d |
diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
|
|
|
0bf83d |
index 16819d8..87f15f4 100644
|
|
|
0bf83d |
--- a/mkfs/xfs_mkfs.c
|
|
|
0bf83d |
+++ b/mkfs/xfs_mkfs.c
|
|
|
0bf83d |
@@ -117,6 +117,7 @@ enum {
|
|
|
0bf83d |
M_UUID,
|
|
|
0bf83d |
M_RMAPBT,
|
|
|
0bf83d |
M_REFLINK,
|
|
|
0bf83d |
+ M_INOBTCNT,
|
|
|
0bf83d |
M_MAX_OPTS,
|
|
|
0bf83d |
};
|
|
|
0bf83d |
|
|
|
0bf83d |
@@ -651,6 +652,7 @@ static struct opt_params mopts = {
|
|
|
0bf83d |
[M_UUID] = "uuid",
|
|
|
0bf83d |
[M_RMAPBT] = "rmapbt",
|
|
|
0bf83d |
[M_REFLINK] = "reflink",
|
|
|
0bf83d |
+ [M_INOBTCNT] = "inobtcount",
|
|
|
0bf83d |
},
|
|
|
0bf83d |
.subopt_params = {
|
|
|
0bf83d |
{ .index = M_CRC,
|
|
|
0bf83d |
@@ -681,6 +683,12 @@ static struct opt_params mopts = {
|
|
|
0bf83d |
.maxval = 1,
|
|
|
0bf83d |
.defaultval = 1,
|
|
|
0bf83d |
},
|
|
|
0bf83d |
+ { .index = M_INOBTCNT,
|
|
|
0bf83d |
+ .conflicts = { { NULL, LAST_CONFLICT } },
|
|
|
0bf83d |
+ .minval = 0,
|
|
|
0bf83d |
+ .maxval = 1,
|
|
|
0bf83d |
+ .defaultval = 1,
|
|
|
0bf83d |
+ },
|
|
|
0bf83d |
},
|
|
|
0bf83d |
};
|
|
|
0bf83d |
|
|
|
0bf83d |
@@ -731,6 +739,7 @@ struct sb_feat_args {
|
|
|
0bf83d |
bool spinodes; /* XFS_SB_FEAT_INCOMPAT_SPINODES */
|
|
|
0bf83d |
bool rmapbt; /* XFS_SB_FEAT_RO_COMPAT_RMAPBT */
|
|
|
0bf83d |
bool reflink; /* XFS_SB_FEAT_RO_COMPAT_REFLINK */
|
|
|
0bf83d |
+ bool inobtcnt; /* XFS_SB_FEAT_RO_COMPAT_INOBTCNT */
|
|
|
0bf83d |
bool nodalign;
|
|
|
0bf83d |
bool nortalign;
|
|
|
0bf83d |
};
|
|
|
0bf83d |
@@ -853,7 +862,8 @@ usage( void )
|
|
|
0bf83d |
{
|
|
|
0bf83d |
fprintf(stderr, _("Usage: %s\n\
|
|
|
0bf83d |
/* blocksize */ [-b size=num]\n\
|
|
|
0bf83d |
-/* metadata */ [-m crc=0|1,finobt=0|1,uuid=xxx,rmapbt=0|1,reflink=0|1]\n\
|
|
|
0bf83d |
+/* metadata */ [-m crc=0|1,finobt=0|1,uuid=xxx,rmapbt=0|1,reflink=0|1,\n\
|
|
|
0bf83d |
+ inobtcnt=0|1]\n\
|
|
|
0bf83d |
/* data subvol */ [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\
|
|
|
0bf83d |
(sunit=value,swidth=value|su=num,sw=num|noalign),\n\
|
|
|
0bf83d |
sectsize=num\n\
|
|
|
0bf83d |
@@ -1607,6 +1617,9 @@ meta_opts_parser(
|
|
|
0bf83d |
case M_REFLINK:
|
|
|
0bf83d |
cli->sb_feat.reflink = getnum(value, opts, subopt);
|
|
|
0bf83d |
break;
|
|
|
0bf83d |
+ case M_INOBTCNT:
|
|
|
0bf83d |
+ cli->sb_feat.inobtcnt = getnum(value, opts, subopt);
|
|
|
0bf83d |
+ break;
|
|
|
0bf83d |
default:
|
|
|
0bf83d |
return -EINVAL;
|
|
|
0bf83d |
}
|
|
|
0bf83d |
@@ -2037,6 +2050,22 @@ _("reflink not supported without CRC support\n"));
|
|
|
0bf83d |
usage();
|
|
|
0bf83d |
}
|
|
|
0bf83d |
cli->sb_feat.reflink = false;
|
|
|
0bf83d |
+
|
|
|
0bf83d |
+ if (cli->sb_feat.inobtcnt && cli_opt_set(&mopts, M_INOBTCNT)) {
|
|
|
0bf83d |
+ fprintf(stderr,
|
|
|
0bf83d |
+_("inode btree counters not supported without CRC support\n"));
|
|
|
0bf83d |
+ usage();
|
|
|
0bf83d |
+ }
|
|
|
0bf83d |
+ cli->sb_feat.inobtcnt = false;
|
|
|
0bf83d |
+ }
|
|
|
0bf83d |
+
|
|
|
0bf83d |
+ if (!cli->sb_feat.finobt) {
|
|
|
0bf83d |
+ if (cli->sb_feat.inobtcnt && cli_opt_set(&mopts, M_INOBTCNT)) {
|
|
|
0bf83d |
+ fprintf(stderr,
|
|
|
0bf83d |
+_("inode btree counters not supported without finobt support\n"));
|
|
|
0bf83d |
+ usage();
|
|
|
0bf83d |
+ }
|
|
|
0bf83d |
+ cli->sb_feat.inobtcnt = false;
|
|
|
0bf83d |
}
|
|
|
0bf83d |
|
|
|
0bf83d |
if ((cli->fsx.fsx_xflags & FS_XFLAG_COWEXTSIZE) &&
|
|
|
0bf83d |
@@ -3002,6 +3031,8 @@ sb_set_features(
|
|
|
0bf83d |
sbp->sb_features_ro_compat |= XFS_SB_FEAT_RO_COMPAT_RMAPBT;
|
|
|
0bf83d |
if (fp->reflink)
|
|
|
0bf83d |
sbp->sb_features_ro_compat |= XFS_SB_FEAT_RO_COMPAT_REFLINK;
|
|
|
0bf83d |
+ if (fp->inobtcnt)
|
|
|
0bf83d |
+ sbp->sb_features_ro_compat |= XFS_SB_FEAT_RO_COMPAT_INOBTCNT;
|
|
|
0bf83d |
|
|
|
0bf83d |
/*
|
|
|
0bf83d |
* Sparse inode chunk support has two main inode alignment requirements.
|
|
|
0bf83d |
@@ -3917,6 +3948,7 @@ main(
|
|
|
0bf83d |
.spinodes = true,
|
|
|
0bf83d |
.rmapbt = false,
|
|
|
0bf83d |
.reflink = true,
|
|
|
0bf83d |
+ .inobtcnt = false,
|
|
|
0bf83d |
.parent_pointers = false,
|
|
|
0bf83d |
.nodalign = false,
|
|
|
0bf83d |
.nortalign = false,
|