linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: NeilBrown <neilb@suse.de>
To: Adam Kwolek <adam.kwolek@intel.com>
Cc: linux-raid@vger.kernel.org, dan.j.williams@intel.com,
	ed.ciechanowski@intel.com, wojciech.neubauer@intel.com
Subject: Re: [PATCH 00/21] IMSM Checkpointing Bug Fix Series
Date: Thu, 9 Jun 2011 13:03:29 +1000	[thread overview]
Message-ID: <20110609130329.42bf05bb@notabene.brown> (raw)
In-Reply-To: <20110608160222.24327.71439.stgit@gklab-128-013.igk.intel.com>

On Wed, 08 Jun 2011 18:09:33 +0200 Adam Kwolek <adam.kwolek@intel.com> wrote:

> The following series fixes problems found in IMSM's checkpointing.
> It contains rework based on Neil's comments to previous/initial checkpointing
> series and tt should be applied on neil_master branch (on the top 
> of previous checkpointing patches).
> 
> BR
> Adam
> 
> 
> ---
> 
> Adam Kwolek (21):
>       MAN: Man update for check-pointing
>       imsm: Optimize expansion speed when no backup is required
>       imsm: FIX: Remove timeout from wait_for_reshape_imsm()
>       imsm: FIX: wait_for_reshape_imsm() cleanup
>       imsm: FIX: Do not continue reshape when backup exists
>       FIX: Move buffer to next location
>       imsm: FIX: Remove unused variables and code
>       imsm: FIX: Move reshape_progress forward
>       imsm: FIX: Detect failed devices during recover_backup_imsm()
>       imsm: FIX: Use metadata information for restore_stripes() and save_stripes()
>       imsm: FIX: Remove unused parameter from save_backup_imsm() interface
>       imsm: FIX: Do not use pba_of_lba0 for copy position calculation
>       imsm: FIX: Do not verify unused parameters
>       imsm: FIX: Calculate backup location based on metadata information
>       imsm: FIX: Use macros to data access
>       imsm: FIX: Check layout for level migration
>       imsm: FIX: Max position could not be rounded to MB
>       imsm: FIX: Detect migration end during migration record saving
>       imsm: FIX: Verify if migration record is loaded correctly
>       imsm: FIX: Opened handle is not closed
>       imsm: FIX: Cannot create volume
> 
> 
>  mdadm.8.in    |    9 ++
>  restripe.c    |    6 +
>  super-intel.c |  237 ++++++++++++++++++++++++++++++++++-----------------------
>  3 files changed, 153 insertions(+), 99 deletions(-)
> 

Thanks.
Apart from the first one which I have already commented on, these look good.
I have applied them all, thanks.

Please confirm that it all still works for you and let me know if you have
any other changes pending that I should know about.

Thanks,
NeilBrown


      parent reply	other threads:[~2011-06-09  3:03 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-08 16:09 [PATCH 00/21] IMSM Checkpointing Bug Fix Series Adam Kwolek
2011-06-08 16:09 ` [PATCH 01/21] imsm: FIX: Cannot create volume Adam Kwolek
2011-06-09  2:42   ` NeilBrown
2011-06-09  6:40     ` Kwolek, Adam
2011-06-08 16:09 ` [PATCH 02/21] imsm: FIX: Opened handle is not closed Adam Kwolek
2011-06-08 16:09 ` [PATCH 03/21] imsm: FIX: Verify if migration record is loaded correctly Adam Kwolek
2011-06-08 16:10 ` [PATCH 04/21] imsm: FIX: Detect migration end during migration record saving Adam Kwolek
2011-06-08 16:10 ` [PATCH 05/21] imsm: FIX: Max position could not be rounded to MB Adam Kwolek
2011-06-08 16:10 ` [PATCH 06/21] imsm: FIX: Check layout for level migration Adam Kwolek
2011-06-08 16:10 ` [PATCH 07/21] imsm: FIX: Use macros to data access Adam Kwolek
2011-06-08 16:10 ` [PATCH 08/21] imsm: FIX: Calculate backup location based on metadata information Adam Kwolek
2011-06-08 16:10 ` [PATCH 09/21] imsm: FIX: Do not verify unused parameters Adam Kwolek
2011-06-08 16:10 ` [PATCH 10/21] imsm: FIX: Do not use pba_of_lba0 for copy position calculation Adam Kwolek
2011-06-08 16:11 ` [PATCH 11/21] imsm: FIX: Remove unused parameter from save_backup_imsm() interface Adam Kwolek
2011-06-08 16:11 ` [PATCH 12/21] imsm: FIX: Use metadata information for restore_stripes() and save_stripes() Adam Kwolek
2011-06-08 16:11 ` [PATCH 13/21] imsm: FIX: Detect failed devices during recover_backup_imsm() Adam Kwolek
2011-06-08 16:11 ` [PATCH 14/21] imsm: FIX: Move reshape_progress forward Adam Kwolek
2011-06-08 16:11 ` [PATCH 15/21] imsm: FIX: Remove unused variables and code Adam Kwolek
2011-06-08 16:11 ` [PATCH 16/21] FIX: Move buffer to next location Adam Kwolek
2011-06-08 16:11 ` [PATCH 17/21] imsm: FIX: Do not continue reshape when backup exists Adam Kwolek
2011-06-08 16:11 ` [PATCH 18/21] imsm: FIX: wait_for_reshape_imsm() cleanup Adam Kwolek
2011-06-08 16:12 ` [PATCH 19/21] imsm: FIX: Remove timeout from wait_for_reshape_imsm() Adam Kwolek
2011-06-08 16:12 ` [PATCH 20/21] imsm: Optimize expansion speed when no backup is required Adam Kwolek
2011-06-08 16:12 ` [PATCH 21/21] MAN: Man update for check-pointing Adam Kwolek
2011-06-09  3: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=20110609130329.42bf05bb@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=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).