[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V3 3/3] tests/test-filter-redirector: Add unit t
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH V3 3/3] tests/test-filter-redirector: Add unit test for filter-redirector |
Date: |
Mon, 7 Mar 2016 16:10:34 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 03/04/2016 08:01 PM, Zhang Chen wrote:
> In this unit test,we will test the filter redirector function.
>
> Case 1, tx traffic flow:
>
> qemu side | test side
> |
> +---------+ | +-------+
> | backend <---------------+ sock0 |
> +----+----+ | +-------+
> | |
> +----v----+ +-------+ |
> | rd0 +->+chardev| |
> +---------+ +---+---+ |
> | |
> +---------+ | |
> | rd1 <------+ |
> +----+----+ |
> | |
> +----v----+ | +-------+
> | rd2 +--------------->sock1 |
> +---------+ | +-------+
> +
>
> Start qemu with:
>
> "-netdev socket,id=qtest-bn0,fd=%d "
> "-device rtl8139,netdev=qtest-bn0,id=qtest-e0 "
> "-chardev socket,id=redirector0,path=%s,server,nowait "
> "-chardev socket,id=redirector1,path=%s,server,nowait "
> "-chardev socket,id=redirector2,path=%s,nowait "
> "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
> "queue=tx,outdev=redirector0 "
> "-object filter-redirector,id=qtest-f1,netdev=qtest-bn0,"
> "queue=tx,indev=redirector2 "
> "-object filter-redirector,id=qtest-f2,netdev=qtest-bn0,"
> "queue=tx,outdev=redirector1 "
Both the patch and the above figure looks good. But it would be still
better to use several sentences to explain the test.
Thanks
>
> --------------------------------------
> Case 2, rx traffic flow
> qemu side | test side
> |
> +---------+ | +-------+
> | backend +---------------> sock1 |
> +----^----+ | +-------+
> | |
> +----+----+ +-------+ |
> | rd0 +<-+chardev| |
> +---------+ +---+---+ |
> ^ |
> +---------+ | |
> | rd1 +------+ |
> +----^----+ |
> | |
> +----+----+ | +-------+
> | rd2 <---------------+sock0 |
> +---------+ | +-------+
>
> Start qemu with:
>
> "-netdev socket,id=qtest-bn0,fd=%d "
> "-device rtl8139,netdev=qtest-bn0,id=qtest-e0 "
> "-chardev socket,id=redirector0,path=%s,server,nowait "
> "-chardev socket,id=redirector1,path=%s,server,nowait "
> "-chardev socket,id=redirector2,path=%s,nowait "
> "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
> "queue=rx,outdev=redirector0 "
> "-object filter-redirector,id=qtest-f1,netdev=qtest-bn0,"
> "queue=rx,indev=redirector2 "
> "-object filter-redirector,id=qtest-f2,netdev=qtest-bn0,"
> "queue=rx,outdev=redirector1 "
>
> Signed-off-by: Zhang Chen <address@hidden>
> Signed-off-by: Wen Congyang <address@hidden>
> ---
> tests/.gitignore | 1 +
> tests/Makefile | 2 +
> tests/test-filter-redirector.c | 214
> +++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 217 insertions(+)
> create mode 100644 tests/test-filter-redirector.c
>
> diff --git a/tests/.gitignore b/tests/.gitignore
> index 10df017..5069d5d 100644
> --- a/tests/.gitignore
> +++ b/tests/.gitignore
> @@ -64,5 +64,6 @@ test-x86-cpuid
> test-xbzrle
> test-netfilter
> test-filter-mirror
> +test-filter-redirector
> *-test
> qapi-schema/*.test.*
> diff --git a/tests/Makefile b/tests/Makefile
> index e56c514..275c4cc 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -213,6 +213,7 @@ check-qtest-x86_64-$(CONFIG_VHOST_NET_TEST_x86_64) +=
> tests/vhost-user-test$(EXE
> endif
> check-qtest-i386-y += tests/test-netfilter$(EXESUF)
> check-qtest-i386-y += tests/test-filter-mirror$(EXESUF)
> +check-qtest-i386-y += tests/test-filter-redirector$(EXESUF)
> check-qtest-x86_64-y = $(check-qtest-i386-y)
> gcov-files-i386-y += i386-softmmu/hw/timer/mc146818rtc.c
> gcov-files-x86_64-y = $(subst
> i386-softmmu/,x86_64-softmmu/,$(gcov-files-i386-y))
> @@ -565,6 +566,7 @@ tests/test-qemu-opts$(EXESUF): tests/test-qemu-opts.o
> $(test-util-obj-y)
> tests/test-write-threshold$(EXESUF): tests/test-write-threshold.o
> $(test-block-obj-y)
> tests/test-netfilter$(EXESUF): tests/test-netfilter.o $(qtest-obj-y)
> tests/test-filter-mirror$(EXESUF): tests/test-filter-mirror.o $(qtest-obj-y)
> +tests/test-filter-redirector$(EXESUF): tests/test-filter-redirector.o
> $(qtest-obj-y)
> tests/ivshmem-test$(EXESUF): tests/ivshmem-test.o
> contrib/ivshmem-server/ivshmem-server.o $(libqos-pc-obj-y)
> tests/vhost-user-bridge$(EXESUF): tests/vhost-user-bridge.o
>
> diff --git a/tests/test-filter-redirector.c b/tests/test-filter-redirector.c
> new file mode 100644
> index 0000000..6074cd7
> --- /dev/null
> +++ b/tests/test-filter-redirector.c
> @@ -0,0 +1,214 @@
> +/*
> + * QTest testcase for filter-redirector
> + *
> + * Copyright (c) 2016 FUJITSU LIMITED
> + * Author: Zhang Chen <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or
> + * later. See the COPYING file in the top-level directory.
> + *
> + * Case 1, tx traffic flow:
> + *
> + * qemu side | test side
> + * |
> + * +---------+ | +-------+
> + * | backend <---------------+ sock0 |
> + * +----+----+ | +-------+
> + * | |
> + * +----v----+ +-------+ |
> + * | rd0 +->+chardev| |
> + * +---------+ +---+---+ |
> + * | |
> + * +---------+ | |
> + * | rd1 <------+ |
> + * +----+----+ |
> + * | |
> + * +----v----+ | +-------+
> + * | rd2 +--------------->sock1 |
> + * +---------+ | +-------+
> + * +
> + *
> + * --------------------------------------
> + * Case 2, rx traffic flow
> + * qemu side | test side
> + * |
> + * +---------+ | +-------+
> + * | backend +---------------> sock1 |
> + * +----^----+ | +-------+
> + * | |
> + * +----+----+ +-------+ |
> + * | rd0 +<-+chardev| |
> + * +---------+ +---+---+ |
> + * ^ |
> + * +---------+ | |
> + * | rd1 +------+ |
> + * +----^----+ |
> + * | |
> + * +----+----+ | +-------+
> + * | rd2 <---------------+sock0 |
> + * +---------+ | +-------+
> + * +
> + */
> +
> +#include <glib.h>
> +#include "libqtest.h"
> +#include "qemu/iov.h"
> +#include "qemu/sockets.h"
> +#include "qemu/error-report.h"
> +#include "qemu/main-loop.h"
> +
> +static void test_redirector_tx(void)
> +{
> +#ifndef _WIN32
> +/* socketpair(PF_UNIX) which does not exist on windows */
> +
> + int backend_sock[2], recv_sock;
> + char *cmdline;
> + uint32_t ret = 0, len = 0;
> + char send_buf[] = "Hello!!";
> + char sock_path0[] = "filter-redirector0.XXXXXX";
> + char sock_path1[] = "filter-redirector1.XXXXXX";
> + char *recv_buf;
> + uint32_t size = sizeof(send_buf);
> + size = htonl(size);
> +
> + ret = socketpair(PF_UNIX, SOCK_STREAM, 0, backend_sock);
> + g_assert_cmpint(ret, !=, -1);
> +
> + ret = mkstemp(sock_path0);
> + g_assert_cmpint(ret, !=, -1);
> + ret = mkstemp(sock_path1);
> + g_assert_cmpint(ret, !=, -1);
> +
> + cmdline = g_strdup_printf("-netdev socket,id=qtest-bn0,fd=%d "
> + "-device rtl8139,netdev=qtest-bn0,id=qtest-e0 "
> + "-chardev socket,id=redirector0,path=%s,server,nowait "
> + "-chardev socket,id=redirector1,path=%s,server,nowait "
> + "-chardev socket,id=redirector2,path=%s,nowait "
> + "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
> + "queue=tx,outdev=redirector0 "
> + "-object filter-redirector,id=qtest-f1,netdev=qtest-bn0,"
> + "queue=tx,indev=redirector2 "
> + "-object filter-redirector,id=qtest-f2,netdev=qtest-bn0,"
> + "queue=tx,outdev=redirector1 "
> + , backend_sock[1], sock_path0, sock_path1, sock_path0);
> + qtest_start(cmdline);
> + g_free(cmdline);
> +
> + recv_sock = unix_connect(sock_path1, NULL);
> + g_assert_cmpint(recv_sock, !=, -1);
> +
> + struct iovec iov[] = {
> + {
> + .iov_base = &size,
> + .iov_len = sizeof(size),
> + }, {
> + .iov_base = send_buf,
> + .iov_len = sizeof(send_buf),
> + },
> + };
> +
> + ret = iov_send(backend_sock[0], iov, 2, 0, sizeof(size) +
> sizeof(send_buf));
> + g_assert_cmpint(ret, ==, sizeof(send_buf) + sizeof(size));
> + close(backend_sock[0]);
> +
> + ret = qemu_recv(recv_sock, &len, sizeof(len), 0);
> + g_assert_cmpint(ret, ==, sizeof(len));
> + len = ntohl(len);
> +
> + g_assert_cmpint(len, ==, sizeof(send_buf));
> + recv_buf = g_malloc(len);
> + ret = qemu_recv(recv_sock, recv_buf, len, 0);
> + g_assert_cmpstr(recv_buf, ==, send_buf);
> +
> + g_free(recv_buf);
> + close(recv_sock);
> + unlink(sock_path0);
> + unlink(sock_path1);
> + qtest_end();
> +
> +#endif
> +}
> +
> +static void test_redirector_rx(void)
> +{
> +#ifndef _WIN32
> +/* socketpair(PF_UNIX) which does not exist on windows */
> +
> + int backend_sock[2],send_sock, recv_sock;
> + char *cmdline;
> + uint32_t ret = 0, len = 0;
> + char send_buf[] = "Hello!!";
> + char sock_path0[] = "filter-redirector0.XXXXXX";
> + char sock_path1[] = "filter-redirector1.XXXXXX";
> + char *recv_buf;
> + uint32_t size = sizeof(send_buf);
> + size = htonl(size);
> +
> + ret = socketpair(PF_UNIX, SOCK_STREAM, 0, backend_sock);
> + g_assert_cmpint(ret, !=, -1);
> +
> + ret = mkstemp(sock_path0);
> + g_assert_cmpint(ret, !=, -1);
> + ret = mkstemp(sock_path1);
> + g_assert_cmpint(ret, !=, -1);
> +
> + cmdline = g_strdup_printf("-netdev socket,id=qtest-bn0,fd=%d "
> + "-device rtl8139,netdev=qtest-bn0,id=qtest-e0 "
> + "-chardev socket,id=redirector0,path=%s,server,nowait "
> + "-chardev socket,id=redirector1,path=%s,server,nowait "
> + "-chardev socket,id=redirector2,path=%s,nowait "
> + "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
> + "queue=rx,indev=redirector0 "
> + "-object filter-redirector,id=qtest-f1,netdev=qtest-bn0,"
> + "queue=rx,outdev=redirector2 "
> + "-object filter-redirector,id=qtest-f2,netdev=qtest-bn0,"
> + "queue=rx,indev=redirector1 "
> + , backend_sock[1], sock_path0, sock_path1, sock_path0);
> + qtest_start(cmdline);
> + g_free(cmdline);
> +
> + struct iovec iov[] = {
> + {
> + .iov_base = &size,
> + .iov_len = sizeof(size),
> + }, {
> + .iov_base = send_buf,
> + .iov_len = sizeof(send_buf),
> + },
> + };
> +
> + send_sock = unix_connect(sock_path1, NULL);
> + ret = iov_send(send_sock, iov, 2, 0, sizeof(size) + sizeof(send_buf));
> + g_assert_cmpint(ret, ==, sizeof(send_buf) + sizeof(size));
> + close(send_sock);
> +
> + ret = qemu_recv(backend_sock[0], &len, sizeof(len), 0);
> + g_assert_cmpint(ret, ==, sizeof(len));
> + len = ntohl(len);
> +
> + g_assert_cmpint(len, ==, sizeof(send_buf));
> + recv_buf = g_malloc(len);
> + ret = qemu_recv(backend_sock[0], recv_buf, len, 0);
> + g_assert_cmpstr(recv_buf, ==, send_buf);
> +
> + g_free(recv_buf);
> + close(recv_sock);
> + unlink(sock_path0);
> + unlink(sock_path1);
> + qtest_end();
> +
> +#endif
> +}
> +
> +int main(int argc, char **argv)
> +{
> + int ret;
> +
> + g_test_init(&argc, &argv, NULL);
> + qtest_add_func("/netfilter/redirector_tx", test_redirector_tx);
> + qtest_add_func("/netfilter/redirector_rx", test_redirector_rx);
> + ret = g_test_run();
> +
> + return ret;
> +}
- [Qemu-devel] [PATCH V3 1/3] net/filter-mirror: Change filter_mirror_send interface, (continued)
[Qemu-devel] [PATCH V3 3/3] tests/test-filter-redirector: Add unit test for filter-redirector, Zhang Chen, 2016/03/04
- Re: [Qemu-devel] [PATCH V3 3/3] tests/test-filter-redirector: Add unit test for filter-redirector,
Jason Wang <=