xfs
[Top] [All Lists]

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

To: Ben Myers <bpm@xxxxxxx>
Subject: Re: [PATCH] xfs: Remove the description about nodelaylog mount option from, the xfs document.
From: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>
Date: Tue, 27 Nov 2012 16:33:51 +0900
Cc: xfs ML <xfs@xxxxxxxxxxx>
In-reply-to: <20121126220736.GG27055@xxxxxxx>
References: <50B30B41.8040400@xxxxxxxxxxxxxx> <20121126155511.GF27055@xxxxxxx> <20121126220736.GG27055@xxxxxxx>
User-agent: Mozilla/5.0 (Windows NT 6.0; rv:15.0) Gecko/20120907 Thunderbird/15.0.1
(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@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.

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@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs
> 


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