mbox series

[PATCHv5,net-next,0/4] selftests: bonding: use slowwait when waiting

Message ID 20240205130048.282087-1-liuhangbin@gmail.com (mailing list archive)
Headers show
Series selftests: bonding: use slowwait when waiting | expand

Message

Hangbin Liu Feb. 5, 2024, 1 p.m. UTC
There are a lot waitings in bonding tests use sleep. Let's replace them with
slowwait(added in the first patch). This could save much test time. e.g.

bond-break-lacpdu-tx.sh
  before: 0m16.346s
  after: 0m2.824s

bond_options.sh
  before: 9m25.299s
  after: 6m14.439s

bond-lladdr-target.sh
  before: 0m7.090s
  after: 0m6.148s

In total, we could save about 180 seconds.

v5: Drop the changes in bond_macvlan test (Jakub Kicinski)
v4: Make sure the client could reach to macvlan2 (Jakub Kicinski)
v3: Rebase to latest net-next
v2: Reduce slowwait sleep time to 0.1 (Paolo Abeni)
    Reduce num_grat_arp() miimon time (Paolo Abeni)
    Use slowwait for ping result in lag_lib.sh

Hangbin Liu (4):
  selftests/net/forwarding: add slowwait functions
  selftests: bonding: use tc filter to check if LACP was sent
  selftests: bonding: reduce garp_test/arp_validate test time
  selftests: bonding: use slowwait instead of hard code sleep

 .../net/bonding/bond-break-lacpdu-tx.sh       | 19 +++++-----
 .../drivers/net/bonding/bond-lladdr-target.sh | 21 ++++++++--
 .../drivers/net/bonding/bond_options.sh       | 38 ++++++++++++++-----
 .../drivers/net/bonding/bond_topo_2d1c.sh     |  6 +--
 .../selftests/drivers/net/bonding/lag_lib.sh  |  7 ++--
 tools/testing/selftests/net/forwarding/lib.sh | 35 +++++++++++++++++
 6 files changed, 97 insertions(+), 29 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Feb. 8, 2024, 2:40 a.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon,  5 Feb 2024 21:00:44 +0800 you wrote:
> There are a lot waitings in bonding tests use sleep. Let's replace them with
> slowwait(added in the first patch). This could save much test time. e.g.
> 
> bond-break-lacpdu-tx.sh
>   before: 0m16.346s
>   after: 0m2.824s
> 
> [...]

Here is the summary with links:
  - [PATCHv5,net-next,1/4] selftests/net/forwarding: add slowwait functions
    https://git.kernel.org/netdev/net-next/c/c8f4b19d64b9
  - [PATCHv5,net-next,2/4] selftests: bonding: use tc filter to check if LACP was sent
    https://git.kernel.org/netdev/net-next/c/9150820c8830
  - [PATCHv5,net-next,3/4] selftests: bonding: reduce garp_test/arp_validate test time
    https://git.kernel.org/netdev/net-next/c/45bf79bc56c4
  - [PATCHv5,net-next,4/4] selftests: bonding: use slowwait instead of hard code sleep
    https://git.kernel.org/netdev/net-next/c/e1f0da9b90fb

You are awesome, thank you!