Re: [PATCH 1/2] modified message on failure to read metadata in Manage

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

 



On Thu, 17 Feb 2011 22:43:19 +0000 "Czarnowska, Anna"
<anna.czarnowska@xxxxxxxxx> wrote:

> >From 347ed6b46a18374cc6743f2fdfccd136ea502331 Mon Sep 17 00:00:00 2001
> From: Anna Czarnowska <anna.czarnowska@xxxxxxxxx>
> Date: Wed, 16 Feb 2011 13:10:29 +0100
> Subject: [PATCH 1/2] modified message on failure to read metadata in Manage
> Cc: linux-raid@xxxxxxxxxxxxxxx, Williams, Dan J <dan.j.williams@xxxxxxxxx>, Ciechanowski, Ed <ed.ciechanowski@xxxxxxxxx>
> 
> Loading container may fail if e.g. one of the disks in container
> has been detached but udev has not realized the change.
> Addition to such array will fail because reading superblock
> from one of disks in array fails.
> Current message is a bit confusing.
> 
> Signed-off-by: Anna Czarnowska <anna.czarnowska@xxxxxxxxx>
> ---
>  Manage.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/Manage.c b/Manage.c
> index 481c165..ea5d70f 100644
> --- a/Manage.c
> +++ b/Manage.c
> @@ -637,7 +637,7 @@ int Manage_subdevs(char *devname, int fd,
>  				/* FIXME this is a bad test to be using */
>  				if (!tst->sb) {
>  					close(tfd);
> -					fprintf(stderr, Name ": cannot find valid superblock in this array - HELP\n");
> +					fprintf(stderr, Name ": cannot load array metadata from %s\n", devname);
>  					return 1;
>  				}
>  


This and the next one both applied - thanks.

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


[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