[PATCH 1/1 v2] Documentation: Add minimum pahole version

[PATCH 1/1 v2] Documentation: Add minimum pahole version

[Arnaldo Carvalho de Melo]

A report was made in https://github.com/acmel/dwarves/issues/26 about
pahole not being listed in the process/changes.rst file as being needed
for building the kernel, address that.

Link: https://github.com/acmel/dwarves/issues/26
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Acked-by: Andrii Nakryiko <andrii@kernel.org>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: bpf@vger.kernel.org
Cc: netdev@vger.kernel.org
Link: http://lore.kernel.org/lkml/YZPQ6+u2wTHRfR+W@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 Documentation/process/changes.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Documentation/process/changes.rst b/Documentation/process/changes.rst
index e35ab74a0f804b04..572465db0db19e8f 100644
--- a/Documentation/process/changes.rst
+++ b/Documentation/process/changes.rst
@@ -35,6 +35,7 @@ GNU make               3.81             make --version
 binutils               2.23             ld -v
 flex                   2.5.35           flex --version
 bison                  2.0              bison --version
+pahole                 1.16             pahole --version
 util-linux             2.10o            fdformat --version
 kmod                   13               depmod -V
 e2fsprogs              1.41.4           e2fsck -V
@@ -108,6 +109,16 @@ Bison
 Since Linux 4.16, the build system generates parsers
 during build.  This requires bison 2.0 or later.
 
+pahole:
+-------
+
+Since Linux 5.2, if CONFIG_DEBUG_INFO_BTF is selected, the build system
+generates BTF (BPF Type Format) from DWARF in vmlinux, a bit later from kernel
+modules as well.  This requires pahole v1.16 or later.
+
+It is found in the 'dwarves' or 'pahole' distro packages or from
+https://fedorapeople.org/~acme/dwarves/.
+
 Perl
 ----
 
-- 
2.31.1

Re: [PATCH 1/1 v2] Documentation: Add minimum pahole version

[Jonathan Corbet]

Arnaldo Carvalho de Melo <acme@kernel.org> writes:

> A report was made in https://github.com/acmel/dwarves/issues/26 about
> pahole not being listed in the process/changes.rst file as being needed
> for building the kernel, address that.
>
> Link: https://github.com/acmel/dwarves/issues/26
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> Acked-by: Andrii Nakryiko <andrii@kernel.org>
> Acked-by: Daniel Borkmann <daniel@iogearbox.net>
> Cc: Alexei Starovoitov <ast@kernel.org>
> Cc: Jiri Olsa <jolsa@redhat.com>
> Cc: Jonathan Corbet <corbet@lwn.net>
> Cc: bpf@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Link: http://lore.kernel.org/lkml/YZPQ6+u2wTHRfR+W@kernel.org
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> ---
>  Documentation/process/changes.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)

Applied (with the duplicate SOB removed), thanks.

jon

Re: [PATCH 1/1 v2] Documentation: Add minimum pahole version

[Arnaldo Carvalho de Melo]

Em Mon, Nov 29, 2021 at 02:48:42PM -0700, Jonathan Corbet escreveu:
> Arnaldo Carvalho de Melo <acme@kernel.org> writes:
> 
> > A report was made in https://github.com/acmel/dwarves/issues/26 about
> > pahole not being listed in the process/changes.rst file as being needed
> > for building the kernel, address that.
> >
> > Link: https://github.com/acmel/dwarves/issues/26
> > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> > Acked-by: Andrii Nakryiko <andrii@kernel.org>
> > Acked-by: Daniel Borkmann <daniel@iogearbox.net>
> > Cc: Alexei Starovoitov <ast@kernel.org>
> > Cc: Jiri Olsa <jolsa@redhat.com>
> > Cc: Jonathan Corbet <corbet@lwn.net>
> > Cc: bpf@vger.kernel.org
> > Cc: netdev@vger.kernel.org
> > Link: http://lore.kernel.org/lkml/YZPQ6+u2wTHRfR+W@kernel.org
> > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> > ---
> >  Documentation/process/changes.rst | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> 
> Applied (with the duplicate SOB removed), thanks.

Oops, one should be enough indeed :-)

Thanks!

- Arnaldo