On Mon 07-09-09 20:45:18, Jens Axboe wrote: > On Mon, Sep 07 2009, Jan Kara wrote: > > Hi Jens, > > > > now I've found just two minor things (see below). Besides them the only > > thing which is blocking my ack is a way to effectively lookup a BDI from a > > superblock so that we can reasonably effectively fsync a superblock... > > Can we side step that as an inclusion criteria, please? I'd really like > to work on adding that quick lookup, but I'd also hate to potentiall > destabilize anything at this point. So we can probably quite easily make > .32 as well for that, but I'd rather not risk doing a version and > include that from the beginning. OK? Well, for now I thought about a simple hack like a pointer to backing_dev_info in a superblock. That should work for every filesystem right now and shouldn't be too complex... But if you promise you'll fix that up before 2.6.32 is released, I'm OK with your patch as is. I'll give your patches some testing with some less usual benchmarks this week and if that works out fine you have my ack. Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- 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
