All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Anton Mitterer <calestyo@scientia.net>
To: Sanidhya Solanki <jpage.lkml@gmail.com>
Cc: linux-btrfs@vger.kernel.org
Subject: Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size
Date: Tue, 29 Dec 2015 05:26:28 +0100	[thread overview]
Message-ID: <1451363188.7094.23.camel@scientia.net> (raw)
In-Reply-To: <20151228190336.59a3f440@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2466 bytes --]

On Mon, 2015-12-28 at 19:03 -0500, Sanidhya Solanki wrote:
> That sounds like an absolutely ghastly idea.
*G* and it probably is ;)


>  Lots of potential for
> mistakes and potential data loss. I take up the offer to implement
> such a feature. 
> Only question is should it be in-place replacement or replace out to
> another disk or storage type. Will wait for comments on that question
> before implementing. 
I guess you really should have a decent discussion with some of the
core btrfs developers (which I am not) before doing any efforts on this
(and possibly wasting great amounts of work).

I spoke largely from the user/admin side,... running a quite big
storage Tier-2, we did many IO benchmarks over time (with different
hardware RAID controllers) and also as our IO patterns changed over
time...
The result was that our preferred RAID chunk sizes changed over
time,...

Being able to to an online conversion (i.e. on the mounted fs) would be
nice of course (from the sysadmin's side of view) but even if that
doesn't seem feasible an offline conversion may be useful (one simply
may not have enough space left elsewhere to move the data of and create
a new fs with different RAID chunk size from scratch)
Both open of course many questions (how to deal with crashes, etc.)...
maybe having a look at how mdadm handles similar problems could be
worth.


> I do not understand this part.
I just wanted to say that being able to set the RAID chunk size makes
sense for any RAID level,... not just for e.g. the parity RAIDs.

>  Is the stripe size not independent of
> the RAID type?
Perhaps one should first do some normative definitions...
Based on e.g. https://en.wikipedia.org/wiki/File:RAID_6.svg I'd use:

RAID chunk size:
the size of A1 which equals the size of A2, and also of B2 and so on
IIRC, e2fsprogs call this the stride size (with a "d" not a "p")

RAID stripe size:
I personally would typically understand this to be the size of A1A2A3
together (which is equal to the one of B1B2B3 and so on).
But I've seen literature, software, people using it as a synonym for
the RAID chunk size  and some consider the stripe size to be also
include the parity, which in the RAID6 example would make it the length
of A1A2A3ApAq.
So I usually try to use "RAID data stripe size" or "RAID data+paritiy
stripe size"


btrfs chunk size = size of a btrfs data or meta-data chunk


Cheers,
Chris.

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5313 bytes --]

  reply	other threads:[~2015-12-29  4:26 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-28 12:24 [PATCH] BTRFS: Adds an option to select RAID Stripe size Sanidhya Solanki
2015-12-28 22:19 ` Christoph Anton Mitterer
2015-12-28 20:38   ` Sanidhya Solanki
2015-12-29  1:21     ` Christoph Anton Mitterer
2015-12-28 21:43       ` Sanidhya Solanki
2015-12-29  3:42         ` Christoph Anton Mitterer
2015-12-29  0:03           ` Sanidhya Solanki
2015-12-29  4:26             ` Christoph Anton Mitterer [this message]
2015-12-29  1:31               ` Sanidhya Solanki
2015-12-29  6:03                 ` Christoph Anton Mitterer
2015-12-29  2:23                   ` Sanidhya Solanki
2015-12-29 15:32                     ` Christoph Anton Mitterer
2015-12-29 16:44                       ` Duncan
2015-12-30  2:56                         ` Christoph Anton Mitterer
2015-12-29 18:06               ` David Sterba
2015-12-30 20:00                 ` Christoph Anton Mitterer
2015-12-30 21:02                   ` Duncan
2015-12-30 21:13                     ` Christoph Anton Mitterer
2016-01-02 11:52                 ` Sanidhya Solanki
2016-01-03  1:37                   ` Qu Wenruo
2016-01-03  2:26                     ` Christoph Anton Mitterer
2016-01-05 10:44                       ` David Sterba
2016-01-05 18:48                         ` Christoph Anton Mitterer
2016-01-10  3:11                     ` Sanidhya Solanki
2016-01-11  1:29                       ` Qu Wenruo
2016-01-11 15:43                       ` Christoph Anton Mitterer
2016-01-11 11:49                         ` Sanidhya Solanki
2016-01-11 15:57                           ` Christoph Anton Mitterer
2016-01-11 16:01                             ` Hugo Mills
2016-01-12 12:23                             ` Austin S. Hemmelgarn
2016-01-12 12:07                               ` Sanidhya Solanki
2015-12-29 13:39 ` David Sterba
2015-12-29 11:15   ` Sanidhya Solanki
2015-12-29 17:06     ` David Sterba
2015-12-29 21:32       ` Sanidhya Solanki
2015-12-30  6:39       ` Sanidhya Solanki
2015-12-30 11:59         ` Qu Wenruo
2015-12-30  9:54           ` Sanidhya Solanki
2015-12-30 14:10             ` Qu Wenruo
2015-12-30 11:15               ` Sanidhya Solanki
2015-12-30 15:58                 ` David Sterba
2015-12-30 21:19                   ` Sanidhya Solanki
2015-12-30 16:17               ` David Sterba
2015-12-30 21:21                 ` Sanidhya Solanki
2016-01-05 10:33                   ` David Sterba
2015-12-31  0:46                 ` Qu Wenruo
2016-01-05 10:16                   ` David Sterba
2015-12-30 19:48               ` Christoph Anton Mitterer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1451363188.7094.23.camel@scientia.net \
    --to=calestyo@scientia.net \
    --cc=jpage.lkml@gmail.com \
    --cc=linux-btrfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.