Re: [PATCH v2 04/19] perf, tools: Tighten detection of BPF events

From: Jiri Olsa
Date: Tue Aug 22 2017 - 04:20:17 EST


On Fri, Aug 11, 2017 at 04:26:19PM -0700, Andi Kleen wrote:
> From: Andi Kleen <ak@xxxxxxxxxxxxxxx>
>
> perf stat -e cpu/uops_executed.core,cmask=1/
>
> would be detected as a BPF source event because the .c matches the .c
> source BPF pattern.
>
> v2:
>
> Originally I tried to use lex lookahead, but it doesn't seem to work.
>
> This now extends the BPF pattern to match longer events, but then
> does an extra check in the C code to reject BPF matches that
> do not end with .c/.o/.obj
>
> This uses REJECT, which makes the flex scanner slower, but
> that shouldn't be a big problem for the perf events.
>
> Signed-off-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>

seems ok, Arnaldo, could you run it with BPF/clang test machinery? ;-)

thanks,
jirka