• Mathieu Poirier's avatar
    perf cs-etm: Add proper header file for symbols · 859dcf64
    Mathieu Poirier authored
    After 'commit e22c1c75 ("perf thread: Don't include symbol.h,
    symbol_conf.h is enough")'
    
    Compilation of the perf tools is broken when using the functionality
    provided by the openCSD library:
    
    [...]
    
    ...                       timerfd: [ on  ]
    ...                  sched_getcpu: [ on  ]
    ...                           sdt: [ OFF ]
    ...                         setns: [ on  ]
    ...                    libopencsd: [ on  ]
    
    [...]
    
      CC       util/arm-spe.o
      CC       util/arm-spe-pkt-decoder.o
      CC       util/s390-cpumsf.o
      CC       util/cs-etm.o
      CC       util/parse-branch-options.o
    util/cs-etm.c: In function ‘cs_etm__mem_access’:
    util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
      struct  addr_location al;
    
    And rightly so since file cs-etm.c doesn't include symbol.h, something
    that is rectified in this patch.
    Signed-off-by: default avatarMathieu Poirier <mathieu.poirier@linaro.org>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Suzuki K Poulouse <suzuki.poulose@arm.com>
    Cc: linux-arm-kernel@lists.infradead.org
    Link: http://lkml.kernel.org/r/20190208223543.31836-1-mathieu.poirier@linaro.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    859dcf64
cs-etm.c 51.5 KB