diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-01-18 14:56:57 -0500 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-01-24 14:40:51 -0500 |
| commit | 23b6339b914ec009ba206584a410039b589243aa (patch) | |
| tree | a5abfabc39badf66472fed0a3034203353a7bec5 /tools/perf/util | |
| parent | 7c3102b843a581b4b84643a18d423f8807364ca0 (diff) | |
perf tools: Fix usage of __ in event parsing struct names
In tools/perf we use a convention where __ separates the struct name
from the function name for functions that operate on a struct instance.
Fix this usage by removing it from the struct names and fix also the
associated functions.
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Namhyung Kim <namhyung@gmail.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-kdcoh7uitivx68otqcz12aaz@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util')
| -rw-r--r-- | tools/perf/util/parse-events.c | 4 | ||||
| -rw-r--r-- | tools/perf/util/parse-events.h | 4 | ||||
| -rw-r--r-- | tools/perf/util/parse-events.y | 26 |
3 files changed, 17 insertions, 17 deletions
diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index e5c0fcbe8dbe..02f6421f03a0 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c | |||
| @@ -856,7 +856,7 @@ static int parse_events__scanner(const char *str, void *data, int start_token) | |||
| 856 | */ | 856 | */ |
| 857 | int parse_events_terms(struct list_head *terms, const char *str) | 857 | int parse_events_terms(struct list_head *terms, const char *str) |
| 858 | { | 858 | { |
| 859 | struct parse_events_data__terms data = { | 859 | struct parse_events_terms data = { |
| 860 | .terms = NULL, | 860 | .terms = NULL, |
| 861 | }; | 861 | }; |
| 862 | int ret; | 862 | int ret; |
| @@ -874,7 +874,7 @@ int parse_events_terms(struct list_head *terms, const char *str) | |||
| 874 | 874 | ||
| 875 | int parse_events(struct perf_evlist *evlist, const char *str) | 875 | int parse_events(struct perf_evlist *evlist, const char *str) |
| 876 | { | 876 | { |
| 877 | struct parse_events_data__events data = { | 877 | struct parse_events_evlist data = { |
| 878 | .list = LIST_HEAD_INIT(data.list), | 878 | .list = LIST_HEAD_INIT(data.list), |
| 879 | .idx = evlist->nr_entries, | 879 | .idx = evlist->nr_entries, |
| 880 | }; | 880 | }; |
diff --git a/tools/perf/util/parse-events.h b/tools/perf/util/parse-events.h index dd2978e7b8c0..2cd2c42a69c5 100644 --- a/tools/perf/util/parse-events.h +++ b/tools/perf/util/parse-events.h | |||
| @@ -61,12 +61,12 @@ struct parse_events_term { | |||
| 61 | struct list_head list; | 61 | struct list_head list; |
| 62 | }; | 62 | }; |
| 63 | 63 | ||
| 64 | struct parse_events_data__events { | 64 | struct parse_events_evlist { |
| 65 | struct list_head list; | 65 | struct list_head list; |
| 66 | int idx; | 66 | int idx; |
| 67 | }; | 67 | }; |
| 68 | 68 | ||
| 69 | struct parse_events_data__terms { | 69 | struct parse_events_terms { |
| 70 | struct list_head *terms; | 70 | struct list_head *terms; |
| 71 | }; | 71 | }; |
| 72 | 72 | ||
diff --git a/tools/perf/util/parse-events.y b/tools/perf/util/parse-events.y index 57ecf9db1f29..9d43c86176ff 100644 --- a/tools/perf/util/parse-events.y +++ b/tools/perf/util/parse-events.y | |||
| @@ -79,7 +79,7 @@ PE_START_TERMS start_terms | |||
| 79 | 79 | ||
| 80 | start_events: groups | 80 | start_events: groups |
| 81 | { | 81 | { |
| 82 | struct parse_events_data__events *data = _data; | 82 | struct parse_events_evlist *data = _data; |
| 83 | 83 | ||
| 84 | parse_events_update_lists($1, &data->list); | 84 | parse_events_update_lists($1, &data->list); |
| 85 | } | 85 | } |
| @@ -186,7 +186,7 @@ event_def: event_pmu | | |||
| 186 | event_pmu: | 186 | event_pmu: |
| 187 | PE_NAME '/' event_config '/' | 187 | PE_NAME '/' event_config '/' |
| 188 | { | 188 | { |
| 189 | struct parse_events_data__events *data = _data; | 189 | struct parse_events_evlist *data = _data; |
| 190 | struct list_head *list = NULL; | 190 | struct list_head *list = NULL; |
| 191 | 191 | ||
| 192 | ABORT_ON(parse_events_add_pmu(&list, &data->idx, $1, $3)); | 192 | ABORT_ON(parse_events_add_pmu(&list, &data->idx, $1, $3)); |
| @@ -202,7 +202,7 @@ PE_VALUE_SYM_SW | |||
| 202 | event_legacy_symbol: | 202 | event_legacy_symbol: |
| 203 | value_sym '/' event_config '/' | 203 | value_sym '/' event_config '/' |
| 204 | { | 204 | { |
| 205 | struct parse_events_data__events *data = _data; | 205 | struct parse_events_evlist *data = _data; |
| 206 | struct list_head *list = NULL; | 206 | struct list_head *list = NULL; |
| 207 | int type = $1 >> 16; | 207 | int type = $1 >> 16; |
| 208 | int config = $1 & 255; | 208 | int config = $1 & 255; |
| @@ -215,7 +215,7 @@ value_sym '/' event_config '/' | |||
| 215 | | | 215 | | |
| 216 | value_sym sep_slash_dc | 216 | value_sym sep_slash_dc |
| 217 | { | 217 | { |
| 218 | struct parse_events_data__events *data = _data; | 218 | struct parse_events_evlist *data = _data; |
| 219 | struct list_head *list = NULL; | 219 | struct list_head *list = NULL; |
| 220 | int type = $1 >> 16; | 220 | int type = $1 >> 16; |
| 221 | int config = $1 & 255; | 221 | int config = $1 & 255; |
| @@ -228,7 +228,7 @@ value_sym sep_slash_dc | |||
| 228 | event_legacy_cache: | 228 | event_legacy_cache: |
| 229 | PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT '-' PE_NAME_CACHE_OP_RESULT | 229 | PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT '-' PE_NAME_CACHE_OP_RESULT |
| 230 | { | 230 | { |
| 231 | struct parse_events_data__events *data = _data; | 231 | struct parse_events_evlist *data = _data; |
| 232 | struct list_head *list = NULL; | 232 | struct list_head *list = NULL; |
| 233 | 233 | ||
| 234 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, $3, $5)); | 234 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, $3, $5)); |
| @@ -237,7 +237,7 @@ PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT '-' PE_NAME_CACHE_OP_RESULT | |||
| 237 | | | 237 | | |
| 238 | PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT | 238 | PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT |
| 239 | { | 239 | { |
| 240 | struct parse_events_data__events *data = _data; | 240 | struct parse_events_evlist *data = _data; |
| 241 | struct list_head *list = NULL; | 241 | struct list_head *list = NULL; |
| 242 | 242 | ||
| 243 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, $3, NULL)); | 243 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, $3, NULL)); |
| @@ -246,7 +246,7 @@ PE_NAME_CACHE_TYPE '-' PE_NAME_CACHE_OP_RESULT | |||
| 246 | | | 246 | | |
| 247 | PE_NAME_CACHE_TYPE | 247 | PE_NAME_CACHE_TYPE |
| 248 | { | 248 | { |
| 249 | struct parse_events_data__events *data = _data; | 249 | struct parse_events_evlist *data = _data; |
| 250 | struct list_head *list = NULL; | 250 | struct list_head *list = NULL; |
| 251 | 251 | ||
| 252 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, NULL, NULL)); | 252 | ABORT_ON(parse_events_add_cache(&list, &data->idx, $1, NULL, NULL)); |
| @@ -256,7 +256,7 @@ PE_NAME_CACHE_TYPE | |||
| 256 | event_legacy_mem: | 256 | event_legacy_mem: |
| 257 | PE_PREFIX_MEM PE_VALUE ':' PE_MODIFIER_BP sep_dc | 257 | PE_PREFIX_MEM PE_VALUE ':' PE_MODIFIER_BP sep_dc |
| 258 | { | 258 | { |
| 259 | struct parse_events_data__events *data = _data; | 259 | struct parse_events_evlist *data = _data; |
| 260 | struct list_head *list = NULL; | 260 | struct list_head *list = NULL; |
| 261 | 261 | ||
| 262 | ABORT_ON(parse_events_add_breakpoint(&list, &data->idx, | 262 | ABORT_ON(parse_events_add_breakpoint(&list, &data->idx, |
| @@ -266,7 +266,7 @@ PE_PREFIX_MEM PE_VALUE ':' PE_MODIFIER_BP sep_dc | |||
| 266 | | | 266 | | |
| 267 | PE_PREFIX_MEM PE_VALUE sep_dc | 267 | PE_PREFIX_MEM PE_VALUE sep_dc |
| 268 | { | 268 | { |
| 269 | struct parse_events_data__events *data = _data; | 269 | struct parse_events_evlist *data = _data; |
| 270 | struct list_head *list = NULL; | 270 | struct list_head *list = NULL; |
| 271 | 271 | ||
| 272 | ABORT_ON(parse_events_add_breakpoint(&list, &data->idx, | 272 | ABORT_ON(parse_events_add_breakpoint(&list, &data->idx, |
| @@ -277,7 +277,7 @@ PE_PREFIX_MEM PE_VALUE sep_dc | |||
| 277 | event_legacy_tracepoint: | 277 | event_legacy_tracepoint: |
| 278 | PE_NAME ':' PE_NAME | 278 | PE_NAME ':' PE_NAME |
| 279 | { | 279 | { |
| 280 | struct parse_events_data__events *data = _data; | 280 | struct parse_events_evlist *data = _data; |
| 281 | struct list_head *list = NULL; | 281 | struct list_head *list = NULL; |
| 282 | 282 | ||
| 283 | ABORT_ON(parse_events_add_tracepoint(&list, &data->idx, $1, $3)); | 283 | ABORT_ON(parse_events_add_tracepoint(&list, &data->idx, $1, $3)); |
| @@ -287,7 +287,7 @@ PE_NAME ':' PE_NAME | |||
| 287 | event_legacy_numeric: | 287 | event_legacy_numeric: |
| 288 | PE_VALUE ':' PE_VALUE | 288 | PE_VALUE ':' PE_VALUE |
| 289 | { | 289 | { |
| 290 | struct parse_events_data__events *data = _data; | 290 | struct parse_events_evlist *data = _data; |
| 291 | struct list_head *list = NULL; | 291 | struct list_head *list = NULL; |
| 292 | 292 | ||
| 293 | ABORT_ON(parse_events_add_numeric(&list, &data->idx, (u32)$1, $3, NULL)); | 293 | ABORT_ON(parse_events_add_numeric(&list, &data->idx, (u32)$1, $3, NULL)); |
| @@ -297,7 +297,7 @@ PE_VALUE ':' PE_VALUE | |||
| 297 | event_legacy_raw: | 297 | event_legacy_raw: |
| 298 | PE_RAW | 298 | PE_RAW |
| 299 | { | 299 | { |
| 300 | struct parse_events_data__events *data = _data; | 300 | struct parse_events_evlist *data = _data; |
| 301 | struct list_head *list = NULL; | 301 | struct list_head *list = NULL; |
| 302 | 302 | ||
| 303 | ABORT_ON(parse_events_add_numeric(&list, &data->idx, | 303 | ABORT_ON(parse_events_add_numeric(&list, &data->idx, |
| @@ -307,7 +307,7 @@ PE_RAW | |||
| 307 | 307 | ||
| 308 | start_terms: event_config | 308 | start_terms: event_config |
| 309 | { | 309 | { |
| 310 | struct parse_events_data__terms *data = _data; | 310 | struct parse_events_terms *data = _data; |
| 311 | data->terms = $1; | 311 | data->terms = $1; |
| 312 | } | 312 | } |
| 313 | 313 | ||
