diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2022-09-02 12:04:34 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2022-09-07 11:01:22 +0200 |
commit | e5e10aff8179b5f410b4e2fe7923303fd183fc75 (patch) | |
tree | 5012fa7c298cafe7658728ea37482a8d947c7423 /test/perf/passt_tcp | |
parent | 29247d0db6953d7a3d45c4da763b487ef695ac1e (diff) | |
download | passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar.gz passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar.bz2 passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar.lz passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar.xz passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.tar.zst passt-e5e10aff8179b5f410b4e2fe7923303fd183fc75.zip |
test: Rewrite test_iperf3
test_iperf3() is a pretty inscrutable mess of nested background processes.
It has a number of ugly sleeps needed to wait for things to complete.
Rewrite it to be cleaner:
* Use the construct (a & b & wait) to run 'a' and 'b' in parallel, but
then wait for them both to complete before continuing
* This allows us to wait for both the server and client to finish, rather
than sleeping
* Use jq to do all the math we need to get the final result, rather than
jq followed by some complicated 'bc' mangling
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'test/perf/passt_tcp')
-rw-r--r-- | test/perf/passt_tcp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/perf/passt_tcp b/test/perf/passt_tcp index a960341..45095b6 100644 --- a/test/perf/passt_tcp +++ b/test/perf/passt_tcp @@ -11,8 +11,8 @@ # Copyright (c) 2021 Red Hat GmbH # Author: Stefano Brivio <sbrivio@redhat.com> -gtools sysctl ip jq nproc seq sleep bc iperf3 tcp_rr tcp_crr # From neper -nstools sysctl ip jq nproc seq sleep bc iperf3 tcp_rr tcp_crr +gtools sysctl ip jq nproc seq sleep iperf3 tcp_rr tcp_crr # From neper +nstools sysctl ip jq nproc seq sleep iperf3 tcp_rr tcp_crr htools bc head sed seq # In this setup, virtio_net TX queue sometimes hangs, still under investigation |