netdev - Re: [PATCH net v2 00/17] selftests: forwarding: Various fixes

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: <3c9c3d17-cd4b-c7e4-0b28-ffa1540ce9dc@blackwall.org>
Date: Wed, 9 Aug 2023 15:10:42 +0300
From: Nikolay Aleksandrov <razor@...ckwall.org>
To: Ido Schimmel <idosch@...dia.com>, netdev@...r.kernel.org
Cc: davem@...emloft.net, kuba@...nel.org, pabeni@...hat.com,
 edumazet@...gle.com, petrm@...dia.com, mirsad.todorovac@....unizg.hr
Subject: Re: [PATCH net v2 00/17] selftests: forwarding: Various fixes

On 8/8/23 17:14, Ido Schimmel wrote:
> Fix various problems with forwarding selftests. See individual patches
> for problem description and solution.
> 
> v2:
> * Patch #10: Probe for MAC Merge support.
> 

Nice! Thanks for taking the time to fix all these.
Acked-by: Nikolay Aleksandrov <razor@...ckwall.org>

> Ido Schimmel (17):
>    selftests: forwarding: Skip test when no interfaces are specified
>    selftests: forwarding: Switch off timeout
>    selftests: forwarding: bridge_mdb: Check iproute2 version
>    selftests: forwarding: bridge_mdb_max: Check iproute2 version
>    selftests: forwarding: Set default IPv6 traceroute utility
>    selftests: forwarding: Add a helper to skip test when using veth pairs
>    selftests: forwarding: ethtool: Skip when using veth pairs
>    selftests: forwarding: ethtool_extended_state: Skip when using veth
>      pairs
>    selftests: forwarding: hw_stats_l3_gre: Skip when using veth pairs
>    selftests: forwarding: ethtool_mm: Skip when MAC Merge is not
>      supported
>    selftests: forwarding: tc_actions: Use ncat instead of nc
>    selftests: forwarding: tc_flower: Relax success criterion
>    selftests: forwarding: tc_tunnel_key: Make filters more specific
>    selftests: forwarding: tc_flower_l2_miss: Fix failing test with old
>      libnet
>    selftests: forwarding: bridge_mdb: Fix failing test with old libnet
>    selftests: forwarding: bridge_mdb_max: Fix failing test with old
>      libnet
>    selftests: forwarding: bridge_mdb: Make test more robust
> 
>   .../selftests/net/forwarding/bridge_mdb.sh    | 59 +++++++++++--------
>   .../net/forwarding/bridge_mdb_max.sh          | 19 ++++--
>   .../selftests/net/forwarding/ethtool.sh       |  2 +
>   .../net/forwarding/ethtool_extended_state.sh  |  2 +
>   .../selftests/net/forwarding/ethtool_mm.sh    | 18 ++++--
>   .../net/forwarding/hw_stats_l3_gre.sh         |  2 +
>   .../net/forwarding/ip6_forward_instats_vrf.sh |  2 +
>   tools/testing/selftests/net/forwarding/lib.sh | 17 ++++++
>   .../testing/selftests/net/forwarding/settings |  1 +
>   .../selftests/net/forwarding/tc_actions.sh    |  6 +-
>   .../selftests/net/forwarding/tc_flower.sh     |  8 +--
>   .../net/forwarding/tc_flower_l2_miss.sh       | 13 ++--
>   .../selftests/net/forwarding/tc_tunnel_key.sh |  9 ++-
>   13 files changed, 109 insertions(+), 49 deletions(-)
>   create mode 100644 tools/testing/selftests/net/forwarding/settings
>