Re: [patch] mempool-2.5.1-D2

From: Benjamin LaHaise (bcrl@redhat.com)
Date: Mon Dec 17 2001 - 15:44:23 EST


On Mon, Dec 17, 2001 at 09:56:07PM +0100, Ingo Molnar wrote:
> sure, the pool will run out under heavy VM load. Will it stay empty
> forever? Nope, because all mempool users are *required* to deallocate the
> buffer after some (reasonable) timeout. (such as IO latency.) This is
> pretty much by definition. (Sure there might be weird cases like IO
> failure timeouts, but sooner or later the buffer will be returned, and it
> will be reused.)

loop. deadlock. kmap. deadlock. You've got a lot of code to fix before
this statement is remotely true.

> (by the way, this is true for every other reservation solution as well,
> just look at the patches. You wont resize on the fly whenever there is
> shortage - thats the problem with shortages, there just wont be more RAM.
> If anyone uses reserved pools and doesnt release those buffers then we are
> deadlocked. Memory reserves *must not* be used as a kmalloc pool. Doing
> that can be considered an advanced form of a 'memory leak'.)

Absolutely. That's why I think we should at least do some work on design
of the code so that we have an idea of what the pitfalls are, plus
documentation before putting it into the kernel.

                -ben

-- 
Fish.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Dec 23 2001 - 21:00:14 EST