Re: [PATCH 0/2] nilfs2: using nobarrier option instead of barrier=off

[Ryusuke Konishi <ryusuke-sG5X7nlA6pw@public.gmane.org>]

Hi!
On Wed, 11 Nov 2009 18:04:16 +0900, Jiro SEKIBA <jir-hfpbi5WX9J54Eiagz67IpQ@public.gmane.org> wrote:
> Hi,
> 
> This is a series of patches to use nobarrier mount option
> instead of barrier=off, for most of fs using nofoobar style option.
> 
> First patch is the actual patch to modify the mount option.
> 
> Second patch is cleanup patch to remove unused match_bool() function
> as a result of modifying mount option.
> 
>  fs/nilfs2/super.c |   28 +++++-----------------------
>  1 files changed, 5 insertions(+), 23 deletions(-)

Well, this change looks quite reasonable, and also helps to resolve
the discordance between the old barrier option and standard linux
options; it also matches the "-o norepair" which I posted recently.

Although the first patch changes user-interface, I believe the impact
is limited because the barrier feature is enabled by default.

So, I'd like to apply your change unless someone dissents.

Sekiba-san, could you modify Documentation/filesystems/nilfs2.txt and
include it in the first patch?

The doc file has description on mount options.

Thanks,
Ryusuke Konishi