On Wed, Oct 29, 2008 at 10:14:35AM +1100, Dave Chinner wrote: > On Wed, Oct 29, 2008 at 01:47:15AM +1100, npiggin@xxxxxxx wrote: > > OK, I'm happier with this patchset now. Note that I've taken your patch > > and mangled it a bit at the end of the series. > > > > This one survives and seems to run OK here, but I'm mainly doing dumb > > stress testing with a handful of filesystems, and data-io error injection > > testing. There are a lot of combinations of ways this function can operate > > and interact obviously, so it would be helpful to get more review. > > > > Chris, would you possibly have time to run your btrfs tests that are > > sensitive to problems in this code? I could provide you a single patch > > rollup against mainline if it helps. > > Nick, after applying the patchset: > > CC mm/page-writeback.o > mm/page-writeback.c: In function write_cache_pages: > mm/page-writeback.c:871: warning: wrteback_index may be used uninitialized in this function > > Looks harmless, but it probably should be cleaned up. > Compiler is 'gcc version 4.3.1 (Debian 4.3.1-9)' Yeah, it's annoying but I couldn't find a way to shut it up nicely (yes AFAIKS it is harmless). I could just put the old "= 0; /* shut up gcc*/" trick there, but that sucks too ;( -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
