netdev - Re: [PATCH bpf-next v3 00/16] selftests: xsk: various simplifications

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20210825182656.GA26792@ranger.igk.intel.com>
Date:   Wed, 25 Aug 2021 20:26:56 +0200
From:   Maciej Fijalkowski <maciej.fijalkowski@...el.com>
To:     Magnus Karlsson <magnus.karlsson@...il.com>
Cc:     magnus.karlsson@...el.com, bjorn@...nel.org, ast@...nel.org,
        daniel@...earbox.net, netdev@...r.kernel.org,
        jonathan.lemon@...il.com, ciara.loftus@...el.com,
        bpf@...r.kernel.org, yhs@...com, andrii@...nel.org
Subject: Re: [PATCH bpf-next v3 00/16] selftests: xsk: various simplifications

On Wed, Aug 25, 2021 at 11:37:06AM +0200, Magnus Karlsson wrote:
> This patch set mainly contains various simplifications to the xsk
> selftests. The only exception is the introduction of packet streams
> that describes what the Tx process should send and what the Rx process
> should receive. If it receives anything else, the test fails. This
> mechanism can be used to produce tests were all packets are not
> received by the Rx thread or modified in some way. An example of this
> is if an XDP program does XDP_PASS on some of the packets.
> 
> This patch set will be followed by another patch set that implements a
> new structure that will facilitate adding new tests. A couple of new
> tests will also be included in that patch set.

I went through the series and besides the typo found by Alexei I have no
objections.

Acked-by: Maciej Fijalkowski <maciej.fijalkowski@...el.com>

Looking forward to next set that you mention above :)

> 
> v2 -> v3:
> 
> * Reworked patch 12 so that it now has functions for creating and
>   destroying ifobjects. Simplifies the code. [Maciej]
> * The packet stream now allocates the supplied buffer array length,
>   instead of the default one. [Maciej]
> * pkt_stream_get_pkt() now returns NULL when indexing a non-existing
>   packet. [Maciej]
> * pkt_validate() is now is_pkt_valid(). [Maciej]
> * Slowed down packet sending speed even more in patch 11 so that slow
>   systems do not silenty drop packets in skb mode.
> 
> v1 -> v2:
> 
> * Dropped the patch with per process limit changes as it is not needed
>   [Yonghong]
> * Improved the commit message of patch 1 [Yonghong]
> * Fixed a spelling error in patch 9
> 
> Thanks: Magnus
> 
> Magnus Karlsson (16):
>   selftests: xsk: remove color mode
>   selftests: xsk: remove the num_tx_packets option
>   selftests: xsk: remove unused variables
>   selftests: xsk: return correct error codes
>   selftests: xsk: simplify the retry code
>   selftests: xsk: remove end-of-test packet
>   selftests: xsk: disassociate umem size with packets sent
>   selftests: xsk: rename worker_* functions that are not thread entry
>     points
>   selftests: xsk: simplify packet validation in xsk tests
>   selftests: xsk: validate tx stats on tx thread
>   selftests: xsk: decrease sending speed
>   selftests: xsk: simplify cleanup of ifobjects
>   selftests: xsk: generate packet directly in umem
>   selftests: xsk: generate packets from specification
>   selftests: xsk: make enums lower case
>   selftests: xsk: preface options with opt
> 
>  tools/testing/selftests/bpf/test_xsk.sh    |  10 +-
>  tools/testing/selftests/bpf/xdpxceiver.c   | 681 ++++++++++-----------
>  tools/testing/selftests/bpf/xdpxceiver.h   |  63 +-
>  tools/testing/selftests/bpf/xsk_prereqs.sh |  30 +-
>  4 files changed, 356 insertions(+), 428 deletions(-)
> 
> 
> base-commit: 3bbc8ee7c363a83aa192d796ad37b6bf462a2947
> --
> 2.29.0