xfs
[Top] [All Lists]

Re: [PATCH] mkfs: add missing noalign suboption information

To: Eric Sandeen <sandeen@xxxxxxxxxxx>
Subject: Re: [PATCH] mkfs: add missing noalign suboption information
From: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
Date: Wed, 10 Jul 2013 10:29:46 -0300
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <51DB52E1.8030508@xxxxxxxxxxx>
References: <1373327110-5533-1-git-send-email-cmaiolino@xxxxxxxxxx> <51DB52E1.8030508@xxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Mon, Jul 08, 2013 at 07:01:37PM -0500, Eric Sandeen wrote:
> On 7/8/13 6:45 PM, Carlos Maiolino wrote:
> > Adds to the mkfs man page, information about noalign mkfs option which is
> > missing until now.
> > 
> > Signed-off-by: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
> > ---
> >  man/man8/mkfs.xfs.8 | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/man/man8/mkfs.xfs.8 b/man/man8/mkfs.xfs.8
> > index 04c19b4..2435814 100644
> > --- a/man/man8/mkfs.xfs.8
> > +++ b/man/man8/mkfs.xfs.8
> > @@ -244,6 +244,12 @@ will automatically query the logical volume for 
> > appropriate
> >  and
> >  .B swidth
> >  values.
> > +.TP
> > +.BI noalign
> > +This option is used to bypass data alignment configuration. In rare
> > +circunstances, data alignment might decrease performance, for such cases,
> > +use this option to avoid data alignment, this will set stripe unit and
> > +stripe width to 0.
> 
> Thanks for this, Carlos.
> 
> I think it might be better to just state plainly that it overrides automatic
> stripe unit detection, and leave the tuning / performance parts out.
> Most of the manpage today is more "what" than "why."
> 
> So maybe simply:
> 
> +This option disables automatic stripe geometry detection, and creates
> +the filesystem without stripe geometry even if the underlying storage
> +device reports this information.
> 
> FWIW, there is a 'noalign' option for the realtime subvolume
> as well, and that should probably get documented too.
> 

Agreed, going to change and re-send it.

> -Eric
> 
> >  .RE
> >  .TP
> >  .B \-f
> > 
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs

-- 
Carlos

<Prev in Thread] Current Thread [Next in Thread>