public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Martin Dalecki <dalecki@evision-ventures.com>
To: Ingo Oeser <ingo.oeser@informatik.tu-chemnitz.de>
Cc: Jonathan Morton <chromi@cyberspace.org>,
	Rogier Wolff <R.E.Wolff@BitWizard.nl>,
	linux-kernel@vger.kernel.org
Subject: Re: OOM killer???
Date: Tue, 27 Mar 2001 21:07:05 +0200	[thread overview]
Message-ID: <3AC0E4D9.E157D407@evision-ventures.com> (raw)
In-Reply-To: <l03130332b6e632432b9f@[192.168.239.101]> <3AC09480.E8317507@evision-ventures.com> <20010327200830.C8133@nightmaster.csn.tu-chemnitz.de>

Ingo Oeser wrote:
> 
> On Tue, Mar 27, 2001 at 03:24:16PM +0200, Martin Dalecki wrote:
> > > @@ -93,6 +95,10 @@
> > >                                 p->uid == 0 || p->euid == 0)
> > >                 points /= 4;
> > >
> > > +       /* Much the same goes for processes with low UIDs */
> > > +       if(p->uid < 100 || p->euid < 100)
> > > +         points /= 2;
> > > +
> >
> > Plase change to 100 to 500 - this would make it consistant with
> > the useradd command, which starts adding new users at the UID 500
> 
> No, useradd reads usally the /etc/login.defs to select the range.
> The oom-killer should have configurables for that, to allow the
> policy decisions in USER space -- where it belongs -- not in KERNEL space

OK sysctl would be more appripriate.

> If we use my OOM killer API, this patch would be a module and
> could have module parameters to select that.
> 
> Johnathan: I URGE you to apply my patch before adding OOM killer
>    stuff. What's wrong with it, that you cannot use it? ;-)
> 
> It is easy to add configurables to a module and play with them
> WITHOUT recompiling.

It's total overkill and therefore not a good design.

> Dynamic sysctl tables would also be possible, IF we had an value
> that is DEFINED to be invalid for sysctrl(2) and only valid for /proc.
> 
> It is also better to include the egid into the decision. There
> are deamons, that I defintely want to be killed on a workstation,
> but not on a server.
> 
> e.g. My important matlab calculation, which runs in user mode
> should not be killed. But killing a local webserver, which serves
> my help system is ok (because I will not loose work, and might
> get it over the net, if there is a problem).
> 
> So as Rik stated: The OOM killer cannot suit all people, so it
> has to be configurable, to be OOM kill, not overkill ;-)

Irony: Why then not store this information permanently - inside
the UID of the application?

  reply	other threads:[~2001-03-27 19:20 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-03-27 10:59 OOM killer??? Rogier Wolff
2001-03-27 12:14 ` Jonathan Morton
2001-03-27 13:24   ` Martin Dalecki
2001-03-27 15:31     ` Jonathan Lundell
2001-03-27 16:07       ` Config bug? In 2.2.19 CONFIG_RTL8139 depends on CONFIG_EXPERIMENTAL Greg Ingram
2001-03-27 16:14         ` Jeff Garzik
2001-03-27 16:37           ` [PATCH] 2.2.19 drivers/net/Config.in Greg Ingram
2001-03-27 18:08     ` OOM killer??? Ingo Oeser
2001-03-27 19:07       ` Martin Dalecki [this message]
2001-03-27 19:55         ` Andreas Dilger
2001-03-27 21:13           ` Andreas Rogge
2001-03-27 18:37     ` Jonathan Morton
2001-03-27 13:57   ` Jonathan Morton
     [not found] <200103282138.f2SLcT824292@webber.adilger.int>
2001-03-29  9:29 ` Dr. Michael Weller
2001-03-29 11:01   ` Guest section DW
2001-03-29 12:02     ` Sean Hunter
2001-03-29 12:57       ` Guest section DW
2001-03-29 15:41     ` David Konerding
2001-03-29 17:52       ` David Lang
2001-03-30  2:26       ` Michael Peddemors
2001-03-30 14:48         ` J. Scott Kasten
2001-03-29 17:21     ` Stephen Satchell
2001-03-29 13:53   ` Szabolcs Szakacsits
2001-03-29 15:01     ` Dr. Michael Weller
2001-03-29 16:29       ` Szabolcs Szakacsits
2001-03-29 16:51       ` Szabolcs Szakacsits
  -- strict thread matches above, loose matches on Subject: below --
2001-03-29 16:22 Jesse Pollard
2001-03-29 19:20 Jesse Pollard

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=3AC0E4D9.E157D407@evision-ventures.com \
    --to=dalecki@evision-ventures.com \
    --cc=R.E.Wolff@BitWizard.nl \
    --cc=chromi@cyberspace.org \
    --cc=ingo.oeser@informatik.tu-chemnitz.de \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox