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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.