xfs
[Top] [All Lists]

Re: [PATCH] xfs: Remove the description about nodelaylog mount option fr

To: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>
Subject: Re: [PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.
From: Ben Myers <bpm@xxxxxxx>
Date: Mon, 26 Nov 2012 16:07:36 -0600
Cc: xfs ML <xfs@xxxxxxxxxxx>
In-reply-to: <20121126155511.GF27055@xxxxxxx>
References: <50B30B41.8040400@xxxxxxxxxxxxxx> <20121126155511.GF27055@xxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
Hey Satoru,

On Mon, Nov 26, 2012 at 09:55:11AM -0600, Ben Myers wrote:
> On Mon, Nov 26, 2012 at 03:25:05PM +0900, Satoru Takeuchi wrote:
> > From: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>
> > 
> > nodelaylog mount option is removed by commit 93b8a585. But there still be
> > the description about it in the xfs document. This patch removes it.
> > 
> > Signed-off-by: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>
> > ---
> >  Documentation/filesystems/xfs.txt |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/Documentation/filesystems/xfs.txt 
> > b/Documentation/filesystems/xfs.txt
> > index 3fc0c31..860b845 100644
> > --- a/Documentation/filesystems/xfs.txt
> > +++ b/Documentation/filesystems/xfs.txt
> > @@ -43,7 +43,7 @@ When mounting an XFS filesystem, the following options 
> > are accepted.
> >         Issue command to let the block device reclaim space freed by the
> >         filesystem.  This is useful for SSD devices, thinly provisioned
> >         LUNs and virtual machine images, but may have a performance
> > -       impact.  This option is incompatible with the nodelaylog option.
> > +       impact.
> 
> Looks good.
> 
> Reviewed-by: Ben Myers <bpm@xxxxxxx>

This has been committed to git://oss.sgi.com/xfs/xfs.git, master and for-next
branches.  It seems like your mailer replaced tabs with spaces so I cleaned
that up.  Just a heads-up... you may want to look into that.

Thanks!
        Ben

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