Commit 90e2b22d authored by Jiri Olsa's avatar Jiri Olsa Committed by Ingo Molnar

perf/tool: Add support to reuse event grammar to parse out terms

We want to reuse the event grammar for parsing aliased terms.
The obvious reason is we dont need to add new code when there's
already support for this in event grammar.

Doing this by adding terms and event start entries into event
parse grammar. The grammar forks on the begining based on the
starting token, which is supplied via bison interface into the
lexer.  The lexer then returns the starting token as the first
token, thus making the grammar switch accordingly.

Currently 2 starting tokens/grammars are supported:

	PE_START_TERMS, PE_START_EVENTS

The PE_START_TERMS related grammar uses 'event_config' part
of the grammar for term parsing.
Signed-off-by: default avatarJiri Olsa <jolsa@redhat.com>
Signed-off-by: default avatarPeter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1339741902-8449-12-git-send-email-zheng.z.yan@intel.comSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent ac20de6f
...@@ -12,6 +12,7 @@ ...@@ -12,6 +12,7 @@
#include "header.h" #include "header.h"
#include "debugfs.h" #include "debugfs.h"
#include "parse-events-bison.h" #include "parse-events-bison.h"
#define YY_EXTRA_TYPE int
#include "parse-events-flex.h" #include "parse-events-flex.h"
#include "pmu.h" #include "pmu.h"
...@@ -788,13 +789,13 @@ int parse_events_modifier(struct list_head *list, char *str) ...@@ -788,13 +789,13 @@ int parse_events_modifier(struct list_head *list, char *str)
return 0; return 0;
} }
static int parse_events__scanner(const char *str, void *data) static int parse_events__scanner(const char *str, void *data, int start_token)
{ {
YY_BUFFER_STATE buffer; YY_BUFFER_STATE buffer;
void *scanner; void *scanner;
int ret; int ret;
ret = parse_events_lex_init(&scanner); ret = parse_events_lex_init_extra(start_token, &scanner);
if (ret) if (ret)
return ret; return ret;
...@@ -811,6 +812,27 @@ static int parse_events__scanner(const char *str, void *data) ...@@ -811,6 +812,27 @@ static int parse_events__scanner(const char *str, void *data)
return ret; return ret;
} }
/*
* parse event config string, return a list of event terms.
*/
int parse_events_terms(struct list_head *terms, const char *str)
{
struct parse_events_data__terms data = {
.terms = NULL,
};
int ret;
ret = parse_events__scanner(str, &data, PE_START_TERMS);
if (!ret) {
list_splice(data.terms, terms);
free(data.terms);
return 0;
}
parse_events__free_terms(data.terms);
return ret;
}
int parse_events(struct perf_evlist *evlist, const char *str, int unset __used) int parse_events(struct perf_evlist *evlist, const char *str, int unset __used)
{ {
struct parse_events_data__events data = { struct parse_events_data__events data = {
...@@ -819,7 +841,7 @@ int parse_events(struct perf_evlist *evlist, const char *str, int unset __used) ...@@ -819,7 +841,7 @@ int parse_events(struct perf_evlist *evlist, const char *str, int unset __used)
}; };
int ret; int ret;
ret = parse_events__scanner(str, &data); ret = parse_events__scanner(str, &data, PE_START_EVENTS);
if (!ret) { if (!ret) {
int entries = data.idx - evlist->nr_entries; int entries = data.idx - evlist->nr_entries;
perf_evlist__splice_list_tail(evlist, &data.list, entries); perf_evlist__splice_list_tail(evlist, &data.list, entries);
......
...@@ -33,6 +33,7 @@ extern int parse_events_option(const struct option *opt, const char *str, ...@@ -33,6 +33,7 @@ extern int parse_events_option(const struct option *opt, const char *str,
int unset); int unset);
extern int parse_events(struct perf_evlist *evlist, const char *str, extern int parse_events(struct perf_evlist *evlist, const char *str,
int unset); int unset);
extern int parse_events_terms(struct list_head *terms, const char *str);
extern int parse_filter(const struct option *opt, const char *str, int unset); extern int parse_filter(const struct option *opt, const char *str, int unset);
#define EVENTS_HELP_MAX (128*1024) #define EVENTS_HELP_MAX (128*1024)
...@@ -68,6 +69,10 @@ struct parse_events_data__events { ...@@ -68,6 +69,10 @@ struct parse_events_data__events {
int idx; int idx;
}; };
struct parse_events_data__terms {
struct list_head *terms;
};
int parse_events__is_hardcoded_term(struct parse_events__term *term); int parse_events__is_hardcoded_term(struct parse_events__term *term);
int parse_events__term_num(struct parse_events__term **_term, int parse_events__term_num(struct parse_events__term **_term,
int type_term, char *config, long num); int type_term, char *config, long num);
......
...@@ -79,6 +79,19 @@ modifier_event [ukhpGH]{1,8} ...@@ -79,6 +79,19 @@ modifier_event [ukhpGH]{1,8}
modifier_bp [rwx] modifier_bp [rwx]
%% %%
%{
{
int start_token;
start_token = (int) parse_events_get_extra(yyscanner);
if (start_token) {
parse_events_set_extra(NULL, yyscanner);
return start_token;
}
}
%}
cpu-cycles|cycles { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_CPU_CYCLES); } cpu-cycles|cycles { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_CPU_CYCLES); }
stalled-cycles-frontend|idle-cycles-frontend { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_STALLED_CYCLES_FRONTEND); } stalled-cycles-frontend|idle-cycles-frontend { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_STALLED_CYCLES_FRONTEND); }
stalled-cycles-backend|idle-cycles-backend { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_STALLED_CYCLES_BACKEND); } stalled-cycles-backend|idle-cycles-backend { return sym(yyscanner, PERF_TYPE_HARDWARE, PERF_COUNT_HW_STALLED_CYCLES_BACKEND); }
......
...@@ -25,6 +25,7 @@ do { \ ...@@ -25,6 +25,7 @@ do { \
%} %}
%token PE_START_EVENTS PE_START_TERMS
%token PE_VALUE PE_VALUE_SYM PE_RAW PE_TERM %token PE_VALUE PE_VALUE_SYM PE_RAW PE_TERM
%token PE_NAME %token PE_NAME
%token PE_MODIFIER_EVENT PE_MODIFIER_BP %token PE_MODIFIER_EVENT PE_MODIFIER_BP
...@@ -60,6 +61,11 @@ do { \ ...@@ -60,6 +61,11 @@ do { \
} }
%% %%
start:
PE_START_EVENTS events
|
PE_START_TERMS terms
events: events:
events ',' event | event events ',' event | event
...@@ -209,6 +215,12 @@ PE_RAW ...@@ -209,6 +215,12 @@ PE_RAW
$$ = list; $$ = list;
} }
terms: event_config
{
struct parse_events_data__terms *data = _data;
data->terms = $1;
}
event_config: event_config:
event_config ',' event_term event_config ',' event_term
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment