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). > > 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. > > 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. > > 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 > > 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. > > 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() > > 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 > > 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(-) > -- 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
signature.asc
Description: PGP signature