public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
* [PATCH] test/perf/pasta_udp: Add host to namespace cases for traffic via tap
@ 2023-01-04  0:08 Stefano Brivio
  2023-01-04  4:27 ` David Gibson
  0 siblings, 1 reply; 2+ messages in thread
From: Stefano Brivio @ 2023-01-04  0:08 UTC (permalink / raw)
  To: passt-dev; +Cc: David Gibson

These were missing as it wasn't clear, when the other tests were
introduced, if using the global address of a namespace, from the
host, should have resulted in traffic being routed via the tap
interface (as opposed to the loopback interface). We now clarified
that's actually the case.

Use same values and thresholds as the tests for loopback traffic, as
throughput figures currently indicate there isn't much difference.

Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
---
Contrary to what I stated a while ago, there's actually no need to
adapt the "code" writing the JavaScript snippets for performance
reports -- the trick is simply to keep those ordered like the
previous ones, and we'll get the right output.

 test/perf/pasta_udp | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/test/perf/pasta_udp b/test/perf/pasta_udp
index 27ea724..112d70d 100644
--- a/test/perf/pasta_udp
+++ b/test/perf/pasta_udp
@@ -179,4 +179,41 @@ nsout	LAT udp_rr --nolog -P 10003 -C 10013 -4 -c -H __GW__ | sed -n 's/^throughp
 hostw
 lat	__LAT__ 200 150
 
+
+tr	UDP throughput over IPv6: host to ns
+nsout	IFNAME ip -j link show | jq -rM '.[] | select(.link_type == "ether").ifname'
+nsout	ADDR6 ip -j -6 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[] | select(.scope == "global" and .prefixlen == 64).local'
+bw	-
+bw	-
+bw	-
+iperf3	BW host ns __ADDR6__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
+bw	__BW__ 7.0 9.0
+
+tl	UDP RR latency over IPv6: host to ns
+lat	-
+lat	-
+lat	-
+nsb	udp_rr --nolog -P 10002 -C 10012 -6
+hout	LAT udp_rr --nolog -P 10002 -C 10012 -6 -c -H __ADDR6__ | sed -n 's/^throughput=\(.*\)/\1/p'
+nsw
+lat	__LAT__ 200 150
+
+
+tr	UDP throughput over IPv4: host to ns
+nsout	ADDR ip -j -4 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[0].local'
+bw	-
+bw	-
+bw	-
+iperf3	BW host ns __ADDR__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
+bw	__BW__ 7.0 9.0
+
+tl	UDP RR latency over IPv4: host to ns
+lat	-
+lat	-
+lat	-
+nsb	udp_rr --nolog -P 10002 -C 10012 -4
+hout	LAT udp_rr --nolog -P 10002 -C 10012 -4 -c -H __ADDR__ | sed -n 's/^throughput=\(.*\)/\1/p'
+nsw
+lat	__LAT__ 200 150
+
 te
-- 
@@ -179,4 +179,41 @@ nsout	LAT udp_rr --nolog -P 10003 -C 10013 -4 -c -H __GW__ | sed -n 's/^throughp
 hostw
 lat	__LAT__ 200 150
 
+
+tr	UDP throughput over IPv6: host to ns
+nsout	IFNAME ip -j link show | jq -rM '.[] | select(.link_type == "ether").ifname'
+nsout	ADDR6 ip -j -6 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[] | select(.scope == "global" and .prefixlen == 64).local'
+bw	-
+bw	-
+bw	-
+iperf3	BW host ns __ADDR6__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
+bw	__BW__ 7.0 9.0
+
+tl	UDP RR latency over IPv6: host to ns
+lat	-
+lat	-
+lat	-
+nsb	udp_rr --nolog -P 10002 -C 10012 -6
+hout	LAT udp_rr --nolog -P 10002 -C 10012 -6 -c -H __ADDR6__ | sed -n 's/^throughput=\(.*\)/\1/p'
+nsw
+lat	__LAT__ 200 150
+
+
+tr	UDP throughput over IPv4: host to ns
+nsout	ADDR ip -j -4 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[0].local'
+bw	-
+bw	-
+bw	-
+iperf3	BW host ns __ADDR__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
+bw	__BW__ 7.0 9.0
+
+tl	UDP RR latency over IPv4: host to ns
+lat	-
+lat	-
+lat	-
+nsb	udp_rr --nolog -P 10002 -C 10012 -4
+hout	LAT udp_rr --nolog -P 10002 -C 10012 -4 -c -H __ADDR__ | sed -n 's/^throughput=\(.*\)/\1/p'
+nsw
+lat	__LAT__ 200 150
+
 te
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] test/perf/pasta_udp: Add host to namespace cases for traffic via tap
  2023-01-04  0:08 [PATCH] test/perf/pasta_udp: Add host to namespace cases for traffic via tap Stefano Brivio
@ 2023-01-04  4:27 ` David Gibson
  0 siblings, 0 replies; 2+ messages in thread
From: David Gibson @ 2023-01-04  4:27 UTC (permalink / raw)
  To: Stefano Brivio; +Cc: passt-dev

[-- Attachment #1: Type: text/plain, Size: 2710 bytes --]

On Wed, Jan 04, 2023 at 01:08:39AM +0100, Stefano Brivio wrote:
> These were missing as it wasn't clear, when the other tests were
> introduced, if using the global address of a namespace, from the
> host, should have resulted in traffic being routed via the tap
> interface (as opposed to the loopback interface). We now clarified
> that's actually the case.
> 
> Use same values and thresholds as the tests for loopback traffic, as
> throughput figures currently indicate there isn't much difference.
> 
> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

> ---
> Contrary to what I stated a while ago, there's actually no need to
> adapt the "code" writing the JavaScript snippets for performance
> reports -- the trick is simply to keep those ordered like the
> previous ones, and we'll get the right output.
> 
>  test/perf/pasta_udp | 37 +++++++++++++++++++++++++++++++++++++
>  1 file changed, 37 insertions(+)
> 
> diff --git a/test/perf/pasta_udp b/test/perf/pasta_udp
> index 27ea724..112d70d 100644
> --- a/test/perf/pasta_udp
> +++ b/test/perf/pasta_udp
> @@ -179,4 +179,41 @@ nsout	LAT udp_rr --nolog -P 10003 -C 10013 -4 -c -H __GW__ | sed -n 's/^throughp
>  hostw
>  lat	__LAT__ 200 150
>  
> +
> +tr	UDP throughput over IPv6: host to ns
> +nsout	IFNAME ip -j link show | jq -rM '.[] | select(.link_type == "ether").ifname'
> +nsout	ADDR6 ip -j -6 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[] | select(.scope == "global" and .prefixlen == 64).local'
> +bw	-
> +bw	-
> +bw	-
> +iperf3	BW host ns __ADDR6__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
> +bw	__BW__ 7.0 9.0
> +
> +tl	UDP RR latency over IPv6: host to ns
> +lat	-
> +lat	-
> +lat	-
> +nsb	udp_rr --nolog -P 10002 -C 10012 -6
> +hout	LAT udp_rr --nolog -P 10002 -C 10012 -6 -c -H __ADDR6__ | sed -n 's/^throughput=\(.*\)/\1/p'
> +nsw
> +lat	__LAT__ 200 150
> +
> +
> +tr	UDP throughput over IPv4: host to ns
> +nsout	ADDR ip -j -4 addr show|jq -rM '.[] | select(.ifname == "__IFNAME__").addr_info[0].local'
> +bw	-
> +bw	-
> +bw	-
> +iperf3	BW host ns __ADDR__ 100${i}2 __THREADS__ __TIME__ __OPTS__ -b 15G
> +bw	__BW__ 7.0 9.0
> +
> +tl	UDP RR latency over IPv4: host to ns
> +lat	-
> +lat	-
> +lat	-
> +nsb	udp_rr --nolog -P 10002 -C 10012 -4
> +hout	LAT udp_rr --nolog -P 10002 -C 10012 -4 -c -H __ADDR__ | sed -n 's/^throughput=\(.*\)/\1/p'
> +nsw
> +lat	__LAT__ 200 150
> +
>  te

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-01-04  4:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-04  0:08 [PATCH] test/perf/pasta_udp: Add host to namespace cases for traffic via tap Stefano Brivio
2023-01-04  4:27 ` David Gibson

Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

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 IMAP folder(s).