Messages in this thread | | | Date | Wed, 28 Aug 2013 11:21:12 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH] perf mem: add priv level filtering support |
| |
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 <acme@redhat.com> 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_symbols hide user symbols > > > > Well, I don't know what perf top does here but I don't want to hide > the samples. I simply don't want to collect them (do not appear > in the perf.data file). If that's what is happening in perf top, then > I'll be glad to use the same options.
Indeed, its for different purposes, 'perf top' when used with one of those options will still collect samples for all priv levels and will just toggle a flag to not zap the ones asked not to show when decaying the samples.
When the user presses 'U' or 'K' on the UI, the flags gets toggled and samples start being considered/zapped.
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
I think it would stay consistent and clear, what do you think?
- Arnaldo
| |