linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Rik van Riel <riel@conectiva.com.br>
To: "Stephen C. Tweedie" <sct@redhat.com>
Cc: Linus Torvalds <torvalds@transmeta.com>, linux-mm@kvack.org
Subject: Re: [patch] VM stable again?
Date: Mon, 15 May 2000 16:10:54 -0300 (BRST)	[thread overview]
Message-ID: <Pine.LNX.4.21.0005151608590.20410-100000@duckman.distro.conectiva> (raw)
In-Reply-To: <20000515200116.E24812@redhat.com>

On Mon, 15 May 2000, Stephen C. Tweedie wrote:
> On Mon, May 15, 2000 at 12:12:03PM -0300, Rik van Riel wrote:
> > 
> > the patch below makes sure processes won't "eat" the pages
> > another process is freeing and seems to avoid the nasty
> > out of memory situations that people have seen.
> 
> One other thought here --- there is another way to achieve this.
> Make try_to_free_pages() return a struct page *.  That will not
> only achieve some measure of SMP locality, it also guarantees
> that the page freed will be reacquired by the task which did the
> work to free it.

I've thought about this but it doesn't seem worth the extra
complexity to me. Just making sure that while our task is
freeing pages nobody else will grab those pages without having
also freed some pages seems to be enough to me.

Furthermore, the "SMP locality" you talk about will probably
be completely overshadowed by the non-locality of the VM
freeing code anyway...

regards,

Rik
--
The Internet is not a network of computers. It is a network
of people. That is its real strength.

Wanna talk about the kernel?  irc.openprojects.net / #kernelnewbies
http://www.conectiva.com/		http://www.surriel.com/

--
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-15 19:10 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-05-15 15:12 Rik van Riel
2000-05-15 15:31 ` Ingo Molnar
2000-05-15 15:27   ` Rik van Riel
2000-05-15 16:15     ` Ingo Molnar
2000-05-15 18:36   ` Linus Torvalds
2000-05-15 18:29 ` Linus Torvalds
2000-05-15 19:01 ` Stephen C. Tweedie
2000-05-15 19:10   ` Rik van Riel [this message]
2000-05-15 19:30     ` Ingo Molnar
2000-05-15 19:27       ` Rik van Riel
2000-05-15 19:22   ` Ingo Molnar

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.21.0005151608590.20410-100000@duckman.distro.conectiva \
    --to=riel@conectiva.com.br \
    --cc=linux-mm@kvack.org \
    --cc=sct@redhat.com \
    --cc=torvalds@transmeta.com \
    /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