[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <09a5c481-883d-48f4-81d4-1028f7cd72d7@amd.com>
Date: Wed, 4 Sep 2024 09:15:22 -0700
From: Brett Creeley <bcreeley@....com>
To: Mohan Prasad J <mohan.prasad@...rochip.com>, netdev@...r.kernel.org,
davem@...emloft.net, kuba@...nel.org
Cc: shuah@...nel.org, bryan.whitehead@...rochip.com,
UNGLinuxDriver@...rochip.com, edumazet@...gle.com, pabeni@...hat.com,
linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org,
horms@...nel.org, brett.creeley@....com, rosenp@...il.com
Subject: Re: [PATCH net-next 3/3] selftests: lan743x: Add testcase to check
throughput of lan743x
On 9/3/2024 3:15 PM, Mohan Prasad J wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
>
>
> Add testcase to check TCP throughput of lan743x network driver.
> Test uses iperf3 to do performance testing of the driver.
> TCP data at different speeds is sent, received and verified.
>
> Signed-off-by: Mohan Prasad J <mohan.prasad@...rochip.com>
> ---
> .../net/hw/microchip/lan743x/lan743x.py | 33 +++++++++++++++++++
> 1 file changed, 33 insertions(+)
>
> diff --git a/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> index 59f0be2a7..a3dcf7896 100755
> --- a/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> +++ b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> @@ -3,6 +3,7 @@
>
> import time
> import re
> +import json
> from lib.py import ksft_run, ksft_exit, ksft_pr, ksft_eq
> from lib.py import KsftFailEx, KsftSkipEx
> from lib.py import NetDrvEpEnv
> @@ -75,6 +76,38 @@ def test_network_speed(cfg) -> None:
> time.sleep(5)
> verify_speed_and_duplex(cfg.ifname, speed, duplex)
>
> +def test_tcp_throughput(cfg) -> None:
> + speeds = ["10", "100", "1000"]
> + """Test duration in seconds"""
> + test_duration = 5
> + target_ip = cfg.remote_addr
> +
> + for speed in speeds:
> + set_speed_and_duplex(cfg.ifname, speed, 'full')
> + time.sleep(5)
> + verify_link_up(cfg.ifname)
> + send_command=f"iperf3 -c {target_ip} -t {test_duration} --json"
> + receive_command=f"iperf3 -c {target_ip} -t {test_duration} --reverse --json"
> + send_result = cmd(send_command)
> + receive_result = cmd(receive_command)
> + if send_result.ret != 0 or receive_result.ret != 0:
> + raise KsftSkipEx("No server is running")
> +
> + send_output = send_result.stdout
> + receive_output = receive_result.stdout
> +
> + send_data = json.loads(send_output)
> + receive_data = json.loads(receive_output)
> + """Convert throughput to Mbps"""
> + send_throughput = round(send_data['end']['sum_sent']['bits_per_second'] / 1e6, 2)
> + receive_throughput = round(receive_data['end']['sum_received']['bits_per_second'] / 1e6, 2)
> +
> + ksft_pr(f"Send throughput: {send_throughput} Mbps, Receive throughput: {receive_throughput} Mbps")
> + """Check whether throughput is not below 0.9 times the set speed"""
> + threshold = float(speed) * 0.9
> + if send_throughput < threshold or receive_throughput < threshold:
> + raise KsftFailEx("Throughput is below threshold")
IMO it would be best to do these checks separately so the failure is
immediately obvious.
Thanks,
Brett
> +
> def main() -> None:
> with NetDrvEpEnv(__file__) as cfg:
> ksft_run(globs=globals(), case_pfx={"test_"}, args=(cfg,))
> --
> 2.43.0
>
Powered by blists - more mailing lists