[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20101117150330.139251f9.akpm@linux-foundation.org>
Date: Wed, 17 Nov 2010 15:03:30 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: Wu Fengguang <fengguang.wu@...el.com>
Cc: Jan Kara <jack@...e.cz>, Christoph Hellwig <hch@....de>,
Dave Chinner <david@...morbit.com>,
"Theodore Ts'o" <tytso@....edu>,
Chris Mason <chris.mason@...cle.com>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Mel Gorman <mel@....ul.ie>, Rik van Riel <riel@...hat.com>,
KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
linux-mm <linux-mm@...ck.org>, <linux-fsdevel@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 00/13] IO-less dirty throttling v2
On Wed, 17 Nov 2010 12:27:20 +0800
Wu Fengguang <fengguang.wu@...el.com> wrote:
> On a simple test of 100 dd, it reduces the CPU %system time from 30% to 3%, and
> improves IO throughput from 38MB/s to 42MB/s.
The changes in CPU consumption are remarkable. I've looked through the
changelogs but cannot find mention of where all that time was being
spent?
How well have these changes been tested with NFS?
The changes are complex and will probably do Bad Things for some
people. Does the code implement sufficient
debug/reporting/instrumentation to enable you to diagnose, understand
and fix people's problems in the minimum possible time? If not, please
add that stuff. Just go nuts with it. Put it in debugfs, add /*
DELETEME */ comments and we can pull it all out again in half a year or
so.
Or perhaps litter the code with temporary tracepoints, provided we can
come up with a way for our testers to trivially gather their output.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists