[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZymSRij76y6bvRi9@mini-arch>
Date: Mon, 4 Nov 2024 19:34:30 -0800
From: Stanislav Fomichev <stfomichev@...il.com>
To: Joe Damato <jdamato@...tly.com>, Stanislav Fomichev <sdf@...ichev.me>,
netdev@...r.kernel.org, davem@...emloft.net, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com, linux-kernel@...r.kernel.org,
linux-kselftest@...r.kernel.org, andrew+netdev@...n.ch,
shuah@...nel.org, horms@...nel.org, almasrymina@...gle.com,
willemb@...gle.com, petrm@...dia.com
Subject: Re: [PATCH net-next v7 12/12] selftests: ncdevmem: Add automated test
On 11/04, Joe Damato wrote:
> On Mon, Nov 04, 2024 at 10:14:30AM -0800, Stanislav Fomichev wrote:
> > Only RX side for now and small message to test the setup.
> > In the future, we can extend it to TX side and to testing
> > both sides with a couple of megs of data.
> >
> > make \
> > -C tools/testing/selftests \
> > TARGETS="drivers/hw/net" \
> > install INSTALL_PATH=~/tmp/ksft
> >
> > scp ~/tmp/ksft ${HOST}:
> > scp ~/tmp/ksft ${PEER}:
> >
> > cfg+="NETIF=${DEV}\n"
> > cfg+="LOCAL_V6=${HOST_IP}\n"
> > cfg+="REMOTE_V6=${PEER_IP}\n"
> > cfg+="REMOTE_TYPE=ssh\n"
> > cfg+="REMOTE_ARGS=root@...EER}\n"
> >
> > echo -e "$cfg" | ssh root@...OST} "cat > ksft/drivers/net/net.config"
> > ssh root@...OST} "cd ksft && ./run_kselftest.sh -t drivers/net:devmem.py"
> >
> > Reviewed-by: Mina Almasry <almasrymina@...gle.com>
> > Signed-off-by: Stanislav Fomichev <sdf@...ichev.me>
> > ---
> > .../testing/selftests/drivers/net/hw/Makefile | 1 +
> > .../selftests/drivers/net/hw/devmem.py | 45 +++++++++++++++++++
> > 2 files changed, 46 insertions(+)
> > create mode 100755 tools/testing/selftests/drivers/net/hw/devmem.py
> >
> > diff --git a/tools/testing/selftests/drivers/net/hw/Makefile b/tools/testing/selftests/drivers/net/hw/Makefile
> > index 182348f4bd40..1c6a77480923 100644
> > --- a/tools/testing/selftests/drivers/net/hw/Makefile
> > +++ b/tools/testing/selftests/drivers/net/hw/Makefile
> > @@ -3,6 +3,7 @@
> > TEST_PROGS = \
> > csum.py \
> > devlink_port_split.py \
> > + devmem.py \
> > ethtool.sh \
> > ethtool_extended_state.sh \
> > ethtool_mm.sh \
> > diff --git a/tools/testing/selftests/drivers/net/hw/devmem.py b/tools/testing/selftests/drivers/net/hw/devmem.py
> > new file mode 100755
> > index 000000000000..1416c31ff81e
> > --- /dev/null
> > +++ b/tools/testing/selftests/drivers/net/hw/devmem.py
> > @@ -0,0 +1,45 @@
> > +#!/usr/bin/env python3
> > +# SPDX-License-Identifier: GPL-2.0
> > +
> > +from lib.py import ksft_run, ksft_exit
> > +from lib.py import ksft_eq, KsftSkipEx
> > +from lib.py import NetDrvEpEnv
> > +from lib.py import bkg, cmd, rand_port, wait_port_listen
> > +from lib.py import ksft_disruptive
> > +
> > +
> > +def require_devmem(cfg):
> > + if not hasattr(cfg, "_devmem_probed"):
> > + port = rand_port()
> > + probe_command = f"./ncdevmem -f {cfg.ifname}"
> > + cfg._devmem_supported = cmd(probe_command, fail=False, shell=True).ret == 0
> > + cfg._devmem_probed = True
> > +
> > + if not cfg._devmem_supported:
> > + raise KsftSkipEx("Test requires devmem support")
> > +
> > +
> > +@...t_disruptive
> > +def check_rx(cfg) -> None:
> > + cfg.require_v6()
> > + require_devmem(cfg)
> > +
> > + port = rand_port()
> > + listen_cmd = f"./ncdevmem -l -f {cfg.ifname} -s {cfg.v6} -p {port}"
> > +
> > + with bkg(listen_cmd) as nc:
> > + wait_port_listen(port)
> > + cmd(f"echo -e \"hello\\nworld\"| nc {cfg.v6} {port}", host=cfg.remote, shell=True)
>
> FWIW, in the v3 of the series I submit, Jakub asked me to replace nc
> with socat due to issues with nc [1].
>
> Your usage of nc seems pretty basic though, so maybe it's fine?
>
> [1]: https://lore.kernel.org/netdev/20241101063426.2e1423a8@kernel.org/
Since I'm doing a respin anyway will try to move to socat as well to keep it
uniform, thanks!
Powered by blists - more mailing lists