lkml.org 
[lkml]   [2023]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 1/2] perf build: Properly guard libbpf includes
Em Fri, Jan 06, 2023 at 11:06:46AM -0800, Ian Rogers escreveu:
> So trying to get build-test working on my Debian derived distro is a
> PITA with broken feature detection for options I don't normally use.

Its really difficult to have perf building with so many dependent
libraries, mowing out some should be in order.

> I'll try to fix this.

Thanks.

> In any case I think I've spotted what is really happening here and it
> isn't a failure but a feature :-D The build is specifying

I get it.

> LIBBPF_DYNAMIC=1 which means you get the libbpf headers from
> /usr/include. I think the build is trying to do this on a system with
> an old libbpf and hence getting the failures above. Previously, even
> though we wanted the dynamic headers we still had a -I, this time for
> the install_headers version. Now you really are using the system
> version and it is broken. This means a few things:
> - the libbpf feature test should fail if code like above is going to fail,

Agreed.

> - we may want to contemplate supporting older libbpfs (I'd rather not),

I'd rather require everybody to be up to the latest trends, but I really
don't think that is a reasonable expectation.

> - does build-test have a way to skip known issues like this?

Unsure, Jiri?

But yeah, previous experiences with Andrii were that we can do not too
costly feature checks, not using .c programs that would fail if some
required feature wasn't present but instead would just do some grep on a
header and if some "smell" wasn't scent, just fail the cap query.

- Arnaldo

\
 
 \ /
  Last update: 2023-03-26 23:33    [W:1.206 / U:0.968 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site