Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size

From	Sanidhya Solanki <jpage.lkml@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size
Date	2015-12-30 15:00 +0100
Message-ID	<qLpse-7lJ-3@gated-at.bofh.it> (permalink)
References	(1 earlier) <qL2P0-1d9-3@gated-at.bofh.it> <qL4e6-2eC-3@gated-at.bofh.it> <qL5Wz-3oe-25@gated-at.bofh.it> <qLmum-5sY-13@gated-at.bofh.it> <qLnJL-6t0-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, 30 Dec 2015 19:59:16 +0800
Qu Wenruo <quwenruo.btrfs@gmx.com> wrote:
> Not really sure about the difference between 2 and 3.

I should have made it clear before, I was asking the exact use case in
mind when listing the choices. Option 2 would be for SysAdmins running
production software and configuring it as they desire.
Option 3 is what we have in the Kernel now, before my patch, where the
option exists, but it is fixed by the code. You can change it, but you
need to be someone fairly involved in the upstream work (like a
distribution Maintainer). This is what my patch implements (well, this
and option 3).
Option 1 leaves it as a compile time option.

> When you mention runtime option, did you mean ioctl/mount/balance 
> convert option?

Yes, that is correct.

> And what's the third one? Default mkfs time option?
> If you can make it mkfs time option, it won't be really hard to make
> it configurable.

This would be ideal for all use-cases, but make the implementation
much larger than it would be for the other options. Hence, I asked
what the exact use case was for the end-user being targeted.

> I didn't consider David means something that.
> As far as I read, he means balance convert option along with mkfs
> option.

Hence, why I asked.

> At least from what I have learned in recent btrfs development,
He> He> either
> we provide a good enough interfaces (normally, balance convert ioctl
> with mkfs time option) to configure some on-disk fields.

Just confirming before starting the implementation.
> So fixed kernel value is not a really good idea, and should at least
> be replace by mkfs time option.

Will do after confirmation.

Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-28 17:30 +0100
  Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size David Sterba <dsterba@suse.cz> - 2015-12-29 14:50 +0100
    Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-29 16:20 +0100
      Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size David Sterba <dsterba@suse.cz> - 2015-12-29 18:10 +0100
        Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-30 02:40 +0100
        Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-30 11:50 +0100
          Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Qu Wenruo <quwenruo.btrfs@gmx.com> - 2015-12-30 13:10 +0100
            Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-30 15:00 +0100
              Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Qu Wenruo <quwenruo.btrfs@gmx.com> - 2015-12-30 15:20 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-30 16:20 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size David Sterba <dsterba@suse.cz> - 2015-12-30 17:10 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-31 02:30 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size David Sterba <dsterba@suse.cz> - 2015-12-30 17:30 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Qu Wenruo <quwenruo@cn.fujitsu.com> - 2015-12-31 01:50 +0100
                Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki <jpage.lkml@gmail.com> - 2015-12-31 02:30 +0100

csiph-web