On Tue, 08 Mar 2011 14:24:55 +0100 Adam Kwolek <adam.kwolek@xxxxxxxxx> wrote:
> Takeover has to occur when md reports different raid level.
> Previously it bases on information from metadata.
> Now metadata reports 'working' level as array level.
> This makes us to check it using information from md.
>
> Signed-off-by: Adam Kwolek <adam.kwolek@xxxxxxxxx>
> ---
>
> Grow.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Grow.c b/Grow.c
> index 88c8ad7..5c5a3a3 100644
> --- a/Grow.c
> +++ b/Grow.c
> @@ -1688,7 +1688,7 @@ static int reshape_array(char *container, int fd, char *devname,
> goto release;
> }
>
> - if (reshape.level != info->array.level) {
> + if (reshape.level != array.level) {
> char *c = map_num(pers, reshape.level);
> int err;
> if (c == NULL)
This patch is correct after all. However the description
at the top doesn't really to justice to the subtly of why it is needed.
I have fixed that in the version that I committed.
NeilBrown
> @@ -1708,7 +1708,7 @@ static int reshape_array(char *container, int fd, char *devname,
> if (!quiet)
> fprintf(stderr, Name ": level of %s changed to %s\n",
> devname, c);
> - orig_level = info->array.level;
> + orig_level = array.level;
> sysfs_freeze_array(info);
>
> if (reshape.level > 0 && st->ss->external) {
--
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
