Re: [PATCH] perf mem: add priv level filtering support

2013-09-05 Thread Andi Kleen
> But my worry here is about consistency accross tools for the single > letter options, so perhaps if you could use: > > -U collect only user level samples > -K collect only kernel level samples Support for this would be nice for perf stat too, to use with t

Re: [PATCH] perf mem: add priv level filtering support

2013-09-05 Thread Sukadev Bhattiprolu
Arnaldo Carvalho de Melo [a...@redhat.com] wrote: | Em Wed, Aug 28, 2013 at 03:38:28PM +0200, Stephane Eranian escreveu: | > On Wed, Aug 28, 2013 at 3:27 PM, Arnaldo Carvalho de Melo wrote: | > > So perhaps we should change both to (and add this to 'report' as well): | > > | > > -U, --hide_ke

Re: [PATCH] perf mem: add priv level filtering support

2013-08-28 Thread Arnaldo Carvalho de Melo
Em Wed, Aug 28, 2013 at 03:38:28PM +0200, Stephane Eranian escreveu: > On Wed, Aug 28, 2013 at 3:27 PM, Arnaldo Carvalho de Melo > wrote: > > So perhaps we should change both to (and add this to 'report' as well): > > > > -U, --hide_kernel_symbols hide kernel symbols > > -K, --hide_user

Re: [PATCH] perf mem: add priv level filtering support

2013-08-28 Thread Stephane Eranian
Arnaldo, So I just checked perf top and those 2 options are doing user level filtering of the samples. This is different from what I want which is hardware level filtering if avail. On Wed, Aug 28, 2013 at 3:38 PM, Stephane Eranian wrote: > On Wed, Aug 28, 2013 at 3:27 PM, Arnaldo Carvalho de

Re: [PATCH] perf mem: add priv level filtering support

2013-08-28 Thread Stephane Eranian
On Wed, Aug 28, 2013 at 3:27 PM, Arnaldo Carvalho de Melo wrote: > Em Mon, Aug 26, 2013 at 04:21:44PM -0600, David Ahern escreveu: >> On 8/26/13 7:11 AM, Stephane Eranian wrote: > >> >This patch adds the -u -and -k options to perf to allow >> >filtering of load/store sampling based on priv levels.

Re: [PATCH] perf mem: add priv level filtering support

2013-08-28 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 26, 2013 at 04:21:44PM -0600, David Ahern escreveu: > On 8/26/13 7:11 AM, Stephane Eranian wrote: > >This patch adds the -u -and -k options to perf to allow > >filtering of load/store sampling based on priv levels. > >This may not be supported by all HW platforms. > >+++ b/tools/perf

Re: [PATCH] perf mem: add priv level filtering support

2013-08-28 Thread Stephane Eranian
On Tue, Aug 27, 2013 at 12:21 AM, David Ahern wrote: > > On 8/26/13 7:11 AM, Stephane Eranian wrote: >> >> perf mem: add priv level filtering support >> >> This patch adds the -u -and -k options to perf to allow >> filtering of load/store sampling based on priv levels. >> This may not be supported

Re: [PATCH] perf mem: add priv level filtering support

2013-08-26 Thread David Ahern
On 8/26/13 7:11 AM, Stephane Eranian wrote: perf mem: add priv level filtering support This patch adds the -u -and -k options to perf to allow filtering of load/store sampling based on priv levels. This may not be supported by all HW platforms. By default, loads/stores are sampled at both user

[PATCH] perf mem: add priv level filtering support

2013-08-26 Thread Stephane Eranian
perf mem: add priv level filtering support This patch adds the -u -and -k options to perf to allow filtering of load/store sampling based on priv levels. This may not be supported by all HW platforms. By default, loads/stores are sampled at both user and kernel privilege levels. To sample only a