[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/6] tests: Run filter-redirector and -mirror te
From: |
Cleber Rosa |
Subject: |
Re: [Qemu-devel] [PATCH 1/6] tests: Run filter-redirector and -mirror test only on POSIX systems |
Date: |
Wed, 30 Aug 2017 15:35:15 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 08/17/2017 02:25 AM, Thomas Huth wrote:
> This way we can get rid of the ugly #ifdefs in the code which makes
> it easier to extend later.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> tests/Makefile.include | 8 ++++----
> tests/test-filter-mirror.c | 5 -----
> tests/test-filter-redirector.c | 10 ----------
> 3 files changed, 4 insertions(+), 19 deletions(-)
>
> diff --git a/tests/Makefile.include b/tests/Makefile.include
> index 37c1bed..8d5991d 100644
> --- a/tests/Makefile.include
> +++ b/tests/Makefile.include
> @@ -283,8 +283,8 @@ ifeq ($(CONFIG_VHOST_USER_NET_TEST_i386),)
> check-qtest-x86_64-$(CONFIG_VHOST_USER_NET_TEST_x86_64) +=
> tests/vhost-user-test$(EXESUF)
> endif
> check-qtest-i386-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
> -check-qtest-i386-y += tests/test-filter-mirror$(EXESUF)
> -check-qtest-i386-y += tests/test-filter-redirector$(EXESUF)
> +check-qtest-i386-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
> +check-qtest-i386-$(CONFIG_POSIX) += tests/test-filter-redirector$(EXESUF)
> check-qtest-i386-y += tests/postcopy-test$(EXESUF)
> check-qtest-i386-y += tests/test-x86-cpuid-compat$(EXESUF)
> check-qtest-i386-y += tests/numa-test$(EXESUF)
> @@ -325,8 +325,8 @@ check-qtest-ppc64-y += tests/usb-hcd-xhci-test$(EXESUF)
> gcov-files-ppc64-y += hw/usb/hcd-xhci.c
> check-qtest-ppc64-y += $(check-qtest-virtio-y)
> check-qtest-ppc64-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
> -check-qtest-ppc64-y += tests/test-filter-mirror$(EXESUF)
> -check-qtest-ppc64-y += tests/test-filter-redirector$(EXESUF)
> +check-qtest-ppc64-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
> +check-qtest-ppc64-$(CONFIG_POSIX) += tests/test-filter-redirector$(EXESUF)
> check-qtest-ppc64-y += tests/display-vga-test$(EXESUF)
> check-qtest-ppc64-y += tests/numa-test$(EXESUF)
> check-qtest-ppc64-$(CONFIG_IVSHMEM) += tests/ivshmem-test$(EXESUF)
> diff --git a/tests/test-filter-mirror.c b/tests/test-filter-mirror.c
> index 9f84402..a1d5865 100644
> --- a/tests/test-filter-mirror.c
> +++ b/tests/test-filter-mirror.c
> @@ -17,9 +17,6 @@
>
> static void test_mirror(void)
> {
> -#ifndef _WIN32
> -/* socketpair(PF_UNIX) which does not exist on windows */
> -
> int send_sock[2], recv_sock;
> char *cmdline;
> uint32_t ret = 0, len = 0;
> @@ -74,8 +71,6 @@ static void test_mirror(void)
> g_free(recv_buf);
> close(recv_sock);
> unlink(sock_path);
> -
> -#endif
> }
>
> int main(int argc, char **argv)
> diff --git a/tests/test-filter-redirector.c b/tests/test-filter-redirector.c
> index 0c4b8d5..69c663b 100644
> --- a/tests/test-filter-redirector.c
> +++ b/tests/test-filter-redirector.c
> @@ -59,9 +59,6 @@
>
> 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;
> @@ -129,15 +126,10 @@ static void test_redirector_tx(void)
> 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;
> char *cmdline;
> uint32_t ret = 0, len = 0;
> @@ -203,8 +195,6 @@ static void test_redirector_rx(void)
> unlink(sock_path0);
> unlink(sock_path1);
> qtest_end();
> -
> -#endif
> }
>
> int main(int argc, char **argv)
>
Tested attempting to cross-compile those:
$ ./configure --cross-prefix=i686-w64-mingw32-
$ make
$ make tests/test-filter-mirror.exe
...
CC tests/test-filter-redirector.o
tests/test-filter-redirector.c: In function 'test_redirector_tx':
tests/test-filter-redirector.c:72:11: error: implicit declaration of
function 'socketpair'; did you mean 'socket_init'?
[-Werror=implicit-function-declaration]
...
Which is supposed to fail.
Tested-by: Cleber Rosa <address@hidden>
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH for-2.11 0/6] Enable more qtests for s390x, Thomas Huth, 2017/08/17
- [Qemu-devel] [PATCH 1/6] tests: Run filter-redirector and -mirror test only on POSIX systems, Thomas Huth, 2017/08/17
- [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Thomas Huth, 2017/08/17
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Cornelia Huck, 2017/08/17
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Thomas Huth, 2017/08/17
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Zhang Chen, 2017/08/17
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Cornelia Huck, 2017/08/18
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Thomas Huth, 2017/08/18
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Cleber Rosa, 2017/08/30
- Re: [Qemu-devel] [PATCH 2/6] tests: Add network filter tests to the check-qtest-s390x list, Cleber Rosa, 2017/08/30
[Qemu-devel] [PATCH 3/6] tests: Enable the drive_del test also on s390x, Thomas Huth, 2017/08/17