Originally posted by starshipeleven
View Post
Announcement
Collapse
No announcement yet.
Btrfs RAID 0/1/5/6/10 Benchmarks On Linux 4.12
Collapse
X
-
-
Originally posted by PuckPoltergeist View PostAre you sure the disk-stripes are checksumed? Doesn't sound reasonable, cause this will be already a layer-break.
Comment
-
Originally posted by starshipeleven View PostNo, what I was saying was that the code checking for checksums when reading parity wasn't working or is still very new (a patch supposed to fix that landed in 4.12, at least for btrfs scrub), not that they need to add checksums on block layer.
As far as I remember from the discussions on btrfs mailinglist, the idea was checksumming the parity too, additional to the checksums of data/metadata. And this doesn't make much sense.
Comment
-
Originally posted by PuckPoltergeist View Post
This doesn't make sense. The FS layer, that does the checksumming, doesn't care about RAID levels. It only cares about redundancy in case of checksum mismatch. If a mismatch happens, we try to get the data from some other place. For RAID1 this means some copy, for RAID5 it reconstructs the stripe from data + parity.
As far as I remember from the discussions on btrfs mailinglist, the idea was checksumming the parity too, additional to the checksums of data/metadata. And this doesn't make much sense.
Without these RAID5/6 is basically retarded bullshit, and for now it's still too early to say if it's all clear.
Comment
-
Originally posted by starshipeleven View Postsince I can't seem to explain myself decently, see here for the fixes I was talking about: https://www.mail-archive.com/linux-b.../msg63365.html
Without these RAID5/6 is basically retarded bullshit, and for now it's still too early to say if it's all clear.
Comment
-
And raid5/6 scrub patches floating around the mailing list get updated again https://www.mail-archive.com/linux-b.../msg64474.html and it explains in more understandable english what scrub can or cannot do atm.
Comment
-
Originally posted by starshipeleven View Post
That if parity isn't checksummed then the guarantee that my data isn't corrupted goes out of the window.
Comment
-
Originally posted by starshipeleven View PostAnd raid5/6 scrub patches floating around the mailing list get updated again https://www.mail-archive.com/linux-b.../msg64474.html and it explains in more understandable english what scrub can or cannot do atm.
> To get a comparable tool for kernel scrub, we need a user-space tool to
> act as benchmark to compare their different behaviors.
The normal user should relay to the kernel implementation, which, unfortunately, is far to be perfect...
Comment
-
Originally posted by kreijack View Post
This is not correct: you still have the data checksum to check if the rebuild data is correct. So even in the case that the parity is corrupted, btrfs is able to detect it.
Comment
-
Originally posted by kreijack View PostThese patches are related to the user space implementation. As described in the first email, the scrub user-space implementation is not related to the kernel space one:
It makes a comparison between btrfs-progs and the kernel's capability, and does so in english with easy ascii graphs, that's why I posted it.
The normal user should relay to the kernel implementation, which, unfortunately, is far to be perfect...
Comment
Comment