Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCHSET 00/20] perf report: Add -F option for specifying output fields (v5) | Date | Fri, 16 May 2014 15:43:37 +0900 |
| |
On Thu, 15 May 2014 15:54:57 +0200, Jiri Olsa wrote: > On Mon, May 12, 2014 at 03:28:33PM +0900, Namhyung Kim wrote: > > SNIP > >> $ perf report -F sym,sample -s overhead >> ... >> # Symbol Samples Overhead >> # .......................... ............ ........ >> # >> [.] strcmp 11 13.75% >> [.] a 8 10.00% >> [.] b 8 10.00% >> [.] c 8 10.00% >> [.] main 7 8.75% >> [.] _setjmp 6 7.50% >> [.] _init 5 6.25% >> [.] frame_dummy 5 6.25% >> [.] __libc_csu_init 4 5.00% >> [.] _dl_name_match_p 4 5.00% >> [.] __new_exitfn 3 3.75% >> [.] __cxa_atexit 2 2.50% >> [.] _dl_check_map_versions 1 1.25% >> [.] _dl_setup_hash 1 1.25% >> [.] _dl_sysdep_start 1 1.25% >> [.] brk 1 1.25% >> [.] calloc@plt 1 1.25% >> [.] dl_main 1 1.25% >> [.] match_symbol 1 1.25% >> [.] sbrk 1 1.25% >> [.] strlen 1 1.25% >> >> >> * changes in v5: >> - add a testcase for hist output sorting > > cool, had just few questions.. otherwise it seems ok to me
Thanks for your review. I'll address your points and resend v6 next week.
Thanks, Namhyung
| |