From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41812) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bwImy-0002j6-Hq for qemu-devel@nongnu.org; Mon, 17 Oct 2016 21:01:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bwImu-0001zk-LA for qemu-devel@nongnu.org; Mon, 17 Oct 2016 21:01:56 -0400 Date: Tue, 18 Oct 2016 12:00:20 +1100 From: David Gibson Message-ID: <20161018010020.GA25390@umbus.fritz.box> References: <1476700224-13186-1-git-send-email-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="309evMHi/619oHyA" Content-Disposition: inline In-Reply-To: <1476700224-13186-1-git-send-email-lvivier@redhat.com> Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v5 0/6] tests: enable virtio tests on SPAPR List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Laurent Vivier Cc: qemu-devel@nongnu.org, dgibson@redhat.com, thuth@redhat.com, qemu-ppc@nongnu.org, Greg Kurz --309evMHi/619oHyA Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 17, 2016 at 12:30:18PM +0200, Laurent Vivier wrote: > This series enables virtio tests on SPAPR by starting > machines using qtest_pc_boot() or qtest_spapr_boot() to > use the good libqos PCI framework (pc or spapr). >=20 > It adds also some byte-swapping in virtio-pci.c as > PCI is always little-endian and the endianness of > the virtio device depends on the endianness of the > guest. >=20 > This series does not enable virtio PCI MSI-X tests on > SPAPR as this needs more work and will be the aim > of another series. Looking good to me. I've tentatively merged this to ppc-for-2.8, unless someone thinks it doesn't belong there. >=20 > v5: > - minor updates: > - remove declaration within the code > - update comment about qtest_vboot() > (terminates on failure) > v4: > - rebase on papr/ppc-for-2.8 > - add a patch to rename target_big_endian() to qvirtio_is_big_endian() > - Add a patch to remove the QVirtioBus arguments from all virtio functions >=20 > v3: > - remove "ppc" from allowed archs for virtio-blk-test > - remove g_assert_nonnull() after g_malloc() > - add a patch to fix "vs" memory leak in virtio-scsi-test > - add a patch to remove g_assert() after qtest_spapr_boot() > and update comment qtest_vboot() to explain it never > returns NULL. >=20 > v2: > - update comments (virtio-1.0, log) > - move g_assert_nonnull() to qtest_boot() > - exit gracefully if the architecture is not supported > - replace qtest_pc_shutdown() by qtest_shutdown() > - move qvirtio_scsi_stop() to qvirtio_scsi_pci_free() >=20 > Laurent Vivier (6): > tests: fix memory leak in virtio-scsi-test > tests: don't check if qtest_spapr_boot() returns NULL > tests: move QVirtioBus pointer into QVirtioDevice > tests: rename target_big_endian() as qvirtio_is_big_endian() > tests: use qtest_pc_boot()/qtest_shutdown() in virtio tests > tests: enable virtio tests on SPAPR >=20 > tests/Makefile.include | 3 +- > tests/libqos/libqos.c | 2 + > tests/libqos/virtio-mmio.c | 1 + > tests/libqos/virtio-pci.c | 28 +++- > tests/libqos/virtio.c | 77 +++++------ > tests/libqos/virtio.h | 57 ++++---- > tests/libqtest.h | 10 -- > tests/rtas-test.c | 1 - > tests/vhost-user-test.c | 33 +++-- > tests/virtio-9p-test.c | 69 +++++----- > tests/virtio-blk-test.c | 322 +++++++++++++++++++++------------------= ------ > tests/virtio-net-test.c | 106 +++++++-------- > tests/virtio-rng-test.c | 7 +- > tests/virtio-scsi-test.c | 91 ++++++------- > 14 files changed, 400 insertions(+), 407 deletions(-) >=20 --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --309evMHi/619oHyA Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJYBXQhAAoJEGw4ysog2bOSUYcP/1HCTmIyClrb99XymnBiiL9X ZSJxEy0HEJ+2brrs5QvhOOvEwS/fDzUaw6tJ/R8/aTH3gI/4yBqnC9S0CuycL5Ks sQMmANooqGq+/NvE5Ee74wr6Qw9bhpqr+Etf5cMSY9oyQTba2/Qfq6yQSD2hxa6E tOGlqmMrufxWx60BBklYGm97oVQ7UG6UynJ2uuC8ybx1vt+AykbSvhAgRMckWHJT /b7Li/CRP2yIQnp2xCutf9lMG3vgwED8fs/+cR9HeA0X/Pv8WwbqdHrzFR46Wfmv oCa6oA3CPnISaFxpQzfRVlETiZxvXiHw9HU0aCOEW9li3lfUzCEvJQPhty6q8Esl FJEn8TQNrdZZrX+G/rnKcWeYkqqdsuAkVkFyqC+XHUNX7X2tWY94aByKob0Z2gI3 8twgtWmxQpcdJJ4/uSxZhRSvQcKS9H3HJ8pSo08xM7IoOtojvrlniIY2Iso8S7hK LCwOVK9oCkB7RsYlvnQlVkMlue+Rz4B+YZUk3gAzB/A5bx6S8o7YJ0UDAW9GE/qc Rgjwqcch7VTBbTKDkHWe9wBpP63ZQTz1Mjy3aka/NGhfwfUu4kG8W/eFWpcAwI/0 sevVCqmrooPZfm2sRB9AnjtX6DJ2X9NyFbaYGxD8kMtsGFGiJCKP4EsfBe4Z2lfi xhPagRCsKYibEiGTb3jV =WqT+ -----END PGP SIGNATURE----- --309evMHi/619oHyA--