Re: [PATCH bpf-next v2] selftests/bpf: Track test_xdp_features DUT processes

["Alexis Lothoré" <alexis.lothore@bootlin.com>]

On Fri Aug 28, 2026 at 8:10 AM CEST, Bochao Cao wrote:
>  Thanks Daniel and Alexis.
>
>   Dropping the procps dependency is not the primary motivation for
>   this change. I agree that Debian can remove the dependency independently.
>   The issue addressed by this patch is process isolation in the standalone
>   test. The current readiness check may observe an unrelated
>   concurrent xdp_features process, while cleanup may terminate every
> xdp_features
>   process on the host. In addition, a DUT which exits before listening can
> leave the test waiting indefinitely.
>
>   Although this script is not currently run by the BPF CI, it remains
> useful  for testing real hardware,
>   so these process lifecycle issues can still  affect users running the
> test manually.
>
>   Would it be acceptable to fix these issues in the script's current
> location?
>   If the preferred direction is to move it under
> tools/testing/selftests/drivers/net/hw/,
>   should the move be submitted first, with this fix rebased on top?

I have no strong feeling about whether it should be moved or kept here,
I'll let BPF/netdev maintainers express their opinions here (Jakub
already suggested in [1] that there are helpers in net testing helpers
that could help making it move to the net tests). I've added Lorenzo in
CC, who is the original author of the script.

Alexis

[1] https://lore.kernel.org/bpf/20240914063828.7bd73c5e@kernel.org/

-- 
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com