From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: Re: [PATCH 04/45] writeback: remove unused nonblocking and congestion checks Date: Sat, 10 Oct 2009 21:47:38 +0800 Message-ID: <20091010134738.GA18238@localhost> References: <20091007073818.318088777@intel.com> <20091007074901.625522882@intel.com> <20091009152629.GH7654@duck.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andrew Morton , Theodore Tso , Christoph Hellwig , Dave Chinner , Chris Mason , Peter Zijlstra , "Li, Shaohua" , Myklebust Trond , "jens.axboe@oracle.com" , Nick Piggin , "linux-fsdevel@vger.kernel.org" , LKML To: Jan Kara Return-path: Received: from mga14.intel.com ([143.182.124.37]:64766 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752968AbZJJNsz (ORCPT ); Sat, 10 Oct 2009 09:48:55 -0400 Content-Disposition: inline In-Reply-To: <20091009152629.GH7654@duck.suse.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri, Oct 09, 2009 at 11:26:29PM +0800, Jan Kara wrote: > On Wed 07-10-09 15:38:22, Wu Fengguang wrote: > > - no one is calling wb_writeback and write_cache_pages with > > wbc.nonblocking=1 any more > > - lumpy pageout will want to do nonblocking writeback without the > > congestion wait > > > > So remove the congestion checks as suggested by Chris. > Looks good. Since encountered_congestion isn't used, you can delete it as > well... BTW, you might need to split this patch to per-fs chunks for the > sake of merging. OK, good suggestions to follow :) Thanks, Fengguang