Re: [PATCH] fix memory leak in mm/slab.c::alloc_kmemlist()

From: Jesper Juhl
Date: Mon Mar 13 2006 - 16:57:13 EST


On 3/13/06, Christoph Lameter <clameter@xxxxxxx> wrote:
> On Mon, 13 Mar 2006, Jesper Juhl wrote:
>
> > Ok, I've been playing around with a few ways to resolve this, but I'm
> > a bit pressed for time and won't have properly tested patches ready
> > tonight. I will however keep at this, so you'll see patches
> > releatively shortly, just give me another day or two and I'll have
> > this fixed in a nice way (nice little task to work at :) ...
>
> Maybe extract a alloc_kmemlist_node and free_kmemlist_node from
> alloc_kmemlist()? It gets a bit complicated if all of this is handled within
> alloc_kmemlist and having these separate may simplify recovery on out of
> memory.
>
Nice Idea. So far I've been trying to handle it all inside the
function, but maybe sepperating stuff out might simplify it..
anyway, i'll keep working on this and will submit patches in a day or two...
It really is a nice little problem to work on and I'll find a solution
for it (well, I already have at least one, but it's ugly as hell, want
to find a better one) - gimme a few days.

--
Jesper Juhl <jesper.juhl@xxxxxxxxx>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/