[PATCH 0/3] deal with imsm spares in Incremental

[PATCH 0/3] deal with imsm spares in Incremental

am 04.01.2011 16:03:04 von anna.czarnowska

Hi Neil,
Some time ago you suggested that Incremental may migrate spares from spare-container
when it is ready to start array as it is impossible to know array's domain before we have all members.
Here are patches that do this.
First two patches modify slightly existing functions to be able to use them in Incremental too.
Third one does the work.
Regards
Anna

0001-Use-one-function-chosing-spares-from-container.patch
0002-move_spare-function-modified-and-moved-to-Manage.c.patc h
0003-Incremental-move-suitable-spares-to-container-when-s.pa tch

Incremental.c | 45 +++++++++++++++++++++++++
Manage.c | 44 ++++++++++++++++++++++++
Monitor.c | 102 +++++++++-----------------------------------------------
mdadm.h | 7 +++-
super-intel.c | 46 ++++++++++++++++++++-----
5 files changed, 149 insertions(+), 95 deletions(-)
--
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

Re: [PATCH 0/3] deal with imsm spares in Incremental

am 05.01.2011 04:44:23 von NeilBrown

On Tue, 4 Jan 2011 15:03:04 +0000 "Czarnowska, Anna"
wrote:

> Hi Neil,
> Some time ago you suggested that Incremental may migrate spares from spare-container
> when it is ready to start array as it is impossible to know array's domain before we have all members.
> Here are patches that do this.
> First two patches modify slightly existing functions to be able to use them in Incremental too.
> Third one does the work.
> Regards
> Anna

Thanks. Looks good.
>
> 0001-Use-one-function-chosing-spares-from-container.patch

I rearrange some of the code in this one as you left a generic function in
super-intel.c. I moved container_chose_spares to util.c and renamed it to
container_choose_spares (and extra 'o').

> 0002-move_spare-function-modified-and-moved-to-Manage.c.patc h
applied.

> 0003-Incremental-move-suitable-spares-to-container-when-s.pa tch

I rearranged the code a little bit to reduce the level of indentation
required - it was getting *really* deep!

Thanks,

NeilBrown



>
> Incremental.c | 45 +++++++++++++++++++++++++
> Manage.c | 44 ++++++++++++++++++++++++
> Monitor.c | 102 +++++++++-----------------------------------------------
> mdadm.h | 7 +++-
> super-intel.c | 46 ++++++++++++++++++++-----
> 5 files changed, 149 insertions(+), 95 deletions(-)
> --
> 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

--
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