Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-17 Thread Andi Kleen
On Wed, Feb 17, 2016 at 11:26:16AM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 16, 2016 at 07:10:37PM -0800, Andi Kleen escreveu: > > > Need to investigate why there are kernel samples when --all-user is used > > > and > > > the other way around as well. > > > This is a known issue due t

Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-17 Thread Arnaldo Carvalho de Melo
Em Wed, Feb 17, 2016 at 03:39:38PM +0100, Jiri Olsa escreveu: > On Tue, Feb 16, 2016 at 01:56:01PM -0300, Arnaldo Carvalho de Melo wrote: > > Em Mon, Feb 15, 2016 at 09:34:31AM +0100, Jiri Olsa escreveu: > > > Allow user to easily switch all events to user or > > > kernel space with simple --all-us

Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-17 Thread Jiri Olsa
On Tue, Feb 16, 2016 at 01:56:01PM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Feb 15, 2016 at 09:34:31AM +0100, Jiri Olsa escreveu: > > Allow user to easily switch all events to user or > > kernel space with simple --all-user or --all-kernel > > options. > > > > This will be handy within per

Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-17 Thread Arnaldo Carvalho de Melo
Em Tue, Feb 16, 2016 at 07:10:37PM -0800, Andi Kleen escreveu: > > Need to investigate why there are kernel samples when --all-user is used and > > the other way around as well. > This is a known issue due to skid. PEBS usually avoids it. Right, thanks for pointing this out, we have to detect tha

Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-16 Thread Andi Kleen
> Need to investigate why there are kernel samples when --all-user is used and > the other way around as well. This is a known issue due to skid. PEBS usually avoids it. -Andi

Re: [PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-16 Thread Arnaldo Carvalho de Melo
Em Mon, Feb 15, 2016 at 09:34:31AM +0100, Jiri Olsa escreveu: > Allow user to easily switch all events to user or > kernel space with simple --all-user or --all-kernel > options. > > This will be handy within perf mem/c2c wrappers to > switch easily monitoring modes. Humm, some oddities: # per

[PATCH 01/23] perf record: Add --all-user/--all-kernel options

2016-02-15 Thread Jiri Olsa
Allow user to easily switch all events to user or kernel space with simple --all-user or --all-kernel options. This will be handy within perf mem/c2c wrappers to switch easily monitoring modes. Link: http://lkml.kernel.org/n/tip-hdd2u0y1o8cwfpplrpzc1...@git.kernel.org Signed-off-by: Jiri Olsa --