[Devel] Re: [PATCH 3/6] perf: add ability to record event period

2011-12-20 Thread Andrew Vagin
On 12/20/2011 12:58 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Dec 16, 2011 at 11:13:07AM +0400, Andrew Vagin escreveu: Hi Arnaldo, Could you review and commit this patch. It's quite common functionality, which allow to get events more effectively and to avoid losing events. All other patches

[Devel] [PATCH] SUNRPC: search for service transports in network namespace context

2011-12-20 Thread Stanislav Kinsbursky
Service transports are parametrized by network namespace. And thus lookup of transport instance have to take network namespace into account. Signed-off-by: Stanislav Kinsbursky --- fs/lockd/svc.c |2 +- fs/nfsd/nfsctl.c|4 ++-- include/linux/sunrpc/svc_x

[Devel] Re: [PATCH 3/6] perf: add ability to record event period

2011-12-20 Thread Peter Zijlstra
On Tue, 2011-12-20 at 12:07 +0400, Andrew Vagin wrote: > Look at the comment below. In it I try describe why we need this > functionality. Its not about comments, a patch has to have a Changelog that covers the why and what. If it doesn't the patch won't go anywhere. Also, its not the maintainer

Re: [Devel] [PATCH] SUNRPC: search for service transports in network namespace context

2011-12-20 Thread Glauber Costa
On 12/20/2011 03:14 PM, Stanislav Kinsbursky wrote: Service transports are parametrized by network namespace. And thus lookup of transport instance have to take network namespace into account. Signed-off-by: Stanislav Kinsbursky --- fs/lockd/svc.c |2 +- fs/nfsd/nfsctl.c

Re: [Devel] [PATCH] SUNRPC: search for service transports in network namespace context

2011-12-20 Thread Stanislav Kinsbursky
20.12.2011 14:25, Glauber Costa пишет: On 12/20/2011 03:14 PM, Stanislav Kinsbursky wrote: Service transports are parametrized by network namespace. And thus lookup of transport instance have to take network namespace into account. Signed-off-by: Stanislav Kinsbursky --- fs/lockd/svc.c

[Devel] Re: [PATCH 3/6] perf: add ability to record event period

2011-12-20 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 20, 2011 at 12:07:41PM +0400, Andrew Vagin escreveu: > On 12/20/2011 12:58 AM, Arnaldo Carvalho de Melo wrote: > >Em Fri, Dec 16, 2011 at 11:13:07AM +0400, Andrew Vagin escreveu: > >>Hi Arnaldo, > >> > >>Could you review and commit this patch. It's quite common > >>functionality, which

[Devel] [PATCH] perf: add ability to record event period

2011-12-20 Thread Andrew Vagin
The problem is that when SAMPLE_PERIOD is not set, kernel generates a number of samples in proportion to an event's period. Number of these samples may be too big and a kernel throttles all samples above a defined limit. E.g.: I want to trace when a process sleeps. I created a process, which sleep

[Devel] Re: [PATCH] perf: add ability to record event period

2011-12-20 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 20, 2011 at 05:32:45PM +0300, Andrew Vagin escreveu: > diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c > index 766fa0a..f8fd14f 100644 > --- a/tools/perf/builtin-record.c > +++ b/tools/perf/builtin-record.c > @@ -700,6 +700,7 @@ const struct option record_options[

[Devel] [PATCH 01/10] perf record: Add ability to record event period

2011-12-20 Thread Arnaldo Carvalho de Melo
From: Andrew Vagin The problem is that when SAMPLE_PERIOD is not set, the kernel generates a number of samples in proportion to an event's period. Number of these samples may be too big and the kernel throttles all samples above a defined limit. E.g.: I want to trace when a process sleeps. I cre

[Devel] [GIT PULL 00/10] perf/core improvements and fixes

2011-12-20 Thread Arnaldo Carvalho de Melo
Hi Ingo, Please consider pulling from: git://github.com/acmel/linux.git perf/core There are still patchkits I need to merge/review, thanks everybody, the more, the better! I'll continue processing them tomorrow, Regards, - Arnaldo Andrew Vagin (1): perf record: Add

[Devel] Re: [GIT PULL 00/10] perf/core improvements and fixes

2011-12-20 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider pulling from: > > git://github.com/acmel/linux.git perf/core > > There are still patchkits I need to merge/review, thanks everybody, the > more, the better! > > I'll continue processing them tomorrow, > > R