On Wed, Sep 25, 2019 at 5:06 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote: > On Wed, Sep 25, 2019 at 12:58 AM Paul Moore <paul@xxxxxxxxxxxxxx> wrote: > > From: Paul Moore <paul@xxxxxxxxxxxxxx> > > > > On a modern Rawhide system builds that include libbpf require libelf > > as well to avoid the following linker errors: > > > > # cc -g -O0 -Wall -D_GNU_SOURCE -DHAVE_BPF \ > > client.c ../bpf/bpf_common.c ../bpf/bpf_common.h -lbpf -o client > > /usr/bin/ld: ... libbpf.so: undefined reference to `gelf_getshdr' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_rawdata' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_getscn' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_begin' > > /usr/bin/ld: ... libbpf.so: undefined reference to `gelf_getrel' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_memory' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_end' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_strptr' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_nextscn' > > /usr/bin/ld: ... libbpf.so: undefined reference to `gelf_getehdr' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_version' > > /usr/bin/ld: ... libbpf.so: undefined reference to `elf_getdata' > > /usr/bin/ld: ... libbpf.so: undefined reference to `gelf_getsym' > > This is a bug in libbpf. I filed a bug against the Fedora package [1] > and also opened a PR [2] against the upstream repo (which is a > modified mirror of the code in the kernel repo under tools/lib/bpf, > where the bug is not present), which I think should fix this problem > also in the Fedora builds. > > [1] https://bugzilla.redhat.com/show_bug.cgi?id=1755317 > [2] https://github.com/libbpf/libbpf/pull/72 Thanks for tracking down the root cause and filing a report upstream; I didn't have time to sort out the "why", I just wanted to get the test working again :) It looks like the best course of action is to hold off on this fix in hopes that it gets fixed upstream, or at least in Fedora. If this issue persists for a week or two we can always go ahead and merge this fix, it shouldn't break anything if/when this gets fixed upstream. Tests that fail to run made me sad :( -- paul moore www.paul-moore.com
