Re: [Libcg-devel] [PATCH 0/4] adds cgsnapshot tool

2010-10-04 Thread dhaval.gi...@gmail.com
On Mon, Oct 4, 2010 at 2:18 PM, Ivana Hutarova Varekova wrote: > The patchset has four patches: > 1/ cgsnapshot tool itself. This tool contains the option to ignore some > variables (blacklist) >    or output only the chosen variables (whitelist). > 2/ the man page for cgsnapshot tool > 3/ the cgs

Re: [Libcg-devel] [PATCH] This patch adds cgsnapshot tool

2010-09-29 Thread dhaval.gi...@gmail.com
On Mon, Sep 27, 2010 at 12:41 PM, Jan Safranek wrote: > On 09/27/2010 08:47 AM, Ivana Hutarova Varekova wrote: >> >> On 09/24/2010 01:53 PM, [email protected] wrote: >>> >>> OK, I am not happy with this patch. I want to change the mechanism >>

Re: [Libcg-devel] [PATCH 2/4] adds cgsnapshot man page

2010-09-24 Thread dhaval.gi...@gmail.com
On Fri, Sep 24, 2010 at 11:37 AM, Ivana Hutarova Varekova wrote: > CHANGELOG v2 >        * incorporate dhavalś feedback >                whitelist -> blacklist > >    CHANGELOG v1 >        * fix jsafranek comments > > > This patch adds cgsnapshot man page > > > Signed-off-by: Ivana Hutarova Vareko

Re: [Libcg-devel] [PATCH] This patch adds cgsnapshot tool

2010-09-24 Thread dhaval.gi...@gmail.com
On Fri, Sep 24, 2010 at 12:16 PM, Ivana Hutarova Varekova wrote: > CHANGELOGS: >   fixes the typos in the previous version > >        CHANGELOG v4 >        * incorporate dhaval feedback: >                * whitelist -> blacklist >                * add cgroup_free_cgroup > >        CHANGELOG v3 >  

Re: [Libcg-devel] sysdefault group ?

2010-09-23 Thread dhaval.gi...@gmail.com
On Thu, Sep 23, 2010 at 2:52 PM, Tommaso Cucinotta wrote: > Il 23/09/2010 12:37, Balbir Singh ha scritto: >> * Tommaso Cucinotta  [2010-09-23 12:26:26]: >> >> >>> Il 23/09/2010 12:01, Balbir Singh ha scritto: >>> There is a simple way to disable this Create /etc/sysconfig/cgconfig and ad

Re: [Libcg-devel] [PATCH 1/4] This patch adds cgsnapshot tool

2010-09-23 Thread dhaval.gi...@gmail.com
> > OK, the patch is fine. I am though wondering if it is possible to > split this patch up into smaller pieces. > BTW, what sort of testing has been done. did you try a file which does a = but no Y or N? What about something like cpu.shares=Y cpu.shares=N in the same file? What happens if the

Re: [Libcg-devel] [PATCH 3/4] the default cgsnapshot configuration file

2010-09-23 Thread dhaval.gi...@gmail.com
On Fri, Sep 17, 2010 at 3:22 PM, Ivana Hutarova Varekova wrote: > CHANGELOG v1 >        * fix balbair commnets: remove devices.(deny/allow) > Balbir :-) > the default cgsnapshot configuration file > contains all variables from  2.6.34 > I am going to ask you to drop this file. Probably just a b

Re: [Libcg-devel] [PATCH 2/4] adds cgsnapshot man page

2010-09-23 Thread dhaval.gi...@gmail.com
On Fri, Sep 17, 2010 at 3:22 PM, Ivana Hutarova Varekova wrote: > CHANGELOG v1 >        * fix jsafranek comments > > > This patch adds cgsnapshot man page > > > Signed-off-by: Ivana Hutarova Varekova > --- > >  doc/man/Makefile.am  |    2 +- >  doc/man/cgsnapshot.1 |   65 > +

Re: [Libcg-devel] [PATCH 1/4] This patch adds cgsnapshot tool

2010-09-23 Thread dhaval.gi...@gmail.com
On Fri, Sep 17, 2010 at 3:22 PM, Ivana Hutarova Varekova wrote: > CHANGELOGS: >        CHANGELOG v3 >        * white/blacklist is cached >        * incorporate jsafraneks feedback > >        CHANGELOG v2 >        * add whitelist which describe the set of variables which can be dealt > by > cgsnap