Re: Recent tarballs missing some files

[Michel Lind <michel@michel-slm.name>]

[-- Attachment #1: Type: text/plain, Size: 2307 bytes --]

Hi Arnaldo,

On Mon, Nov 25, 2024 at 10:50:49PM -0300, Arnaldo Carvalho de Melo wrote:
> On Mon, Nov 25, 2024, 8:50 PM Michel Lind <michel@michel-slm.name> wrote:
> 
> > On Mon, Nov 25, 2024 at 12:08:28PM -0600, Michel Lind wrote:
> > > Hello all,
> > >
> > > Per discussions with acme back at LPC, I'm working on packaging a
> > > standalone perf RPM, initially for the CentOS Hyperscale SIG and
> > > potentially for Fedora later on.
> > >
> > > I noticed that, while perf 6.9.0 builds fine from the source tarball,
> > > more recent versions (at least 6.11 and 6.12) fails due to a missing
> > > header:
> > >
> > > ```
> > > In file included from
> > /builddir/build/BUILD/perf-6.12.0/tools/include/uapi/linux/bpf_perf_event.h:11,
> > >                  from libbpf.c:36:
> > >
> > /builddir/build/BUILD/perf-6.12.0/tools/include/uapi/asm/bpf_perf_event.h:2:10:
> > fatal error: ../../arch/arm64/include/uapi/asm/bpf_perf_event.h: No such
> > file or directory
> > >     2 | #include "../../arch/arm64/include/uapi/asm/bpf_perf_event.h"
> > >       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > compilation terminated.
> > > ```
> > >
> > > I can submit a patch to fix this (and any other missing files), if
> > > someone could point me to where the script is that generates the perf
> > > tarballs.
> > >
> > Update - on digging in this only happens on aarch64 (probably why this
> > was not caught earlier). And it also affects 6.9.0. In all cases the
> > alternatives are either copying in that one header *or* passing
> > NO_LIBBPF=1
> >
> 
> Maybe the case of adding the missing file to:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/log/tools/perf/MANIFEST
> 
> Look at the other csets for explanation.
> 
> You could also try building with the system 's libbpf, look at:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/perf/Makefile.perf#n120
>

Thank you!

Building with the system libbpf might not be an option when building for
older Enterprise Linux releases, but I'll add that as an option in the
spec for new enough releases.

Best regards,

-- 
 _o) Michel Lind
_( ) identities: https://keyoxide.org/5dce2e7e9c3b1cffd335c1d78b229d2f7ccc04f2

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]