Re: [PATCH md 000 of 7] Introduction

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

 



Neil Brown <neilb@xxxxxxxxxxxxxxx> wrote:
>
> On Wednesday August 3, akpm@xxxxxxxx wrote:
> > NeilBrown <neilb@xxxxxxxxxxxxxxx> wrote:
> > >
> > > 
> > > Following are 7 patches for md in 2.6.13-rc4
> > > They are all fairly well tested, with the possible exception of '4' -
> > > I haven't actually tried throwing BIO_RW_BARRIER requests are any md
> > > devices.  However the code is very straight forward.
> > > 
> > > I'm happy (even keen) for these to go into 2.6.13.
> > > If it's getting a bit late, then 2 is probably the most important.
> > > The others we can probably live without.
> > > 
> > 
> > hm, OK.  Merging 1) and 2) seems sane.  I must say that I worry about 4)
> > and would prefer to defer things if poss.
> 
> Fair enough.  4 can wait until 2.6.14 (only a couple of months away,
> right :-)

Thereabouts..

> > 
> > If there are others there which you really would prefer to see in 2.6.13
> > then please let me know.
> > 
> 
> 5, 6, and 7 I would really prefer to be in 2.6.13.

OK.

> The intent-bitmap stuff is broken (in small but potentially
> significant ways) without them, and they are complete no-ops if
> bitmaps aren't enabled:
>   5 only touches bitmap.c
>   6 removes the setting for R1BIO_Degraded which is not used and
>      slightly re-arranges an 'if' statement.  All other changes are 
>      complete no-ops if bitmaps aren't enabled.
>   7: if bitmaps aren't enabled, all this does is call "wait_event"
>      exactly the same way as will very possibly be called a few lines
>      later when md_update_sb is called.
> 
> So I'm convinced (and hopefully convincing) that they don't have any
> significant effect if bitmaps aren't enabled, and fix genuine problems
> with bitmaps, and so are appropriate for 2.6.13...

We'll see ;)
-
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