Re: [perfmon2] [PATCH] Handle API changes for python

2011-02-15 Thread Arun Sharma
On Tue, Feb 15, 2011 at 11:48:17PM +0100, Stephane Eranian wrote: > > Why do you need to remove #if 0 code? I don't need to. Feel free to ignore that part of the patch :) -Arun -- The ultimate all-in-one performance to

Re: [perfmon2] [PATCH] Handle API changes for python

2011-02-15 Thread Stephane Eranian
On Tue, Feb 15, 2011 at 9:20 PM, Arun Sharma wrote: > > * Remove unimplemented functions from headers (so swig doesn't generate code > for them) > * Rename the output argument so swig can handle it properly. > * Pass the new pfm_os_t arg properly > --- >  include/perfmon/pfmlib.h |    3 +-- >  li

[perfmon2] [PATCH] Handle API changes for python

2011-02-15 Thread Arun Sharma
* Remove unimplemented functions from headers (so swig doesn't generate code for them) * Rename the output argument so swig can handle it properly. * Pass the new pfm_os_t arg properly --- include/perfmon/pfmlib.h |3 +-- lib/pfmlib_common.c | 44 --

Re: [perfmon2] [PATCH 1/2] perf_events: add cgroup support (v9)

2011-02-15 Thread stephane eranian
On Tue, Feb 15, 2011 at 3:55 PM, Peter Zijlstra wrote: > On Mon, 2011-02-14 at 11:20 +0200, Stephane Eranian wrote: >> +                       if (mode & PERF_CGROUP_SWOUT) { >> +                               cpu_ctx_sched_out(cpuctx, EVENT_ALL); >> +                               /* >> +        

Re: [perfmon2] [PATCH 1/2] perf_events: add cgroup support (v9)

2011-02-15 Thread Peter Zijlstra
On Mon, 2011-02-14 at 11:20 +0200, Stephane Eranian wrote: > + if (mode & PERF_CGROUP_SWOUT) { > + cpu_ctx_sched_out(cpuctx, EVENT_ALL); > + /* > +* must not be done before ctxswout dur