Re: How to implement raid1 repair

On 03/17/2011 06:09 PM, Andrey Kuzmin wrote:

On Thu, Mar 17, 2011 at 5:46 PM, Jan Schmidt <list.btrfs@jan-o-sch.net
<mailto:list.btrfs@jan-o-sch.net>> wrote:
    - Is it acceptable to retry reading a block immediately after the disk
    said it won't work? Or in case of a successful read followed by a
    checksum error? (Which is already being done right now in btrfs.)


These are two pretty different cases. When disk firmware fails read, it
means it has retried number of times but gave up (suggesting media
error), so an upper layer retry would hardly make sense. Checksum error
catches on-disk EDC fault, so retry is on the contrary quite reasonable.

Agreed.

    - Is it acceptable to always write both mirrors if one is found to be
    bad (also consider ssds)?


Writing on read path bypassing file-system transaction mechanism doesn't
seem a good idea to me. Just imaging loosing power while overwriting
last good copy.

Okay, sounds reasonable to me. Let's say we're bypassing transaction
mechanism in the same rude manner, but only write the bad mirror. Does
that seem reasonable?

Thanks,
Jan