* [PATCH v2] mkfs: add noalign option to usage() @ 2013-08-29 16:03 Carlos Maiolino 2013-08-29 16:34 ` Eric Sandeen 0 siblings, 1 reply; 3+ messages in thread From: Carlos Maiolino @ 2013-08-29 16:03 UTC (permalink / raw) To: xfs Although it has been added to manpage, there is no information about the existence of noalign option into the usage(). Changelog: V2: Remove space in comma separated options Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com> --- mkfs/xfs_mkfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c index 4bdacee..ef1d758 100644 --- a/mkfs/xfs_mkfs.c +++ b/mkfs/xfs_mkfs.c @@ -3033,7 +3033,7 @@ usage( void ) /* metadata */ [-m crc=[0|1]\n\ /* data subvol */ [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\ (sunit=value,swidth=value|su=num,sw=num),\n\ - sectlog=n|sectsize=num\n\ + noalign,sectlog=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]\n\ -- 1.8.1.4 _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] mkfs: add noalign option to usage() 2013-08-29 16:03 [PATCH v2] mkfs: add noalign option to usage() Carlos Maiolino @ 2013-08-29 16:34 ` Eric Sandeen 2013-08-30 17:03 ` Carlos Maiolino 0 siblings, 1 reply; 3+ messages in thread From: Eric Sandeen @ 2013-08-29 16:34 UTC (permalink / raw) To: Carlos Maiolino; +Cc: xfs On 8/29/13 11:03 AM, Carlos Maiolino wrote: > Although it has been added to manpage, there is no information about the > existence of noalign option into the usage(). > > Changelog: > > V2: Remove space in comma separated options > > Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com> > --- > mkfs/xfs_mkfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c > index 4bdacee..ef1d758 100644 > --- a/mkfs/xfs_mkfs.c > +++ b/mkfs/xfs_mkfs.c > @@ -3033,7 +3033,7 @@ usage( void ) > /* metadata */ [-m crc=[0|1]\n\ > /* data subvol */ [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\ > (sunit=value,swidth=value|su=num,sw=num),\n\ > - sectlog=n|sectsize=num\n\ > + noalign,sectlog=n|sectsize=num\n\ I'd really rather see it as I suggested, for clarity. su/sw vs. sunit/swidth vs. noalign are mutually exclusive. su/sw & sunit/swidth are shown to be so via the (A|B) notation. Adding noalign outside the (A|B) makes it look like it can be specified in addition to alignment options, which really makes no sense. Can you do: - (sunit=value,swidth=value|su=num,sw=num),\n\ + (sunit=value,swidth=value|su=num,sw=num|noalign),\n\ instead? -Eric > /* force overwrite */ [-f]\n\ > /* inode size */ [-i log=n|perblock=n|size=num,maxpct=n,attr=0|1|2,\n\ > projid32bit=0|1]\n\ > _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] mkfs: add noalign option to usage() 2013-08-29 16:34 ` Eric Sandeen @ 2013-08-30 17:03 ` Carlos Maiolino 0 siblings, 0 replies; 3+ messages in thread From: Carlos Maiolino @ 2013-08-30 17:03 UTC (permalink / raw) To: Eric Sandeen; +Cc: xfs On Thu, Aug 29, 2013 at 11:34:54AM -0500, Eric Sandeen wrote: > On 8/29/13 11:03 AM, Carlos Maiolino wrote: > > Although it has been added to manpage, there is no information about the > > existence of noalign option into the usage(). > > > > Changelog: > > > > V2: Remove space in comma separated options > > > > Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com> > > --- > > mkfs/xfs_mkfs.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c > > index 4bdacee..ef1d758 100644 > > --- a/mkfs/xfs_mkfs.c > > +++ b/mkfs/xfs_mkfs.c > > @@ -3033,7 +3033,7 @@ usage( void ) > > /* metadata */ [-m crc=[0|1]\n\ > > /* data subvol */ [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\ > > (sunit=value,swidth=value|su=num,sw=num),\n\ > > - sectlog=n|sectsize=num\n\ > > + noalign,sectlog=n|sectsize=num\n\ > > I'd really rather see it as I suggested, for clarity. > > su/sw vs. sunit/swidth vs. noalign are mutually exclusive. > > su/sw & sunit/swidth are shown to be so via the (A|B) notation. > > Adding noalign outside the (A|B) makes it look like it can be specified in > addition to alignment options, which really makes no sense. > > Can you do: > > - (sunit=value,swidth=value|su=num,sw=num),\n\ > + (sunit=value,swidth=value|su=num,sw=num|noalign),\n\ > > instead? Sure, didn't think about mutual exclusive options being packed together, will send a v3 > > -Eric > > > /* force overwrite */ [-f]\n\ > > /* inode size */ [-i log=n|perblock=n|size=num,maxpct=n,attr=0|1|2,\n\ > > projid32bit=0|1]\n\ > > > -- Carlos _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-08-30 17:03 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-08-29 16:03 [PATCH v2] mkfs: add noalign option to usage() Carlos Maiolino 2013-08-29 16:34 ` Eric Sandeen 2013-08-30 17:03 ` Carlos Maiolino
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox