All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.