From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: linux-xfs <linux-xfs@vger.kernel.org>
Subject: Re: [PATCH 3/5] mkfs: invert project id width boolean name
Date: Thu, 7 Dec 2017 20:18:28 -0800 [thread overview]
Message-ID: <20171208041828.GH19219@magnolia> (raw)
In-Reply-To: <c22e3212-3e1c-ed6d-e1cc-4f0396265c52@sandeen.net>
On Thu, Dec 07, 2017 at 10:15:58PM -0600, Eric Sandeen wrote:
> It's a bit nuts that we have a projid32bit mkfs option, but
> we carry around the inverse of its value in "projid16bit" -
> just flip it around for sanity.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Looks ok,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> mkfs/xfs_mkfs.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> index d30f73d..f6e2fad 100644
> --- a/mkfs/xfs_mkfs.c
> +++ b/mkfs/xfs_mkfs.c
> @@ -763,7 +763,7 @@ struct sb_feat_args {
> bool inode_align;
> bool nci;
> bool lazy_sb_counters;
> - bool projid16bit;
> + bool projid32bit;
> bool crcs_enabled;
> bool dirftype;
> bool finobt;
> @@ -1551,7 +1551,7 @@ inode_opts_parser(
> cli->sb_feat.attr_version = getnum(value, &iopts, I_ATTR);
> break;
> case I_PROJID32BIT:
> - cli->sb_feat.projid16bit = !getnum(value, &iopts, I_PROJID32BIT);
> + cli->sb_feat.projid32bit = getnum(value, &iopts, I_PROJID32BIT);
> break;
> case I_SPINODES:
> cli->sb_feat.spinodes = getnum(value, &iopts, I_SPINODES);
> @@ -2037,7 +2037,7 @@ _("V2 attribute format always enabled on CRC enabled filesytems\n"));
>
> /* 32 bit project quota always on */
> /* attr2 always on */
> - if (cli->sb_feat.projid16bit) {
> + if (!cli->sb_feat.projid32bit) {
> fprintf(stderr,
> _("32 bit Project IDs always enabled on CRC enabled filesytems\n"));
> usage();
> @@ -2905,7 +2905,7 @@ sb_set_features(
> sbp->sb_features2 = 0;
> if (fp->lazy_sb_counters)
> sbp->sb_features2 |= XFS_SB_VERSION2_LAZYSBCOUNTBIT;
> - if (!fp->projid16bit)
> + if (fp->projid32bit)
> sbp->sb_features2 |= XFS_SB_VERSION2_PROJID32BIT;
> if (fp->parent_pointers)
> sbp->sb_features2 |= XFS_SB_VERSION2_PARENTBIT;
> @@ -3219,7 +3219,7 @@ print_mkfs_cfg(
> "realtime =%-22s extsz=%-6d blocks=%lld, rtextents=%lld\n"),
> dfile, cfg->inodesize, (long long)cfg->agcount,
> (long long)cfg->agsize,
> - "", cfg->sectorsize, fp->attr_version, !fp->projid16bit,
> + "", cfg->sectorsize, fp->attr_version, fp->projid32bit,
> "", fp->crcs_enabled, fp->finobt, fp->spinodes, fp->rmapbt,
> fp->reflink,
> "", cfg->blocksize, (long long)cfg->dblocks, cfg->imaxpct,
> @@ -3854,7 +3854,7 @@ main(
> .inode_align = XFS_IFLAG_ALIGN,
> .nci = false,
> .lazy_sb_counters = true,
> - .projid16bit = false,
> + .projid32bit = true,
> .crcs_enabled = true,
> .dirftype = true,
> .finobt = true,
> --
> 1.8.3.1
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2017-12-08 4:18 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-08 4:11 [PATCH 0/5] mkfs: minor tidyups Eric Sandeen
2017-12-08 4:13 ` [PATCH 1/5] mkfs: Don't emit default config message yet Eric Sandeen
2017-12-08 4:21 ` Darrick J. Wong
2017-12-08 4:54 ` Dave Chinner
2017-12-08 13:37 ` Eric Sandeen
2017-12-08 4:13 ` [PATCH 2/5] mkfs: remove unused m_uuid in sb_feat_args Eric Sandeen
2017-12-08 4:22 ` Darrick J. Wong
2017-12-08 4:15 ` [PATCH 3/5] mkfs: invert project id width boolean name Eric Sandeen
2017-12-08 4:18 ` Darrick J. Wong [this message]
2017-12-08 4:17 ` [PATCH 4/5] mkfs: document sb_feat_args members Eric Sandeen
2017-12-08 4:27 ` Darrick J. Wong
2017-12-08 4:59 ` Dave Chinner
2017-12-08 4:18 ` [PATCH 5/5] mkfs: remove use-once default macros Eric Sandeen
2017-12-08 4:27 ` Darrick J. Wong
2017-12-08 5:01 ` [PATCH 0/5] mkfs: minor tidyups Dave Chinner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171208041828.GH19219@magnolia \
--to=darrick.wong@oracle.com \
--cc=linux-xfs@vger.kernel.org \
--cc=sandeen@sandeen.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox