Netdev List
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Maciej Fijalkowski <maciej.fijalkowski@intel.com>,
	Alexei Starovoitov <ast@kernel.org>,
	Daniel Borkmann <daniel@iogearbox.net>
Cc: netdev@vger.kernel.org, magnus.karlsson@intel.com,
	stfomichev@gmail.com, pabeni@redhat.com,
	tushar.vyavahare@intel.com, kerneljasonxing@gmail.com,
	bjorn@kernel.org, bpf@vger.kernel.org
Subject: Re: [PATCH v2 net-next 0/4] selftests: move the AF_XDP test suite from bpf to net
Date: Wed, 11 Nov 2026 15:57:48 -0800	[thread overview]
Message-ID: <20261111155748.11b92b7b@kernel.org> (raw)
In-Reply-To: <20261109193048.1006986-1-maciej.fijalkowski@intel.com>

On Mon,  9 Nov 2026 20:30:44 +0100 Maciej Fijalkowski wrote:
> As the maintainers asked, this series is the first step of moving the
> AF_XDP selftests from selftests/bpf to selftests/net: the code moves and
> builds from its new location, without changing what the tests do. The
> rework [0] posted as "selftests: net: migrate AF_XDP test suite over to
> net" (v4) will follow on top of this move.

Close enough, but why did you skip CCing bpf@ :(

BPF folks, I made a tag for both of us to merge / pull to avoid
Makefile conflicts in selftests, the original series is here:
https://lore.kernel.org/all/20261109193048.1006986-1-maciej.fijalkowski@intel.com/
PR text follows. Please LMK if / how you'd like to proceed.

The following changes since commit 7475ea8b58616b33c04afb976cf0fbaad9dd63fb:

  Linux 7.4-rc2 (2026-11-08 16:22:20 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/kuba/linux.git tags/bpf-net-xsk-move

for you to fetch changes up to d01fb5c8113055e3d1b7f573261ae6a30abf4e53:

  selftests: net: move xdp_hw_metadata from selftests/bpf (2026-11-11 15:50:02 -0800)

----------------------------------------------------------------
Maciej says:

As the maintainers asked, this series is the first step of moving the
AF_XDP selftests from selftests/bpf to selftests/net: the code moves and
builds from its new location, without changing what the tests do. The
rework [0] posted as "selftests: net: migrate AF_XDP test suite over to
net" (v4) will follow on top of this move.

Patch 1 drops the test_progs wrapper of the AF_XDP tests, patch 2 adds a
generic BPF skeleton rule to net/bpf.mk, patch 3 moves xskxceiver, its
library and test_xsk.sh to selftests/net, and patch 4 (new, not covered
at [0]) moves xdp_hw_metadata, which is built from the same library.

The xdp_metadata test stays in test_progs: it covers BPF core
behaviour, and moving it would mean rewriting it for the kselftest
harness. It keeps using xsk.c from net/lib/xsk.

xskxceiver and xdp_hw_metadata are the first users of bpftool at build
time in selftests/net. NIPA has bpftool, but 0-day and KernelCI build
net without it, so both tools are skipped with a warning when bpftool
is missing or does not run.

test_xsk.sh is installed but not run by run_tests, so NIPA does not pick
it up: it runs every case, including flaky ones, and some cases leak
page_pool pages after skb_pp_cow_data() in generic XDP mode on
veth-based runs. Until a later step makes the suite fit for NIPA, no CI
runs the AF_XDP tests. test_xsk.sh is being removed by [0] but it is
kept here as there is no other feasible way to run the test suite (if
needed) until we come up with an upstreamable Python runner.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>

----------------------------------------------------------------
Maciej Fijalkowski (4):
      selftests/bpf: drop the test_progs AF_XDP wrapper
      selftests: net: add a generic rule for BPF skeletons
      selftests: xsk: move the AF_XDP test suite to selftests/net
      selftests: net: move xdp_hw_metadata from selftests/bpf

 Documentation/networking/af_xdp.rst                |   6 +-
 Documentation/networking/xsk-tx-metadata.rst       |   2 +-
 MAINTAINERS                                        |   4 +-
 tools/testing/selftests/bpf/.gitignore             |   2 -
 tools/testing/selftests/bpf/Makefile               |  29 +---
 tools/testing/selftests/bpf/Makefile.buildvars     |   6 +-
 tools/testing/selftests/bpf/network_helpers.c      |  48 ------
 tools/testing/selftests/bpf/network_helpers.h      |   2 -
 tools/testing/selftests/bpf/prog_tests/xsk.c       | 170 ---------------------
 tools/testing/selftests/net/.gitignore             |   2 +
 tools/testing/selftests/net/Makefile               |  22 +++
 tools/testing/selftests/net/bpf.mk                 |  13 +-
 tools/testing/selftests/net/config                 |   1 +
 tools/testing/selftests/net/lib/.gitignore         |   2 +
 tools/testing/selftests/net/lib/Makefile           |  25 +++
 .../{bpf/prog_tests => net/lib/xsk}/test_xsk.c     |   2 +-
 .../{bpf/prog_tests => net/lib/xsk}/test_xsk.h     |   5 +-
 tools/testing/selftests/{bpf => net/lib/xsk}/xsk.c |  49 +++++-
 tools/testing/selftests/{bpf => net/lib/xsk}/xsk.h |   4 +
 .../{bpf => net/lib/xsk}/xsk_xdp_common.h          |   0
 .../lib/xsk/xsk_xdp_progs.bpf.c}                   |   2 +-
 .../selftests/{bpf => net/lib/xsk}/xskxceiver.c    |  17 +--
 .../selftests/{bpf => net/lib/xsk}/xskxceiver.h    |   0
 tools/testing/selftests/{bpf => net}/test_xsk.sh   |   5 +
 .../xdp_hw_metadata.bpf.c}                         |  35 +++--
 .../selftests/{bpf => net}/xdp_hw_metadata.c       |  81 +++++++++-
 tools/testing/selftests/net/xdp_metadata.h         |  67 ++++++++
 .../testing/selftests/{bpf => net}/xsk_prereqs.sh  |   2 +-
 28 files changed, 309 insertions(+), 294 deletions(-)
 delete mode 100644 tools/testing/selftests/bpf/prog_tests/xsk.c
 rename tools/testing/selftests/{bpf/prog_tests => net/lib/xsk}/test_xsk.c (99%)
 rename tools/testing/selftests/{bpf/prog_tests => net/lib/xsk}/test_xsk.h (99%)
 rename tools/testing/selftests/{bpf => net/lib/xsk}/xsk.c (95%)
 rename tools/testing/selftests/{bpf => net/lib/xsk}/xsk.h (97%)
 rename tools/testing/selftests/{bpf => net/lib/xsk}/xsk_xdp_common.h (100%)
 rename tools/testing/selftests/{bpf/progs/xsk_xdp_progs.c => net/lib/xsk/xsk_xdp_progs.bpf.c} (98%)
 rename tools/testing/selftests/{bpf => net/lib/xsk}/xskxceiver.c (96%)
 rename tools/testing/selftests/{bpf => net/lib/xsk}/xskxceiver.h (100%)
 rename tools/testing/selftests/{bpf => net}/test_xsk.sh (98%)
 rename tools/testing/selftests/{bpf/progs/xdp_hw_metadata.c => net/xdp_hw_metadata.bpf.c} (75%)
 rename tools/testing/selftests/{bpf => net}/xdp_hw_metadata.c (92%)
 create mode 100644 tools/testing/selftests/net/xdp_metadata.h
 rename tools/testing/selftests/{bpf => net}/xsk_prereqs.sh (98%)

      parent reply	other threads:[~2026-11-11 23:57 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-09 19:30 [PATCH v2 net-next 0/4] selftests: move the AF_XDP test suite from bpf to net Maciej Fijalkowski
2026-11-09 19:30 ` [PATCH v2 net-next 1/4] selftests/bpf: drop the test_progs AF_XDP wrapper Maciej Fijalkowski
2026-11-09 19:30 ` [PATCH v2 net-next 2/4] selftests: net: add a generic rule for BPF skeletons Maciej Fijalkowski
2026-11-09 19:30 ` [PATCH v2 net-next 3/4] selftests: xsk: move the AF_XDP test suite to selftests/net Maciej Fijalkowski
2026-11-09 19:30 ` [PATCH v2 net-next 4/4] selftests: net: move xdp_hw_metadata from selftests/bpf Maciej Fijalkowski
2026-11-11 23:57 ` Jakub Kicinski [this message]

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=20261111155748.11b92b7b@kernel.org \
    --to=kuba@kernel.org \
    --cc=ast@kernel.org \
    --cc=bjorn@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=daniel@iogearbox.net \
    --cc=kerneljasonxing@gmail.com \
    --cc=maciej.fijalkowski@intel.com \
    --cc=magnus.karlsson@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=stfomichev@gmail.com \
    --cc=tushar.vyavahare@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox