public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Rik van Riel <riel@redhat.com>
To: Nick Piggin <nickpiggin@yahoo.com.au>
Cc: linux-mm <linux-mm@kvack.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [RFC][PATCH] split file and anonymous page queues #2
Date: Tue, 20 Mar 2007 08:27:59 -0400	[thread overview]
Message-ID: <45FFD34F.10809@redhat.com> (raw)
In-Reply-To: <45FF7B3A.70709@yahoo.com.au>

Nick Piggin wrote:
> Rik van Riel wrote:

>> We apply pressure to each of sets of the pageout queues based on:
>> - the size of each queue
>> - the fraction of recently referenced pages in each queue,
>>    not counting used-once file pages
>> - swappiness (file IO is more efficient than swap IO)

> This ignores whether a file page is mapped, doesn't it?

> Even so, it could be a good approach anyway.

It does, but once it gets the file list down to the size
where it finds that a fair number of the pages were
referenced, it will back off the pressure automatically.

Also, we do not apply the used-once algorithm to mapped
pages, meaning that mapped pages with the accessed bit
set always get rotated back onto the active list, while
unmapped pages do not.

> There are a couple of little nice improvements you have there, such as
> treating shmem pages in the same class as anon pages. We found that we
> needed something similar, so some of those things should go upstream
> on their own.

It will be hard to merge that "on its own" without the
split queues.  I can't really think of a good way to
split this patch up into multiple functional bits...

-- 
Politics is the struggle between those who want to make their country
the best in the world, and those who believe it already is.  Each group
calls the other unpatriotic.

  reply	other threads:[~2007-03-20 12:28 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-20  0:52 [RFC][PATCH] split file and anonymous page queues #2 Rik van Riel
2007-03-20  1:06 ` Rik van Riel
2007-03-20  6:12 ` Nick Piggin
2007-03-20 12:27   ` Rik van Riel [this message]
2007-03-20 16:24 ` Lee Schermerhorn
2007-03-20 17:19   ` Rik van Riel
2007-03-20 18:39 ` Bob Picco
2007-03-20 18:06   ` Rik van Riel

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=45FFD34F.10809@redhat.com \
    --to=riel@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=nickpiggin@yahoo.com.au \
    /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