From: Leigh Orf <orf@mailbag.com>
To: Rik van Riel <riel@conectiva.com.br>
Cc: Ken Brownfield <brownfld@irridia.com>,
Mike Galbraith <mikeg@wen-online.de>,
"M.H.VanLeeuwen" <vanl@megsinet.net>,
Mark Hahn <hahn@physics.mcmaster.ca>,
Andrew Morton <akpm@zip.com.au>,
linux-kernel@vger.kernel.org
Subject: Re: 2.4.16 memory badness (fixed?)
Date: Mon, 10 Dec 2001 10:49:24 -0500 [thread overview]
Message-ID: <200112101549.fBAFnOq08395@orp.orf.cx> (raw)
In-Reply-To: Your message of "Mon, 10 Dec 2001 09:10:42 -0200." <Pine.LNX.4.33L.0112100909250.4755-100000@duckman.distro.conectiva>
Rik van Riel wrote:
| On Mon, 10 Dec 2001, Ken Brownfield wrote:
|
| > What about moving the calls to shrink_[di]cache_memory()
| > after the nr_pages check after the call to kmem_cache_reap?
| > Or perhaps keep it at the beginning, but only call it
| > after priority has gone a number of notches down from
| > DEF_PRIORITY?
| >
| > Something like that seems like the only obvious way to
| > balance how soon these caches are flushed without over- or
| > under-kill.
|
| So obvious that it's been re-introduced 3 times now even
| though it broke each time. ;)
And in fact, after furthur playing around with the "fixed" version
(moving shrink_[id]cache_memory to the top of vmscan.c::shrink_caches)
I find that I still will get ENOMEM after updatedb occasionally. Less
often than before, but it still happens.
| The only way to get stuff balanced somewhat is to call
| the shrink functions unconditionally. It's not optimally
| balanced, but at least the cache will stay reasonably small
| while still being able to grow under load.
I just can't understand why the kernel wouldn't tag application memory
as being more important tan buff/cache and free up some of that stuff
when an application calls for it. I mean, it won't even use the gobs of
swap I have. That just seems to be a plain ol' bug to me.
Leigh Orf
next prev parent reply other threads:[~2001-12-10 15:51 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <Pine.LNX.4.33.0112090808250.6883-100000@mikeg.weiden.de>
2001-12-09 16:07 ` Leigh Orf
2001-12-09 17:17 ` Mike Galbraith
2001-12-10 7:24 ` Ken Brownfield
2001-12-10 11:10 ` Rik van Riel
2001-12-10 15:49 ` Leigh Orf [this message]
2001-12-10 16:29 ` M. Edward Borasky
2001-12-12 9:04 ` Helge Hafting
2001-12-14 22:54 ` Mike Galbraith
2001-12-09 17:32 ` Mike Galbraith
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=200112101549.fBAFnOq08395@orp.orf.cx \
--to=orf@mailbag.com \
--cc=akpm@zip.com.au \
--cc=brownfld@irridia.com \
--cc=hahn@physics.mcmaster.ca \
--cc=linux-kernel@vger.kernel.org \
--cc=mikeg@wen-online.de \
--cc=riel@conectiva.com.br \
--cc=vanl@megsinet.net \
/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®