Re: [PATCH 01/13] Manage reshape process in manage_reshape vector.

[NeilBrown <neilb@suse.de>]

On Mon, 10 Jan 2011 12:27:33 +0100 Adam Kwolek <adam.kwolek@intel.com> wrote:

> The manage_reshape vector is intended to be be used as
> reshape process supervisor for external metadata types.
> Sometimes there is no need for separate control,
> and main code for native meta can be used instead.
> 
> Signed-off-by: Krzysztof Wojcik <krzysztof.wojcik@intel.com>
> Signed-off-by: Adam Kwolek <adam.kwolek@intel.com>
> ---


Thanks.
I have address this issue a different way.
->manage_reshape is still required, but it can trivially be implemented by
calling child_monitor to use the native backup mechanism.

NeilBrown


> 
>  Grow.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/Grow.c b/Grow.c
> index 1222263..4cbe427 100644
> --- a/Grow.c
> +++ b/Grow.c
> @@ -574,8 +574,7 @@ static int reshape_super(struct supertype *st, long long size, int level,
>  	/* nothing extra to check in the native case */
>  	if (!st->ss->external)
>  		return 0;
> -	if (!st->ss->reshape_super ||
> -	    !st->ss->manage_reshape) {
> +	if (!st->ss->reshape_super) {
>  		fprintf(stderr, Name ": %s metadata does not support reshape\n",
>  			st->ss->name);
>  		return 1;
> @@ -1940,7 +1939,7 @@ static int reshape_array(char *container, int fd, char *devname,
>  	}
>  
>  	start_reshape(sra);
> -	if (st->ss->external) {
> +	if ((st->ss->external) && (st->ss->manage_reshape)) {
>  		/* metadata handler takes it from here */
>  		ping_manager(container);
>  		st->ss->manage_reshape(st, backup_file);