From mboxrd@z Thu Jan 1 00:00:00 1970 From: Miao Xie Subject: Re: [PATCH 1/4] vfs: introduce try_to_writeback_inodes_sb(_nr) Date: Fri, 27 Apr 2012 16:06:00 +0800 Message-ID: <4F9A5368.5000904@gmail.com> References: <4F98B9A7.8060209@gmail.com> <20120426031156.GY9541@dastard> <20120426151159.GB25603@dhcp231-144.rdu.redhat.com> Reply-To: miaoxie1984@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Xie Miao , Dave Chinner , Alexander Viro , Christoph Hellwig , Linux FSDevel , Linux Btrfs , miaox@cn.fujitsu.com, Kamal Mostafa , Linux Ext4 To: Josef Bacik Return-path: In-Reply-To: <20120426151159.GB25603@dhcp231-144.rdu.redhat.com> Sender: linux-ext4-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org =E4=BA=8E 2012=E5=B9=B404=E6=9C=8826=E6=97=A5 23:12, Josef Bacik =E5=86= =99=E9=81=93: > On Thu, Apr 26, 2012 at 03:55:52PM +0800, Xie Miao wrote: >> On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner = 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 introduc= e 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 m= odule. >> >=20 > I agree with David, there's no sense in making something completely s= eperate, > 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 behavio= r they can > creating something different for themselves. Thanks, OK, I'll make them together. Thanks Miao -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html