From: NeilBrown <neilb@suse.de>
To: Adam Kwolek <adam.kwolek@intel.com>
Cc: linux-raid@vger.kernel.org, ed.ciechanowski@intel.com,
marcin.labun@intel.com, dan.j.williams@intel.com,
lukasz.dorau@intel.com
Subject: Re: [PATCH 00/12] Problems during restart container reshape
Date: Thu, 9 Feb 2012 12:39:23 +1100 [thread overview]
Message-ID: <20120209123923.14638b58@notabene.brown> (raw)
In-Reply-To: <20120207135940.20627.33309.stgit@gklab-128-013.igk.intel.com>
[-- Attachment #1: Type: text/plain, Size: 2185 bytes --]
On Tue, 07 Feb 2012 15:02:55 +0100 Adam Kwolek <adam.kwolek@intel.com> wrote:
> This series corrects problems occurred during restart multi arrays reshape:
> 1. Assembled incrementally array during reshape is assembled degraded
> 2. Reshape cannot be continued after reboot. '--continue' gives an error
> 3. When raid0 arrays are reshaped, first array is reshaped only
> 4. Size doesn't change on second raid0 array
> Please note that patch 'imsm: FIX: No new missing disks are allowed during general migration' can be easily extended by:
> - Narrow it to incremental assembly only
> - Array can be pushed to be assembled by using e.g. 'force' option during incremental assembly.
>
> Other issue addressed in this series is chunk size change from 128k to 4k problem.
>
> BR
> Adam
>
> ---
>
> Adam Kwolek (12):
> FIX: Do not try to (continue) reshape using inactive array
> FIX: restart reshape when reshape process is stopped just between 2 reshapes
> imsm: FIX: Clear migration record when migration switches to next volume.
> imsm: FIX: use md position to reshape restart
> FIX: use md position to reshape restart
> imsm: FIX: Chunk size migration problem
> Flush mdmon before next reshape step during container operation
> Fix: Sometimes mdmon throws core dump during reshape
> imsm: FIX: imsm_get_allowed_degradation() doesn't count degradation for raid1
> FIX: Array is not run when expansion disks are added
> imsm: FIX: No new missing disks are allowed during general migration
> FIX: NULL pointer to strdup() can be passed
>
>
> Assemble.c | 2 -
> Grow.c | 124 ++++++++++++++++++++++++++++++++++++++++++++-------------
> managemon.c | 6 +++
> msg.c | 10 +++++
> msg.h | 1
> super-intel.c | 47 +++++++++++++++++++---
> util.c | 5 +-
> 7 files changed, 159 insertions(+), 36 deletions(-)
>
Thanks for these. I have provisionally applied them all however I won't push
them out until I get confirmation about the '<=' or '<' issue that I
mentioned separately.
Thanks,
NeilBrown
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]
prev parent reply other threads:[~2012-02-09 1:39 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-07 14:02 [PATCH 00/12] Problems during restart container reshape Adam Kwolek
2012-02-07 14:03 ` [PATCH 01/12] FIX: NULL pointer to strdup() can be passed Adam Kwolek
2012-02-07 14:03 ` [PATCH 02/12] imsm: FIX: No new missing disks are allowed during general migration Adam Kwolek
2012-02-07 14:03 ` [PATCH 03/12] FIX: Array is not run when expansion disks are added Adam Kwolek
2012-02-07 14:03 ` [PATCH 04/12] imsm: FIX: imsm_get_allowed_degradation() doesn't count degradation for raid1 Adam Kwolek
2012-02-07 14:03 ` [PATCH 05/12] Fix: Sometimes mdmon throws core dump during reshape Adam Kwolek
2012-02-07 14:03 ` [PATCH 06/12] Flush mdmon before next reshape step during container operation Adam Kwolek
2012-02-07 14:03 ` [PATCH 07/12] imsm: FIX: Chunk size migration problem Adam Kwolek
2012-02-07 14:03 ` [PATCH 08/12] FIX: use md position to reshape restart Adam Kwolek
2012-02-09 1:36 ` NeilBrown
2012-02-09 8:16 ` Kwolek, Adam
2012-02-09 9:19 ` NeilBrown
2012-02-09 9:49 ` Kwolek, Adam
2012-02-09 10:19 ` NeilBrown
2012-02-09 10:23 ` Kwolek, Adam
2012-02-09 10:53 ` Kwolek, Adam
2012-02-07 14:04 ` [PATCH 09/12] imsm: " Adam Kwolek
2012-02-07 14:04 ` [PATCH 10/12] imsm: FIX: Clear migration record when migration switches to next volume Adam Kwolek
2012-02-07 14:04 ` [PATCH 11/12] FIX: restart reshape when reshape process is stopped just between 2 reshapes Adam Kwolek
2012-02-07 14:04 ` [PATCH 12/12] FIX: Do not try to (continue) reshape using inactive array Adam Kwolek
2012-02-09 1:39 ` 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=20120209123923.14638b58@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=lukasz.dorau@intel.com \
--cc=marcin.labun@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).