perf tools: Add symbolic events support for parse_events_error
Allowing symbolic events processing to report back error. $ perf record -e 'cycles/period=krava/' ls event syntax error: '../period=krava/' \___ expected numeric value $ perf record -e 'cycles/name=1/' ls event syntax error: '..es/name=1/' \___ expected string value Signed-off-by: Jiri Olsa <jolsa@kernel.org> Cc: David Ahern <dsahern@gmail.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/1429729824-13932-10-git-send-email-jolsa@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:

committed by
Arnaldo Carvalho de Melo

parent
492d977444
commit
87d650be1d
@@ -625,7 +625,8 @@ static int config_attr(struct perf_event_attr *attr,
|
||||
return 0;
|
||||
}
|
||||
|
||||
int parse_events_add_numeric(struct list_head *list, int *idx,
|
||||
int parse_events_add_numeric(struct parse_events_evlist *data,
|
||||
struct list_head *list,
|
||||
u32 type, u64 config,
|
||||
struct list_head *head_config)
|
||||
{
|
||||
@@ -636,10 +637,10 @@ int parse_events_add_numeric(struct list_head *list, int *idx,
|
||||
attr.config = config;
|
||||
|
||||
if (head_config &&
|
||||
config_attr(&attr, head_config, NULL))
|
||||
config_attr(&attr, head_config, data->error))
|
||||
return -EINVAL;
|
||||
|
||||
return add_event(list, idx, &attr, NULL);
|
||||
return add_event(list, &data->idx, &attr, NULL);
|
||||
}
|
||||
|
||||
static int parse_events__is_name_term(struct parse_events_term *term)
|
||||
|
Reference in New Issue
Block a user