Received-SPF: pass (google.com: domain of linux-kernel+bounces-79109-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240222100656.0a644254@canb.auug.org.au> <Zdj3FyPjE5ezyfsM@x1> <Zdj74Zo10vYTZNMl@x1>
In-Reply-To: <Zdj74Zo10vYTZNMl@x1>
From: Namhyung Kim <namhyung@kernel.org>
Date: Fri, 23 Feb 2024 12:34:40 -0800
Message-ID: <CAM9d7ciaj80QZL0AS_T2HNBdMOyS-j1wBHQSYs=U3kHQimY1mQ@mail.gmail.com>
Subject: Re: [solved] Re: linux-next: build failure after merge of the perf tree
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>, Ian Rogers <irogers@google.com>, 
	Arnaldo Carvalho de Melo <arnaldo.melo@gmail.com>, Arnaldo Carvalho de Melo <acme@redhat.com>, 
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, 
	Linux Next Mailing List <linux-next@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Arnaldo,

On Fri, Feb 23, 2024 at 12:11=E2=80=AFPM Arnaldo Carvalho de Melo
<acme@kernel.org> wrote:
>
> On Fri, Feb 23, 2024 at 04:50:47PM -0300, Arnaldo Carvalho de Melo wrote:
> > On Thu, Feb 22, 2024 at 10:06:56AM +1100, Stephen Rothwell wrote:
> > > Hi all,
> > >
> > > After merging the perf tree, today's linux-next build (native perf)
> > > failed like this:
> > >
> > > util/bpf_skel/augmented_raw_syscalls.bpf.c:329:15: error: invalid app=
lication of 'sizeof' to an incomplete type 'struct timespec64'
> > >         __u32 size =3D sizeof(struct timespec64);
> > >                      ^     ~~~~~~~~~~~~~~~~~~~
> > > util/bpf_skel/augmented_raw_syscalls.bpf.c:329:29: note: forward decl=
aration of 'struct timespec64'
> > >         __u32 size =3D sizeof(struct timespec64);
> > >                                    ^
> > >
> > > Caused by commit
> > >
> > >   29d16de26df1 ("perf augmented_raw_syscalls.bpf: Move 'struct timesp=
ec64' to vmlinux.h")
> > >
> > > This is a ppc64 le build.
> > >
> > > I have used the perf tree from next-20240221 for today.
> >
> > Ok, finally I managed to secure a ppc64 machine to test this and
> > sometimes I reproduce just like you reported, but sometimes I can't do
> > it, didn't manage to isolate what is that makes it fail sometimes, make
> > -C tools/perf clean, nuking the O=3D target directory, etc, when I
> > reproduce it:
>
> So I think I see the problem, I now left the build directory with a
> previous build from torvalds/master, then switched to the
> perf-tools-branch and tried to build from there, without first removing
> the old build, it fails:
>
>   CLANG   /tmp/build/perf-tools-next/util/bpf_skel/.tmp/augmented_raw_sys=
calls.bpf.o
> util/bpf_skel/augmented_raw_syscalls.bpf.c:329:15: error: invalid applica=
tion of 'sizeof' to an incomplete type 'struct timespec64'
>   329 |         __u32 size =3D sizeof(struct timespec64);
>       |                      ^     ~~~~~~~~~~~~~~~~~~~
> util/bpf_skel/augmented_raw_syscalls.bpf.c:329:29: note: forward declarat=
ion of 'struct timespec64'
>   329 |         __u32 size =3D sizeof(struct timespec64);
>       |                                    ^
> 1 error generated.
> make[2]: *** [Makefile.perf:1161: /tmp/build/perf-tools-next/util/bpf_ske=
l/.tmp/augmented_raw_syscalls.bpf.o] Error 1
> make[1]: *** [Makefile.perf:264: sub-make] Error 2
>
>
> Because it will use what was installed before in the build dir:
>
> [acme@ibm-p9z-16-lp5 perf-tools-next]$ ls -la /tmp/build/perf-tools-next/=
util/bpf_skel/vmlinux.h
> -rw-r--r--. 1 acme acme 4319 Feb 23 14:59 /tmp/build/perf-tools-next/util=
/bpf_skel/vmlinux.h
> [acme@ibm-p9z-16-lp5 perf-tools-next]$
>
> And that one doesn't have 'struct timespec64':
>
> [acme@ibm-p9z-16-lp5 perf-tools-next]$ grep timespec64 /tmp/build/perf-to=
ols-next/util/bpf_skel/vmlinux.h
> [acme@ibm-p9z-16-lp5 perf-tools-next]$
>
> If I remove that directory contents:
>
> [acme@ibm-p9z-16-lp5 perf-tools-next]$ rm -rf /tmp/build/perf-tools-next/
> [acme@ibm-p9z-16-lp5 perf-tools-next]$ mkdir /tmp/build/perf-tools-next/
> [acme@ibm-p9z-16-lp5 perf-tools-next]$
>
> And then try to build again:
>
> make -k O=3D/tmp/build/perf-tools-next/ -C tools/perf install-bin
>
> It works.
>
> I reproduced the problem on x86_64, so, on this transition period, the
> problem happens, probably we need to robustify the installation of
> tools/perf/util/bpf_skel/vmlinux/vmlinux.h in the O=3D target directory,
> but if you just make sure the build directory is clean before trying to
> build it, this time, it should work, wdyt?

Can we add a dependency to the minimal vmlinux.h?

Thanks,
Namhyung

---8<---
diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
index 3cecd51b2397..33621114135e 100644
--- a/tools/perf/Makefile.perf
+++ b/tools/perf/Makefile.perf
@@ -1147,7 +1147,7 @@ ifeq ($(VMLINUX_H),)
   endif
 endif

-$(SKEL_OUT)/vmlinux.h: $(VMLINUX_BTF) $(BPFTOOL)
+$(SKEL_OUT)/vmlinux.h: $(VMLINUX_BTF) $(BPFTOOL) $(VMLINUX_H)
 ifeq ($(VMLINUX_H),)
        $(QUIET_GEN)$(BPFTOOL) btf dump file $< format c > $@
 else