linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: NeilBrown <neilb@suse.de>
To: "Mathias Burén" <mathias.buren@gmail.com>
Cc: Adam Kwolek <adam.kwolek@intel.com>,
	linux-raid@vger.kernel.org, dan.j.williams@intel.com,
	ed.ciechanowski@intel.com, wojciech.neubauer@intel.com
Subject: Re: [PATCH 1/2] man mdadm: add information for MDADM_EXPERIMENTAL flag
Date: Wed, 23 Mar 2011 12:03:42 +1100	[thread overview]
Message-ID: <20110323120342.682693dd@notabene.brown> (raw)
In-Reply-To: <AANLkTi=dqmKeXeHH2wJUfv3wrXc5uLHVFaCnuNONvNFi@mail.gmail.com>

On Tue, 22 Mar 2011 13:10:06 +0000 Mathias Burén <mathias.buren@gmail.com>
wrote:

> On 22 March 2011 11:46, Adam Kwolek <adam.kwolek@intel.com> wrote:
> > Update man for MDADM_EXPERIMENTAL flag.
> >
> > Signed-off-by: Adam Kwolek <adam.kwolek@intel.com>
> > ---
> >
> >  mdadm.8.in |   12 ++++++++++++
> >  1 files changed, 12 insertions(+), 0 deletions(-)
> >
> > diff --git a/mdadm.8.in b/mdadm.8.in
> > index 4b70e20..96945ae 100644
> > --- a/mdadm.8.in
> > +++ b/mdadm.8.in
> > @@ -2229,6 +2229,18 @@ in a container can be converted between levels where those levels are
> >  supported by the container, and the conversion is on of those listed
> >  above.
> >
> > +Grow functionality (e.g. expand number of raid devices) for Intel's
> > +IMSM container format has experimental status. It is guarded by
> > +.B MDADM_EXPERIMENTAL
> > +environment variable. This is due to the following reasons:
> > +
> > +1. Native Intel's IMSM check-pointing is not fully implemented yet.
> > +This causes IMSM incompatibility during grow process: grew array cannot
> > +be roamed between Windows(R) and Linux systems.
> > +
> > +2. Interrupting grow operation is not recommended, because
> > +for Intel's IMSM container format it is not fully tested yet.
> > +
> >  .SS SIZE CHANGES
> >  Normally when an array is built the "size" is taken from the smallest
> >  of the drives.  If all the small drives in an arrays are, one at a
> >
> > --
> > 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
> >
> 
> My small modifications (spelling?), which might not be correct.
> 
>  mdadm.8.in |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/mdadm.8.in b/mdadm.8.in
> index 4b70e20..96945ae 100644
> --- a/mdadm.8.in
> +++ b/mdadm.8.in
> @@ -2229,6 +2229,18 @@ in a container can be converted between levels
> where those levels are
>  supported by the container, and the conversion is on of those listed
>  above.
> 
> +Grow functionality (e.g. expand a number of raid devices) for Intel's
> +IMSM container format has an experimental status. It is guarded by the
> +.B MDADM_EXPERIMENTAL
> +environment variable. This is due to the following reasons:
> +
> +1. Native Intel's IMSM check-pointing is not fully implemented yet.
> +This causes IMSM incompatibility during the grow process: a grown array cannot
> +roam between Windows(R) and Linux systems.
> +
> +2. Interrupting a grow operation is not recommended, because it
> +has not been fully tested for Intel's IMSM container format yet.
> +
>  .SS SIZE CHANGES
>  Normally when an array is built the "size" is taken from the smallest
>  of the drives.  If all the small drives in an arrays are, one at a

Thanks. I included your revisions and made some more of my own :-)

See below.

NeilBrown


commit ca24ddb08da4c86b60f7e049f0b0a2c30b19c95a
Author: Adam Kwolek <adam.kwolek@intel.com>
Date:   Wed Mar 23 11:45:03 2011 +1100

    man mdadm: add information for MDADM_EXPERIMENTAL flag
    
    Update man for MDADM_EXPERIMENTAL flag.
    
    Minor revisions by Mathias Burén <mathias.buren@gmail.com> and Neil Brown.
    
    Signed-off-by: Adam Kwolek <adam.kwolek@intel.com>
    Signed-off-by: NeilBrown <neilb@suse.de>

diff --git a/mdadm.8.in b/mdadm.8.in
index 4b70e20..9537304 100644
--- a/mdadm.8.in
+++ b/mdadm.8.in
@@ -2229,6 +2229,23 @@ in a container can be converted between levels where those levels are
 supported by the container, and the conversion is on of those listed
 above.
 
+Grow functionality (e.g. expand a number of raid devices) for Intel's
+IMSM container format has an experimental status. It is guarded by the
+.B MDADM_EXPERIMENTAL
+environment variable which must be set to '1' for a GROW command to
+succeed.
+This is for the following reasons:
+
+.IP 1.
+Intel's native IMSM check-pointing is not fully implemented yet.
+This causes IMSM incompatibility during the grow process: an array
+which is growing cannot roam between Microsoft Windows(R) and Linux
+systems.
+
+.IP 2.
+Interrupting a grow operation is not recommended, because it
+has not been fully tested for Intel's IMSM container format yet.
+
 .SS SIZE CHANGES
 Normally when an array is built the "size" is taken from the smallest
 of the drives.  If all the small drives in an arrays are, one at a
--
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

      reply	other threads:[~2011-03-23  1:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-22 11:46 [PATCH 1/2] man mdadm: add information for MDADM_EXPERIMENTAL flag Adam Kwolek
2011-03-22 11:46 ` [PATCH 2/2] man mdadm: Add note about auto-assembly during array reshape Adam Kwolek
2011-03-23  1:04   ` NeilBrown
2011-03-22 13:10 ` [PATCH 1/2] man mdadm: add information for MDADM_EXPERIMENTAL flag Mathias Burén
2011-03-23  1:03   ` NeilBrown [this message]

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=20110323120342.682693dd@notabene.brown \
    --to=neilb@suse.de \
    --cc=adam.kwolek@intel.com \
    --cc=dan.j.williams@intel.com \
    --cc=ed.ciechanowski@intel.com \
    --cc=linux-raid@vger.kernel.org \
    --cc=mathias.buren@gmail.com \
    --cc=wojciech.neubauer@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).