From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 2/2] writeback: elevate queue_io() into wb_writeback() Date: Wed, 4 May 2011 07:17:22 -0400 Message-ID: <20110504111722.GB19261@infradead.org> References: <20110426144218.GA14862@localhost> <20110426144402.GA15166@localhost> <20110504110849.GC4646@infradead.org> <20110504111547.GA5441@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , Andrew Morton , Jan Kara , Mel Gorman , Dave Chinner , 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: <20110504111547.GA5441@localhost> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Wed, May 04, 2011 at 07:15:47PM +0800, Wu Fengguang wrote: > > This should move to writeback_inodes_wb and be unconditional as > > wb_writeback already always initializes it. > > Never mind :) wbc->wb_start has been killed in a later patch named > "writeback: avoid extra sync work at enqueue time". Even better. I was already wondering why we'd need two different jiffies and jiffies + offset value in the writeback code. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org