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

From: Jiri Olsa
Date: Fri Feb 12 2016 - 09:09:59 EST


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>

thanks,
jirka