perf symbol: Move sym_entry->skip to symbol->ignore
Commit-ID: 171b3be9c42e97cd4530706654242f6a3efb6ac3 Gitweb: http://git.kernel.org/tip/171b3be9c42e97cd4530706654242f6a3efb6ac3 Author: Arnaldo Carvalho de Melo <[email protected]> AuthorDate: Fri, 11 Mar 2011 13:36:01 -0300 Committer: Arnaldo Carvalho de Melo <[email protected]> CommitDate: Fri, 11 Mar 2011 13:36:01 -0300
perf symbol: Move sym_entry->skip to symbol->ignore
While going thru each of the sym_entry fields looking to reduce it to the set of entries needed when in an active symbols list, 'skip' should really be in symbol, as we set it when loading the symtab.
And the space used by the basic symbol allocation remains the same as we had 5 bytes of padding.
Cc: Frederic Weisbecker <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Mike Galbraith <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Tom Zanussi <[email protected]>
LKML-Reference: <new-submission>
Signed-off-by Arnaldo Carvalho de Melo <[email protected]>
--- tools/perf/builtin-top.c | 4 ++-- tools/perf/util/symbol.h | 5 +++++ tools/perf/util/top.h | 1 - 3 files changed, 7 insertions(+), 3 deletions(-)
tools/perf/builtin-top.c +2 -2
@@ -676,7 +676,7 @@ static int symbol_filter(struct map *map, struct symbol *sym)for (i =0 ; skip_symbols[i]; i++) {if (!strcmp (skip_symbols[i], name)) { - syme->skip =1 ; + sym->ignore = true;break ; } }
@@ -779,7 +779,7 @@ static void perf_event__process_sample(const union perf_event *event, } syme =symbol__priv (al.sym); -if (!syme->skip) { +if (!al.sym->ignore) {struct perf_evsel *evsel; evsel =perf_evlist__id2evsel (top.evlist, sample->id);
tools/perf/util/symbol.h +5 -0
@@ -48,12 +48,17 @@ char *strxfrchar(char *s, char from, char to);#define BUILD_ID_SIZE 20 +/** struct symbol - symtab entry +* +* @ignore - resolvable but tools ignore it (e.g. idle routines) +*/ struct symbol {struct rb_node rb_node;u64 start;u64 end;u16 namelen;u8 binding; +bool ignore;char name[0 ]; };
tools/perf/util/top.h +0 -1
@@ -16,7 +16,6 @@ struct sym_entry {struct list_head node;unsigned long snap_count;double weight; -int skip;struct map *map;unsigned long count[0 ]; };