From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 3/6] writeback: sync expired inodes first in background writeback Date: Thu, 21 Apr 2011 00:36:06 -0400 Message-ID: <20110421043606.GB22423@infradead.org> References: <20110419073523.GF23985@dastard> <20110419095740.GC5257@quack.suse.cz> <20110419125616.GA20059@localhost> <20110420012120.GK23985@dastard> <20110420025321.GA14398@localhost> <20110421004547.GD1814@dastard> <20110421020617.GB12191@localhost> <20110421030152.GG1814@dastard> <20110421035954.GA15461@localhost> <20110421041010.GA18710@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Dave Chinner , Jan Kara , Andrew Morton , Mel Gorman , Mel Gorman , Trond Myklebust , Itaru Kitayama , Minchan Kim , LKML , "linux-fsdevel@vger.kernel.org" , Linux Memory Management List To: Wu Fengguang Return-path: Content-Disposition: inline In-Reply-To: <20110421041010.GA18710@localhost> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Thu, Apr 21, 2011 at 12:10:11PM +0800, Wu Fengguang wrote: > OK, let's try moving up the lock too. Do you like this change? :) I like it, especially as it kills the horribly named __writeback_inodes_sb that I added a while ago.