All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Larsson <roger.larsson@norran.net>
To: riel@nl.linux.org
Cc: linux-mm@kvack.org
Subject: Re: PG_referenced and lru_cache (cpu%)...
Date: Sun, 07 May 2000 00:16:40 +0200	[thread overview]
Message-ID: <391499C8.D4FDA2E6@norran.net> (raw)
In-Reply-To: Pine.LNX.4.21.0005061529280.4627-100000@duckman.conectiva

Rik van Riel wrote:
> 
> On Sat, 6 May 2000, Roger Larsson wrote:
> 
> > When _add_to_page_cache adds a page to the lru_cache
> > it forces it to be referenced.
> > In addition it will be added as youngest in list.
> 
> Which is IMHO a good thing, since the page *was* referenced
> and was referenced last.

Ok, I can buy that.

> 
> > When a page is needed it is very likely that a lot of
> > the youngest pages are marked as referenced.
> 
> > order=0 is the only that tries to search the full list.
> 
> No. Referenced pages are not counted, so if we encounter
> a lot of them we will happily age them all without decreasing
> the value of count.

But if it is first in the order==0 run that they are found...

Extreme example:
Suppose all DMA pages are referenced and last in list. No other zones
with pressure.
order = [6..1] will not find them.
order = 0 will, but since they are referenced they are put in young.
shrink_mmap will loop (256 times) without finding them...
And return 0.
order is 0
=> no more shrink_mmap will be called...
=> the pages were not found.

I would feel a lot safer if the young pages was inserted at top
of lru_cache instead of in another local list.
With them in lru_cache they will be searched if it needs to
loop that far...

/RogerL

PS.
  This feels a little like archaeology with a new algorithm
  soon to be released...
  But at leas I can learn a lot from it - Rik thanks for your replies!
DS

--
Home page:
  http://www.norran.net/nra02596/
--
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.eu.org/Linux-MM/

      reply	other threads:[~2000-05-06 22:16 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <8evk0f$7jote$1@fido.engr.sgi.com>
2000-05-06 17:12 ` [DATAPOINT] pre7-6 will not swap Rajagopal Ananthanarayanan
2000-05-06  4:25   ` Benjamin Redelings I
2000-05-06 19:35   ` Linus Torvalds
2000-05-06  5:35     ` Benjamin Redelings I
2000-05-06 21:46       ` Rik van Riel
2000-05-06 22:24         ` Rajagopal Ananthanarayanan
2000-05-06 14:03           ` Benjamin Redelings I
2000-05-07  0:22           ` Rik van Riel
2000-05-07  2:23           ` Linus Torvalds
2000-05-07 17:40             ` Rik van Riel
2000-05-07 17:53               ` Linus Torvalds
2000-05-07 19:13                 ` Rajagopal Ananthanarayanan
2000-05-07 19:30                   ` Linus Torvalds
2000-05-08 20:40                     ` gprof data for pre7-6 Rajagopal Ananthanarayanan
2000-05-09  1:52     ` [DATAPOINT] pre7-6 will not swap Quintela Carreira Juan J.
2000-05-09  2:28       ` Rajagopal Ananthanarayanan
2000-05-09  2:33       ` Linus Torvalds
2000-05-09  3:31         ` Rajagopal Ananthanarayanan
2000-05-09 15:56           ` [DATAPOINT] pre7-8 swaps with FREE mem? Benjamin Redelings I
2000-05-06 20:12   ` PG_referenced and lru_cache (cpu%) Roger Larsson
2000-05-06 18:31     ` Rik van Riel
2000-05-06 22:16       ` Roger Larsson [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=391499C8.D4FDA2E6@norran.net \
    --to=roger.larsson@norran.net \
    --cc=linux-mm@kvack.org \
    --cc=riel@nl.linux.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 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.