netdev - Re: [PATCH v3 bpf-next 0/3] Fixes for bpftool-prog-profile

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <9a91e2a9-1668-99fc-927a-39b2f1188477@iogearbox.net>
Date:   Fri, 13 Mar 2020 00:11:41 +0100
From:   Daniel Borkmann <daniel@...earbox.net>
To:     Song Liu <songliubraving@...com>, netdev@...r.kernel.org,
        bpf@...r.kernel.org
Cc:     john.fastabend@...il.com, quentin@...valent.com,
        kernel-team@...com, ast@...nel.org, arnaldo.melo@...il.com,
        jolsa@...nel.org
Subject: Re: [PATCH v3 bpf-next 0/3] Fixes for bpftool-prog-profile

On 3/12/20 7:23 PM, Song Liu wrote:
> 1. Fix build for older clang;
> 2. Fix skeleton's dependency on libbpf;
> 3. Add files to .gitignore.
> 
> Changes v2 => v3:
> 1. Add -I$(LIBBPF_PATH) to Makefile (Quentin);
> 2. Use p_err() for error message (Quentin).
> 
> Changes v1 => v2:
> 1. Rewrite patch 1 with real feature detection (Quentin, Alexei).
> 2. Add files to .gitignore (Andrii).
> 
> Song Liu (3):
>    bpftool: only build bpftool-prog-profile if supported by clang
>    bpftool: skeleton should depend on libbpf
>    bpftool: add _bpftool and profiler.skel.h to .gitignore
> 
>   tools/bpf/bpftool/.gitignore                  |  2 ++
>   tools/bpf/bpftool/Makefile                    | 20 +++++++++++++------
>   tools/bpf/bpftool/prog.c                      |  1 +
>   tools/build/feature/Makefile                  |  9 ++++++++-
>   .../build/feature/test-clang-bpf-global-var.c |  4 ++++
>   5 files changed, 29 insertions(+), 7 deletions(-)
>   create mode 100644 tools/build/feature/test-clang-bpf-global-var.c

Tested with clang-7 and clang-11; looks good, applied, thanks!