Re: [PATCH bpf-next] selftests, bpf: remove AF_XDP samples

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 30/06/2022 16.20, Magnus Karlsson wrote:
On Thu, Jun 30, 2022 at 3:44 PM Jesper Dangaard Brouer
<jbrouer@xxxxxxxxxx> wrote:


On 30/06/2022 11.37, Magnus Karlsson wrote:
From: Magnus Karlsson <magnus.karlsson@xxxxxxxxx>

Remove the AF_XDP samples from samples/bpf as they are dependent on
the AF_XDP support in libbpf. This support has now been removed in the
1.0 release, so these samples cannot be compiled anymore. Please start
to use libxdp instead. It is backwards compatible with the AF_XDP
support that was offered in libbpf. New samples can be found in the
various xdp-project repositories connected to libxdp and by googling.

Signed-off-by: Magnus Karlsson <magnus.karlsson@xxxxxxxxx>

Will you (or Maciej) be submitting these samples to XDP-tools[1] which
is the current home for libxdp or maybe BPF-examples[2] ?

   [1] https://github.com/xdp-project/xdp-tools
   [2] https://github.com/xdp-project/bpf-examples

I know Toke is ready to take over maintaining these, but we will
appreciate someone to open a PR with this code...

---
   MAINTAINERS                     |    2 -
   samples/bpf/Makefile            |    9 -
   samples/bpf/xdpsock.h           |   19 -
   samples/bpf/xdpsock_ctrl_proc.c |  190 ---
   samples/bpf/xdpsock_kern.c      |   24 -
   samples/bpf/xdpsock_user.c      | 2019 -------------------------------
   samples/bpf/xsk_fwd.c           | 1085 -----------------

The code in samples/bpf/xsk_fwd.c is interesting, because it contains a
buffer memory manager, something I've seen people struggle with getting
right and performant (at the same time).

I can push xsk_fwd to BPF-examples. Though I do think that xdpsock has
become way too big to serve as a sample. It slowly turned into a catch
all demonstrating every single feature of AF_XDP. We need a minimal
example and then likely other samples for other features that should
be demoed. So I suggest that xdpsock dies here and we start over with
something minimal and use xsk_fwd for the forwarding and mempool
example.

I trust that we in bpf-examples[0] will see a PR with this from you/Intel, so:

Acked-by: Jesper Dangaard Brouer <brouer@xxxxxxxxxx>


Toke, I think you told me at Recipes in Paris that someone from RedHat
was working on an example. Did I remember correctly?

You can get my ACK if someone commits to port this to [1] or [2], or a
3rd place that have someone what will maintain this in the future.


[0] https://github.com/xdp-project/bpf-examples/




[Index of Archives]     [Linux Networking Development]     [Fedora Linux Users]     [Linux SCTP]     [DCCP]     [Gimp]     [Yosemite Campsites]

  Powered by Linux