linux-kernel - Re: [PATCH v2 01/10] perf tools: Integrating the CoreSight decoding library

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 16 Jan 2018 13:15:00 +0100
From:   Jiri Olsa <jolsa@...hat.com>
To:     Mathieu Poirier <mathieu.poirier@...aro.org>
Cc:     acme@...nel.org, peterz@...radead.org, mingo@...hat.com,
        alexander.shishkin@...ux.intel.com, namhyung@...nel.org,
        adrian.hunter@...el.com, mike.leach@....com,
        suzuki.poulosi@....com, kim.phillips@....com,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 01/10] perf tools: Integrating the CoreSight decoding
 library

On Mon, Jan 15, 2018 at 11:13:05AM -0700, Mathieu Poirier wrote:
> The Open CoreSight Decoding Library (openCSD) is a free and open
> library to decode traces collected by the CoreSight hardware
> infrastructure.
> 
> This patch adds the required mechanic to recognise the presence
> of the openCSD library on a system and set up miscellaneous flags
> to be used in the compilation of the trace decoding feature.
> 
> Signed-off-by: Mathieu Poirier <mathieu.poirier@...aro.org>
> ---
>  tools/build/Makefile.feature          |  6 ++++--
>  tools/build/feature/Makefile          |  7 ++++++-
>  tools/build/feature/test-all.c        |  5 +++++
>  tools/build/feature/test-libopencsd.c |  8 ++++++++
>  tools/perf/Makefile.config            | 25 +++++++++++++++++++++++++
>  tools/perf/Makefile.perf              |  2 ++
>  6 files changed, 50 insertions(+), 3 deletions(-)
>  create mode 100644 tools/build/feature/test-libopencsd.c
> 
> diff --git a/tools/build/Makefile.feature b/tools/build/Makefile.feature
> index e52fcefee379..dc3d97b248ab 100644
> --- a/tools/build/Makefile.feature
> +++ b/tools/build/Makefile.feature
> @@ -66,7 +66,8 @@ FEATURE_TESTS_BASIC :=                  \
>          bpf                             \
>          sched_getcpu			\
>          sdt				\
> -        setns
> +        setns				\
> +	libopencsd
>  
>  # FEATURE_TESTS_BASIC + FEATURE_TESTS_EXTRA is the complete list
>  # of all feature tests
> @@ -108,7 +109,8 @@ FEATURE_DISPLAY ?=              \
>           zlib                   \
>           lzma                   \
>           get_cpuid              \
> -         bpf
> +         bpf			\
> +	 libopencsd

we put in this list only generic libraries, this one seems arch
specific please put it into FEATURE_TESTS_EXTRA list

the state is displayed by running make 'VF=1', with this patch:
  https://marc.info/?l=linux-kernel&m=151549001914932&w=2

Arnaldo, could you please check on that one ^^^?

thanks,
jirka