mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrea Arcangeli <andrea@suse.de>
To: Roland Bless <bless@tm.uka.de>
Cc: miquels@cistron.nl, linux-kernel@vger.kernel.org,
	walter@tm.uka.de, winter@tm.uka.de, doll@tm.uka.de,
	Marc-Christian Petersen <m.c.p@wolk-project.de>
Subject: Re: Fix for wrong OOM killer trigger?
Date: Mon, 22 Sep 2003 15:02:02 +0200	[thread overview]
Message-ID: <20030922130202.GB9936@velociraptor.random> (raw)
In-Reply-To: <20030922121140.73f81627.bless@tm.uka.de>

On Mon, Sep 22, 2003 at 12:11:40PM +0200, Roland Bless wrote:
> was a fix for the particular bug. My suggestion is that the log entry
> below describes the bug fix for it:

the kmem cleanup wasn't a bug. So in theory it could be even the leaking
of pagetables that went from -aa to mainline in 23pre1, but I think it
really was the removal of the oom killer with the -aa VM merges that
went into 2.4.23pre[2-5] that really fixed your problem (if it's true
that you had no swap, which I understood it's the case, and no swap puts
at the light the brokeness of the oom killer), that leak is a minor one,
many other places shrinks the per-cpu queues, so it's unlikely to be
able to leak lots of ram in a misc workload.

It's good to hear that pre5 is fixed. thanks.


> 
> Summary of changes from v2.4.22 to v2.4.23-pre1
> ============================================
> ...
> Marc-Christian Petersen:
>   o Cleanup kmem_cache_reap()
> or was it related to this one:
>   o Avoid potentially leaking pagetables into the per-cpu queues
> 
> I hope that it was also fixed in 2.6, or is there a different mechanism
> used?

dunno, but the oom killer certainly has not enough information in 2.6
either to be able to do a reliable decision.

Andrea - If you prefer relying on open source software, check these links:
	    rsync.kernel.org::pub/scm/linux/kernel/bkcvs/linux-2.[45]/
	    http://www.cobite.com/cvsps/
	    svn://svn.kernel.org/linux-2.[46]/trunk

      reply	other threads:[~2003-09-22 13:01 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-19 17:16 Roland Bless
2003-09-19 17:30 ` Marc-Christian Petersen
2003-09-19 19:25 ` Andrea Arcangeli
2003-09-19 19:35   ` Russell King
2003-09-19 20:01     ` Andrea Arcangeli
2003-09-19 20:52       ` Larry McVoy
2003-09-19 21:05         ` Formal complaint , " Mr. James W. Laferriere
2003-09-20  6:36           ` David S. Miller
2003-09-20  3:31         ` Andrea Arcangeli
2003-09-20  4:30           ` Larry McVoy
2003-09-20 12:22             ` Bernd Schmidt
2003-09-20 13:52               ` Willy Tarreau
2003-09-20 13:54               ` Larry McVoy
2003-09-20 19:56                 ` Gateways (was Re: Fix for wrong OOM killer trigger?) Jamie Lokier
2003-09-20 20:14                   ` Larry McVoy
2003-09-21 11:52                     ` David S. Miller
2003-09-20 16:07               ` Fix for wrong OOM killer trigger? Valdis.Kletnieks
2003-09-20 14:23             ` Andrea Arcangeli
2003-09-20 15:13               ` Larry McVoy
2003-09-20 17:14                 ` Flames (was: Fix for wrong OOM killer trigger?) Stephen Satchell
2003-09-20 17:46                   ` Alan Cox
2003-09-21 10:40                 ` Fix for wrong OOM killer trigger? Eric W. Biederman
2003-09-21 14:22                   ` Andrea Arcangeli
2003-09-21 14:52                     ` Larry McVoy
2003-09-21 15:53                       ` Andrea Arcangeli
2003-09-23 12:46                     ` Bas Mevissen
2003-09-20 11:09   ` Roland Bless
2003-09-20 14:34     ` Andrea Arcangeli
2003-09-22 10:11       ` Roland Bless
2003-09-22 13:02         ` Andrea Arcangeli [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=20030922130202.GB9936@velociraptor.random \
    --to=andrea@suse.de \
    --cc=bless@tm.uka.de \
    --cc=doll@tm.uka.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m.c.p@wolk-project.de \
    --cc=miquels@cistron.nl \
    --cc=walter@tm.uka.de \
    --cc=winter@tm.uka.de \
    /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

all inboxes | Powered by JetHome®