On Wed, Dec 1, 2010 at 6:06 PM, Arnaldo Carvalho de Melo
<a...@ghostprotocols.net> wrote:
> Em Wed, Dec 01, 2010 at 05:00:05PM +0200, Stephane Eranian escreveu:
>> This patch adds an option (-x) to print counts using a CSV-style output.
>> This makes it very easy to import counts directly into your favorite
>> spreadsheet without having to write scripts.
>
> I was about to work on this :-) I think we should use the same option
> 'report' uses:
>
> OPT_STRING('t', "field-separator", &symbol_conf.field_sep, "separator",
> "separator for columns, no spaces will be added between "
> "columns '.' is reserved."),
>
> [r...@mica ~]# perf record -F 100000 ls > /dev/null
> [ perf record: Woken up 1 times to write data ]
> [ perf record: Captured and wrote 0.041 MB perf.data (~1798 samples) ]
> [r...@mica ~]# perf report --stdio -t, | head -10
> # Events: 1K cycles
> #
> # Overhead,Command,Shared Object,Symbol
> 52.57, ls,libc-2.5.so ,[.] __GI___strcoll_l
> 3.97, ls,ls ,[.] 24a2
> 3.48, ls,libc-2.5.so ,[.] __GI_strlen
> 2.33, ls,[ext3] ,[k] ext3fs_dirhash
> 2.21, ls,[kernel.kallsyms],[k] clear_page_c
> 1.94, ls,[ext3] ,[k] ext3_htree_store_dirent
> 1.81, ls,[kernel.kallsyms],[k] rt_spin_lock_fastunlock
> [r...@mica ~]#
>
> Spaces are being added, gack, will fix. Tried to use the same option letter
> and
> long option name as in 'sort':
>
> -t, --field-separator=SEP
> use SEP instead of non-blank to blank transition
>
> But then 'perf stat' already uses -t for --tid, so in 'stat' we would have to
> use '-x'/--field-separator.
>
Fine with me.
I can re-spin the patch to add the flexibility to name your SEP.
> Argh, I think we should stop using short options, only assigning something
> when
> it gets from seldomly used to just before making it the default 8-)
I tend to use the short options....
------------------------------------------------------------------------------
Increase Visibility of Your 3D Game App & Earn a Chance To Win $500!
Tap into the largest installed PC base & get more eyes on your game by
optimizing for Intel(R) Graphics Technology. Get started today with the
Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs.
http://p.sf.net/sfu/intelisp-dev2dev
_______________________________________________
perfmon2-devel mailing list
perfmon2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel
