Linux Btrfs filesystem development
 help / color / mirror / Atom feed
From: "Austin S. Hemmelgarn" <ahferroin7@gmail.com>
To: Tomasz Pala <gotar@polanet.pl>,
	Linux fs Btrfs <linux-btrfs@vger.kernel.org>
Subject: Re: Unexpected raid1 behaviour
Date: Tue, 19 Dec 2017 07:25:49 -0500	[thread overview]
Message-ID: <7ff86029-5b0f-1d02-778a-af78c6f3e461@gmail.com> (raw)
In-Reply-To: <20171218194351.GA25245@polanet.pl>

On 2017-12-18 14:43, Tomasz Pala wrote:
> On Mon, Dec 18, 2017 at 08:06:57 -0500, Austin S. Hemmelgarn wrote:
> 
>> The fact is, the only cases where this is really an issue is if you've
>> either got intermittently bad hardware, or are dealing with external
> 
> Well, the RAID1+ is all about the failing hardware.
About catastrophically failing hardware, not intermittent failure.
> 
>> storage devices.  For the majority of people who are using multi-device
>> setups, the common case is internally connected fixed storage devices
>> with properly working hardware, and for that use case, it works
>> perfectly fine.
> 
> If you're talking about "RAID"-0 or storage pools (volume management)
> that is true.
> But if you imply, that RAID1+ "works perfectly fine as long as hardware
> works fine" this is fundamentally wrong. If the hardware needs to work
> properly for the RAID to work properly, noone would need this RAID in
> the first place.
I never said the hardware needed to not fail, just that it needed to 
fail in a consistent manner.  BTRFS handles catastrophic failures of 
storage devices just fine right now.  It has issues with intermittent 
failures, but so does hardware RAID, and so do MD and LVM to a lesser 
degree.
> 
>> that BTRFS should not care.  At the point at which a device is dropping
>> off the bus and reappearing with enough regularity for this to be an
>> issue, you have absolutely no idea how else it's corrupting your data,
>> and support of such a situation is beyond any filesystem (including ZFS).
> 
> Support for such situation is exactly what RAID performs. So don't blame
> people for expecting this to be handled as long as you call the
> filesystem feature a 'RAID'.
No, classical RAID (other than RAID0) is supposed to handle catastrophic 
failure of component devices.  That is the entirety of the original 
design purpose, and that is the entirety of what you should be using it 
for in production.  The point at which you are getting random corruption 
on a disk and you're using anything but BTRFS for replication, you 
_NEED_ to replace that disk, and if you don't you risk it causing 
corruption on the other disk.  As of right now, BTRFS is no different in 
that respect, but I agree that it _should_ be able to handle such a 
situation eventually.
> 
> If this feature is not going to mitigate hardware hiccups by design (as
> opposed to "not implemented yet, needs some time", which is perfectly
> understandable), just don't call it 'RAID'.
It shouldn't have been called RAID in the first place, that we can agree 
on (even if for different reasons).
> 
> All the features currently working, like bit-rot mitigation for
> duplicated data (dup/raid*) using checksums, are something different
> than RAID itself. RAID means "survive failure of N devices/controllers"
> - I got one "RAID1" stuck in r/o after degraded mount, not nice... Not
> _expected_ to happen after single disk failure (without any reappearing).
And that's a known bug on older kernels (not to mention that you should 
not be mounting writable and degraded for any purpose other than fixing 
the volume).

  parent reply	other threads:[~2017-12-19 12:25 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-16 19:50 Unexpected raid1 behaviour Dark Penguin
2017-12-17 11:58 ` Duncan
2017-12-17 15:48   ` Peter Grandi
2017-12-17 20:42     ` Chris Murphy
2017-12-18  8:49       ` Anand Jain
2017-12-18  8:49     ` Anand Jain
2017-12-18 10:36       ` Peter Grandi
2017-12-18 12:10       ` Nikolay Borisov
2017-12-18 13:43         ` Anand Jain
2017-12-18 22:28       ` Chris Murphy
2017-12-18 22:29         ` Chris Murphy
2017-12-19 12:30         ` Adam Borowski
2017-12-19 12:54         ` Andrei Borzenkov
2017-12-19 12:59         ` Peter Grandi
2017-12-18 13:06     ` Austin S. Hemmelgarn
2017-12-18 19:43       ` Tomasz Pala
2017-12-18 22:01         ` Peter Grandi
2017-12-19 12:46           ` Austin S. Hemmelgarn
2017-12-19 12:25         ` Austin S. Hemmelgarn [this message]
2017-12-19 14:46           ` Tomasz Pala
2017-12-19 16:35             ` Austin S. Hemmelgarn
2017-12-19 17:56               ` Tomasz Pala
2017-12-19 19:47                 ` Chris Murphy
2017-12-19 21:17                   ` Tomasz Pala
2017-12-20  0:08                     ` Chris Murphy
2017-12-23  4:08                       ` Tomasz Pala
2017-12-23  5:23                         ` Duncan
2017-12-20 16:53                   ` Andrei Borzenkov
2017-12-20 16:57                     ` Austin S. Hemmelgarn
2017-12-20 20:02                     ` Chris Murphy
2017-12-20 20:07                       ` Chris Murphy
2017-12-20 20:14                         ` Austin S. Hemmelgarn
2017-12-21  1:34                           ` Chris Murphy
2017-12-21 11:49                         ` Andrei Borzenkov
2017-12-19 20:11                 ` Austin S. Hemmelgarn
2017-12-19 21:58                   ` Tomasz Pala
2017-12-20 13:10                     ` Austin S. Hemmelgarn
2017-12-19 23:53                   ` Chris Murphy
2017-12-20 13:12                     ` Austin S. Hemmelgarn
2017-12-19 18:31             ` George Mitchell
2017-12-19 20:28               ` Tomasz Pala
2017-12-19 19:35             ` Chris Murphy
2017-12-19 20:41               ` Tomasz Pala
2017-12-19 20:47                 ` Austin S. Hemmelgarn
2017-12-19 22:23                   ` Tomasz Pala
2017-12-20 13:33                     ` Austin S. Hemmelgarn
2017-12-20 17:28                       ` Duncan
2017-12-21 11:44                   ` Andrei Borzenkov
2017-12-21 12:27                     ` Austin S. Hemmelgarn
2017-12-22 16:05                       ` Tomasz Pala
2017-12-22 21:04                         ` Chris Murphy
2017-12-23  2:52                           ` Tomasz Pala
2017-12-23  5:40                             ` Duncan
2017-12-19 23:59                 ` Chris Murphy
2017-12-20  8:34                   ` Tomasz Pala
2017-12-20  8:51                     ` Tomasz Pala
2017-12-20 19:49                     ` Chris Murphy
2017-12-18  5:11   ` Anand Jain
2017-12-18  1:20 ` Qu Wenruo
2017-12-18 13:31 ` Austin S. Hemmelgarn
2018-01-12 12:26   ` Dark Penguin

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=7ff86029-5b0f-1d02-778a-af78c6f3e461@gmail.com \
    --to=ahferroin7@gmail.com \
    --cc=gotar@polanet.pl \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox