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 <[email protected]>

thanks,
jirka

Reply via email to