Re: [PATCH] imsm: fix, the second array need to have the whole available space on devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 27 Jan 2012 15:28:36 +0000 "Labun, Marcin" <Marcin.Labun@xxxxxxxxx>
wrote:

> Fix the case with creating an array with given container in command line
> instead of real devices:
> mdadm -CR /dev/md/raid0 -l 0 -n 2 -z5G /dev/md/imsm
> 
> Signed-off-by: Marcin Labun <marcin.labun@xxxxxxxxx>
> ---
>  super-intel.c |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
> 
> diff --git a/super-intel.c b/super-intel.c
> index 0e9269f..03e10bd 100644
> --- a/super-intel.c
> +++ b/super-intel.c
> @@ -5525,8 +5525,15 @@ static int reserve_space(struct supertype *st, int raiddisks,
>  			size /= 2 * chunk;
>  			size *= 2 * chunk;
>  		}
> +		maxsize = size;
> +	}
> +	if (!check_env("IMSM_NO_PLATFORM") &&
> +	    mpb->num_raid_devs > 0 && size && size != maxsize) {
> +		fprintf(stderr, Name ": attempting to create a second "
> +			"volume with size less then remaining space. "
> +			"Aborting...\n");
> +		return 0;
>  	}
> -
>  	cnt = 0;
>  	for (dl = super->disks; dl; dl = dl->next)
>  		if (dl->e)

It is a shame that we need the same test in two different places...

I've applied this patch,
Thanks,
NeilBrown

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux