[PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.

[PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.

[Satoru Takeuchi]

From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>

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@jp.fujitsu.com>
---
 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.
 
   dmapi
        Enable the DMAPI (Data Management API) event callouts.

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

Re: [PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.

[Ben Myers]

On Mon, Nov 26, 2012 at 03:25:05PM +0900, Satoru Takeuchi wrote:
> From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
> 
> 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@jp.fujitsu.com>
> ---
>  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@sgi.com>

Thanks Satoru!

-Ben

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

Re: [PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.

[Ben Myers]

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@jp.fujitsu.com>
> > 
> > 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@jp.fujitsu.com>
> > ---
> >  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@sgi.com>

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

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

Re: [PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.

[Satoru Takeuchi]

(2012/11/27 7:07), Ben Myers wrote:
> 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@jp.fujitsu.com>
>>>
>>> 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@jp.fujitsu.com>
>>> ---
>>>  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@sgi.com>
> 
> 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.

I apologize to you. I copy-and-past this patch from my terminal and it converts
tab to spaces. I'll be careful from now on.

Thanks,
Satoru

> 
> Thanks!
> 	Ben
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
> 


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