Re: [PATCH] imsm: fix: thunderdome may drop 2tb attribute

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

 



On Fri, 27 Apr 2012 08:58:14 +0000 "Czarnowska, Anna"
<anna.czarnowska@xxxxxxxxx> wrote:

> Hi Neil,
> this patch is quite important. It seems to have been overlooked

Yes I had over-looked it.  Thanks for the reminder.  It is applied now.

NeilBrown



> Regards
> Anna
> 
> > -----Original Message-----
> > From: linux-raid-owner@xxxxxxxxxxxxxxx [mailto:linux-raid-
> > owner@xxxxxxxxxxxxxxx] On Behalf Of Czarnowska, Anna
> > Sent: Monday, April 02, 2012 10:59 PM
> > To: NeilBrown
> > Cc: linux-raid@xxxxxxxxxxxxxxx; Williams, Dan J; Ciechanowski, Ed;
> > Grabowski, Grzegorz
> > Subject: [PATCH] imsm: fix: thunderdome may drop 2tb attribute
> > 
> > Spare superblock doesn't depend on other spares in container.
> > When loading container metadata thunderdome
> > may pick a small disk for the champion. This will result in incorrect
> > interpretation of sizes of other disks in container when joint
> > superblock
> > is returned. If any disk in container has the 2TB attribute set, the
> > result
> > must have it set too.
> > 
> > Signed-off-by: Anna Czarnowska <anna.czarnowska@xxxxxxxxx>
> > ---
> >  super-intel.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/super-intel.c b/super-intel.c
> > index ed13542..869a39c 100644
> > --- a/super-intel.c
> > +++ b/super-intel.c
> > @@ -4079,6 +4079,8 @@ imsm_thunderdome(struct intel_super **super_list,
> > int len)
> >  		if (s == champion)
> >  			continue;
> > 
> > +		mpb->attributes |= s->anchor->attributes &
> > MPB_ATTRIB_2TB_DISK;
> > +
> >  		for (i = 0; i < mpb->num_disks; i++) {
> >  			struct imsm_disk *disk;
> > 
> > --
> > 1.6.4.2
> > 
> > --
> > 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
> --
> 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

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