All of lore.kernel.org
 help / color / mirror / Atom feed
From: Romain Naour <romain.naour@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [RFC PATCH 0/4] add BCC and bpftrace packages
Date: Tue, 29 Sep 2020 23:08:34 +0200	[thread overview]
Message-ID: <b25b6c1c-b013-9a84-8a57-bd44738c6453@gmail.com> (raw)
In-Reply-To: <20200929102618.28571-1-qais.yousef@arm.com>

Hi Qais,

Le 29/09/2020 ? 12:26, Qais Yousef a ?crit?:
> *** RESEND AFTER SUBSCRIBING TO THE LIST ***
> 
> The following series adds 2 new packages:
> 
> 	* BPF Compiler Collection (BCC)
> 	* bpftrace
> 
> It is based on 2020.08 tag. Can rebase on tip of master after this initial RFC
> review.
> 
> The support was validated on x86_64 and aarch64 (little endian). Hence
> restricted to these 2 archs only. aarch64 big endian is set as valid, but when
> I tried to compile that I think LLVM or clang support for aarch64_be + BPF was
> generating some errors; or something else was missing. So it might be a better
> idea to remove BR2_aarch64_be from Config.in dependency.
> 
> I tried to enable i386 and arm (aarch32); but I encountered some issues and
> decided to narrow the initial support to what I got working in hope that other
> archs support can follow through later if people are interested.
> 
> IIRC the issues were:
> 
> 	* bpftrace doesn't support these 2 archs.
> 	* BCC had some build failures on arm.
> 	* BCC had runtime issues on i386.
> 
> Marking this as RFC as it is something that was done over several weekends and
> my last contribution to buildroot was many many years ago. So I am sure there
> are many rough edges to tackle. Beside from my vague description above, you
> might have gotten the hint it's been a while since I last touched it. But I'd
> like to get the support merged, so hopefully starting a discussion with what
> I have now is a better start.

I reviewed your RFC series. I'm agree that bcc is difficult to cross-compile...

I suggest adding bpftrace to the bcc series from Jugurtha and keeping you as
author of this package.

@jugurtha can you have a look at bpftrace package ?

I'll mark your RFC series "Changes requested" in patchwork.
Thanks!

Best regards,
Romain


> 
> Qais Yousef (4):
>   llvm: Add BPF backend
>   Add BPF Compiler Collection (BCC) Package
>   Add bpftrace package
>   clang: Fix undefined reference to dladdr error
> 
>  package/Config.in                             |  2 ++
>  package/bcc/0001-fix-dlinfo.patch             | 17 ++++++++++
>  .../0002-fix-undefined-gzgets-runtime.patch   | 15 ++++++++
>  .../0003-fix-loading-libc-python-perf.patch   | 17 ++++++++++
>  .../0004-fix-loading-librt-python-init.patch  | 17 ++++++++++
>  .../0005-fix-python-installation-path.patch   | 15 ++++++++
>  package/bcc/Config.in                         | 34 +++++++++++++++++++
>  package/bcc/bcc.mk                            | 25 ++++++++++++++
>  .../0001-fix-no-sys-auxv-header.patch         | 22 ++++++++++++
>  ...02-fix-missing-def-ADDR_NO_RANDOMIZE.patch | 13 +++++++
>  package/bpftrace/0003-install-libparser.patch | 12 +++++++
>  .../bpftrace/0004-install-libresources.patch  | 12 +++++++
>  package/bpftrace/0005-install-libarch.patch   | 10 ++++++
>  package/bpftrace/0006-install-libast.patch    | 11 ++++++
>  package/bpftrace/Config.in                    | 27 +++++++++++++++
>  package/bpftrace/bpftrace.mk                  | 17 ++++++++++
>  ...01-fix-undefined-reference-to-dladdr.patch | 13 +++++++
>  package/llvm/Config.in                        |  6 ++++
>  package/llvm/llvm.mk                          |  5 +++
>  19 files changed, 290 insertions(+)
>  create mode 100644 package/bcc/0001-fix-dlinfo.patch
>  create mode 100644 package/bcc/0002-fix-undefined-gzgets-runtime.patch
>  create mode 100644 package/bcc/0003-fix-loading-libc-python-perf.patch
>  create mode 100644 package/bcc/0004-fix-loading-librt-python-init.patch
>  create mode 100644 package/bcc/0005-fix-python-installation-path.patch
>  create mode 100644 package/bcc/Config.in
>  create mode 100644 package/bcc/bcc.mk
>  create mode 100644 package/bpftrace/0001-fix-no-sys-auxv-header.patch
>  create mode 100644 package/bpftrace/0002-fix-missing-def-ADDR_NO_RANDOMIZE.patch
>  create mode 100644 package/bpftrace/0003-install-libparser.patch
>  create mode 100644 package/bpftrace/0004-install-libresources.patch
>  create mode 100644 package/bpftrace/0005-install-libarch.patch
>  create mode 100644 package/bpftrace/0006-install-libast.patch
>  create mode 100644 package/bpftrace/Config.in
>  create mode 100644 package/bpftrace/bpftrace.mk
>  create mode 100644 package/clang/0001-fix-undefined-reference-to-dladdr.patch
> 

  parent reply	other threads:[~2020-09-29 21:08 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-29 10:26 [Buildroot] [RFC PATCH 0/4] add BCC and bpftrace packages Qais Yousef
2020-09-29 10:26 ` [Buildroot] [RFC PATCH 1/4] llvm: Add BPF backend Qais Yousef
2020-09-29 19:35   ` Romain Naour
2020-09-30 14:38     ` Qais Yousef
2020-09-29 10:26 ` [Buildroot] [RFC PATCH 2/4] Add BPF Compiler Collection (BCC) Package Qais Yousef
2020-09-29 20:33   ` Romain Naour
2020-10-03 20:39     ` Qais Yousef
2020-09-29 10:26 ` [Buildroot] [RFC PATCH 3/4] Add bpftrace package Qais Yousef
2020-09-29 20:55   ` Romain Naour
2020-10-03 20:56     ` Qais Yousef
2020-10-04 20:10       ` Qais Yousef
2020-10-08 22:31     ` Qais Yousef
2020-09-29 10:26 ` [Buildroot] [RFC PATCH 4/4] clang: Fix undefined reference to dladdr error Qais Yousef
2020-09-29 20:59   ` Romain Naour
2020-10-03 20:58     ` Qais Yousef
2020-10-04 16:12       ` Qais Yousef
2020-09-29 21:08 ` Romain Naour [this message]
2020-10-01 20:46   ` [Buildroot] [RFC PATCH 0/4] add BCC and bpftrace packages Qais Yousef
     [not found] <20200926193133.31390-1-qais.yousef@arm.com>
2020-09-26 21:06 ` Romain Naour
2020-09-27 19:38   ` Qais Yousef
2020-09-28 15:34     ` Romain Naour

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b25b6c1c-b013-9a84-8a57-bd44738c6453@gmail.com \
    --to=romain.naour@gmail.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.