Messages in this thread | | | Date | Tue, 23 Jun 2009 10:27:12 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH 2/2 -tip] perf_counter: parse-events.c introduce alias member in event_symbol |
| |
* Jaswinder Singh Rajput <jaswinder@kernel.org> wrote:
> L1-data-Cache-Load-Referencees 398303881 > L1-data-Cache-Load-Misses 3552374 > L1-data-Cache-Store-Referencees 270178 > L1-data-Cache-Store-Misses <inactive> > L1-data-Cache-Prefetch-Referencees 611622 > L1-data-Cache-Prefetch-Misses 399730 > L1-instruction-Cache-Load-Referencees 124696447 > L1-instruction-Cache-Load-Misses 2912802 > L1-instruction-Cache-Store-Referencees <inactive> > L1-instruction-Cache-Store-Misses <inactive> > L1-instruction-Cache-Prefetch-Referencees 156576 > L1-instruction-Cache-Prefetch-Misses <inactive> > L2-Cache-Load-Referencees 4312353 > L2-Cache-Load-Misses 470382 > L2-Cache-Store-Referencees 4392945 > L2-Cache-Store-Misses <inactive> > L2-Cache-Prefetch-Referencees <inactive> > L2-Cache-Prefetch-Misses <inactive> > Data-TLB-Cache-Load-Referencees 127076487 > Data-TLB-Cache-Load-Misses 1930048 > Data-TLB-Cache-Store-Referencees <inactive> > Data-TLB-Cache-Store-Misses <inactive> > Data-TLB-Cache-Prefetch-Referencees <inactive> > Data-TLB-Cache-Prefetch-Misses <inactive> > Instruction-TLB-Cache-Load-Referencees 132768077 > Instruction-TLB-Cache-Load-Misses 6406 > Instruction-TLB-Cache-Store-Referencees <inactive> > Instruction-TLB-Cache-Store-Misses <inactive> > Instruction-TLB-Cache-Prefetch-Referencees <inactive>
there's a typo somewhere: it's References, not Referencees.
> Instruction-TLB-Cache-Prefetch-Misses <inactive> > Branch-Cache-Load-Referencees 58030210 > Branch-Cache-Load-Misses 3257804 > Branch-Cache-Store-Referencees <inactive> > Branch-Cache-Store-Misses <inactive> > Branch-Cache-Prefetch-Referencees <inactive> > Branch-Cache-Prefetch-Misses <inactive>
btw., these names should be shortened somehow.
Instruction-TLB-Cache-Prefetch-Referencees could become:
iTLB-prefetch-refs
Data-TLB-Cache-Store-Referencees could become:
dTLB-store-refs
etc. (You might want to send us a list of abbreviations first, before coding it, to avoid unnecessary work - it will need a few iterations.)
The long versions should be recognized too, if someone prefers those (i.e. our alias detection should be extensive and permissive), but the names we _display_ should be short.
Ingo
| |