linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Neil Brown <neilb@suse.de>
To: Krzysztof Wojcik <krzysztof.wojcik@intel.com>
Cc: linux-raid@vger.kernel.org, wojciech.neubauer@intel.com,
	adam.kwolek@intel.com, dan.j.williams@intel.com,
	ed.ciechanowski@intel.com
Subject: Re: [PATCH 0/2] Release 3.2 summary
Date: Fri, 28 Jan 2011 11:26:06 +1000	[thread overview]
Message-ID: <20110128112606.076f6d49@nbeee.brown> (raw)
In-Reply-To: <20110127154905.11703.56960.stgit@gklab-128-111.igk.intel.com>

On Thu, 27 Jan 2011 17:46:42 +0100
Krzysztof Wojcik <krzysztof.wojcik@intel.com> wrote:

> Neil,
> 
> Intel plans to include following features in mdadm-3.2 release:
> - Array Auto Rebuild,
> - Online Capacity Expansion,
> - Takeover raid10->raid0,
> - Takeover raid0->raid10,
> - Takeover raid1->raid0
> 
> They are almost incorporated into your repository and tested.
> Some of patches are waiting for your acceptation and some will be send
> today.
> We're getting closer to the end of release and I would like to
> summarize and explain what exactly needs to be done.
> 
> To finish our features following patches have to be appyied:
> mdadm:
> - FIX: Meet SET_ARRAY_INFO ioctl requirements (this series)
> - FIX: Remove disks in mdmon for external metadata (this series)
> - WORKAROUND: mdadm hangs during reshape (PART #2)
> - FIX: delta_disk can have UnSet value
> - imsm: FIX: spare cannot be added (very imprtand fix- blocks many
> tests)
> - FIX: start_reshape status should be checked
> - FIX: Container can be left frozen
> - FIX: Array after takeover has to be frozen
> - imsm: FIX: do not allow for container operation for the same disks
> number
> - imsm: Update metadata for second array
> kernel:
> - Add raid1->raid0 takeover support
> - md/raid5: FIX: reshape on degraded devices has wrong configuration
> - md: do not write resync checkpoint, if max_sector has beeen reached
> - unblock the creation of an external metadata RAID if native one
> exists
> 
> Could you review patches above and eventually apply them, please?
> 
> Besides "Assemble: allow to assemble spares on their own" patch has
> to be discused and final solution has to be found.
> 
> After all patches will be applyied we expect all planned featues will
> be working and we can finish the 3.2 release. Of course we will
> continue our testing after official release and new defects may be
> discovered.
> 
> And question for you. When do you plan to announce official 3.2
> release? Can we assume that this will be at the beginning of next
> week?

I think I have applied and pushed out all of the patches in my inbox
that I can agree with or easily fix.  There are a few that raise real
issues but that I am not happy with the approach.  And some that I
don't agree with at all.  But as you say, most have been resolved.

One issue that hasn't been addressed much at all is restarting an array
that is in the middle of a reshape.  I want to at least get that
working for native arrays before I release 3.2.  Getting that working
for external metadata will probably have to wait for 3.2.1.

I very much hope to release 3.2 on Monday.  I will announce it as a
'developer' release - it mostly works but needs lots of testing before
anyone depends on it.

My minimum requirement for 3.2 is that the mdadm test suite passes.

Thanks,
NeilBrown

> 
> ---
> 
> Krzysztof Wojcik (2):
>       FIX: Remove disks in mdmon for external metadata
>       FIX: Meet SET_ARRAY_INFO ioctl requirements
> 
> 
>  Grow.c |   35 +++++++++++++++++++++++------------
>  1 files changed, 23 insertions(+), 12 deletions(-)
> 


  parent reply	other threads:[~2011-01-28  1:26 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-27 16:46 [PATCH 0/2] Release 3.2 summary Krzysztof Wojcik
2011-01-27 16:46 ` [PATCH 1/2] FIX: Remove disks in mdmon for external metadata Krzysztof Wojcik
2011-01-27 16:46 ` [PATCH 2/2] FIX: Meet SET_ARRAY_INFO ioctl requirements Krzysztof Wojcik
2011-01-28  1:26 ` Neil Brown [this message]
2011-01-29 17:52   ` [PATCH 0/2] Release 3.2 summary Wojcik, Krzysztof
2011-01-30 23:44     ` NeilBrown
2011-01-31  6:34     ` NeilBrown

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=20110128112606.076f6d49@nbeee.brown \
    --to=neilb@suse.de \
    --cc=adam.kwolek@intel.com \
    --cc=dan.j.williams@intel.com \
    --cc=ed.ciechanowski@intel.com \
    --cc=krzysztof.wojcik@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).