Re: [PATCH 09/19] mkfs: use getnum_checked for all ranged parameters

[Jan Tulak <jtulak@xxxxxxxxxx>]

On Thu, Apr 7, 2016 at 9:02 PM, Eric Sandeen <sandeen@xxxxxxxxxxx> wrote:

On 3/24/16 6:15 AM, jtulak@xxxxxxxxxx wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> Now that getnum_checked can handle min/max checking, use this for
> all parameters that take straight numbers and don't require unit
> conversions.
>
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> Signed-off-by: Jan Tulak <jtulak@xxxxxxxxxx>

Looks ok.

Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>

âOr Reviewed-by? :-)

âThanks,

Janâ

â

Â

> ---
>Â include/xfs_multidisk.h |Â Â5 +-
> mkfs/xfs_mkfs.c    Â| 148 ++++++++++++++++++++++++------------------------
>Â 2 files changed, 76 insertions(+), 77 deletions(-)
>
> diff --git a/include/xfs_multidisk.h b/include/xfs_multidisk.h
> index af35100..8e81d90 100644
> --- a/include/xfs_multidisk.h
> +++ b/include/xfs_multidisk.h
> @@ -42,8 +42,9 @@
>
>Â #define XFS_AG_BYTES(bblog)Â ((long long)BBSIZE << (bblog))
> #define   XFS_AG_MIN_BYTES    ((XFS_AG_BYTES(15)))  /* 16 MB */
> -#define XFS_AG_MIN_BLOCKS(blog)Â Â Â ((XFS_AG_BYTES(15)) >> (blog))
> -#define XFS_AG_MAX_BLOCKS(blog)Â Â Â ((XFS_AG_BYTES(31) - 1) >> (blog))
> +#define   XFS_AG_MAX_BYTES    ((XFS_AG_BYTES(31)))  /* 1 TB */
> +#define XFS_AG_MIN_BLOCKS(blog)Â Â Â (XFS_AG_MIN_BYTES >> (blog))
> +#define XFS_AG_MAX_BLOCKS(blog)Â Â Â ((XFS_AG_MAX_BYTES - 1) >> (blog))
>
>Â #define XFS_MAX_AGNUMBERÂ Â Â((xfs_agnumber_t)(NULLAGNUMBER - 1))
>
> diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> index 0e2cfac..021d682 100644
> --- a/mkfs/xfs_mkfs.c
> +++ b/mkfs/xfs_mkfs.c
> @@ -163,6 +163,8 @@ struct opt_params dopts = {
>Â Â Â Â},
>Â Â Â Â.subopt_params = {
>Â Â Â Â Â Â Â Â{ .index = D_AGCOUNT,
> +Â Â Â Â Â Â Â Â.minval = 1,
> +Â Â Â Â Â Â Â Â.maxval = XFS_MAX_AGNUMBER,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_FILE,
> @@ -177,18 +179,26 @@ struct opt_params dopts = {
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_SUNIT,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_SWIDTH,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_AGSIZE,
> +Â Â Â Â Â Â Â Â.minval = XFS_AG_MIN_BYTES,
> +Â Â Â Â Â Â Â Â.maxval = XFS_AG_MAX_BYTES,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_SU,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_SW,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_SECTLOG,
> @@ -207,12 +217,18 @@ struct opt_params dopts = {
>Â Â Â Â Â Â Â Â Â.defaultval = 1,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_RTINHERIT,
> -Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
> +Â Â Â Â Â Â Â Â.minval = 1,
> +Â Â Â Â Â Â Â Â.maxval = 1,
> +Â Â Â Â Â Â Â Â.defaultval = 1,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_PROJINHERIT,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = D_EXTSZINHERIT,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â},
> @@ -252,15 +268,23 @@ struct opt_params iopts = {
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = I_MAXPCT,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = 100,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = I_PERBLOCK,
> +Â Â Â Â Â Â Â Â.minval = XFS_MIN_INODE_PERBLOCK,
> +Â Â Â Â Â Â Â Â.maxval = XFS_MAX_BLOCKSIZE / XFS_DINODE_MIN_SIZE,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = I_SIZE,
> +Â Â Â Â Â Â Â Â.minval = XFS_DINODE_MIN_SIZE,
> +Â Â Â Â Â Â Â Â.maxval = XFS_DINODE_MAX_SIZE,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = I_ATTR,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = 2,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = I_PROJID32BIT,
> @@ -307,6 +331,8 @@ struct opt_params lopts = {
>Â Â Â Â},
>Â Â Â Â.subopt_params = {
>Â Â Â Â Â Â Â Â{ .index = L_AGNUM,
> +Â Â Â Â Â Â Â Â.minval = 0,
> +Â Â Â Â Â Â Â Â.maxval = UINT_MAX,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = L_INTERNAL,
> @@ -318,9 +344,13 @@ struct opt_params lopts = {
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = L_VERSION,
> +Â Â Â Â Â Â Â Â.minval = 1,
> +Â Â Â Â Â Â Â Â.maxval = 2,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = L_SUNIT,
> +Â Â Â Â Â Â Â Â.minval = BTOBB(XLOG_MIN_RECORD_BSIZE),
> +Â Â Â Â Â Â Â Â.maxval = BTOBB(XLOG_MAX_RECORD_BSIZE),
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = L_SU,
> @@ -380,6 +410,8 @@ struct opt_params nopts = {
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = N_VERSION,
> +Â Â Â Â Â Â Â Â.minval = 2,
> +Â Â Â Â Â Â Â Â.maxval = 2,
>Â Â Â Â Â Â Â Â Â.defaultval = SUBOPT_NEEDS_VAL,
>Â Â Â Â Â Â Â Â},
>Â Â Â Â Â Â Â Â{ .index = N_FTYPE,
> @@ -1560,13 +1592,11 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âswitch (getsubopt(&p, (constpp)subopts,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â&value)) {
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_AGCOUNT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_AGCOUNT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (daflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('d', subopts, D_AGCOUNT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âagcount = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif ((__int64_t)agcount <= 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "d agcount");
> +
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âagcount = getnum_checked(value, &dopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_AGCOUNT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdaflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_AGSIZE:
> @@ -1601,28 +1631,22 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdsize = value;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_SUNIT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_SUNIT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dsunit)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('d', subopts, D_SUNIT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (nodsflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('d', subopts, D_NOALIGN,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SUNIT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdsunit = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dsunit < 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "d sunit");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdsunit = getnum_checked(value, &dopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SUNIT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_SWIDTH:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_SWIDTH);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dswidth)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('d', subopts, D_SWIDTH);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (nodsflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('d', subopts, D_NOALIGN,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SWIDTH);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdswidth = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dswidth < 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "d swidth");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdswidth = getnum_checked(value, &dopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SWIDTH);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_SU:
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> @@ -1638,16 +1662,13 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "d su");
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_SW:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_SW);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dsw)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('d', subopts, D_SW);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (nodsflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('d', subopts, D_NOALIGN,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SW);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdsw = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (dsw < 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "d sw");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdsw = getnum_checked(value, &dopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_SW);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_NOALIGN:
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ânodsflag = getnum_checked(value,
> @@ -1696,21 +1717,22 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âssflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_RTINHERIT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |= \
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂXFS_DIFLAG_RTINHERIT;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âc = getnum_checked(value, &dopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â D_RTINHERIT);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (c)
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |=
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂXFS_DIFLAG_RTINHERIT;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_PROJINHERIT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_PROJINHERIT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_projid = atoi(value);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |= \
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_projid = getnum_checked(value,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â&dopts, D_PROJINHERIT);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |=
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂXFS_DIFLAG_PROJINHERIT;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase D_EXTSZINHERIT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('d', subopts, D_EXTSZINHERIT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_extsize = atoi(value);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |= \
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_extsize = getnum_checked(value,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â&dopts, D_EXTSZINHERIT);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âfsx.fsx_xflags |=
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂXFS_DIFLAG_EXTSZINHERIT;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdefault:
> @@ -1745,18 +1767,13 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âilflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase I_MAXPCT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('i', subopts, I_MAXPCT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (imflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('i', subopts, I_MAXPCT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âimaxpct = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (imaxpct < 0 || imaxpct > 100)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "i maxpct");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âimaxpct = getnum_checked(
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âvalue, &iopts, I_MAXPCT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âimflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase I_PERBLOCK:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('i', subopts, I_PERBLOCK);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (ilflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('i', subopts, I_LOG,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_PERBLOCK);
> @@ -1765,16 +1782,13 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (isflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('i', subopts, I_SIZE,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_PERBLOCK);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âinopblock = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (inopblock <
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂXFS_MIN_INODE_PERBLOCK ||
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â!ispow2(inopblock))
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âinopblock = getnum_checked(value, &iopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_PERBLOCK);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!ispow2(inopblock))
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "i perblock");
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âipflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase I_SIZE:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('i', subopts, I_SIZE);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (ilflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('i', subopts, I_LOG,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_SIZE);
> @@ -1783,19 +1797,16 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_SIZE);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (isflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('i', subopts, I_SIZE);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âisize = getnum(value, 0, 0, true);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (isize <= 0 || !ispow2(isize))
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âisize = getnum_checked(value, &iopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â I_SIZE);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!ispow2(isize))
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "i size");
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âinodelog = libxfs_highbit32(isize);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âisflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase I_ATTR:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('i', subopts, I_ATTR);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âc = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (c < 0 || c > 2)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "i attr");
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.attr_version = c;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.attr_version = getnum_checked(
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âvalue, &iopts, I_ATTR);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase I_PROJID32BIT:
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.projid16bit =
> @@ -1817,21 +1828,16 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Âwhile (*p != '\0') {
>               Âchar  **subopts = (char **)lopts.subopts;
>               Âchar  *value;
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlong long tmp_num;
>
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âswitch (getsubopt(&p, (constpp)subopts,
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â&value)) {
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_AGNUM:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('l', subopts, L_AGNUM);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (laflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('l', subopts, L_AGNUM);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (ldflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('l', subopts, L_AGNUM, L_DEV);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âtmp_num = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (tmp_num < 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "l agno");
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlogagno = (xfs_agnumber_t)tmp_num;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlogagno = getnum_checked(value, &lopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â L_AGNUM);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlaflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_FILE:
> @@ -1868,13 +1874,10 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlsuflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_SUNIT:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('l', subopts, L_SUNIT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (lsunit)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('l', subopts, L_SUNIT);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlsunit = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (lsunit < 0)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "l sunit");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlsunit = getnum_checked(value, &lopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â L_SUNIT);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlsunitflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_NAME:
> @@ -1893,14 +1896,10 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âxi.logname = value;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_VERSION:
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (!value || *value == '\0')
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreqval('l', subopts, L_VERSION);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (lvflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('l', subopts, L_VERSION);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âc = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (c < 1 || c > 2)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "l version");
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.log_version = c;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.log_version = getnum_checked(
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âvalue, &lopts, L_VERSION);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlvflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_SIZE:
> @@ -2035,11 +2034,10 @@ _("cannot specify both -m crc=1 and -n ftype\n"));
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â/* ASCII CI mode */
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.nci = true;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â} else {
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âc = getnum(value, 0, 0, false);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (c != 2)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value,
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"n version");
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.dir_version = c;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsb_feat.dir_version =
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âgetnum_checked(value,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â&nopts,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂN_VERSION);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â}
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ânvflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs

--

Jan Tulak

jtulak@xxxxxxxxxxÂ/ jan@xxxxxxxx