[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <56C561DE.5070700@huawei.com>
Date: Thu, 18 Feb 2016 14:17:02 +0800
From: "Wangnan (F)" <wangnan0@...wei.com>
To: Jiri Olsa <jolsa@...hat.com>
CC: Alexei Starovoitov <ast@...nel.org>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Brendan Gregg <brendan.d.gregg@...il.com>,
Adrian Hunter <adrian.hunter@...el.com>,
"Cody P Schafer" <dev@...yps.com>,
"David S. Miller" <davem@...emloft.net>,
"He Kuang" <hekuang@...wei.com>,
Jérémie Galarneau
<jeremie.galarneau@...icios.com>, Jiri Olsa <jolsa@...nel.org>,
"Kirill Smelkov" <kirr@...edi.com>, Li Zefan <lizefan@...wei.com>,
Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>,
Namhyung Kim <namhyung@...nel.org>,
Peter Zijlstra <peterz@...radead.org>, <pi3orama@....com>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 07/54] perf tools: Enable BPF object configure syntax
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@...nel.org>
You have already acked this patch before :)
Thank you.
Powered by blists - more mailing lists