On Tue, Jun 15, 2010 at 07:54:13PM +0200, Jan Kara wrote: > But this is buggy isn't it? writeback_inodes_sb() returns as soon as > flusher threads start submitting the IO. So we release s_umount too early - > flusher threads can be still writing out inodes for our work. It's exactly > cases like this why I don't like the work to be called "done" when flusher > threads actually still do it... Now that writeback_inodes_sb uses bdi_queue_work_onstack, wb_do_writeback doesn't signal completion until I/O is submitted. So the locking should be safe. -- 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