On Mon, 10 Jan 2011 12:27:42 +0100 Adam Kwolek <adam.kwolek@xxxxxxxxx> wrote: > Any request for no size change has to be accepted. > > Signed-off-by: Adam Kwolek <adam.kwolek@xxxxxxxxx> > --- > > super-intel.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/super-intel.c b/super-intel.c > index d70a7df..58a776e 100644 > --- a/super-intel.c > +++ b/super-intel.c > @@ -6322,7 +6322,7 @@ static int imsm_reshape_is_allowed_on_container(struct supertype *st, > "st->devnum = (%i)\n", > st->devnum); > > - if (geo->size != -1 || > + if (geo->size >= 0 || > geo->level != UnSet || > geo->layout != UnSet || > geo->chunksize != 0 || As far as I can tell, this is a not op. 'size' will only ever be -1, 0, or positive. Given that, "size != -1" and "size >= 0" produce identical results. If there is some way that 'size' can be some other negative number, we should fix that. NeilBrown -- 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
