From: Nikolay Aleksandrov <razor@blackwall.org>
To: Ido Schimmel <idosch@nvidia.com>, netdev@vger.kernel.org
Cc: davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com,
edumazet@google.com, petrm@nvidia.com,
mirsad.todorovac@alu.unizg.hr
Subject: Re: [PATCH net v2 00/17] selftests: forwarding: Various fixes
Date: Wed, 9 Aug 2023 15:10:42 +0300 [thread overview]
Message-ID: <3c9c3d17-cd4b-c7e4-0b28-ffa1540ce9dc@blackwall.org> (raw)
In-Reply-To: <20230808141503.4060661-1-idosch@nvidia.com>
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@blackwall.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
>
next prev parent reply other threads:[~2023-08-09 12:10 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-08 14:14 [PATCH net v2 00/17] selftests: forwarding: Various fixes Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 01/17] selftests: forwarding: Skip test when no interfaces are specified Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 02/17] selftests: forwarding: Switch off timeout Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 03/17] selftests: forwarding: bridge_mdb: Check iproute2 version Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 04/17] selftests: forwarding: bridge_mdb_max: " Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 05/17] selftests: forwarding: Set default IPv6 traceroute utility Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 06/17] selftests: forwarding: Add a helper to skip test when using veth pairs Ido Schimmel
2023-08-09 1:55 ` Hangbin Liu
2023-08-09 5:47 ` Ido Schimmel
2023-08-09 6:46 ` Hangbin Liu
2023-08-08 14:14 ` [PATCH net v2 07/17] selftests: forwarding: ethtool: Skip " Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 08/17] selftests: forwarding: ethtool_extended_state: " Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 09/17] selftests: forwarding: hw_stats_l3_gre: " Ido Schimmel
2023-08-09 2:02 ` Hangbin Liu
2023-08-09 5:45 ` Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 10/17] selftests: forwarding: ethtool_mm: Skip when MAC Merge is not supported Ido Schimmel
2023-08-10 17:28 ` Vladimir Oltean
2023-08-08 14:14 ` [PATCH net v2 11/17] selftests: forwarding: tc_actions: Use ncat instead of nc Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 12/17] selftests: forwarding: tc_flower: Relax success criterion Ido Schimmel
2023-08-08 14:14 ` [PATCH net v2 13/17] selftests: forwarding: tc_tunnel_key: Make filters more specific Ido Schimmel
2023-08-08 14:15 ` [PATCH net v2 14/17] selftests: forwarding: tc_flower_l2_miss: Fix failing test with old libnet Ido Schimmel
2023-08-08 14:15 ` [PATCH net v2 15/17] selftests: forwarding: bridge_mdb: " Ido Schimmel
2023-08-08 14:15 ` [PATCH net v2 16/17] selftests: forwarding: bridge_mdb_max: " Ido Schimmel
2023-08-08 14:15 ` [PATCH net v2 17/17] selftests: forwarding: bridge_mdb: Make test more robust Ido Schimmel
2023-08-09 6:47 ` [PATCH net v2 00/17] selftests: forwarding: Various fixes Hangbin Liu
2023-08-09 12:10 ` Nikolay Aleksandrov [this message]
2023-08-09 22:00 ` patchwork-bot+netdevbpf
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=3c9c3d17-cd4b-c7e4-0b28-ffa1540ce9dc@blackwall.org \
--to=razor@blackwall.org \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=idosch@nvidia.com \
--cc=kuba@kernel.org \
--cc=mirsad.todorovac@alu.unizg.hr \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=petrm@nvidia.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;
as well as URLs for NNTP newsgroup(s).