• Ian Rogers's avatar
    perf test: Parse events workaround for dash/minus · 146edff3
    Ian Rogers authored
    Skip an event configuration for event names with a dash/minus in them.
    Events with a dash/minus in their name cause parsing issues as legacy
    encoding of events would use a dash/minus as a separator.
    
    The parser separates events with dashes into prefixes and suffixes and
    then recombines them. Unfortunately if an event has part of its name
    that matches a legacy token then the recombining fails.
    
    This is seen for branch-brs where branch is a legacy token. branch-brs
    was introduced to sysfs in:
    
      https://lore.kernel.org/all/20220322221517.2510440-5-eranian@google.com/
    
    The failure is shown below as well as the workaround to use a config
    where the dash/minus isn't treated specially:
    
    ```
      $ perf stat -e branch-brs true
      event syntax error: 'branch-brs'
                                 \___ parser error
    
      $ perf stat -e cpu/branch-brs/ true
    
       Performance counter stats for 'true':
    
                  46,179      cpu/branch-brs/
    ```
    Signed-off-by: default avatarIan Rogers <irogers@google.com>
    Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
    Cc: Jiri Olsa <jolsa@kernel.org>
    Cc: John Garry <john.garry@huawei.com>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Ravi Bangoria <ravi.bangoria@amd.com>
    Cc: Stephane Eranian <eranian@google.com>
    Link: http://lore.kernel.org/lkml/20221013011205.3151391-1-irogers@google.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    146edff3
parse-events.c 81.7 KB