From: Dan Williams <dan.j.williams@intel.com>
To: Neil Brown <neilb@suse.de>
Cc: Doug Ledford <dledford@redhat.com>,
"Labun, Marcin" <Marcin.Labun@intel.com>,
Linux RAID Mailing List <linux-raid@vger.kernel.org>,
"Ciechanowski, Ed" <ed.ciechanowski@intel.com>,
"Hawrylewicz Czarnowski,
Przemyslaw" <przemyslaw.hawrylewicz.czarnowski@intel.com>
Subject: Re: More Hot Unplug/Plug work
Date: Thu, 29 Apr 2010 14:55:23 -0700 [thread overview]
Message-ID: <l2he9c3a7c21004291455w936bab77gea816e681f72a943@mail.gmail.com> (raw)
In-Reply-To: <20100429123717.2144a341@notabene.brown>
On Wed, Apr 28, 2010 at 7:37 PM, Neil Brown <neilb@suse.de> wrote:
>> So something like DOMAIN spanning=imsm, to tell mdadm to follow imsm
>> rules for this domain. Where 'spanning' is policy tag??
>>
>
> Thanks.
>
> So we have two different ideas here.
>
> 1/ A given set of devices (paths) are all attached to the one controller.
> 2/ A given array is not allowed to span controllers
>
> The first statement is somewhat similar to a statement about sparegroups.
> It groups devices together is some way.
>
> The second is a policy statement, and is metadata specific to some extent.
> If I create a native-metadata array using the controller, then adding other
> devices from a different controller is a non-issue. It is only when an
> IMSM array is created that it is an issue (and then - only if the array is to
> be used for boot for for multi-boot).
>
> So the ARRAY line could have "exclusive-group=foo" where 'foo' is a group
> name similar to those used for 'spare-group='
> But that isn't much fun for auto-detect and auto-assembly.
>
> Maybe we want to extend the 'auto' line. It gives policy on a per-metadata
> basis. Maybe:
>
> POLICY auto-assemble +1.x -all
> POLICY same-group imsm
>
> where 'same-group' means that all the devices in an array must be in the
> same spare-group. The 'domain' lines assign spare-groups to devices.
>
> Maybe "same-group" could be "same-$tag" so we could have different tags
> for different metadatas....
>
> Is this working for anyone else, or have I lost the plot??
>
I am not grokking the separate POLICY line, especially for defining
the spare-migration border because that is already what DOMAIN is
specifying.
Here is what I think we need to allow for simple honoring of platform
constraints but without needing to expose all the nuances of those
constraints in config-file syntax... yet.
1/ Allow path= to take a metadata name this allows the handler to
identify its known controller ports, alleviating the user from needing
to track which ports are allowed, especially as it may change over
time. If someone really wants to see which ports a metadata handler
cares about we could have a DOMAIN line dumped by --detail-platform
--brief -e imsm. However for simplicity I would rather just dump:
DOMAIN path=imsm action=spare-same-port spare-migration=imsm
2/ I think we should always block configurations that cross domain
boundaries. One can always append more path= lines to override this.
3/ The metadata handler may want to restrict/control where spares are
placed in a domain. To enable interaction with CIM we are looking to
add a storage-pool id to the metadata. The primary usage of this will
be to essentially encode a spare-group number in the metadata. This
seems to require a spare-migration= option to the DOMAIN line. By
default it is 'all' but it can be set to a metadata-name to let the
handler apply its internal migration policy.
That should cover everything I would like to expose Comments?
--
Dan
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2010-04-29 21:55 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-27 16:45 More Hot Unplug/Plug work Doug Ledford
2010-04-27 19:41 ` Christian Gatzemeier
2010-04-28 16:08 ` Labun, Marcin
2010-04-28 17:47 ` Doug Ledford
2010-04-28 18:34 ` Labun, Marcin
2010-04-28 21:05 ` Doug Ledford
2010-04-28 21:13 ` Dan Williams
2010-04-30 13:38 ` Doug Ledford
2010-04-29 1:01 ` Neil Brown
2010-04-29 1:19 ` Dan Williams
2010-04-29 2:37 ` Neil Brown
2010-04-29 18:22 ` Labun, Marcin
2010-04-29 21:55 ` Dan Williams [this message]
2010-05-03 5:58 ` Neil Brown
2010-05-08 1:06 ` Dan Williams
2010-04-30 16:13 ` Doug Ledford
2010-04-30 11:14 ` John Robinson
2010-04-30 15:52 ` Doug Ledford
2010-04-28 20:59 ` Luca Berra
2010-04-28 21:16 ` Doug Ledford
2010-04-29 20:32 ` Dan Williams
2010-04-29 21:22 ` Dan Williams
2010-04-30 16:26 ` Doug Ledford
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=l2he9c3a7c21004291455w936bab77gea816e681f72a943@mail.gmail.com \
--to=dan.j.williams@intel.com \
--cc=Marcin.Labun@intel.com \
--cc=dledford@redhat.com \
--cc=ed.ciechanowski@intel.com \
--cc=linux-raid@vger.kernel.org \
--cc=neilb@suse.de \
--cc=przemyslaw.hawrylewicz.czarnowski@intel.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).