From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38903) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WEOwW-00037B-A4 for qemu-devel@nongnu.org; Fri, 14 Feb 2014 15:01:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WEOwQ-0007LN-Kx for qemu-devel@nongnu.org; Fri, 14 Feb 2014 15:01:00 -0500 Received: from cantor2.suse.de ([195.135.220.15]:51847 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WEOwQ-0007LE-E7 for qemu-devel@nongnu.org; Fri, 14 Feb 2014 15:00:54 -0500 Message-ID: <52FE75F0.7090207@suse.de> Date: Fri, 14 Feb 2014 21:00:48 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1392057426-31990-1-git-send-email-afaerber@suse.de> <1392057426-31990-27-git-send-email-afaerber@suse.de> In-Reply-To: <1392057426-31990-27-git-send-email-afaerber@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL 26/48] tests: Add virtio-net qtest List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Stefan Hajnoczi , "Michael S. Tsirkin" Am 10.02.2014 19:36, schrieb Andreas F=C3=A4rber: > Cc: Michael S. Tsirkin > Reviewed-by: Stefan Hajnoczi > Signed-off-by: Andreas F=C3=A4rber > --- > tests/Makefile | 7 +++++++ > tests/virtio-net-test.c | 33 +++++++++++++++++++++++++++++++++ > 2 files changed, 40 insertions(+) > create mode 100644 tests/virtio-net-test.c >=20 > diff --git a/tests/Makefile b/tests/Makefile > index 62ae7f8..2acd358 100644 > --- a/tests/Makefile > +++ b/tests/Makefile > @@ -65,6 +65,10 @@ check-block-$(CONFIG_POSIX) +=3D tests/qemu-iotests-= quick.sh > # All QTests for now are POSIX-only, but the dependencies are > # really in libqtest, not in the testcases themselves. > =20 > +gcov-files-virtio-y +=3D hw/virtio/virtio.c > +check-qtest-virtio-y +=3D tests/virtio-net-test$(EXESUF) > +gcov-files-virtio-y +=3D hw/net/virtio-net.c > + > check-qtest-pci-y +=3D tests/e1000-test$(EXESUF) > gcov-files-pci-y +=3D hw/net/e1000.c > check-qtest-pci-y +=3D tests/rtl8139-test$(EXESUF) This does not work as intended. Needs path fixes: diff --git a/tests/Makefile b/tests/Makefile index 5424574..cfd0052 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -65,9 +65,9 @@ check-block-$(CONFIG_POSIX) +=3D tests/qemu-iotests-qui= ck.sh # All QTests for now are POSIX-only, but the dependencies are # really in libqtest, not in the testcases themselves. -gcov-files-virtio-y +=3D hw/virtio/virtio.c +gcov-files-virtio-y +=3D i386-softmmu/hw/virtio/virtio.c check-qtest-virtio-y +=3D tests/virtio-net-test$(EXESUF) -gcov-files-virtio-y +=3D hw/net/virtio-net.c +gcov-files-virtio-y +=3D i386-softmmu/hw/net/virtio-net.c check-qtest-pci-y +=3D tests/e1000-test$(EXESUF) gcov-files-pci-y +=3D hw/net/e1000.c Fixing for v2. Andreas > @@ -74,6 +78,8 @@ gcov-files-pci-y +=3D hw/net/pcnet.c > gcov-files-pci-y +=3D hw/net/pcnet-pci.c > check-qtest-pci-y +=3D tests/eepro100-test$(EXESUF) > gcov-files-pci-y +=3D hw/net/eepro100.c > +check-qtest-pci-y +=3D $(check-qtest-virtio-y) > +gcov-files-pci-y +=3D $(gcov-files-virtio-y) hw/virtio/virtio-pci.c > =20 > check-qtest-i386-y =3D tests/endianness-test$(EXESUF) > check-qtest-i386-y +=3D tests/fdc-test$(EXESUF) > @@ -219,6 +225,7 @@ tests/pcnet-test$(EXESUF): tests/pcnet-test.o > tests/eepro100-test$(EXESUF): tests/eepro100-test.o > tests/vmxnet3-test$(EXESUF): tests/vmxnet3-test.o > tests/ne2000-test$(EXESUF): tests/ne2000-test.o > +tests/virtio-net-test$(EXESUF): tests/virtio-net-test.o > tests/qom-test$(EXESUF): tests/qom-test.o > tests/blockdev-test$(EXESUF): tests/blockdev-test.o $(libqos-pc-obj-y) > tests/qdev-monitor-test$(EXESUF): tests/qdev-monitor-test.o $(libqos-p= c-obj-y) --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg