Messages in this thread | | | Date | Wed, 31 Jul 2013 11:17:32 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH V2 1/9] perf tools: add test for reading object code |
| |
Em Wed, Jul 31, 2013 at 12:13:50AM +0300, Adrian Hunter escreveu: > Using the information in mmap events, perf tools can read object > code associated with sampled addresses. A test is added that > compares bytes read by perf with the same bytes read using > objdump.
So this parses objdump output, and we also already have the annotation logic that does that too, have you thought about having common routines for these two cases?
I mean the disasm_line, ins, ins_ops, ins_operands classes, that now lives in util/annotate.h but could be moved somewhere else, disconnecting it as much as possible from annotation, because probably there are more cool things we could do with that... :-)
We could certainly do it incrementally, merging your current patch series and then working on sharing code on these two use cases, but perhaps you can do it now?
What do you think?
- Arnaldo
| |