Re: [0/1] [patch -mm] Add containerstats (v3)

From: Paul Menage
Date: Sat Jun 09 2007 - 03:10:18 EST


On 6/8/07, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
On Fri, 08 Jun 2007 23:43:46 +0530
Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> wrote:

> This patch implements per container statistics infrastructure and re-uses
> code from the taskstats interface.

boggle.

Symbol: CONTAINERS [=y]
Selected by: CONTAINER_DEBUG || CPUSETS && SMP || CONTAINER_CPUACCT

Paul, that's just bizarre. How come it was done this way?

Containers on their own without any subsystems aren't hugely useful.
So the plan was that any container subsystem would "select CONTAINERS"
to cause the container framework to be built. I'm happy to change it
to invert the dependencies if you'd prefer.

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/