netdev - Re: [PATCH bpf-next v2] bpf: btf: add btf documentation

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: <CAADnVQ+NKMCxcf7fSs0SCTfa4w7Be6kHdp+Ek0+5t-hbhd2L8g@mail.gmail.com>
Date:   Tue, 22 Jan 2019 23:51:21 -0800
From:   Alexei Starovoitov <alexei.starovoitov@...il.com>
To:     Yonghong Song <yhs@...com>
Cc:     Alexei Starovoitov <ast@...com>,
        Daniel Borkmann <daniel@...earbox.net>,
        Network Development <netdev@...r.kernel.org>,
        Edward Cree <ecree@...arflare.com>,
        Kernel Team <kernel-team@...com>
Subject: Re: [PATCH bpf-next v2] bpf: btf: add btf documentation

On Fri, Jan 18, 2019 at 2:00 PM Yonghong Song <yhs@...com> wrote:
>
> This patch added documentation for BTF (BPF Debug Format).
> The document is placed under linux:Documentation/bpf directory.
>
> Signed-off-by: Yonghong Song <yhs@...com>
> ---
>  Documentation/bpf/btf.rst   | 870 ++++++++++++++++++++++++++++++++++++
>  Documentation/bpf/index.rst |   7 +
>  2 files changed, 877 insertions(+)
>  create mode 100644 Documentation/bpf/btf.rst
>
> Changelogs:
>   v1 -> v2:
>     address comments from Edward, mainly including:
>     . more detailed explanation of BTF_INT_OFFSET().
>     . more explanation about array dimensions.
>     . better wording refers to "type" field in btf_type
>       for typedef/const/volatile/restrict.
>     . better explanation for BTF_KIND_FUNC.
>     . explanation of what is btf_id.
>     . more cross references inside the document.

Looks great to me. Applied to bpf-next.