From: Bill Davidsen <davidsen@tmr.com>
To: linux-kernel@vger.kernel.org
Subject: Re: [VM PATCH 2.6.6-rc3-bk5] Dirty balancing in the presence of mapped pages
Date: Sun, 09 May 2004 13:24:54 -0400 [thread overview]
Message-ID: <c7lovl$4ic$1@gatekeeper.tmr.com> (raw)
In-Reply-To: <40985C91.9080809@yahoo.com.au>
Nick Piggin wrote:
>> So we need to understand why it was written, and what effects were
>> observed, with what workload, and all that good stuff.
>>
>
> I guess it is an attempt to do somewhat better at dirty page accounting
> for mmap'ed pages. The balance_dirty_pages_ratelimited writeout path
> also has the same problem as you describe. Maybe usage patterns means
> this is less of an issue here?
>
> But I suppose it wouldn't be nice to change without seeing some
> improvement somewhere.
Lots of issues here, writing in random blocks can lead to fragmentation
if the data is newly allocated, but won't change fragmenting if the page
mapped is alread allocated on the disk.
Is it practical or desirable to be writing mapped pages of allready
allocated files back more readily, since it avoid all the allocation
issues? But you still need to limit dirty pages, so at some point it
will be necessary to do the allocation, preferably in an optimal way.
>
>>
>>> It doesn't do the wakeup_bdflush thing, but that sounds
>>> like a good idea. What does wakeup_bdflush(-1) mean?
>>
>>
>>
>> It appears that it will cause pdflush to write out down to
>> dirty_background_ratio.
>>
>
> Yeah. So wakeup_bdflush(0) would be more consistent?
--
bill davidsen <davidsen@tmr.com>
CTO TMR Associates, Inc
Doing interesting things with small computers since 1979
next prev parent reply other threads:[~2004-05-09 17:18 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-05-05 0:20 [VM PATCH 2.6.6-rc3-bk5] Dirty balancing in the presence of mapped pages Shantanu Goel
2004-05-05 1:03 ` Andrew Morton
2004-05-05 2:16 ` Nick Piggin
2004-05-05 2:57 ` Andrew Morton
2004-05-05 3:16 ` Nick Piggin
2004-05-05 3:24 ` Nick Piggin
2004-05-09 17:24 ` Bill Davidsen [this message]
2004-05-05 4:31 ` Shantanu Goel
2004-05-05 5:25 ` Andrew Morton
2004-05-05 16:56 ` Nikita Danilov
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='c7lovl$4ic$1@gatekeeper.tmr.com' \
--to=davidsen@tmr.com \
--cc=linux-kernel@vger.kernel.org \
/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