On Thu, 10 Feb 2011 14:55:53 +0100 Adam Kwolek <adam.kwolek@xxxxxxxxx> wrote:
> This few patches prepares mdadm for level and chunk size migration using external metadata.
> 1. FIX: delta_disk can have UnSet value
> A while ago I've reported problem that delta_disks can have UnSet value, so adding it
> to raid_disks can produce error in mdadm. Looking my first patch you proposed
> to address this problem by changing reshape_super() interface rather than
> set condition on function call. This patch implements your idea.
>
> 2. FIX: Get spares from external metadata
> imsm: FIX: Add spare disks information to array description
> Those 2 patches addresses problem with not initializes spares counter in mdinfo
> using external metadata.
>
> 3. FIX: Add raid5 to raid0 case to analyse_change()
> This patch adds missing of level transition in Grow.c
I've applied all of these though I changed the last one to:
--- a/Grow.c
+++ b/Grow.c
@@ -1056,6 +1056,7 @@ char *analyse_change(struct mdinfo *info, struct reshape *re)
info->array.layout = ALGORITHM_PARITY_N;
case 5:
switch (info->new_level) {
+ case 0:
case 4:
re->level = info->array.level;
re->before.data_disks = info->array.raid_disks - 1;
as I think that is all that is required.
Thanks,
NeilBrown
>
> This series should be applied on devel-3.2 branch with my yesterday's 4 patches
>
> BR
> Adam
>
> ---
>
> Adam Kwolek (4):
> FIX: Add raid5 to raid0 case to analyse_change()
> imsm: FIX: Add spare disks information to array description
> FIX: Get spares from external metadata
> FIX: delta_disk can have UnSet value
>
>
> Grow.c | 36 ++++++++++++++++++++++++++++++------
> mdadm.h | 3 ++-
> super-intel.c | 14 +++++++++++++-
> 3 files changed, 45 insertions(+), 8 deletions(-)
>
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
