[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <p73y7n3fm7t.fsf@bingen.suse.de>
Date: 12 Feb 2007 16:07:50 +0100
From: Andi Kleen <ak@...e.de>
To: Nadia.Derbey@...l.net
Cc: akpm@...l.org, randy.dunlap@...cle.com,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/6] AKT - Tunable structure and registration routines
Nadia.Derbey@...l.net writes:
> +
> +This feature aims at making the kernel automatically change the tunables
> +values as it sees resources running out.
The only reason we have resource limit is to avoid DOS when one
resource consumes too much memory. When there is no such danger then
there isn't any reason to have a limit at all and it could be just
eliminated (or set to unlimited by default)
Your feature doesn't address the DOS and without that there isn't
any reason to have limits at all. So what's the point?
I agree that some of the default limits we have are not very useful
on modern machines. I guess you're trying to address that.
I would suggest the following strategy:
- Review any limits we have and make sure they make sense.
- Figure out if they actually serve a useful purpose
e.g. what happens when they are exceeded, is there a DOS?.
If yes can the DOS be addressed in a better way (e.g. by allowing to shrink
the resource by a shrinker callback).
Some of the existing limits are clearly bogus, e.g. the limit
on shared memory.
For others i don't see a good alternative. e.g. if you don't limit
the number of files allocated the only alternative would be to kill
processes when they allocate too many files. Is that really preferable
to a errno?
- If they serve a useful purpose then check if the default is useful
on a modern machine. Or make them scale with the amount of memory
like many limits already do.
-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists