* [RFC] perf script: modify field selection option
@ 2017-11-20 20:53 Stephane Eranian
2017-11-20 22:37 ` David Ahern
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Stephane Eranian @ 2017-11-20 20:53 UTC (permalink / raw)
To: LKML
Cc: Jiri Olsa, Arnaldo Carvalho de Melo, Peter Zijlstra, mingo,
Andi Kleen, David Ahern
Hi,
I have been using the perf script -F option on the latest perf and I
find it not very convenient to use. I appreciate the + and - prefix to
field names to add or suppress them. But most of the time, I want to
print only one or two fields and I have to guess which ones are there
by default so I can suppress them. I think there should be a way to
say: start from no fields. I understand why you have default to
maintain compatibility with older perf script but I would like a
syntax to say: remove defaults. For instance:
$ perf script -F --,+ip,+syms .....
Where -- would mean drop all defaults.
Any better suggestions?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC] perf script: modify field selection option
2017-11-20 20:53 [RFC] perf script: modify field selection option Stephane Eranian
@ 2017-11-20 22:37 ` David Ahern
2017-11-20 22:50 ` Milian Wolff
2017-11-21 14:35 ` Arnaldo Carvalho de Melo
2 siblings, 0 replies; 6+ messages in thread
From: David Ahern @ 2017-11-20 22:37 UTC (permalink / raw)
To: Stephane Eranian, LKML
Cc: Jiri Olsa, Arnaldo Carvalho de Melo, Peter Zijlstra, mingo,
Andi Kleen
On 11/20/17 1:53 PM, Stephane Eranian wrote:
> Hi,
>
> I have been using the perf script -F option on the latest perf and I
> find it not very convenient to use. I appreciate the + and - prefix to
> field names to add or suppress them. But most of the time, I want to
> print only one or two fields and I have to guess which ones are there
> by default so I can suppress them. I think there should be a way to
> say: start from no fields. I understand why you have default to
> maintain compatibility with older perf script but I would like a
> syntax to say: remove defaults. For instance:
>
> $ perf script -F --,+ip,+syms .....
>
> Where -- would mean drop all defaults.
>
> Any better suggestions?
>
-all,... ?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC] perf script: modify field selection option
2017-11-20 20:53 [RFC] perf script: modify field selection option Stephane Eranian
2017-11-20 22:37 ` David Ahern
@ 2017-11-20 22:50 ` Milian Wolff
2017-11-20 23:21 ` Stephane Eranian
2017-11-21 14:35 ` Arnaldo Carvalho de Melo
2 siblings, 1 reply; 6+ messages in thread
From: Milian Wolff @ 2017-11-20 22:50 UTC (permalink / raw)
To: Stephane Eranian
Cc: LKML, Jiri Olsa, Arnaldo Carvalho de Melo, Peter Zijlstra, mingo,
Andi Kleen, David Ahern
On Montag, 20. November 2017 21:53:04 CET Stephane Eranian wrote:
> Hi,
>
> I have been using the perf script -F option on the latest perf and I
> find it not very convenient to use. I appreciate the + and - prefix to
> field names to add or suppress them. But most of the time, I want to
> print only one or two fields and I have to guess which ones are there
> by default so I can suppress them. I think there should be a way to
> say: start from no fields. I understand why you have default to
> maintain compatibility with older perf script but I would like a
> syntax to say: remove defaults. For instance:
>
> $ perf script -F --,+ip,+syms .....
>
> Where -- would mean drop all defaults.
>
> Any better suggestions?
Isn't `perf script -F ip,sym` what you want? Note the lack of any '+':
$ perf script -F ip,sym | head -n 5
206aad x86_pmu_enable
380591 ctx_resched
380b46 __perf_event_enable
378716 event_function
Cheers
--
Milian Wolff | milian.wolff@kdab.com | Senior Software Engineer
KDAB (Deutschland) GmbH&Co KG, a KDAB Group company
Tel: +49-30-521325470
KDAB - The Qt Experts
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC] perf script: modify field selection option
2017-11-20 22:50 ` Milian Wolff
@ 2017-11-20 23:21 ` Stephane Eranian
2017-11-21 14:36 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 6+ messages in thread
From: Stephane Eranian @ 2017-11-20 23:21 UTC (permalink / raw)
To: Milian Wolff
Cc: LKML, Jiri Olsa, Arnaldo Carvalho de Melo, Peter Zijlstra, mingo,
Andi Kleen, David Ahern
On Mon, Nov 20, 2017 at 2:50 PM, Milian Wolff <milian.wolff@kdab.com> wrote:
> On Montag, 20. November 2017 21:53:04 CET Stephane Eranian wrote:
>> Hi,
>>
>> I have been using the perf script -F option on the latest perf and I
>> find it not very convenient to use. I appreciate the + and - prefix to
>> field names to add or suppress them. But most of the time, I want to
>> print only one or two fields and I have to guess which ones are there
>> by default so I can suppress them. I think there should be a way to
>> say: start from no fields. I understand why you have default to
>> maintain compatibility with older perf script but I would like a
>> syntax to say: remove defaults. For instance:
>>
>> $ perf script -F --,+ip,+syms .....
>>
>> Where -- would mean drop all defaults.
>>
>> Any better suggestions?
>
> Isn't `perf script -F ip,sym` what you want? Note the lack of any '+':
>
> $ perf script -F ip,sym | head -n 5
>
> 206aad x86_pmu_enable
> 380591 ctx_resched
> 380b46 __perf_event_enable
> 378716 event_function
>
Ah, yes, looks like it.
problem solved then.
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC] perf script: modify field selection option
2017-11-20 20:53 [RFC] perf script: modify field selection option Stephane Eranian
2017-11-20 22:37 ` David Ahern
2017-11-20 22:50 ` Milian Wolff
@ 2017-11-21 14:35 ` Arnaldo Carvalho de Melo
2 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2017-11-21 14:35 UTC (permalink / raw)
To: Stephane Eranian
Cc: LKML, Jiri Olsa, Peter Zijlstra, mingo, Andi Kleen, David Ahern
Em Mon, Nov 20, 2017 at 12:53:04PM -0800, Stephane Eranian escreveu:
> Hi,
>
> I have been using the perf script -F option on the latest perf and I
> find it not very convenient to use. I appreciate the + and - prefix to
> field names to add or suppress them. But most of the time, I want to
> print only one or two fields and I have to guess which ones are there
> by default so I can suppress them. I think there should be a way to
> say: start from no fields. I understand why you have default to
> maintain compatibility with older perf script but I would like a
> syntax to say: remove defaults. For instance:
>
> $ perf script -F --,+ip,+syms .....
>
> Where -- would mean drop all defaults.
$ perf script -F -*,+ip,+syms
But perhaps using a new option and do away with all those prefixes?
- Arnaldo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC] perf script: modify field selection option
2017-11-20 23:21 ` Stephane Eranian
@ 2017-11-21 14:36 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2017-11-21 14:36 UTC (permalink / raw)
To: Stephane Eranian
Cc: Milian Wolff, LKML, Jiri Olsa, Peter Zijlstra, mingo, Andi Kleen,
David Ahern
Em Mon, Nov 20, 2017 at 03:21:01PM -0800, Stephane Eranian escreveu:
> On Mon, Nov 20, 2017 at 2:50 PM, Milian Wolff <milian.wolff@kdab.com> wrote:
> > On Montag, 20. November 2017 21:53:04 CET Stephane Eranian wrote:
> >> Hi,
> >>
> >> I have been using the perf script -F option on the latest perf and I
> >> find it not very convenient to use. I appreciate the + and - prefix to
> >> field names to add or suppress them. But most of the time, I want to
> >> print only one or two fields and I have to guess which ones are there
> >> by default so I can suppress them. I think there should be a way to
> >> say: start from no fields. I understand why you have default to
> >> maintain compatibility with older perf script but I would like a
> >> syntax to say: remove defaults. For instance:
> >>
> >> $ perf script -F --,+ip,+syms .....
> >>
> >> Where -- would mean drop all defaults.
> >>
> >> Any better suggestions?
> >
> > Isn't `perf script -F ip,sym` what you want? Note the lack of any '+':
> >
> > $ perf script -F ip,sym | head -n 5
> >
> > 206aad x86_pmu_enable
> > 380591 ctx_resched
> > 380b46 __perf_event_enable
> > 378716 event_function
> >
> Ah, yes, looks like it.
> problem solved then.
> Thanks.
Even more intuitive :-)
- Arnaldo
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-11-21 14:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-20 20:53 [RFC] perf script: modify field selection option Stephane Eranian
2017-11-20 22:37 ` David Ahern
2017-11-20 22:50 ` Milian Wolff
2017-11-20 23:21 ` Stephane Eranian
2017-11-21 14:36 ` Arnaldo Carvalho de Melo
2017-11-21 14:35 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).