Re: [PATCH net-next v8 12/12] selftests: ncdevmem: Add automated test

From: Joe Damato
Date: Thu Nov 07 2024 - 13:38:10 EST


On Thu, Nov 07, 2024 at 10:12:11AM -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@${PEER}\n"
>
> echo -e "$cfg" | ssh root@${HOST} "cat > ksft/drivers/net/net.config"
> ssh root@${HOST} "cd ksft && ./run_kselftest.sh -t drivers/net:devmem.py"
>
> Reviewed-by: Mina Almasry <almasrymina@xxxxxxxxxx>
> Signed-off-by: Stanislav Fomichev <sdf@xxxxxxxxxxx>
> ---
> .../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

I'm not a python expert, but the changes make sense to me.

Reviewed-by: Joe Damato <jdamato@xxxxxxxxxx>