On Mon, 24 Jan 2011 15:17:04 +0100 Krzysztof Wojcik <krzysztof.wojcik@xxxxxxxxx> wrote: > If reshape_array does not success we should not leave > array freezed. > > Signed-off-by: Krzysztof Wojcik <krzysztof.wojcik@xxxxxxxxx> > --- > Grow.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/Grow.c b/Grow.c > index c5f83a8..7c5edae 100644 > --- a/Grow.c > +++ b/Grow.c > @@ -1567,7 +1567,10 @@ int Grow_reshape(char *devname, int fd, int > quiet, char *backup_file, sync_metadata(st); > rv = reshape_array(container, cfd, fd, devname, > st, &info, force, backup_file, > quiet, 0); > - frozen = 0; > + if (rv) > + frozen = 1; > + else > + frozen = 0; reshape_array is responsible for unfreezing the array in this case, and I'm fairly sure that it already does! NeilBrown > } > release: > if (frozen > 0) -- 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
