On Tue, Sep 18, 2012 at 09:19:40AM +0100, Hugo Mills wrote:
> On Tue, Sep 18, 2012 at 10:19:30AM +0600, Roman Mamedov wrote:
> > On Tue, 18 Sep 2012 10:31:41 +0800
> > Miao Xie <mi...@cn.fujitsu.com> wrote:
> > 
> > > On tue, 18 Sep 2012 10:30:17 +0900, Hidetoshi Seto wrote:
> > > > This patch adds mount-option command.
> > > > The command can set/get default mount options.
> > > > Now, the command can set/get 24 options.
> > > > These options are equal to mount options which store
> > > > in fs_info/mount-opt.
> > > 
> > > I don't think we need implement a separate command to do this,
> > > we can add it into btrfstune just like ext3/4. If so, the users
> > > who used ext3/4 before can be familiar with btrfs command as soon
> > > as possible.
> > 
> > btrfstune currently only does one thing:
> > 
> > $ sudo btrfstune
> > usage: btrfstune [options] device
> >     -S value        enable/disable seeding
> > 
> > To me it'd seem more logical the other way, why not move this operation to 
> > the
> > base "btrfs" utility under some command, and remove "btrfstune" completely.
> 
>    I would suggest picking up and extending Alex Block's "btrfs
> property set/get" idea for this.

And I'm with you here. I'm not sure if btrfstune has widespread use, the
functionality should go into a single management tool that everybody
uses.

david
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to