Re: [PATCH 07/54] perf tools: Enable BPF object configure syntax

From: Wangnan (F)
Date: Thu Feb 18 2016 - 01:19:37 EST




On 2016/2/12 22:09, Jiri Olsa wrote:
On Fri, Feb 05, 2016 at 02:01:32PM +0000, Wang Nan wrote:

SNIP

}
|
-PE_BPF_SOURCE
+PE_BPF_SOURCE opt_event_config
{
struct parse_events_evlist *data = _data;
struct list_head *list;
ALLOC_LIST(list);
- ABORT_ON(parse_events_load_bpf(data, list, $1, true));
+ ABORT_ON(parse_events_load_bpf(data, list, $1, true, $2));
+ parse_events__free_terms($2);
$$ = list;
}
+opt_event_config:
+'/' event_config '/'
+{
+ $$ = $2;
+}
+|
+{
+ $$ = NULL;
+}
can't judge the bpf part, but for the parser part:

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

You have already acked this patch before :)

Thank you.