On Thu, Apr 21, 2011 at 12:10:11PM +0800, Wu Fengguang wrote: > > > Still, given wb_writeback() is the only caller of both > > > __writeback_inodes_sb and writeback_inodes_wb(), I'm wondering if > > > moving the queue_io calls up into wb_writeback() would clean up this > > > logic somewhat. I think Jan mentioned doing something like this as > > > well elsewhere in the thread... > > > > Unfortunately they call queue_io() inside the lock.. > > OK, let's try moving up the lock too. Do you like this change? :) Yes, very much ;) Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- 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
