On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner <david@xxxxxxxxxxxxx> wrote: > > writeback_inodes_sb(_nr) grabs s_umount lock when it want to start > > writeback, > > it may bring us deadlock problem when doing umount. So we introduce new > > functions -- try_to_writeback_inodes_sb(_nr) -- which use > > down_read_trylock() > > instead of down_read() to avoid that deadlock problem. > > > > This idea came from Christoph Hellwig. > > Some code is from the patch of Kamal Mostafa. > > This just re-implements writeback_inodes_[nr]_sb_if_idle() with a > trylock instead of a blocking lock. > > Just replace the blocking lock in writeback_inodes_[nr]_sb_if_idle() > with a trylock and use that. The change of these two functions is relative to three modules, so I think the patch set now is easy to be reviewed by the developers of each module. Thanks Miao -- 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