From: Doug Ledford <dledford@redhat.com>
To: Neil Brown <neilb@suse.de>
Cc: Bill Davidsen <davidsen@tmr.com>,
Joe Bryant <tenminjoe@yahoo.com>,
linux-raid@vger.kernel.org
Subject: Re: Spare drive won't spin down
Date: Mon, 17 May 2010 14:11:42 -0400 [thread overview]
Message-ID: <4BF186DE.5050502@redhat.com> (raw)
In-Reply-To: <20100512065318.44e934d4@notabene.brown>
[-- Attachment #1: Type: text/plain, Size: 1338 bytes --]
On 05/11/2010 04:53 PM, Neil Brown wrote:
> Theoretically, when the spares are one behind the active array and we need to
> update them all, we should update the spares first, then the rest. If we
> don't and there is a crash at the wrong time, some spares could be 2 events
> behind the most recent device. However that is a fairly unlikely race to
> lose and the cost is only having a spare device fall out of the array, which
> is quite easy to put back it, that I might not worry to much about it.
>
> So if you haven't seen a patch to fix this in a week or two, please remind me.
They're spares. They don't need to have an in-sync generation count.
Change the semantics so that spares have a 0 generation count and only
when they've been activated does their count get brought into sync with
the rest of the array. The only thing needed to make that work is to
not kick them on assembly because their generation count doesn't match,
but that should be trivial (and sane) to do based on the fact that if
they are a spare, they don't contain data, so the generation count
really is meaningless.
--
Doug Ledford <dledford@redhat.com>
GPG KeyID: CFBFF194
http://people.redhat.com/dledford
Infiniband specific RPMs available at
http://people.redhat.com/dledford/Infiniband
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2010-05-17 18:11 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-06 17:49 Spare drive won't spin down Joe Bryant
2010-05-07 5:07 ` Michael Evans
2010-05-07 7:39 ` Joe Bryant
2010-05-07 6:20 ` Neil Brown
2010-05-07 7:40 ` Joe Bryant
2010-05-07 9:47 ` Neil Brown
2010-05-07 10:05 ` Joe Bryant
2010-05-10 16:59 ` Bill Davidsen
2010-05-11 20:53 ` Neil Brown
2010-05-17 18:11 ` Doug Ledford [this message]
2010-05-18 0:23 ` Neil Brown
2010-05-18 0:38 ` Michael Evans
2010-05-18 0:50 ` Neil Brown
2010-05-18 0:20 ` Neil Brown
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=4BF186DE.5050502@redhat.com \
--to=dledford@redhat.com \
--cc=davidsen@tmr.com \
--cc=linux-raid@vger.kernel.org \
--cc=neilb@suse.de \
--cc=tenminjoe@yahoo.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).