Messages in this thread | | | Date | Wed, 23 Jan 2013 14:59:49 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 08/13] perf gtk/browser: Add support for event group view |
| |
Em Tue, Jan 22, 2013 at 02:58:36PM +0900, Namhyung Kim escreveu: > Hi Arnaldo, > > On Wed, 16 Jan 2013 15:28:59 -0300, Arnaldo Carvalho de Melo wrote: > > Em Wed, Jan 16, 2013 at 07:25:02PM +0900, Namhyung Kim escreveu: > >> Adding current header name to event name will fix the problem but it > >> probably occupies too much screen width especially for long named > >> tracepoint or PMU-specific events like > >> "compaction:mm_compaction_isolate_migratepages". > >> > >> Overhead/branches Overhead/branch-misses sys/branches sys/branch-misses usr/branches usr/branch-misses Command Shared Object Symbol > >> ................. ...................... ............ ................. ............ ................. ....... ............. ....... > >> 98.32% 31.16% 0.00% 0.00% 98.32% 31.16% a.out a.out [.] foo > >> > >> > >> If you have a better idea or other way to place the cursor without > >> printing bogus 0.00% on GTK, please let me know. > > > > Compacting it using an extra line: > > > > Overhead............... sys.................... usr.................... > > branches branch-misses branches branch-misses branches branch-misses Command Shared Object Symbol > > ........ ............. ........ ............. ........ ............. ....... ............. ....... > > 98.32% 31.16% 0.00% 0.00% 98.32% 31.16% a.out a.out [.] foo > > > > It could even use some reference: > > > > Overhead..................... sys......... usr........... > > branches(1) branch-misses(2) (1) (2) (1) (2) Command Shared Object Symbol > > ........... ................ ..... ..... ...... ...... ....... ............. ....... > > 98.32% 31.16% 0.00% 0.00% 98.32% 31.16% a.out a.out [.] foo > > > > The (1) could be done with a superscript number or even just using a > > different fore/background color, to use fewer columns. > > > > One other way, that would scale for really long event names, would be to > > have the event list in the first few lines and then: > > > > Events: > > 1. branches > > 2. branch-misses > > > > Overhead...... sys......... usr........... > > (1) (2) (1) (2) (1) (2) Command Shared Object Symbol > > ...... ...... ..... ..... ...... ...... ....... ............. ....... > > 98.32% 31.16% 0.00% 0.00% 98.32% 31.16% a.out a.out [.] foo > > > > I think you could switch to/from each of these forms using a hotkey, > > that would influence how the hist_entry__snprintf() routine would work, > > either using perf_evsel__name() or evsel->idx :-) > > > > This way if at some point the user wants to expand/compress the lines, > > it will be possible to do so quickly, just pressing the hotkey. > > By saying hotkey, I guess you meant to use it for TUI. However TUI > doesn't provide those header lines. ;-)
See? I was trying to hit two rabbits with one stone, but you realised that...
> As this extra line (and hotkey) thing might add complexity to the > patchset, I'd like to separate it to a different work and to focus on > the basic feature with current behavior. Is it acceptable for you?
Sure thing.
- Arnaldo
| |