Skip to content

selftests: Assortment of fixes

Petr Machata linux-kselftest netdev 1 version Applied

Status from this message: This series was applied to netdev/net-next.git (main)

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 7 repliesApplied 10/10

Patches in v1 All 10 applied

#Subject and filesSizeRepliesApplied
1/10selftests: net: Unify code of busywait() and slowwait()tools/testing/selftests/net/forwarding/lib.sh tools/testing/selftests/net/lib.sh+15 -231 replyApplied
2/10selftests: forwarding: lib.sh: Validate NETIFStools/testing/selftests/net/forwarding/lib.sh+17 -51 replyApplied
3/10selftests: forwarding: bail_on_lldpad() should SKIPtools/testing/selftests/net/forwarding/lib.sh+4 -11 replyApplied
4/10selftests: drivers: hw: Fix ethtool_rmontools/testing/selftests/drivers/net/hw/ethtool_rmon.sh+1 -00 repliesApplied
5/10selftests: drivers: hw: ethtool.sh: Adjust outputtools/testing/selftests/drivers/net/hw/ethtool.sh+6 -90 repliesApplied
6/10selftests: drivers: hw: Include tc_common.sh in hw_stats_l3tools/testing/selftests/drivers/net/hw/hw_stats_l3.sh tools/testing/selftests/drivers/net/hw/hw_stats_l3_gre.sh+2 -00 repliesApplied
7/10selftests: mlxsw: ethtool_lanes: Wait for lanes parameter dump explicitlytools/testing/selftests/drivers/net/mlxsw/ethtool_lanes.sh+8 -60 repliesApplied
8/10selftests: forwarding: router_mpath_nh: Add a diagramtools/testing/selftests/net/forwarding/router_mpath_nh.sh+35 -02 repliesApplied
9/10selftests: forwarding: router_mpath_nh_res: Add a diagramtools/testing/selftests/net/forwarding/router_mpath_nh_res.sh+35 -01 replyApplied
10/10selftests: forwarding: router_nh: Add a diagramtools/testing/selftests/net/forwarding/router_nh.sh+14 -01 replyApplied

Reviews trailers per patch

7 of 10 patches have a Reviewed-by or Acked-by.

PatchBenjamin PoirierPetr MachataHangbin LiuDavid Ahern
1/10R-b R-b
2/10R-b R-b
3/10R-b R-b
4/10
5/10
6/10
7/10R-b
8/10R-b R-b
9/10R-b
10/10R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.

Read the v1 thread

Cover letter [PATCH net-next 00/10] selftests: Assortment of fixes

This is a loose follow-up to the Kernel CI patchset posted recently. It contains various fixes that were supposed to be part of said patchset, but didn't fit due to its size. The latter 4 patches were written independently of the CI effort, but again didn't fit in their intended patchsets.

- Patch #1 unifies code of two very similar looking functions, busywait() and slowwait().

- Patch #2 adds sanity checks around the setting of NETIFS, which carries list of interfaces to run on.

- Patch #3 changes bail_on_lldpad() to SKIP instead of FAILing.

- Patches #4 to #7 fix issues in selftests.

- Patches #8 to #10 add topology diagrams to several selftests. This should have been part of the mlxsw leg of NH group stats patches, but again, it did not fit in due to size.

Danielle Ratson (1):
selftests: mlxsw: ethtool_lanes: Wait for lanes parameter dump
explicitly

Petr Machata (9):
selftests: net: Unify code of busywait() and slowwait()
selftests: forwarding: lib.sh: Validate NETIFS
selftests: forwarding: bail_on_lldpad() should SKIP
selftests: drivers: hw: Fix ethtool_rmon
selftests: drivers: hw: ethtool.sh: Adjust output
selftests: drivers: hw: Include tc_common.sh in hw_stats_l3
selftests: forwarding: router_mpath_nh: Add a diagram
selftests: forwarding: router_mpath_nh_res: Add a diagram
selftests: forwarding: router_nh: Add a diagram

 .../selftests/drivers/net/hw/ethtool.sh       | 15 +++---
 .../selftests/drivers/net/hw/ethtool_rmon.sh  |  1 +
 .../selftests/drivers/net/hw/hw_stats_l3.sh   |  1 +
 .../drivers/net/hw/hw_stats_l3_gre.sh         |  1 +
 .../drivers/net/mlxsw/ethtool_lanes.sh        | 14 +++---
 tools/testing/selftests/net/forwarding/lib.sh | 49 +++++++++----------
 .../net/forwarding/router_mpath_nh.sh         | 35 +++++++++++++
 .../net/forwarding/router_mpath_nh_res.sh     | 35 +++++++++++++
 .../selftests/net/forwarding/router_nh.sh     | 14 ++++++
 tools/testing/selftests/net/lib.sh            | 16 ++++--
 10 files changed, 137 insertions(+), 44 deletions(-)
Signature · 1 line
2.43.0