linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Tuomas Leikola <tuomas.leikola@gmail.com>
Cc: Neil Bortnak <linux-raid@moro.us>, linux-raid@vger.kernel.org
Subject: Re: Feature Request/Suggestion - "Drive Linking"
Date: Sun, 03 Sep 2006 22:35:25 +0400	[thread overview]
Message-ID: <44FB206D.6000206@tls.msk.ru> (raw)
In-Reply-To: <5b170a7d0609030759u2b145237m85757ec649f86488@mail.gmail.com>

Tuomas Leikola wrote:
[]
> Here's an alternate description. On first 'unrecoverable' error, the
> disk is marked as FAILING, which means that a spare is immediately
> taken into use to replace the failing one. The disk is not kicked, and
> readable blocks can still be used to rebuild other blocks (from other
> FAILING disks).
> 
> The rebuild can be more like a ddrescue type operation, which is
> probably a lot faster in the case of raid6, and the disk can be
> automatically kicked after the sync is done. If there is no read
> access to the FAILING disk, the rebuild will be faster just because
> seeks are avoided in a busy system.

It's not that simple.  The issue is with writes.  If there's a "failing"
disk, md code will need to keep track of "up-to-date", or "good" sectors
of it vs "obsolete" ones.  Ie, when write fails, the data in that block
is either unreadable (but can become readable on the next try, say, after
themperature change or whatnot), or readable but contains old data, or
is readable but contains some random garbage.  So at least that block(s)
of the disk should not be copied to the spare during resync, and should
not be read at all, to avoid returning wrong data to userspace.  In short,
if the array isn't stopped (or changed to read-only), we should watch for
writes, and remember which ones are failed.  Which is some non-trivial
change.  Yes, bitmaps somewhat helps here.

/mjt

-- 
VGER BF report: H 0.418675

  reply	other threads:[~2006-09-03 18:35 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-29 16:21 Feature Request/Suggestion - "Drive Linking" Neil Bortnak
2006-08-29 17:43 ` dean gaudet
2006-09-03 14:59 ` Tuomas Leikola
2006-09-03 18:35   ` Michael Tokarev [this message]
2006-09-04 16:55     ` Bill Davidsen
2006-09-05  6:33       ` dean gaudet

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=44FB206D.6000206@tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=linux-raid@moro.us \
    --cc=linux-raid@vger.kernel.org \
    --cc=tuomas.leikola@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).