From: Andrea Arcangeli <andrea@suse.de>
To: Nikita Danilov <nikita@clusterfs.com>
Cc: Linux Kernel Mailing List <Linux-Kernel@vger.kernel.org>
Subject: Re: [PATCH] per-task-predictive-write-throttling-1
Date: Fri, 16 Sep 2005 17:34:10 +0200 [thread overview]
Message-ID: <20050916153410.GS4122@opteron.random> (raw)
In-Reply-To: <17193.3830.498682.626783@gargle.gargle.HOWL>
On Thu, Sep 15, 2005 at 10:04:38AM +0400, Nikita Danilov wrote:
> Why is that useful? Don't we want (in general) to cache as many dirty
It allows all tasks to have a chance to take advantage of the writeback
cache, not only the first one like now.
> pages as possible in the hope that some of them will be re-dirtied (thus
> avoiding additional write) or truncated (avoiding write altogether)?
We could add an option to increase the per-task future_dirty pages only
if we had to allocate a new pagecache (i.e. cache miss), that
would leave the rewrite behaviour unchanged, but still if we've a huge
write-hog, we should reserve some cache for other people anyway, so I'm
unsure if we should allow rewriter-hogs to use half the ram and leave
nothing to the other potential small writers. Certainly accounting for
future_dirty only on cache-misses on the radix-tree would still catch
the bad untarring and cp beahviour just fine, so that might be good
enough and I would certainly agree that's more conservative approach.
Initially every task has access to the whole dirty cache, even "cp
/dev/zero ." initially allocates half the ram, but then if it keeps
writing that fast, the dirty cache decreases, to give a chance to other
tasks to avoid blocking while writing.
If the write-hog becomes reasoanable and it slowdown writing, it can
then use more dirty cache again.
prev parent reply other threads:[~2005-09-16 15:33 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-14 22:03 [PATCH] per-task-predictive-write-throttling-1 Andrea Arcangeli
2005-09-15 0:44 ` Con Kolivas
2005-09-15 17:14 ` Andrea Arcangeli
2005-09-16 10:42 ` Anton Blanchard
2005-09-16 15:12 ` Andrea Arcangeli
2005-09-15 6:04 ` Nikita Danilov
2005-09-16 15:34 ` Andrea Arcangeli [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20050916153410.GS4122@opteron.random \
--to=andrea@suse.de \
--cc=Linux-Kernel@vger.kernel.org \
--cc=nikita@clusterfs.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox