public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

  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