From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stanislav Fomichev Subject: Re: [PATCH bpf-next v2] bpftool: make libbfd optional Date: Fri, 16 Nov 2018 21:06:53 -0800 Message-ID: <20181117050653.wgcv6u4hsqldgb22@mini-arch> References: <20181112214410.14270-1-sdf@google.com> <20181117045712.i6ierstm66jbnckx@ast-mbp.dhcp.thefacebook.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Stanislav Fomichev , netdev@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net, jakub.kicinski@netronome.com, quentin.monnet@netronome.com To: Alexei Starovoitov Return-path: Received: from mail-pf1-f193.google.com ([209.85.210.193]:42866 "EHLO mail-pf1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728817AbeKQPWV (ORCPT ); Sat, 17 Nov 2018 10:22:21 -0500 Received: by mail-pf1-f193.google.com with SMTP id 64so7876802pfr.9 for ; Fri, 16 Nov 2018 21:06:55 -0800 (PST) Content-Disposition: inline In-Reply-To: <20181117045712.i6ierstm66jbnckx@ast-mbp.dhcp.thefacebook.com> Sender: netdev-owner@vger.kernel.org List-ID: On 11/16, Alexei Starovoitov wrote: > On Fri, Nov 16, 2018 at 08:52:23PM -0800, Stanislav Fomichev wrote: > > I actually wanted to follow up with a v2 when > > https://lkml.org/lkml/2018/11/16/243 reaches bpf-next (I got an ack > > already). > > it will go via perf tree, so not related. My understanding was that you periodically merge whatever goes to Linus back to bpf-next, so my plan was to wait for that event and propose a v3. > > Alternatively, I can follow up with another patch on top of that to fix > > libbfd feature detection (it's semi broken on ubuntu/fedora now). > > I think you'd need to wait until all trees merge. Sure, that's no problem. I'll follow up with another patch whenever that happens. > pls don't top post. Sorry, Gmail :-/ > > On Fri, Nov 16, 2018 at 8:47 PM Alexei Starovoitov < > > alexei.starovoitov@gmail.com> wrote: > > > > > On Mon, Nov 12, 2018 at 1:44 PM Stanislav Fomichev wrote: > > > > > > > > Make it possible to build bpftool without libbfd. libbfd and libopcodes > > > are > > > > typically provided in dev/dbg packages (binutils-dev in debian) which we > > > > usually don't have installed on the fleet machines and we'd like a way > > > to have > > > > bpftool version that works without installing any additional packages. > > > > This excludes support for disassembling jit-ted code and prints an error > > > if > > > > the user tries to use these features. > > > > > > > > Tested by: > > > > cat > FEATURES_DUMP.bpftool < > > > feature-libbfd=0 > > > > feature-disassembler-four-args=1 > > > > feature-reallocarray=0 > > > > feature-libelf=1 > > > > feature-libelf-mmap=1 > > > > feature-bpf=1 > > > > EOF > > > > FEATURES_DUMP=$PWD/FEATURES_DUMP.bpftool make > > > > ldd bpftool | grep libbfd > > > > > > > > Signed-off-by: Stanislav Fomichev > > > > > > applied, thanks > > >