On Thu, Apr 26, 2012 at 03:55:52PM +0800, Xie Miao wrote: > 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. > I agree with David, there's no sense in making something completely seperate, this function was introduced soley to kick off background writeout if we could with no garuntees, if the other users suddenly don't like the behavior they can creating something different for themselves. Thanks, Josef -- 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
