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

2007-06-09 Thread Paul Menage
On 6/9/07, Andrew Morton <[EMAIL PROTECTED]> wrote: Would it not be simplest to have CONTAINERS as the top-level user-configurable item and to then have everything else depend on it? Yes, OK - it can go that way around too. I guess my thought was that people would be more interested in enabli

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

2007-06-09 Thread Andrew Morton
On Sat, 9 Jun 2007 01:07:53 -0700 "Paul Menage" <[EMAIL PROTECTED]> wrote: > On 6/9/07, Andrew Morton <[EMAIL PROTECTED]> wrote: > > > > - CONTAINER_DEBUG should depend on CONTAINERS > > CONTAINER_DEBUG is actually a container subsystem whose sole purpose > is to provide debugging information abo

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

2007-06-09 Thread Paul Menage
On 6/9/07, Andrew Morton <[EMAIL PROTECTED]> wrote: - CONTAINER_DEBUG should depend on CONTAINERS CONTAINER_DEBUG is actually a container subsystem whose sole purpose is to provide debugging information about any hierarchy that it's mounted as a part of. So in some senses it's in the same boat

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

2007-06-09 Thread Andrew Morton
On Sat, 9 Jun 2007 00:09:55 -0700 "Paul Menage" <[EMAIL PROTECTED]> wrote: > On 6/8/07, Andrew Morton <[EMAIL PROTECTED]> wrote: > > On Fri, 08 Jun 2007 23:43:46 +0530 > > Balbir Singh <[EMAIL PROTECTED]> wrote: > > > > > This patch implements per container statistics infrastructure and re-uses >

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

2007-06-09 Thread Paul Menage
On 6/8/07, Andrew Morton <[EMAIL PROTECTED]> wrote: On Fri, 08 Jun 2007 23:43:46 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > This patch implements per container statistics infrastructure and re-uses > code from the taskstats interface. boggle. Symbol: CONTAINERS [=y] Selected by: CONTAIN

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

2007-06-08 Thread Balbir Singh
Andrew Morton wrote: > On Fri, 08 Jun 2007 23:43:46 +0530 > Balbir Singh <[EMAIL PROTECTED]> wrote: > >> This patch implements per container statistics infrastructure and re-uses >> code from the taskstats interface. > > boggle. > > Symbol: CONTAINERS [=y] > Selected by: CONTAINER_DEBUG || CPU

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

2007-06-08 Thread Balbir Singh
Changelog (v3) 1. Use the container iterator (Paul Menage) 2. delayacct_set_flag() has now been moved to delayacct_blkio_start (Andrew Morton) Changelog (v2) 1. Instead of parsing long container path's use the dentry to match the container for which stats are