linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@digeo.com>
To: Rik van Riel <riel@conectiva.com.br>
Cc: lkml <linux-kernel@vger.kernel.org>, linux-mm@kvack.org
Subject: Re: 2.5.49-mm2
Date: Wed, 27 Nov 2002 10:28:16 -0800	[thread overview]
Message-ID: <3DE50EC0.31354C37@digeo.com> (raw)
In-Reply-To: <Pine.LNX.4.44L.0211270930510.4103-100000@imladris.surriel.com>

Rik van Riel wrote:
> 
> On Wed, 27 Nov 2002, Andrew Morton wrote:
> 
> > +pf_memdie.patch
> >
> >  Fix the PF_MEMDIE logic
> 
> The first part of the patch looks suspicious. If PF_MEMALLOC
> is set we shouldn't be allowed to go into try_to_free_pages()
> in the first place, should we ?

Long story.  Someone sent out a 2.4 patch quite a long time ago to
preserve PF_MEMALLOC in there because they were running userspace
processes as PF_MEMALLOC.  These were realtime "userspace device drivers"
which actually provided block driver services to the kernel.

When you think about it, that's not totally dumb, and all the recursion
protection etc works OK.  Supporting it is just a two-liner, so...

hm.  OK, let's forget that idea ;)

> ...
> > page-reclaim-motion.patch
> >   Move reclaimable pages to the tail ofthe inactive list on IO completion
> 
> Very nice, though if you're worried about effective reclaiming
> you might be interested in Arjan's O(1) VM code, which I'll
> probably forward-port to 2.5 once I've got it properly tuned.

2.5 tends to refile pages more than 2.4, in preference to blocking
on them (the latency thing).  Of course this blows CPU and perverts
page aging (not that the LRU lists add much value in page aging under 
heavy loads anyway...)

Under stupid qsbench loads this patch took the reclaimed/scanned ratio
from ~15% to ~25% and reduced runtime from 7min 45sec to 6min 42sec.

Yup, splitting the lists up would make sense.  Of course, the interrupt-time
motion is "ideal" in that the right pages are placed in the right place
at the right time - we never have to scan past pages which are still
under IO due to elevator reordering, device speed differences, etc...
  
> > activate-unreleaseable-pages.patch
> >   Move unreleasable pages onto the active list
> 
> Interesting, does this make much difference ?

My notes are not clear :(  No, I wouldn't expect it to make a lot
of difference.  I was seeing quite a lot of normal zone pages which
were pinned by buffers getting churned around on the inactive list.
Things like ext2 group descriptor blocks, etc. 

There shouldn't normally be many of these, but there may be some
scenarios in which there are a lot of these, and the inactive list
gets really small due to large amounts of pinned memory.
--
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/

  reply	other threads:[~2002-11-27 18:28 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-27  9:11 2.5.49-mm2 Andrew Morton
2002-11-27 11:38 ` 2.5.49-mm2 Rik van Riel
2002-11-27 18:28   ` Andrew Morton [this message]
2002-11-27 16:10 ` [PATCH] page walker bugfix (was: 2.5.49-mm2) Ingo Oeser
2002-11-27 19:36   ` Andrew Morton
2002-11-27 20:01 ` 2.5.49-mm2 Rasmus Andersen
2002-11-27 20:11   ` 2.5.49-mm2 Andrew Morton
2002-11-27 20:24     ` 2.5.49-mm2 Rasmus Andersen
2002-11-27 21:19     ` 2.5.49-mm2 Rasmus Andersen

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=3DE50EC0.31354C37@digeo.com \
    --to=akpm@digeo.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=riel@conectiva.com.br \
    /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