From: Stanislav Fomichev <sdf.kernel@gmail.com>
To: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
Cc: netdev@vger.kernel.org, magnus.karlsson@intel.com,
stfomichev@gmail.com, kuba@kernel.org, pabeni@redhat.com,
tushar.vyavahare@intel.com, kerneljasonxing@gmail.com,
bjorn@kernel.org
Subject: Re: [PATCH net-next 0/4] selftests: move the AF_XDP test suite from bpf to net
Date: Mon, 9 Nov 2026 08:52:20 -0800 [thread overview]
Message-ID: <avH47_V-WbLm35cN@devvm16445.scu0.facebook.com> (raw)
In-Reply-To: <avHAGLwtyucllh/M@boxer>
On 11/09, Maciej Fijalkowski wrote:
> On Thu, Nov 05, 2026 at 12:43:36PM -0800, Stanislav Fomichev wrote:
> > On 11/05, Maciej Fijalkowski wrote:
> > > Hi,
> > >
> > > 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.
> > >
> > > Thanks,
> > > Maciej
> > >
> > > [0]: https://lore.kernel.org/bpf/20261016174338.787135-1-maciej.fijalkowski@intel.com/
> > >
> > > 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
> >
> > Acked-by: Stanislav Fomichev <sdf@fomichev.me>
>
> Thanks for acking! However for some reason clashiko missed this set, can
> someone remind me what could be the case here?
>
> I also see in SUIE that check_selftest failed - is it mandatory to repost
> in this case?
```
Bad format: tools/testing/selftests/net/Makefile
```
Not super descriptive :-(
Maybe check on your side if this is complaining about something legit?
https://github.com/linux-netdev/nipa/blob/main/tests/patch/check_selftest/validate_makefile_format.py
next prev parent reply other threads:[~2026-11-09 16:52 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-05 13:41 [PATCH net-next 0/4] selftests: move the AF_XDP test suite from bpf to net Maciej Fijalkowski
2026-11-05 13:41 ` [PATCH net-next 1/4] selftests/bpf: drop the test_progs AF_XDP wrapper Maciej Fijalkowski
2026-11-05 13:41 ` [PATCH net-next 2/4] selftests: net: add a generic rule for BPF skeletons Maciej Fijalkowski
2026-11-05 13:41 ` [PATCH net-next 3/4] selftests: xsk: move the AF_XDP test suite to selftests/net Maciej Fijalkowski
2026-11-05 13:41 ` [PATCH net-next 4/4] selftests: net: move xdp_hw_metadata from selftests/bpf Maciej Fijalkowski
2026-11-05 20:43 ` [PATCH net-next 0/4] selftests: move the AF_XDP test suite from bpf to net Stanislav Fomichev
2026-11-09 12:44 ` Maciej Fijalkowski
2026-11-09 16:52 ` Stanislav Fomichev [this message]
2026-11-09 19:12 ` Maciej Fijalkowski
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=avH47_V-WbLm35cN@devvm16445.scu0.facebook.com \
--to=sdf.kernel@gmail.com \
--cc=bjorn@kernel.org \
--cc=kerneljasonxing@gmail.com \
--cc=kuba@kernel.org \
--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 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.