linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Lameter <clameter@sgi.com>
To: Mel Gorman <mel@skynet.ie>
Cc: linux-mm@kvack.org, apw@shadowen.org
Subject: Re: [RFC] Page allocator: Get rid of the list of cold pages
Date: Thu, 15 Nov 2007 10:16:27 -0800 (PST)	[thread overview]
Message-ID: <Pine.LNX.4.64.0711151010530.27140@schroedinger.engr.sgi.com> (raw)
In-Reply-To: <20071115104004.GC5128@skynet.ie>

On Thu, 15 Nov 2007, Mel Gorman wrote:

> It would improve readahead but if there are active processes looking
> for hot pages, they could be impacted because readahead has used up hot
> pages. Basically, it could go either way but justifying that splitting the
> lists is the right thing to do in all situations is difficult to justify
> too. I think you could justify either approach with about the same amount
> of hand-waving and not be able to prove anything conclusively.

Readahead is a rather slow process so its likely that the competing fast 
allocating process that is faulting in anonymous pages will replenish the 
pcp pages multiple times between accesses of readahead to the pcp pages.

I guess this is some handwaving. However, if there is no conclusive proof 
either way then lets remove it.

> You're welcome. The PPC64 results came through as well. The difference
> between the two kernels is negligible. There are very slight
> improvements with your patch but it's in the noise.
> 
> What I have seen so far is that things are no worse with your patch than
> without which is the important thing.

Right.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

      reply	other threads:[~2007-11-15 18:16 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-13  4:42 Christoph Lameter
2007-11-14 18:41 ` Mel Gorman
2007-11-14 18:51   ` Christoph Lameter
2007-11-15 10:40     ` Mel Gorman
2007-11-15 18:16       ` Christoph Lameter [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=Pine.LNX.4.64.0711151010530.27140@schroedinger.engr.sgi.com \
    --to=clameter@sgi.com \
    --cc=apw@shadowen.org \
    --cc=linux-mm@kvack.org \
    --cc=mel@skynet.ie \
    /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