Re: [PATCH] cgroup(fix critical bug): new handling for tasks file

From: Paul Menage
Date: Tue Aug 26 2008 - 18:44:38 EST


On Mon, Aug 25, 2008 at 10:22 PM, Lai Jiangshan <laijs@xxxxxxxxxxxxxx> wrote:
> It's complicated than necessary and change too much code IMO.

What about the problem that maintaining a single pid array can still
fail for a really large cgroup? I guess we could just say "don't
create such large cgroups" but someone's bound to want to do that.
Perhaps use an array of pages rather than a single large kmalloc?

Paul
--
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/