Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755952Ab0FOBlc (ORCPT ); Mon, 14 Jun 2010 21:41:32 -0400 Received: from smtp1.linux-foundation.org ([140.211.169.13]:36925 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755332Ab0FOBlb (ORCPT ); Mon, 14 Jun 2010 21:41:31 -0400 Date: Mon, 14 Jun 2010 18:39:57 -0700 From: Andrew Morton To: Dave Chinner Cc: Mel Gorman , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Chris Mason , Nick Piggin , Rik van Riel , Johannes Weiner , Christoph Hellwig , KAMEZAWA Hiroyuki Subject: Re: [PATCH 11/12] vmscan: Write out dirty pages in batch Message-Id: <20100614183957.ad0cdb58.akpm@linux-foundation.org> In-Reply-To: <20100615003943.GK6590@dastard> References: <1276514273-27693-1-git-send-email-mel@csn.ul.ie> <1276514273-27693-12-git-send-email-mel@csn.ul.ie> <20100614231144.GG6590@dastard> <20100614162143.04783749.akpm@linux-foundation.org> <20100615003943.GK6590@dastard> X-Mailer: Sylpheed 2.7.1 (GTK+ 2.18.9; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3376 Lines: 72 On Tue, 15 Jun 2010 10:39:43 +1000 Dave Chinner wrote: > On Mon, Jun 14, 2010 at 04:21:43PM -0700, Andrew Morton wrote: > > On Tue, 15 Jun 2010 09:11:44 +1000 > > > 10-15% reduction in seeks on simple kernel compile workloads. This > > > shows that if we optimise IO patterns at higher layers where the > > > sort window is much, much larger than in the IO scheduler, then > > > overall system performance improves.... > > > > Yup. > > > > But then, this all really should be done at the block layer so other > > io-submitting-paths can benefit from it. > > That was what we did in the past with really, really deep IO > scheduler queues. That leads to IO latency and OOM problems because > we could lock gigabytes of memory away under IO and take minutes to > clean it. > > Besides, there really isn't the right context in the block layer to > be able to queue and prioritise large amounts of IO without > significant penalties to some higher layer operation. > > > IOW, maybe "the sort queue is the submission queue" wasn't a good idea. > > Perhaps, but IMO sorting should be done where the context allows it > to be done most efficiently. Sorting is most effective when ever a > significant queue of IO is formed, whether it be in the filesystem, > the VFS, the VM or the block layer because the IO stack is very much > a GIGO queue. > > Simply put, there's nothing the lower layers can do to optimise bad > IO patterns from the higher layers because they have small sort > windows which are necessary to keep IO latency in check. Hence if > the higher layers feed the lower layers crap they simply don't have > the context or depth to perform the same level of optimistations we > can do easily higher up the stack. > > IOWs, IMO anywhere there is a context with significant queue of IO, > that's where we should be doing a better job of sorting before that > IO is dispatched to the lower layers. This is still no guarantee of > better IO (e.g. if the filesystem fragments the file) but it does > give the lower layers a far better chance at optimal allocation and > scheduling of IO... None of what you said had much to do with what I said. What you've described are implementation problems in the current block layer because it conflates "sorting" with "queueing". I'm saying "fix that". And... sorting at the block layer will always be superior to sorting at the pagecache layer because the block layer sorts at the physical block level and can handle not-well-laid-out files and can sort and merge pages from different address_spaces. Still, I suspect none of it will improve anything anyway. Those pages are still dirty, possibly-locked and need to go to disk. It doesn't matter from the MM POV whether they sit in some VM list or in the request queue. Possibly there may be some benefit to not putting so many of these unreclaimable pages into the queue all at the the same time. But that's a shortcoming in the block code: we should be able to shove arbitrary numbers of dirty page (segments) into the queue and not gum the system up. Don't try to work around that in the VM. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/