]> git.ipfire.org Git - thirdparty/xfsprogs-dev.git/blobdiff - mkfs/xfs_mkfs.c
xfsprogs: make static things static
[thirdparty/xfsprogs-dev.git] / mkfs / xfs_mkfs.c
index dbe85d7f8ae3cfc51a736efe5191469766d73c43..d1387ddf0539173606ed60d03a6a0354a8e2b09c 100644 (file)
@@ -1,26 +1,14 @@
+// SPDX-License-Identifier: GPL-2.0
 /*
  * Copyright (c) 2000-2005 Silicon Graphics, Inc.
  * All Rights Reserved.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it would be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write the Free Software Foundation,
- * Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
  */
 #include "libfrog.h"
 #include "libxfs.h"
 #include <ctype.h>
 #include "xfs_multidisk.h"
 #include "libxcmd.h"
-
+#include "fsgeom.h"
 
 
 #define TERABYTES(count, blog) ((uint64_t)(count) << (40 - (blog)))
@@ -43,8 +31,8 @@
  * XXX: The configured block and sector sizes are defined as global variables so
  * that they don't need to be passed to getnum/cvtnum().
  */
-unsigned int           blocksize;
-unsigned int           sectorsize;
+static unsigned int            blocksize;
+static unsigned int            sectorsize;
 
 /*
  * Enums for each CLI parameter type are declared first so we can calculate the
@@ -228,9 +216,9 @@ struct opt_params {
  * about tables that haven't yet been defined. Work around this ordering
  * issue with extern definitions here.
  */
-extern struct opt_params sopts;
+static struct opt_params sopts;
 
-struct opt_params bopts = {
+static struct opt_params bopts = {
        .name = 'b',
        .subopts = {
                [B_SIZE] = "size",
@@ -247,7 +235,7 @@ struct opt_params bopts = {
        },
 };
 
-struct opt_params dopts = {
+static struct opt_params dopts = {
        .name = 'd',
        .subopts = {
                [D_AGCOUNT] = "agcount",
@@ -384,7 +372,7 @@ struct opt_params dopts = {
 };
 
 
-struct opt_params iopts = {
+static struct opt_params iopts = {
        .name = 'i',
        .subopts = {
                [I_ALIGN] = "align",
@@ -445,7 +433,7 @@ struct opt_params iopts = {
        },
 };
 
-struct opt_params lopts = {
+static struct opt_params lopts = {
        .name = 'l',
        .subopts = {
                [L_AGNUM] = "agnum",
@@ -507,6 +495,7 @@ struct opt_params lopts = {
                },
                { .index = L_DEV,
                  .conflicts = { { &lopts, L_AGNUM },
+                                { &lopts, L_NAME },
                                 { &lopts, L_INTERNAL },
                                 { NULL, LAST_CONFLICT } },
                  .defaultval = SUBOPT_NEEDS_VAL,
@@ -529,6 +518,7 @@ struct opt_params lopts = {
                },
                { .index = L_NAME,
                  .conflicts = { { &lopts, L_AGNUM },
+                                { &lopts, L_DEV },
                                 { &lopts, L_INTERNAL },
                                 { NULL, LAST_CONFLICT } },
                  .defaultval = SUBOPT_NEEDS_VAL,
@@ -542,7 +532,7 @@ struct opt_params lopts = {
        },
 };
 
-struct opt_params nopts = {
+static struct opt_params nopts = {
        .name = 'n',
        .subopts = {
                [N_SIZE] = "size",
@@ -573,7 +563,7 @@ struct opt_params nopts = {
        },
 };
 
-struct opt_params ropts = {
+static struct opt_params ropts = {
        .name = 'r',
        .subopts = {
                [R_EXTSIZE] = "extsize",
@@ -599,7 +589,8 @@ struct opt_params ropts = {
                  .defaultval = SUBOPT_NEEDS_VAL,
                },
                { .index = R_DEV,
-                 .conflicts = { { NULL, LAST_CONFLICT } },
+                 .conflicts = { { &ropts, R_NAME },
+                                { NULL, LAST_CONFLICT } },
                  .defaultval = SUBOPT_NEEDS_VAL,
                },
                { .index = R_FILE,
@@ -609,7 +600,8 @@ struct opt_params ropts = {
                  .conflicts = { { NULL, LAST_CONFLICT } },
                },
                { .index = R_NAME,
-                 .conflicts = { { NULL, LAST_CONFLICT } },
+                 .conflicts = { { &ropts, R_DEV },
+                                { NULL, LAST_CONFLICT } },
                  .defaultval = SUBOPT_NEEDS_VAL,
                },
                { .index = R_NOALIGN,
@@ -621,7 +613,7 @@ struct opt_params ropts = {
        },
 };
 
-struct opt_params sopts = {
+static struct opt_params sopts = {
        .name = 's',
        .subopts = {
                [S_SIZE] = "size",
@@ -651,7 +643,7 @@ struct opt_params sopts = {
        },
 };
 
-struct opt_params mopts = {
+static struct opt_params mopts = {
        .name = 'm',
        .subopts = {
                [M_CRC] = "crc",
@@ -860,25 +852,24 @@ static void __attribute__((noreturn))
 usage( void )
 {
        fprintf(stderr, _("Usage: %s\n\
-/* blocksize */                [-b log=n|size=num]\n\
+/* blocksize */                [-b size=num]\n\
 /* metadata */         [-m crc=0|1,finobt=0|1,uuid=xxx,rmapbt=0|1,reflink=0|1]\n\
 /* data subvol */      [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\
                            (sunit=value,swidth=value|su=num,sw=num|noalign),\n\
-                           sectlog=n|sectsize=num\n\
+                           sectsize=num\n\
 /* force overwrite */  [-f]\n\
 /* inode size */       [-i log=n|perblock=n|size=num,maxpct=n,attr=0|1|2,\n\
                            projid32bit=0|1,sparse=0|1]\n\
 /* no discard */       [-K]\n\
 /* log subvol */       [-l agnum=n,internal,size=num,logdev=xxx,version=n\n\
-                           sunit=value|su=num,sectlog=n|sectsize=num,\n\
-                           lazy-count=0|1]\n\
+                           sunit=value|su=num,sectsize=num,lazy-count=0|1]\n\
 /* label */            [-L label (maximum 12 characters)]\n\
-/* naming */           [-n log=n|size=num,version=2|ci,ftype=0|1]\n\
+/* naming */           [-n size=num,version=2|ci,ftype=0|1]\n\
 /* no-op info only */  [-N]\n\
 /* prototype file */   [-p fname]\n\
 /* quiet */            [-q]\n\
 /* realtime subvol */  [-r extsize=num,size=num,rtdev=xxx]\n\
-/* sectorsize */       [-s log=n|size=num]\n\
+/* sectorsize */       [-s size=num]\n\
 /* version */          [-V]\n\
                        devicename\n\
 <devicename> is required unless -d name=xxx is given.\n\
@@ -908,7 +899,7 @@ illegal(
        const char      *value,
        const char      *opt)
 {
-       fprintf(stderr, _("Illegal value %s for -%s option\n"), value, opt);
+       fprintf(stderr, _("Invalid value %s for -%s option\n"), value, opt);
        usage();
 }
 
@@ -1021,7 +1012,6 @@ check_device_type(
        bool            no_size,
        bool            no_name,
        int             *create,
-       bool            force_overwrite,
        const char      *optname)
 {
        struct stat statbuf;
@@ -1052,13 +1042,6 @@ check_device_type(
                return;
        }
 
-       if (!force_overwrite && check_overwrite(name)) {
-               fprintf(stderr,
-       _("%s: Use the -f option to force overwrite.\n"),
-                       progname);
-               exit(1);
-       }
-
        /*
         * We only want to completely truncate and recreate an existing file if
         * we were specifically told it was a file. Set the create flag only in
@@ -1088,6 +1071,20 @@ check_device_type(
        usage();
 }
 
+static void
+validate_overwrite(
+       const char      *name,
+       bool            force_overwrite)
+{
+       if (!force_overwrite && check_overwrite(name)) {
+               fprintf(stderr,
+       _("%s: Use the -f option to force overwrite.\n"),
+                       progname);
+               exit(1);
+       }
+
+}
+
 static void
 validate_ag_geometry(
        int             blocklog,
@@ -1264,9 +1261,9 @@ illegal_option(
        const char              *reason)
 {
        fprintf(stderr,
-               _("Illegal value %s for -%c %s option. %s\n"),
+               _("Invalid value %s for -%c %s option. %s\n"),
                value, opts->name, opts->subopts[index],
-               reason ? reason : "");
+               reason);
        usage();
 }
 
@@ -1357,18 +1354,20 @@ getnum(
 
                c = strtoll(str, &str_end, 0);
                if (c == 0 && str_end == str)
-                       illegal_option(str, opts, index, NULL);
+                       illegal_option(str, opts, index,
+                                       _("Value not recognized as number."));
                if (*str_end != '\0')
-                       illegal_option(str, opts, index, NULL);
+                       illegal_option(str, opts, index,
+                                       _("Unit suffixes are not allowed."));
        }
 
        /* Validity check the result. */
        if (c < sp->minval)
-               illegal_option(str, opts, index, _("value is too small"));
+               illegal_option(str, opts, index, _("Value is too small."));
        else if (c > sp->maxval)
-               illegal_option(str, opts, index, _("value is too large"));
+               illegal_option(str, opts, index, _("Value is too large."));
        if (sp->is_power_2 && !ispow2(c))
-               illegal_option(str, opts, index, _("value must be a power of 2"));
+               illegal_option(str, opts, index, _("Value must be a power of 2."));
        return c;
 }
 
@@ -1401,7 +1400,7 @@ block_opts_parser(
 {
        switch (subopt) {
        case B_SIZE:
-               cli->blocksize = getnum(value, opts, B_SIZE);
+               cli->blocksize = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1418,52 +1417,52 @@ data_opts_parser(
 {
        switch (subopt) {
        case D_AGCOUNT:
-               cli->agcount = getnum(value, opts, D_AGCOUNT);
+               cli->agcount = getnum(value, opts, subopt);
                break;
        case D_AGSIZE:
-               cli->agsize = getstr(value, opts, D_AGSIZE);
+               cli->agsize = getstr(value, opts, subopt);
                break;
        case D_FILE:
-               cli->xi->disfile = getnum(value, opts, D_FILE);
+               cli->xi->disfile = getnum(value, opts, subopt);
                break;
        case D_NAME:
-               cli->xi->dname = getstr(value, opts, D_NAME);
+               cli->xi->dname = getstr(value, opts, subopt);
                break;
        case D_SIZE:
-               cli->dsize = getstr(value, opts, D_SIZE);
+               cli->dsize = getstr(value, opts, subopt);
                break;
        case D_SUNIT:
-               cli->dsunit = getnum(value, opts, D_SUNIT);
+               cli->dsunit = getnum(value, opts, subopt);
                break;
        case D_SWIDTH:
-               cli->dswidth = getnum(value, opts, D_SWIDTH);
+               cli->dswidth = getnum(value, opts, subopt);
                break;
        case D_SU:
-               cli->dsu = getstr(value, opts, D_SU);
+               cli->dsu = getstr(value, opts, subopt);
                break;
        case D_SW:
-               cli->dsw = getnum(value, opts, D_SW);
+               cli->dsw = getnum(value, opts, subopt);
                break;
        case D_NOALIGN:
-               cli->sb_feat.nodalign = getnum(value, opts, D_NOALIGN);
+               cli->sb_feat.nodalign = getnum(value, opts, subopt);
                break;
        case D_SECTSIZE:
-               cli->sectorsize = getnum(value, opts, D_SECTSIZE);
+               cli->sectorsize = getnum(value, opts, subopt);
                break;
        case D_RTINHERIT:
-               if (getnum(value, opts, D_RTINHERIT))
+               if (getnum(value, opts, subopt))
                        cli->fsx.fsx_xflags |= XFS_DIFLAG_RTINHERIT;
                break;
        case D_PROJINHERIT:
-               cli->fsx.fsx_projid = getnum(value, opts, D_PROJINHERIT);
+               cli->fsx.fsx_projid = getnum(value, opts, subopt);
                cli->fsx.fsx_xflags |= XFS_DIFLAG_PROJINHERIT;
                break;
        case D_EXTSZINHERIT:
-               cli->fsx.fsx_extsize = getnum(value, opts, D_EXTSZINHERIT);
+               cli->fsx.fsx_extsize = getnum(value, opts, subopt);
                cli->fsx.fsx_xflags |= XFS_DIFLAG_EXTSZINHERIT;
                break;
        case D_COWEXTSIZE:
-               cli->fsx.fsx_cowextsize = getnum(value, opts, D_COWEXTSIZE);
+               cli->fsx.fsx_cowextsize = getnum(value, opts, subopt);
                cli->fsx.fsx_xflags |= FS_XFLAG_COWEXTSIZE;
                break;
        default:
@@ -1481,25 +1480,25 @@ inode_opts_parser(
 {
        switch (subopt) {
        case I_ALIGN:
-               cli->sb_feat.inode_align = getnum(value, opts, I_ALIGN);
+               cli->sb_feat.inode_align = getnum(value, opts, subopt);
                break;
        case I_MAXPCT:
-               cli->imaxpct = getnum(value, opts, I_MAXPCT);
+               cli->imaxpct = getnum(value, opts, subopt);
                break;
        case I_PERBLOCK:
-               cli->inopblock = getnum(value, opts, I_PERBLOCK);
+               cli->inopblock = getnum(value, opts, subopt);
                break;
        case I_SIZE:
-               cli->inodesize = getnum(value, opts, I_SIZE);
+               cli->inodesize = getnum(value, opts, subopt);
                break;
        case I_ATTR:
-               cli->sb_feat.attr_version = getnum(value, opts, I_ATTR);
+               cli->sb_feat.attr_version = getnum(value, opts, subopt);
                break;
        case I_PROJID32BIT:
-               cli->sb_feat.projid32bit = getnum(value, opts, I_PROJID32BIT);
+               cli->sb_feat.projid32bit = getnum(value, opts, subopt);
                break;
        case I_SPINODES:
-               cli->sb_feat.spinodes = getnum(value, opts, I_SPINODES);
+               cli->sb_feat.spinodes = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1516,36 +1515,36 @@ log_opts_parser(
 {
        switch (subopt) {
        case L_AGNUM:
-               cli->logagno = getnum(value, opts, L_AGNUM);
+               cli->logagno = getnum(value, opts, subopt);
                break;
        case L_FILE:
-               cli->xi->lisfile = getnum(value, opts, L_FILE);
+               cli->xi->lisfile = getnum(value, opts, subopt);
                break;
        case L_INTERNAL:
-               cli->loginternal = getnum(value, opts, L_INTERNAL);
+               cli->loginternal = getnum(value, opts, subopt);
                break;
        case L_SU:
-               cli->lsu = getstr(value, opts, L_SU);
+               cli->lsu = getstr(value, opts, subopt);
                break;
        case L_SUNIT:
-               cli->lsunit = getnum(value, opts, L_SUNIT);
+               cli->lsunit = getnum(value, opts, subopt);
                break;
        case L_NAME:
        case L_DEV:
-               cli->xi->logname = getstr(value, opts, L_NAME);
+               cli->xi->logname = getstr(value, opts, subopt);
                cli->loginternal = 0;
                break;
        case L_VERSION:
-               cli->sb_feat.log_version = getnum(value, opts, L_VERSION);
+               cli->sb_feat.log_version = getnum(value, opts, subopt);
                break;
        case L_SIZE:
-               cli->logsize = getstr(value, opts, L_SIZE);
+               cli->logsize = getstr(value, opts, subopt);
                break;
        case L_SECTSIZE:
-               cli->lsectorsize = getnum(value, opts, L_SECTSIZE);
+               cli->lsectorsize = getnum(value, opts, subopt);
                break;
        case L_LAZYSBCNTR:
-               cli->sb_feat.lazy_sb_counters = getnum(value, opts, L_LAZYSBCNTR);
+               cli->sb_feat.lazy_sb_counters = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1562,24 +1561,24 @@ meta_opts_parser(
 {
        switch (subopt) {
        case M_CRC:
-               cli->sb_feat.crcs_enabled = getnum(value, opts, M_CRC);
+               cli->sb_feat.crcs_enabled = getnum(value, opts, subopt);
                if (cli->sb_feat.crcs_enabled)
                        cli->sb_feat.dirftype = true;
                break;
        case M_FINOBT:
-               cli->sb_feat.finobt = getnum(value, opts, M_FINOBT);
+               cli->sb_feat.finobt = getnum(value, opts, subopt);
                break;
        case M_UUID:
                if (!value || *value == '\0')
-                       reqval('m', opts->subopts, M_UUID);
+                       reqval('m', opts->subopts, subopt);
                if (platform_uuid_parse(value, &cli->uuid))
                        illegal(value, "m uuid");
                break;
        case M_RMAPBT:
-               cli->sb_feat.rmapbt = getnum(value, opts, M_RMAPBT);
+               cli->sb_feat.rmapbt = getnum(value, opts, subopt);
                break;
        case M_REFLINK:
-               cli->sb_feat.reflink = getnum(value, opts, M_REFLINK);
+               cli->sb_feat.reflink = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1596,19 +1595,19 @@ naming_opts_parser(
 {
        switch (subopt) {
        case N_SIZE:
-               cli->dirblocksize = getstr(value, opts, N_SIZE);
+               cli->dirblocksize = getstr(value, opts, subopt);
                break;
        case N_VERSION:
-               value = getstr(value, &nopts, N_VERSION);
+               value = getstr(value, &nopts, subopt);
                if (!strcasecmp(value, "ci")) {
                        /* ASCII CI mode */
                        cli->sb_feat.nci = true;
                } else {
-                       cli->sb_feat.dir_version = getnum(value, opts, N_VERSION);
+                       cli->sb_feat.dir_version = getnum(value, opts, subopt);
                }
                break;
        case N_FTYPE:
-               cli->sb_feat.dirftype = getnum(value, opts, N_FTYPE);
+               cli->sb_feat.dirftype = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1625,20 +1624,20 @@ rtdev_opts_parser(
 {
        switch (subopt) {
        case R_EXTSIZE:
-               cli->rtextsize = getstr(value, opts, R_EXTSIZE);
+               cli->rtextsize = getstr(value, opts, subopt);
                break;
        case R_FILE:
-               cli->xi->risfile = getnum(value, opts, R_FILE);
+               cli->xi->risfile = getnum(value, opts, subopt);
                break;
        case R_NAME:
        case R_DEV:
-               cli->xi->rtname = getstr(value, opts, R_NAME);
+               cli->xi->rtname = getstr(value, opts, subopt);
                break;
        case R_SIZE:
-               cli->rtsize = getstr(value, opts, R_SIZE);
+               cli->rtsize = getstr(value, opts, subopt);
                break;
        case R_NOALIGN:
-               cli->sb_feat.nortalign = getnum(value, opts, R_NOALIGN);
+               cli->sb_feat.nortalign = getnum(value, opts, subopt);
                break;
        default:
                return -EINVAL;
@@ -1665,10 +1664,13 @@ sector_opts_parser(
        return 0;
 }
 
-struct subopts {
+static struct subopts {
        char            opt;
        struct opt_params *opts;
-       int             (*parser)();
+       int             (*parser)(struct opt_params     *opts,
+                                 int                   subopt,
+                                 char                  *value,
+                                 struct cli_params     *cli);
 } subopt_tab[] = {
        { 'b', &bopts, block_opts_parser },
        { 'd', &dopts, data_opts_parser },
@@ -1738,18 +1740,15 @@ validate_sectorsize(
         * files or block devices and set the control parameters correctly.
         */
        check_device_type(dfile, &cli->xi->disfile, !cli->dsize, !dfile,
-                         dry_run ? NULL : &cli->xi->dcreat,
-                         force_overwrite, "d");
+                         dry_run ? NULL : &cli->xi->dcreat, "d");
        if (!cli->loginternal)
                check_device_type(cli->xi->logname, &cli->xi->lisfile,
                                  !cli->logsize, !cli->xi->logname,
-                                 dry_run ? NULL : &cli->xi->lcreat,
-                                 force_overwrite, "l");
+                                 dry_run ? NULL : &cli->xi->lcreat, "l");
        if (cli->xi->rtname)
                check_device_type(cli->xi->rtname, &cli->xi->risfile,
                                  !cli->rtsize, !cli->xi->rtname,
-                                 dry_run ? NULL : &cli->xi->rcreat,
-                                 force_overwrite, "r");
+                                 dry_run ? NULL : &cli->xi->rcreat, "r");
 
        /*
         * Explicitly disable direct IO for image files so we don't error out on
@@ -1934,28 +1933,28 @@ _("Minimum inode size for CRCs is %d bytes\n"),
                /* inodes always aligned */
                if (!cli->sb_feat.inode_align) {
                        fprintf(stderr,
-_("Inodes always aligned for CRC enabled filesytems\n"));
+_("Inodes always aligned for CRC enabled filesystems\n"));
                        usage();
                }
 
                /* lazy sb counters always on */
                if (!cli->sb_feat.lazy_sb_counters) {
                        fprintf(stderr,
-_("Lazy superblock counted always enabled for CRC enabled filesytems\n"));
+_("Lazy superblock counters always enabled for CRC enabled filesystems\n"));
                        usage();
                }
 
                /* version 2 logs always on */
                if (cli->sb_feat.log_version != 2) {
                        fprintf(stderr,
-_("V2 logs always enabled for CRC enabled filesytems\n"));
+_("V2 logs always enabled for CRC enabled filesystems\n"));
                        usage();
                }
 
                /* attr2 always on */
                if (cli->sb_feat.attr_version != 2) {
                        fprintf(stderr,
-_("V2 attribute format always enabled on CRC enabled filesytems\n"));
+_("V2 attribute format always enabled on CRC enabled filesystems\n"));
                        usage();
                }
 
@@ -1963,14 +1962,14 @@ _("V2 attribute format always enabled on CRC enabled filesytems\n"));
                /* attr2 always on */
                if (!cli->sb_feat.projid32bit) {
                        fprintf(stderr,
-_("32 bit Project IDs always enabled on CRC enabled filesytems\n"));
+_("32 bit Project IDs always enabled on CRC enabled filesystems\n"));
                        usage();
                }
 
                /* ftype always on */
                if (!cli->sb_feat.dirftype) {
                        fprintf(stderr,
-_("Directory ftype field always enabled on CRC enabled filesytems\n"));
+_("Directory ftype field always enabled on CRC enabled filesystems\n"));
                        usage();
                }
 
@@ -1991,7 +1990,7 @@ _("finobt not supported without CRC support\n"));
                }
                cli->sb_feat.finobt = false;
 
-               if (cli->sb_feat.spinodes) {
+               if (cli->sb_feat.spinodes && cli_opt_set(&iopts, I_SPINODES)) {
                        fprintf(stderr,
 _("sparse inodes not supported without CRC support\n"));
                        usage();
@@ -2020,6 +2019,13 @@ _("cowextsize not supported without reflink support\n"));
                usage();
        }
 
+       if (cli->sb_feat.reflink && cli->xi->rtname) {
+               fprintf(stderr,
+_("reflink not supported with realtime devices\n"));
+               usage();
+               cli->sb_feat.reflink = false;
+       }
+
        if (cli->sb_feat.rmapbt && cli->xi->rtname) {
                fprintf(stderr,
 _("rmapbt not supported with realtime devices\n"));
@@ -2097,7 +2103,7 @@ validate_inodesize(
                int     maxsz;
 
                fprintf(stderr, _("illegal inode size %d\n"), cfg->inodesize);
-               maxsz = MIN(cfg->blocksize / XFS_MIN_INODE_PERBLOCK,
+               maxsz = min(cfg->blocksize / XFS_MIN_INODE_PERBLOCK,
                            XFS_DINODE_MAX_SIZE);
                if (XFS_DINODE_MIN_SIZE == maxsz)
                        fprintf(stderr,
@@ -2208,6 +2214,7 @@ calc_stripe_factors(
        struct cli_params       *cli,
        struct fs_topology      *ft)
 {
+       long long int   big_dswidth;
        int             dsunit = 0;
        int             dswidth = 0;
        int             lsunit = 0;
@@ -2227,7 +2234,7 @@ calc_stripe_factors(
                dsw = cli->dsw;
 
        /* data sunit/swidth options */
-       if ((dsunit && !dswidth) || (!dsunit && dswidth)) {
+       if (cli_opt_set(&dopts, D_SUNIT) != cli_opt_set(&dopts, D_SWIDTH)) {
                fprintf(stderr,
 _("both data sunit and data swidth options must be specified\n"));
                usage();
@@ -2235,7 +2242,7 @@ _("both data sunit and data swidth options must be specified\n"));
 
        /* convert dsu/dsw to dsunit/dswidth and use them from now on */
        if (dsu || dsw) {
-               if ((dsu && !dsw) || (!dsu && dsw)) {
+               if (cli_opt_set(&dopts, D_SU) != cli_opt_set(&dopts, D_SW)) {
                        fprintf(stderr,
 _("both data su and data sw options must be specified\n"));
                        usage();
@@ -2248,10 +2255,18 @@ _("data su must be a multiple of the sector size (%d)\n"), cfg->sectorsize);
                }
 
                dsunit  = (int)BTOBBT(dsu);
-               dswidth = dsunit * dsw;
+               big_dswidth = (long long int)dsunit * dsw;
+               if (big_dswidth > INT_MAX) {
+                       fprintf(stderr,
+_("data stripe width (%lld) is too large of a multiple of the data stripe unit (%d)\n"),
+                               big_dswidth, dsunit);
+                       usage();
+               }
+               dswidth = big_dswidth;
        }
 
-       if (dsunit && (dswidth % dsunit != 0)) {
+       if ((dsunit && !dswidth) || (!dsunit && dswidth) ||
+           (dsunit && (dswidth % dsunit != 0))) {
                fprintf(stderr,
 _("data stripe width (%d) must be a multiple of the data stripe unit (%d)\n"),
                        dswidth, dsunit);
@@ -2272,11 +2287,20 @@ _("data stripe width (%d) must be a multiple of the data stripe unit (%d)\n"),
 
        /* if no stripe config set, use the device default */
        if (!dsunit) {
-               dsunit = ft->dsunit;
-               dswidth = ft->dswidth;
-               use_dev = true;
+               /* Ignore nonsense from device.  XXX add more validation */
+               if (ft->dsunit && ft->dswidth == 0) {
+                       fprintf(stderr,
+_("%s: Volume reports stripe unit of %d bytes and stripe width of 0, ignoring.\n"),
+                               progname, BBTOB(ft->dsunit));
+                       ft->dsunit = 0;
+                       ft->dswidth = 0;
+               } else {
+                       dsunit = ft->dsunit;
+                       dswidth = ft->dswidth;
+                       use_dev = true;
+               }
        } else {
-               /* check and warn is alignment is sub-optimal */
+               /* check and warn if user-specified alignment is sub-optimal */
                if (ft->dsunit && ft->dsunit != dsunit) {
                        fprintf(stderr,
 _("%s: Specified data stripe unit %d is not the same as the volume stripe unit %d\n"),
@@ -2371,8 +2395,7 @@ _("log stripe unit (%d bytes) is too large (maximum is 256KiB)\n"
 static void
 open_devices(
        struct mkfs_params      *cfg,
-       struct libxfs_xinit     *xi,
-       bool                    discard)
+       struct libxfs_xinit     *xi)
 {
        uint64_t                sector_mask;
 
@@ -2397,14 +2420,19 @@ open_devices(
         * So, we reduce the size (in basic blocks) to a perfect
         * multiple of the sector size, or 1024, whichever is larger.
         */
-       sector_mask = (uint64_t)-1 << (MAX(cfg->sectorlog, 10) - BBSHIFT);
+       sector_mask = (uint64_t)-1 << (max(cfg->sectorlog, 10) - BBSHIFT);
        xi->dsize &= sector_mask;
        xi->rtsize &= sector_mask;
-       xi->logBBsize &= (uint64_t)-1 << (MAX(cfg->lsectorlog, 10) - BBSHIFT);
-
+       xi->logBBsize &= (uint64_t)-1 << (max(cfg->lsectorlog, 10) - BBSHIFT);
+}
 
-       if (!discard)
-               return;
+static void
+discard_devices(
+       struct libxfs_xinit     *xi)
+{
+       /*
+        * This function has to be called after libxfs has been initialized.
+        */
 
        if (!xi->disfile)
                discard_blocks(xi->ddev, xi->dsize);
@@ -2943,7 +2971,7 @@ _("Due to stripe alignment, the internal log size (%lld) is too large.\n"
        }
 }
 
-void
+static void
 validate_log_size(uint64_t logblocks, int blocklog, int min_logblocks)
 {
        if (logblocks < min_logblocks) {
@@ -2984,12 +3012,12 @@ calculate_log_size(
        libxfs_umount(&mount);
 
        ASSERT(min_logblocks);
-       min_logblocks = MAX(XFS_MIN_LOG_BLOCKS, min_logblocks);
+       min_logblocks = max(XFS_MIN_LOG_BLOCKS, min_logblocks);
 
        /* if we have lots of blocks, check against XFS_MIN_LOG_BYTES, too */
        if (!cli->logsize &&
            cfg->dblocks >= (1024*1024*1024) >> cfg->blocklog)
-               min_logblocks = MAX(min_logblocks,
+               min_logblocks = max(min_logblocks,
                                    XFS_MIN_LOG_BYTES >> cfg->blocklog);
 
        /*
@@ -3025,7 +3053,7 @@ _("external log device %lld too small, must be at least %lld blocks\n"),
                         * XFS_MIN_LOG_BYTES for filesystems smaller than 16G if
                         * at all possible, ramping up to 128MB at 256GB.
                         */
-                       cfg->logblocks = MIN(XFS_MIN_LOG_BYTES >> cfg->blocklog,
+                       cfg->logblocks = min(XFS_MIN_LOG_BYTES >> cfg->blocklog,
                                        min_logblocks * XFS_DFL_LOG_FACTOR);
                } else {
                        /*
@@ -3039,7 +3067,7 @@ _("external log device %lld too small, must be at least %lld blocks\n"),
                }
 
                /* Ensure the chosen size meets minimum log size requirements */
-               cfg->logblocks = MAX(min_logblocks, cfg->logblocks);
+               cfg->logblocks = max(min_logblocks, cfg->logblocks);
 
                /*
                 * Make sure the log fits wholly within an AG
@@ -3051,11 +3079,11 @@ _("external log device %lld too small, must be at least %lld blocks\n"),
                 * opened to decide what is the valid maximum size of a log in
                 * an AG.
                 */
-               cfg->logblocks = MIN(cfg->logblocks,
+               cfg->logblocks = min(cfg->logblocks,
                                     libxfs_alloc_ag_max_usable(mp) - 1);
 
                /* and now clamp the size to the maximum supported size */
-               cfg->logblocks = MIN(cfg->logblocks, XFS_MAX_LOG_BLOCKS);
+               cfg->logblocks = min(cfg->logblocks, XFS_MAX_LOG_BLOCKS);
                if ((cfg->logblocks << cfg->blocklog) > XFS_MAX_LOG_BYTES)
                        cfg->logblocks = XFS_MAX_LOG_BYTES >> cfg->blocklog;
 
@@ -3152,40 +3180,6 @@ initialise_mount(
        mp->m_sectbb_log = sbp->sb_sectlog - BBSHIFT;
 }
 
-static void
-print_mkfs_cfg(
-       struct mkfs_params      *cfg,
-       char                    *dfile,
-       char                    *logfile,
-       char                    *rtfile)
-{
-       struct sb_feat_args     *fp = &cfg->sb_feat;
-
-       printf(_(
-"meta-data=%-22s isize=%-6d agcount=%lld, agsize=%lld blks\n"
-"         =%-22s sectsz=%-5u attr=%u, projid32bit=%u\n"
-"         =%-22s crc=%-8u finobt=%u, sparse=%u, rmapbt=%u, reflink=%u\n"
-"data     =%-22s bsize=%-6u blocks=%llu, imaxpct=%u\n"
-"         =%-22s sunit=%-6u swidth=%u blks\n"
-"naming   =version %-14u bsize=%-6u ascii-ci=%d ftype=%d\n"
-"log      =%-22s bsize=%-6d blocks=%lld, version=%d\n"
-"         =%-22s sectsz=%-5u sunit=%d blks, lazy-count=%d\n"
-"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->projid32bit,
-               "", fp->crcs_enabled, fp->finobt, fp->spinodes, fp->rmapbt,
-                       fp->reflink,
-               "", cfg->blocksize, (long long)cfg->dblocks, cfg->imaxpct,
-               "", cfg->dsunit, cfg->dswidth,
-               fp->dir_version, cfg->dirblocksize, fp->nci, fp->dirftype,
-               logfile, cfg->blocksize, (long long)cfg->logblocks,
-                       fp->log_version,
-               "", cfg->lsectorsize, cfg->lsunit, fp->lazy_sb_counters,
-               rtfile, (int)cfg->rtextblocks << cfg->blocklog,
-                       (long long)cfg->rtblocks, (long long)cfg->rtextents);
-}
-
 /*
  * Format everything from the generated config into the superblock that
  * will be used to initialise the on-disk superblock. This is the in-memory
@@ -3279,7 +3273,7 @@ prepare_devices(
         */
        buf = libxfs_getbuf(mp->m_ddev_targp, (xi->dsize - whack_blks),
                            whack_blks);
-       memset(XFS_BUF_PTR(buf), 0, WHACK_SIZE);
+       memset(buf->b_addr, 0, WHACK_SIZE);
        libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE);
        libxfs_purgebuf(buf);
 
@@ -3290,15 +3284,15 @@ prepare_devices(
         * ext[2,3] and reiserfs (64k) - and hopefully all else.
         */
        buf = libxfs_getbuf(mp->m_ddev_targp, 0, whack_blks);
-       memset(XFS_BUF_PTR(buf), 0, WHACK_SIZE);
+       memset(buf->b_addr, 0, WHACK_SIZE);
        libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE);
        libxfs_purgebuf(buf);
 
        /* OK, now write the superblock... */
        buf = libxfs_getbuf(mp->m_ddev_targp, XFS_SB_DADDR, XFS_FSS_TO_BB(mp, 1));
        buf->b_ops = &xfs_sb_buf_ops;
-       memset(XFS_BUF_PTR(buf), 0, cfg->sectorsize);
-       libxfs_sb_to_disk((void *)XFS_BUF_PTR(buf), sbp);
+       memset(buf->b_addr, 0, cfg->sectorsize);
+       libxfs_sb_to_disk(buf->b_addr, sbp);
        libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE);
        libxfs_purgebuf(buf);
 
@@ -3318,7 +3312,7 @@ prepare_devices(
                buf = libxfs_getbuf(mp->m_rtdev_targp,
                                    XFS_FSB_TO_BB(mp, cfg->rtblocks - 1LL),
                                    BTOBB(cfg->blocksize));
-               memset(XFS_BUF_PTR(buf), 0, cfg->blocksize);
+               memset(buf->b_addr, 0, cfg->blocksize);
                libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE);
                libxfs_purgebuf(buf);
        }
@@ -3362,8 +3356,8 @@ initialise_ag_headers(
                        XFS_AG_DADDR(mp, agno, XFS_SB_DADDR),
                        XFS_FSS_TO_BB(mp, 1));
        buf->b_ops = &xfs_sb_buf_ops;
-       memset(XFS_BUF_PTR(buf), 0, cfg->sectorsize);
-       libxfs_sb_to_disk((void *)XFS_BUF_PTR(buf), sbp);
+       memset(buf->b_addr, 0, cfg->sectorsize);
+       libxfs_sb_to_disk(buf->b_addr, sbp);
        libxfs_writebuf(buf, LIBXFS_EXIT_ON_FAILURE);
 
        /*
@@ -3401,7 +3395,7 @@ initialise_ag_headers(
        }
 
        agf->agf_flfirst = 0;
-       agf->agf_fllast = cpu_to_be32(XFS_AGFL_SIZE(mp) - 1);
+       agf->agf_fllast = cpu_to_be32(libxfs_agfl_size(mp) - 1);
        agf->agf_flcount = 0;
        agblocks = (xfs_agblock_t)(agsize - libxfs_prealloc_blocks(mp));
        agf->agf_freeblks = cpu_to_be32(agblocks);
@@ -3433,7 +3427,7 @@ initialise_ag_headers(
                agfl->agfl_magicnum = cpu_to_be32(XFS_AGFL_MAGIC);
                agfl->agfl_seqno = cpu_to_be32(agno);
                platform_uuid_copy(&agfl->agfl_uuid, &sbp->sb_uuid);
-               for (bucket = 0; bucket < XFS_AGFL_SIZE(mp); bucket++)
+               for (bucket = 0; bucket < libxfs_agfl_size(mp); bucket++)
                        agfl->agfl_bno[bucket] = cpu_to_be32(NULLAGBLOCK);
        }
 
@@ -3690,10 +3684,9 @@ initialise_ag_freespace(
 {
        struct xfs_alloc_arg    args;
        struct xfs_trans        *tp;
-       struct xfs_trans_res tres = {0};
        int                     c;
 
-       c = libxfs_trans_alloc(mp, &tres, worst_freelist, 0, 0, &tp);
+       c = -libxfs_trans_alloc_rollable(mp, worst_freelist, &tp);
        if (c)
                res_failed(c);
 
@@ -3706,7 +3699,9 @@ initialise_ag_freespace(
 
        libxfs_alloc_fix_freelist(&args, 0);
        libxfs_perag_put(args.pag);
-       libxfs_trans_commit(tp);
+       c = -libxfs_trans_commit(tp);
+       if (c)
+               res_failed(c);
 }
 
 /*
@@ -3791,7 +3786,7 @@ main(
                        .crcs_enabled = true,
                        .dirftype = true,
                        .finobt = true,
-                       .spinodes = false,
+                       .spinodes = true,
                        .rmapbt = false,
                        .reflink = false,
                        .parent_pointers = false,
@@ -3917,7 +3912,8 @@ main(
        /*
         * Open and validate the device configurations
         */
-       open_devices(&cfg, &xi, (discard && !dry_run));
+       open_devices(&cfg, &xi);
+       validate_overwrite(dfile, force_overwrite);
        validate_datadev(&cfg, &cli);
        validate_logdev(&cfg, &cli, &logfile);
        validate_rtdev(&cfg, &cli, &rtfile);
@@ -3947,12 +3943,32 @@ main(
         */
        calculate_log_size(&cfg, &cli, mp);
 
+       finish_superblock_setup(&cfg, mp, sbp);
+
+       /* Print the intended geometry of the fs. */
        if (!quiet || dry_run) {
-               print_mkfs_cfg(&cfg, dfile, logfile, rtfile);
+               struct xfs_fsop_geom    geo;
+               int                     error;
+
+               error = -libxfs_fs_geometry(sbp, &geo,
+                               XFS_FS_GEOM_MAX_STRUCT_VER);
+               if (error) {
+                       fprintf(stderr,
+       _("%s: failed to generate filesystem geometry\n"),
+                               progname);
+                       exit(1);
+               }
+
+               xfs_report_geom(&geo, dfile, logfile, rtfile);
                if (dry_run)
                        exit(0);
        }
-       finish_superblock_setup(&cfg, mp, sbp);
+
+       /*
+        * All values have been validated, discard the old device layout.
+        */
+       if (discard && !dry_run)
+               discard_devices(&xi);
 
        /*
         * we need the libxfs buffer cache from here on in.
@@ -4025,6 +4041,7 @@ main(
        if (xi.logdev && xi.logdev != xi.ddev)
                libxfs_device_close(xi.logdev);
        libxfs_device_close(xi.ddev);
+       libxfs_destroy();
 
        return 0;
 }