Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: Domenico Andreoli <domenico.andreoli.it@gmail.com>
Date:   Wed, 17 Feb 2021 13:08:23 +0100
From:   Domenico Andreoli <cavok@debian.org>
To:     Arnaldo Carvalho de Melo <acme@kernel.org>
Cc:     Sedat Dilek <sedat.dilek@gmail.com>, dwarves@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        bpf@vger.kernel.org, Jiri Olsa <jolsa@kernel.org>,
        Jan Engelhardt <jengelh@inai.de>,
        Matthias Schwarzott <zzam@gentoo.org>,
        Andrii Nakryiko <andriin@fb.com>, Yonghong Song <yhs@fb.com>,
        Mark Wieelard <mjw@redhat.com>,
        Paul Moore <paul@paul-moore.com>,
        Ondrej Mosnacek <omosnace@redhat.com>,
        Daniel =?iso-8859-1?Q?P=2E_Berrang=E9?= <berrange@redhat.com>,
        Tom Stellard <tstellar@redhat.com>, 705969@bugs.debian.org
Subject: Re: ANNOUNCE: pahole v1.20 (gcc11 DWARF5's default, lots of ELF
 sections, BTF)
Message-ID: <YC0HN+0Tva0lOPIt@m4>
References: <20210204220741.GA920417@kernel.org>
 <CA+icZUXngJL2WXRyeWDjTyBYbXc0uC0_C69nBH9bq4sr_TAx5g@mail.gmail.com>
 <20210208123253.GI920417@kernel.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="gIYVh1ASy4tAHzNX"
Content-Disposition: inline
In-Reply-To: <20210208123253.GI920417@kernel.org>
Precedence: bulk


--gIYVh1ASy4tAHzNX
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Feb 08, 2021 at 09:32:53AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Feb 08, 2021 at 03:44:54AM +0100, Sedat Dilek escreveu:
> > On Thu, Feb 4, 2021 at 11:07 PM Arnaldo Carvalho de Melo
> > <arnaldo.melo@gmail.com> wrote:
> > >
> > > Hi,
> > >
> > >         The v1.20 release of pahole and its friends is out, mostly
> > > addressing problems related to gcc 11 defaulting to DWARF5 for -g,
> > > available at the usual places:
> > >
> > > Main git repo:
> > >
> > >    git://git.kernel.org/pub/scm/devel/pahole/pahole.git
> > >
> > > Mirror git repo:
> > >
> > >    https://github.com/acmel/dwarves.git
> > >
> > > tarball + gpg signature:
> > >
> > >    https://fedorapeople.org/~acme/dwarves/dwarves-1.20.tar.xz
> > >    https://fedorapeople.org/~acme/dwarves/dwarves-1.20.tar.bz2
> > >    https://fedorapeople.org/~acme/dwarves/dwarves-1.20.tar.sign
> > >
> >=20
> > FYI:
> > Debian now ships dwarves package version 1.20-1 in unstable.
> >=20
> > Just a small nit to this release and its tagging:
> >=20
> > You did:
> > commit 0d415f68c468b77c5bf8e71965cd08c6efd25fc4 ("pahole: Prep 1.20")
> >=20
> > Is this new?
> >=20
> > The release before:
> > commit dd15aa4b0a6421295cbb7c3913429142fef8abe0 ("dwarves: Prep v1.19")
>=20
> Its minor but intentional, pahole is by far the most well known tool in
> dwarves, so using that name more frequently (the git repo is pahole.git
> , for instance) may help more quickly associate with the tool needed for
> BTF encoding, data analysis, etc. And since its not about only DWARF,
> perhaps transitioning to using 'pahole' more widely is interesting.

Any plan to switch also the release tarball name?

We are planning to rename the Debian package once the Bullseye is
released, currently it's dwarves-dfsg for legacy/unclear reasons.

Would it be a good idea to switch directly to pahole then?

Dom

>=20
> - Arnaldo
> =20
> > - Sedat -
> >=20
> > > Best Regards,
> > >
> > >  - Arnaldo
> > >
> > > v1.20:
> > >
> > > BTF encoder:
> > >
> > >   - Improve ELF error reporting using elf_errmsg(elf_errno()).
> > >
> > >   - Improve objcopy error handling.
> > >
> > >   - Fix handling of 'restrict' qualifier, that was being treated as a=
 'const'.
> > >
> > >   - Support SHN_XINDEX in st_shndx symbol indexes, to handle ELF obje=
cts with
> > >     more than 65534 sections, for instance, which happens with kernel=
s built
> > >     with 'KCFLAGS=3D"-ffunction-sections -fdata-sections", Other case=
s may
> > >     include when using FG-ASLR, LTO.
> > >
> > >   - Cope with functions without a name, as seen sometimes when buildi=
ng kernel
> > >     images with some versions of clang, when a SEGFAULT was taking pl=
ace.
> > >
> > >   - Fix BTF variable generation for kernel modules, not skipping vari=
ables at
> > >     offset zero.
> > >
> > >   - Fix address size to match what is in the ELF file being processed=
, to fix using
> > >     a 64-bit pahole binary to generate BTF for a 32-bit vmlinux image.
> > >
> > >   - Use kernel module ftrace addresses when finding which functions t=
o encode,
> > >     which increases the number of functions encoded.
> > >
> > > libbpf:
> > >
> > >   - Allow use of packaged version, for distros wanting to dynamically=
 link with
> > >     the system's libbpf package instead of using the libbpf git submo=
dule shipped
> > >     in pahole's source code.
> > >
> > > DWARF loader:
> > >
> > >   - Support DW_AT_data_bit_offset
> > >
> > >     This appeared in DWARF4 but is supported only in gcc's -gdwarf-5,
> > >     support it in a way that makes the output be the same for both ca=
ses.
> > >
> > >       $ gcc -gdwarf-5 -c examples/dwarf5/bf.c
> > >       $ pahole bf.o
> > >       struct pea {
> > >             long int                   a:1;                  /*     0=
: 0  8 */
> > >             long int                   b:1;                  /*     0=
: 1  8 */
> > >             long int                   c:1;                  /*     0=
: 2  8 */
> > >
> > >             /* XXX 29 bits hole, try to pack */
> > >             /* Bitfield combined with next fields */
> > >
> > >             int                        after_bitfield;       /*     4=
     4 */
> > >
> > >             /* size: 8, cachelines: 1, members: 4 */
> > >             /* sum members: 4 */
> > >             /* sum bitfield members: 3 bits, bit holes: 1, sum bit ho=
les: 29 bits */
> > >             /* last cacheline: 8 bytes */
> > >       };
> > >
> > >   - DW_FORM_implicit_const in attr_numeric() and attr_offset()
> > >
> > >   - Support DW_TAG_GNU_call_site, its the standardized rename of the =
previously supported
> > >     DW_TAG_GNU_call_site.
> > >
> > > build:
> > >
> > >     - Fix compilation on 32-bit architectures.
> > >
> > > Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>=20
> --=20
>=20
> - Arnaldo

--=20
rsa4096: 3B10 0CA1 8674 ACBA B4FE  FCD2 CE5B CF17 9960 DE13
ed25519: FFB4 0CC3 7F2E 091D F7DA  356E CC79 2832 ED38 CB05

--gIYVh1ASy4tAHzNX
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE0znebYyV6RAN/q8htwRzp/vsqYEFAmAtBzMACgkQtwRzp/vs
qYFwaQ//cmKhFeLG2+gdsqQq4giRbZXq2lnrUpPV/WHtlMQ/VHjwicMP80GqHZ1h
7SUDggas/62cxXZ2+9ffyQYUbgvpNwk5MVPsAUOzCJF6EQ/1SDMQEsS2lGEU6SVc
qtYXB7YTaKCxFNBeh6mcNqpPZnJ4qwezVs52e36Rk0tiYJNneHrhOmXrox16vI1/
X5q7NaGvfmsB/WpyWiecw7MpuI3aoZq+D7Y5rOoEgCML4s/13rUuybJlAYnsuww5
9OZomNOWypQpPTmdGBd2APBsK/VaGZkoYtGVGMXqg+XqiYh/1aUlmo95blbsFMuB
sRKTRoHkY6Hf4JjalAKgUfYAsHnBxBxMdV3nmLypEeEX9YPROBj6e3c7HlkgsWbb
oxL7WpqxWrVNPl5FtchzMaBD9xt+tWgzhTFLnKNsEbprHAuLzp/p8fPC91HCf7c7
Ox1gVR2UY0Lju+aByffZlRxJ/pbSYZpTYtKZUXrTw/pGoAqKer4ntNBiRfl4vG97
PbMKV49faixLwpkTQr1VuS7RpE2nkrnT9mRMVZk4aFxGq5sgpfLWfXmvOtVX6Zq0
/kyoan585ro4qtlQ5Wxq2TcSU5QNcLCtI4N89wqENyzTbFqZjU4LE/jhtoSYpYOO
fBf5WybiqIKJSHOArgCXKWQvg7DAfGCP5pkTS2zCye0r3J2MGx8=
=aQcl
-----END PGP SIGNATURE-----

--gIYVh1ASy4tAHzNX--