Re: [PATCH dwarves] cmake: make libbpf's Linux UAPI headers available to all binaries

[Arnaldo Carvalho de Melo <acme@kernel.org>]

Em Mon, Oct 26, 2020 at 04:26:22PM -0700, Andrii Nakryiko escreveu:
> On Mon, Oct 26, 2020 at 12:49 PM Arnaldo Carvalho de Melo
> <acme@kernel.org> wrote:
> >
> > Em Sat, Oct 24, 2020 at 11:36:53AM -0700, Andrii Nakryiko escreveu:
> > > Now that libbpf is used to implement deduplicated strings container, all of
> > > the binaries will need linux/btf.h header to compile properly. libbpf is
> > > distributed with its own copies of Linux UAPI headers, so use them during
> > > compilation.
> >
> > Thanks, applied.
> >
> 
> Still don't see it at
> https://git.kernel.org/pub/scm/devel/pahole/pahole.git/, did you
> forget to push? This is breaking libbpf and kernel CIs at the moment
> :(

That got delayed due to that 'void variable found' bug that Hao fixed, I
just tested built a kernel with it, rebooting with it, some more testing
and it'll go to the public repo,

- Arnaldo
 
> > - Arnaldo
> >
> > > Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
> > > ---
> > >  CMakeLists.txt | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/CMakeLists.txt b/CMakeLists.txt
> > > index 8e9fe4710a0c..84693514e0c8 100644
> > > --- a/CMakeLists.txt
> > > +++ b/CMakeLists.txt
> > > @@ -2,7 +2,9 @@ project(pahole C)
> > >  cmake_minimum_required(VERSION 2.8.8)
> > >  cmake_policy(SET CMP0005 NEW)
> > >
> > > -INCLUDE_DIRECTORIES( ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_CURRENT_SOURCE_DIR} )
> > > +INCLUDE_DIRECTORIES(${CMAKE_CURRENT_BINARY_DIR}
> > > +                 ${CMAKE_CURRENT_SOURCE_DIR}
> > > +                 ${CMAKE_CURRENT_SOURCE_DIR}/lib/bpf/include/uapi)
> > >
> > >  # Try to parse this later, Helio just showed me a KDE4 example to support
> > >  # x86-64 builds.
> > > --
> > > 2.24.1
> > >
> >
> > --
> >
> > - Arnaldo

-- 

- Arnaldo